From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: [PATCH v5 48/48] kernel: Remove pm_power_off Date: Thu, 6 Nov 2014 08:43:32 -0800 Message-ID: <1415292213-28652-49-git-send-email-linux@roeck-us.net> References: <1415292213-28652-1-git-send-email-linux@roeck-us.net> Return-path: In-Reply-To: <1415292213-28652-1-git-send-email-linux@roeck-us.net> Sender: linux-kernel-owner@vger.kernel.org To: linux-kernel@vger.kernel.org Cc: linux-pm@vger.kernel.org, Guenter Roeck , "Rafael J. Wysocki" , Pavel Machek , Len Brown List-Id: linux-pm@vger.kernel.org No users of pm_power_off are left, so it is safe to remove the function. Cc: Rafael J. Wysocki Cc: Pavel Machek Cc: Len Brown Acked-by: Rafael J. Wysocki Acked-by: Pavel Machek 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 v2: - poweroff -> power_off for API functions include/linux/pm.h | 1 - kernel/power/power_off_handler.c | 13 +------------ 2 files changed, 1 insertion(+), 13 deletions(-) diff --git a/include/linux/pm.h b/include/linux/pm.h index a4d6bf8..8fcf352 100644 --- a/include/linux/pm.h +++ b/include/linux/pm.h @@ -31,7 +31,6 @@ /* * Callbacks for platform drivers to implement. */ -extern void (*pm_power_off)(void); extern void (*pm_power_off_prepare)(void); struct device; /* we have a circular dep with device.h */ diff --git a/kernel/power/power_off_handler.c b/kernel/power/power_off_handler.c index e283ea1..d67ab57 100644 --- a/kernel/power/power_off_handler.c +++ b/kernel/power/power_off_handler.c @@ -23,12 +23,6 @@ #include /* - * If set, calling this function will power off the system immediately. - */ -void (*pm_power_off)(void); -EXPORT_SYMBOL(pm_power_off); - -/* * List of handlers for kernel code which wants to be called * to power off the system. */ @@ -277,10 +271,6 @@ void do_kernel_power_off(void) * removed while the call chain is traversed, but we'll have to accept * that risk. */ - - if (pm_power_off) - pm_power_off(); - p = rcu_dereference_raw(power_off_handler_list); while (p) { next_p = rcu_dereference_raw(p->next); @@ -296,7 +286,6 @@ void do_kernel_power_off(void) */ bool have_kernel_power_off(void) { - return pm_power_off != NULL || - rcu_dereference_raw(power_off_handler_list) != NULL; + return rcu_dereference_raw(power_off_handler_list) != NULL; } EXPORT_SYMBOL(have_kernel_power_off); -- 1.9.1