Re: [PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-11-10 Thread Dmitry Osipenko
10.11.2021 13:43, Jonathan Neuschäfer пишет: > On Mon, Nov 08, 2021 at 02:36:42PM +0300, Dmitry Osipenko wrote: >> 08.11.2021 14:22, Jonathan Neuschäfer пишет: >>> On Sun, Nov 07, 2021 at 08:42:33PM +0300, Dmitry Osipenko wrote: >>> [...] EC drivers tend to use higher priority in general. Jona

Re: [PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-11-10 Thread Jonathan Neuschäfer
On Mon, Nov 08, 2021 at 02:36:42PM +0300, Dmitry Osipenko wrote: > 08.11.2021 14:22, Jonathan Neuschäfer пишет: > > On Sun, Nov 07, 2021 at 08:42:33PM +0300, Dmitry Osipenko wrote: > > [...] > >> EC drivers tend to use higher priority in general. Jonathan, could you > >> please confirm that NTXEC d

Re: [PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-11-08 Thread Dmitry Osipenko
08.11.2021 14:22, Jonathan Neuschäfer пишет: > On Sun, Nov 07, 2021 at 08:42:33PM +0300, Dmitry Osipenko wrote: > [...] >> EC drivers tend to use higher priority in general. Jonathan, could you >> please confirm that NTXEC driver is a more preferable restart method >> than the watchdog? > > Yes. T

Re: [PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-11-08 Thread Jonathan Neuschäfer
On Sun, Nov 07, 2021 at 08:42:33PM +0300, Dmitry Osipenko wrote: [...] > EC drivers tend to use higher priority in general. Jonathan, could you > please confirm that NTXEC driver is a more preferable restart method > than the watchdog? Yes. The original firmware uses the NTXEC to restart, and it w

Re: [PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-11-07 Thread Dmitry Osipenko
07.11.2021 20:32, Guenter Roeck пишет: > On 11/7/21 9:16 AM, Dmitry Osipenko wrote: >> 07.11.2021 20:08, Guenter Roeck пишет: >>> On 11/7/21 8:53 AM, Dmitry Osipenko wrote: 06.11.2021 23:54, Jonathan Neuschäfer пишет: > Hi, > > On Thu, Oct 28, 2021 at 12:16:57AM +0300, Dmitry Osipe

Re: [PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-11-07 Thread Guenter Roeck
On 11/7/21 9:16 AM, Dmitry Osipenko wrote: 07.11.2021 20:08, Guenter Roeck пишет: On 11/7/21 8:53 AM, Dmitry Osipenko wrote: 06.11.2021 23:54, Jonathan Neuschäfer пишет: Hi, On Thu, Oct 28, 2021 at 12:16:57AM +0300, Dmitry Osipenko wrote: Use devm_register_power_handler() that replaces globa

Re: [PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-11-07 Thread Dmitry Osipenko
07.11.2021 20:08, Guenter Roeck пишет: > On 11/7/21 8:53 AM, Dmitry Osipenko wrote: >> 06.11.2021 23:54, Jonathan Neuschäfer пишет: >>> Hi, >>> >>> On Thu, Oct 28, 2021 at 12:16:57AM +0300, Dmitry Osipenko wrote: Use devm_register_power_handler() that replaces global pm_power_off variable

Re: [PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-11-07 Thread Guenter Roeck
On 11/7/21 8:53 AM, Dmitry Osipenko wrote: 06.11.2021 23:54, Jonathan Neuschäfer пишет: Hi, On Thu, Oct 28, 2021 at 12:16:57AM +0300, Dmitry Osipenko wrote: Use devm_register_power_handler() that replaces global pm_power_off variable and allows to register multiple power-off handlers. It also

Re: [PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-11-07 Thread Dmitry Osipenko
06.11.2021 23:54, Jonathan Neuschäfer пишет: > Hi, > > On Thu, Oct 28, 2021 at 12:16:57AM +0300, Dmitry Osipenko wrote: >> Use devm_register_power_handler() that replaces global pm_power_off >> variable and allows to register multiple power-off handlers. It also >> provides restart-handler support

Re: [PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-11-06 Thread Jonathan Neuschäfer
Hi, On Thu, Oct 28, 2021 at 12:16:57AM +0300, Dmitry Osipenko wrote: > Use devm_register_power_handler() that replaces global pm_power_off > variable and allows to register multiple power-off handlers. It also > provides restart-handler support, i.e. all in one API. > > Signed-off-by: Dmitry Osip

[PATCH v2 27/45] mfd: ntxec: Use devm_register_power_handler()

2021-10-27 Thread Dmitry Osipenko
Use devm_register_power_handler() that replaces global pm_power_off variable and allows to register multiple power-off handlers. It also provides restart-handler support, i.e. all in one API. Signed-off-by: Dmitry Osipenko --- drivers/mfd/ntxec.c | 50 ++--