Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Thu, Dec 10, 2015 at 4:11 PM, Arnd Bergmann wrote: > On Thursday 10 December 2015 13:43:16 John Stultz wrote: >> On Thu, Dec 10, 2015 at 12:24 PM, Rob Herring wrote: >> > The fact that we are using notifiers for reset reason and triggering >> > is probably

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Wed 09 Dec 17:32 PST 2015, John Stultz wrote: > On Tue, Dec 8, 2015 at 2:07 PM, Bjorn Andersson > wrote: > > On Tue 08 Dec 13:29 PST 2015, John Stultz wrote: > >> diff --git a/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts > >>

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Thu, Dec 10, 2015 at 1:20 AM, Tomas Winkler wrote: > Intel uses EFI variables for that on some AOS platforms. There is a > need for persistent storage abstraction and generalize the reboot > reasons strings. Yea. I've been told there isn't any sort of standardized method

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Thu, Dec 10, 2015 at 6:52 AM, Arnd Bergmann wrote: > On Wednesday 09 December 2015 17:19:52 John Stultz wrote: >> >> If the concern is that since DT is basically ABI, one might not want >> to have such a wide interface that specifies all the different >> reasons, I can

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Thu, Dec 10, 2015 at 11:57 AM, One Thousand Gnomes wrote: > On Thu, 10 Dec 2015 11:04:03 -0800 > John Stultz wrote: >> On Thu, Dec 10, 2015 at 1:20 AM, Tomas Winkler wrote: >> > Second, I wonder why this is submitted under

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Thu, Dec 10, 2015 at 11:05 AM, Arnd Bergmann wrote: > On Wednesday 09 December 2015 17:32:02 John Stultz wrote: >> On Tue, Dec 8, 2015 at 2:07 PM, Bjorn Andersson >> wrote: >> > On Tue 08 Dec 13:29 PST 2015, John Stultz wrote: >> >> diff --git

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Wednesday 09 December 2015 17:32:02 John Stultz wrote: > On Tue, Dec 8, 2015 at 2:07 PM, Bjorn Andersson > wrote: > > On Tue 08 Dec 13:29 PST 2015, John Stultz wrote: > >> diff --git a/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts > >>

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Thu, Dec 10, 2015 at 12:24 PM, Rob Herring wrote: > The fact that we are using notifiers for reset reason and triggering > is probably some indication that some infrastructure is needed. But I > don't think you need to do that here as long as it is all kernel > internals.

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Thu, Dec 10, 2015 at 12:56 PM, John Stultz wrote: > On Thu, Dec 10, 2015 at 6:52 AM, Arnd Bergmann wrote: >> On Wednesday 09 December 2015 17:19:52 John Stultz wrote: >>> >>> If the concern is that since DT is basically ABI, one might not want >>> to

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Wednesday 09 December 2015 17:19:52 John Stultz wrote: > On Wed, Dec 9, 2015 at 2:07 AM, Arnd Bergmann wrote: > > On Tuesday 08 December 2015 16:22:40 John Stultz wrote: > >> >> diff --git a/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts > >> >>

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Tuesday 08 December 2015 16:22:40 John Stultz wrote: > >> diff --git a/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts > >> b/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts > >> index 5183d18..ee5dcb7 100644 > >> --- a/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts > >> +++

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Tue, Dec 8, 2015 at 2:07 PM, Bjorn Andersson wrote: > On Tue 08 Dec 13:29 PST 2015, John Stultz wrote: >> diff --git a/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts >> b/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts >> index 5183d18..ee5dcb7 100644 >> ---

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Tue, Dec 8, 2015 at 2:07 PM, Bjorn Andersson wrote: > On Tue 08 Dec 13:29 PST 2015, John Stultz wrote: > >> This patch adds a basic driver to allow for commands like >> "reboot bootloader" and "reboot recovery" to communicate this >> reboot-reason to the

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

On Tue 08 Dec 13:29 PST 2015, John Stultz wrote: > This patch adds a basic driver to allow for commands like > "reboot bootloader" and "reboot recovery" to communicate this > reboot-reason to the bootloader. > > This is commonly done on Android devices, in order to reboot > the device into