From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754896AbcDMSJI (ORCPT ); Wed, 13 Apr 2016 14:09:08 -0400 Received: from mail-qg0-f52.google.com ([209.85.192.52]:34941 "EHLO mail-qg0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754861AbcDMSIv (ORCPT ); Wed, 13 Apr 2016 14:08:51 -0400 From: Ezequiel Garcia To: , , , Cc: Richard Purdie , Jacek Anaszewski , Kumar Gala , Ian Campbell , Mark Rutland , Pawel Moll , Rob Herring , Ezequiel Garcia Subject: [PATCH v2 3/3] leds: gpio: Support the "panic-indicator" firmware property Date: Wed, 13 Apr 2016 15:08:19 -0300 Message-Id: <1460570899-24642-4-git-send-email-ezequiel@vanguardiasur.com.ar> X-Mailer: git-send-email 2.7.0 In-Reply-To: <1460570899-24642-1-git-send-email-ezequiel@vanguardiasur.com.ar> References: <1460570899-24642-1-git-send-email-ezequiel@vanguardiasur.com.ar> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Calling a GPIO LEDs is quite likely to work even if the kernel has paniced, so they are ideal to blink in this situation. This commit adds support for the new "panic-indicator" firmware property, allowing to mark a given LED to blink on a kernel panic. Signed-off-by: Ezequiel Garcia --- drivers/leds/leds-gpio.c | 4 ++++ include/linux/leds.h | 1 + 2 files changed, 5 insertions(+) diff --git a/drivers/leds/leds-gpio.c b/drivers/leds/leds-gpio.c index 61143f55597e..8229f063b483 100644 --- a/drivers/leds/leds-gpio.c +++ b/drivers/leds/leds-gpio.c @@ -127,6 +127,8 @@ static int create_gpio_led(const struct gpio_led *template, led_dat->cdev.brightness = state ? LED_FULL : LED_OFF; if (!template->retain_state_suspended) led_dat->cdev.flags |= LED_CORE_SUSPENDRESUME; + if (template->panic_indicator) + led_dat->cdev.flags |= LED_PANIC_INDICATOR; ret = gpiod_direction_output(led_dat->gpiod, state); if (ret < 0) @@ -200,6 +202,8 @@ static struct gpio_leds_priv *gpio_leds_create(struct platform_device *pdev) if (fwnode_property_present(child, "retain-state-suspended")) led.retain_state_suspended = 1; + if (fwnode_property_present(child, "panic-indicator")) + led.panic_indicator = 1; ret = create_gpio_led(&led, &priv->leds[priv->num_leds], dev, NULL); diff --git a/include/linux/leds.h b/include/linux/leds.h index 49adf9c6e326..1067fb5f9296 100644 --- a/include/linux/leds.h +++ b/include/linux/leds.h @@ -359,6 +359,7 @@ struct gpio_led { unsigned gpio; unsigned active_low : 1; unsigned retain_state_suspended : 1; + unsigned panic_indicator : 1; unsigned default_state : 2; /* default_state should be one of LEDS_GPIO_DEFSTATE_(ON|OFF|KEEP) */ struct gpio_desc *gpiod; -- 2.7.0