From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: [PATCH v5 23/48] power/reset: gpio-poweroff: Register with kernel power-off handler Date: Thu, 6 Nov 2014 08:43:07 -0800 Message-ID: <1415292213-28652-24-git-send-email-linux@roeck-us.net> References: <1415292213-28652-1-git-send-email-linux@roeck-us.net> Return-path: Received: from bh-25.webhostbox.net ([208.91.199.152]:56279 "EHLO bh-25.webhostbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751976AbaKFQog (ORCPT ); Thu, 6 Nov 2014 11:44:36 -0500 Received: from mailnull by bh-25.webhostbox.net with sa-checked (Exim 4.82) (envelope-from ) id 1XmQAl-000Tvl-UI for linux-pm@vger.kernel.org; Thu, 06 Nov 2014 16:44:36 +0000 In-Reply-To: <1415292213-28652-1-git-send-email-linux@roeck-us.net> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: linux-kernel@vger.kernel.org Cc: linux-pm@vger.kernel.org, Guenter Roeck , Sebastian Reichel , Dmitry Eremin-Solenikov , David Woodhouse Register with kernel power-off handler instead of setting pm_power_off directly. Register with low priority to reflect that the original code only sets pm_power_off if it was not already set. Other changes: Drop note that there can not be an additional instance of this driver. The original reason no longer applies, it should be obvious that there can only be one instance of the driver if static variables are used to reflect its state, and support for multiple instances can now be added easily if needed by avoiding static variables. Do not create an error message if another power-off handler has already been registered. This is perfectly normal and acceptable. Drop remove function since it is no longer needed. Cc: Sebastian Reichel Cc: Dmitry Eremin-Solenikov Cc: David Woodhouse Acked-by: Andrew Lunn Acked-by: Sebastian Reichel Signed-off-by: Guenter Roeck --- v5: - Rebase to v3.18-rc3 v4: - Do not use notifiers but internal functions and data structures to manage the list of power-off handlers. Drop unused parameters from callbacks, and make the power-off function type void v3: - Replace poweroff in all newly introduced variables and in text with power_off or power-off as appropriate - Replace POWEROFF_PRIORITY_xxx with POWER_OFF_PRIORITY_xxx v2; - Use define to specify poweroff handler priority - Use devm_register_power_off_handler - Drop remove function as it is no longer needed drivers/power/reset/gpio-poweroff.c | 36 +++++++++++++----------------------- 1 file changed, 13 insertions(+), 23 deletions(-) diff --git a/drivers/power/reset/gpio-poweroff.c b/drivers/power/reset/gpio-poweroff.c index ce849bc..4d480d2 100644 --- a/drivers/power/reset/gpio-poweroff.c +++ b/drivers/power/reset/gpio-poweroff.c @@ -14,18 +14,16 @@ #include #include #include +#include #include #include #include #include -/* - * Hold configuration here, cannot be more than one instance of the driver - * since pm_power_off itself is global. - */ static struct gpio_desc *reset_gpio; -static void gpio_poweroff_do_poweroff(void) +static void gpio_poweroff_do_poweroff(struct power_off_handler_block *this) + { BUG_ON(!reset_gpio); @@ -45,17 +43,15 @@ static void gpio_poweroff_do_poweroff(void) WARN_ON(1); } +static struct power_off_handler_block gpio_power_off_hb = { + .handler = gpio_poweroff_do_poweroff, + .priority = POWER_OFF_PRIORITY_LOW, +}; + static int gpio_poweroff_probe(struct platform_device *pdev) { bool input = false; - - /* If a pm_power_off function has already been added, leave it alone */ - if (pm_power_off != NULL) { - dev_err(&pdev->dev, - "%s: pm_power_off function already registered", - __func__); - return -EBUSY; - } + int err; reset_gpio = devm_gpiod_get(&pdev->dev, NULL); if (IS_ERR(reset_gpio)) @@ -77,16 +73,11 @@ static int gpio_poweroff_probe(struct platform_device *pdev) } } - pm_power_off = &gpio_poweroff_do_poweroff; - return 0; -} - -static int gpio_poweroff_remove(struct platform_device *pdev) -{ - if (pm_power_off == &gpio_poweroff_do_poweroff) - pm_power_off = NULL; + err = devm_register_power_off_handler(&pdev->dev, &gpio_power_off_hb); + if (err) + dev_err(&pdev->dev, "Failed to register power-off handler\n"); - return 0; + return err; } static const struct of_device_id of_gpio_poweroff_match[] = { @@ -96,7 +87,6 @@ static const struct of_device_id of_gpio_poweroff_match[] = { static struct platform_driver gpio_poweroff_driver = { .probe = gpio_poweroff_probe, - .remove = gpio_poweroff_remove, .driver = { .name = "poweroff-gpio", .owner = THIS_MODULE, -- 1.9.1