From mboxrd@z Thu Jan 1 00:00:00 1970 From: geert@linux-m68k.org (Geert Uytterhoeven) Date: Thu, 4 Dec 2014 14:36:27 +0100 Subject: [PATCH v7 08/11] arm/arm64: Unexport restart handlers In-Reply-To: <1408495538-27480-9-git-send-email-linux@roeck-us.net> References: <1408495538-27480-1-git-send-email-linux@roeck-us.net> <1408495538-27480-9-git-send-email-linux@roeck-us.net> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi G?nther, On Wed, Aug 20, 2014 at 2:45 AM, Guenter Roeck wrote: > Implementing a restart handler in a module don't make sense > as there would be no guarantee that the module is loaded when > a restart is needed. Unexport arm_pm_restart to ensure that > no one gets the idea to do it anyway. Why not? I was just going to do that, but I got greeted by: ERROR: "arm_pm_restart" [drivers/power/reset/rmobile-reset.ko] undefined! So now we have to make sure all reset drivers for a zillion different hardware devices are builtin, and can't be modular? > Signed-off-by: Guenter Roeck > Acked-by: Catalin Marinas > Acked-by: Heiko Stuebner > --- > v7: No change > v6: No change > v5: No change > v4: No change > v3: No change > v2: No change > > arch/arm/kernel/process.c | 1 - > arch/arm64/kernel/process.c | 1 - > 2 files changed, 2 deletions(-) > > diff --git a/arch/arm/kernel/process.c b/arch/arm/kernel/process.c > index ea279f7..250b6f6 100644 > --- a/arch/arm/kernel/process.c > +++ b/arch/arm/kernel/process.c > @@ -121,7 +121,6 @@ void (*pm_power_off)(void); > EXPORT_SYMBOL(pm_power_off); > > void (*arm_pm_restart)(enum reboot_mode reboot_mode, const char *cmd); > -EXPORT_SYMBOL_GPL(arm_pm_restart); > > /* > * This is our default idle handler. > diff --git a/arch/arm64/kernel/process.c b/arch/arm64/kernel/process.c > index 0d3fb9f..398ab05 100644 > --- a/arch/arm64/kernel/process.c > +++ b/arch/arm64/kernel/process.c > @@ -98,7 +98,6 @@ void (*pm_power_off)(void); > EXPORT_SYMBOL_GPL(pm_power_off); > > void (*arm_pm_restart)(enum reboot_mode reboot_mode, const char *cmd); > -EXPORT_SYMBOL_GPL(arm_pm_restart); > > /* > * This is our default idle handler. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds