Re: [PATCH v3 2/2] gpio_wdt: Add option for early registration

2015-06-06 Thread Guenter Roeck
On 06/06/2015 12:46 AM, Jean-Baptiste Theou wrote: In some situation, mainly when it's not possible to disable a watchdog, you may want the watchdog driver to be started as soon as possible. Adding GPIO_WATCHDOG_ARCH_INITCALL to raise initcall from module_init to arch_initcall. This patch requi

Re: [PATCH v3 2/2] gpio_wdt: Add option for early registration

2015-06-06 Thread Jean-Baptiste Theou
Thanks for the feedback. I tested a version before cleanup, and the mistake was introduce during it. It's my mistake, sorry about that. Will fix it on v4. On Sat, 6 Jun 2015 16:11:18 +0200 Paul Bolle wrote: > On Sat, 2015-06-06 at 14:56 +0200, Valentin Rothberg wrote: > > On Sat, Jun 06, 2015

Re: [PATCH v3 2/2] gpio_wdt: Add option for early registration

2015-06-06 Thread Paul Bolle
On Sat, 2015-06-06 at 14:56 +0200, Valentin Rothberg wrote: > On Sat, Jun 06, 2015 at 12:03:06PM +0200, Paul Bolle wrote: > > On Sat, 2015-06-06 at 00:46 -0700, Jean-Baptiste Theou wrote: > > > +#ifdef GPIO_WATCHDOG_ARCH_INITCALL > > > > You meant > > #ifdef CONFIG_GPIO_WATCHDOG_ARCH_INITCALL

Re: [PATCH v3 2/2] gpio_wdt: Add option for early registration

2015-06-06 Thread Valentin Rothberg
On Sat, Jun 06, 2015 at 12:03:06PM +0200, Paul Bolle wrote: > On Sat, 2015-06-06 at 00:46 -0700, Jean-Baptiste Theou wrote: > > --- a/drivers/watchdog/Kconfig > > +++ b/drivers/watchdog/Kconfig > > > +config GPIO_WATCHDOG_ARCH_INITCALL > > + bool "Register the watchdog as early as possible" > >

Re: [PATCH v3 2/2] gpio_wdt: Add option for early registration

2015-06-06 Thread Paul Bolle
On Sat, 2015-06-06 at 00:46 -0700, Jean-Baptiste Theou wrote: > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > +config GPIO_WATCHDOG_ARCH_INITCALL > + bool "Register the watchdog as early as possible" > + depends on GPIO_WATCHDOG=y > + help > + In some situatio

[PATCH v3 2/2] gpio_wdt: Add option for early registration

2015-06-06 Thread Jean-Baptiste Theou
In some situation, mainly when it's not possible to disable a watchdog, you may want the watchdog driver to be started as soon as possible. Adding GPIO_WATCHDOG_ARCH_INITCALL to raise initcall from module_init to arch_initcall. This patch require watchdog registration deferral mechanism Signed-o