Message ID | 1573256452-14838-2-git-send-email-eberman@codeaurora.org (mailing list archive) |
---|---|
State | Not Applicable, archived |
Headers | show |
Series | Support reboot power-off driver as module | expand |
On Fri 08 Nov 15:40 PST 2019, Elliot Berman wrote: > Export reboot_mode to support kernel modules wishing to modify reboot_mode. > Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> > Signed-off-by: Elliot Berman <eberman@codeaurora.org> > --- > kernel/reboot.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/kernel/reboot.c b/kernel/reboot.c > index c4d472b..b1fbc22 100644 > --- a/kernel/reboot.c > +++ b/kernel/reboot.c > @@ -32,7 +32,9 @@ EXPORT_SYMBOL(cad_pid); > #define DEFAULT_REBOOT_MODE > #endif > enum reboot_mode reboot_mode DEFAULT_REBOOT_MODE; > +EXPORT_SYMBOL_GPL(reboot_mode); > enum reboot_mode panic_reboot_mode = REBOOT_UNDEFINED; > +EXPORT_SYMBOL_GPL(panic_reboot_mode); > > /* > * This variable is used privately to keep track of whether or not > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > a Linux Foundation Collaborative Project >
On Fri, Nov 08, 2019 at 03:40:51PM -0800, Elliot Berman wrote: > Export reboot_mode to support kernel modules wishing to modify reboot_mode. > > Signed-off-by: Elliot Berman <eberman@codeaurora.org> Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
On Mon, May 24, 2021 at 01:09:18PM +0100, Lee Jones wrote: > On Fri, 8 Nov 2019 at 23:41, Elliot Berman <eberman@codeaurora.org> wrote: > > > Export reboot_mode to support kernel modules wishing to modify reboot_mode. > > > > Signed-off-by: Elliot Berman <eberman@codeaurora.org> > > --- > > kernel/reboot.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > Reviewed by Bjorn and GregKH, but never merged. > > I guess all the parties are still solving this downstream. > > Andrew, is this still something you'd consider? If there is no in-kernel user, it does not need to be exported. thanks, greg k-h
On Mon 24 May 07:09 CDT 2021, Lee Jones wrote: > On Fri, 8 Nov 2019 at 23:41, Elliot Berman <eberman@codeaurora.org> wrote: > > > Export reboot_mode to support kernel modules wishing to modify reboot_mode. > > > > Signed-off-by: Elliot Berman <eberman@codeaurora.org> > > --- > > kernel/reboot.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > Reviewed by Bjorn and GregKH, but never merged. > > I guess all the parties are still solving this downstream. > > Andrew, is this still something you'd consider? > Seems more reasonable that it would go together with patch 2, which Sebastian pointed out doesn't build. Regards, Bjorn > diff --git a/kernel/reboot.c b/kernel/reboot.c > > index c4d472b..b1fbc22 100644 > > --- a/kernel/reboot.c > > +++ b/kernel/reboot.c > > @@ -32,7 +32,9 @@ EXPORT_SYMBOL(cad_pid); > > #define DEFAULT_REBOOT_MODE > > #endif > > enum reboot_mode reboot_mode DEFAULT_REBOOT_MODE; > > +EXPORT_SYMBOL_GPL(reboot_mode); > > enum reboot_mode panic_reboot_mode = REBOOT_UNDEFINED; > > +EXPORT_SYMBOL_GPL(panic_reboot_mode); > > > > /* > > * This variable is used privately to keep track of whether or not > > -- > > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > > a Linux Foundation Collaborative Project > > > > > > -- > Lee Jones [?????????] > Linaro Services Senior Technical Lead > Linaro.org ??? Open source software for ARM SoCs > Follow Linaro: Facebook | Twitter | Blog
diff --git a/kernel/reboot.c b/kernel/reboot.c index c4d472b..b1fbc22 100644 --- a/kernel/reboot.c +++ b/kernel/reboot.c @@ -32,7 +32,9 @@ EXPORT_SYMBOL(cad_pid); #define DEFAULT_REBOOT_MODE #endif enum reboot_mode reboot_mode DEFAULT_REBOOT_MODE; +EXPORT_SYMBOL_GPL(reboot_mode); enum reboot_mode panic_reboot_mode = REBOOT_UNDEFINED; +EXPORT_SYMBOL_GPL(panic_reboot_mode); /* * This variable is used privately to keep track of whether or not
Export reboot_mode to support kernel modules wishing to modify reboot_mode. Signed-off-by: Elliot Berman <eberman@codeaurora.org> --- kernel/reboot.c | 2 ++ 1 file changed, 2 insertions(+)