Re: [PATCH 1/2] watchdog: sama5d4: Cache MR instead of a partial config

2017-02-04 Thread Guenter Roeck
On 01/30/2017 09:18 AM, Alexandre Belloni wrote: .config is used to cache a part of WDT_MR at probe time and is not used afterwards. Instead of doing that, actually cache MR and avoid reading it every time it is modified. Signed-off-by: Alexandre Belloni

Re: [PATCH 1/2] watchdog: sama5d4: Cache MR instead of a partial config

2017-02-04 Thread Guenter Roeck
On 01/30/2017 09:18 AM, Alexandre Belloni wrote: .config is used to cache a part of WDT_MR at probe time and is not used afterwards. Instead of doing that, actually cache MR and avoid reading it every time it is modified. Signed-off-by: Alexandre Belloni Reviewed-by: Guenter Roeck [ hoping

Re: [PATCH 1/2] watchdog: sama5d4: Cache MR instead of a partial config

2017-02-01 Thread Alexandre Belloni
On 30/01/2017 at 09:58:13 -0800, Guenter Roeck wrote: > On Mon, Jan 30, 2017 at 06:18:47PM +0100, Alexandre Belloni wrote: > > .config is used to cache a part of WDT_MR at probe time and is not used > > afterwards. Instead of doing that, actually cache MR and avoid reading it > > every time it is

Re: [PATCH 1/2] watchdog: sama5d4: Cache MR instead of a partial config

2017-02-01 Thread Alexandre Belloni
On 30/01/2017 at 09:58:13 -0800, Guenter Roeck wrote: > On Mon, Jan 30, 2017 at 06:18:47PM +0100, Alexandre Belloni wrote: > > .config is used to cache a part of WDT_MR at probe time and is not used > > afterwards. Instead of doing that, actually cache MR and avoid reading it > > every time it is

Re: [PATCH 1/2] watchdog: sama5d4: Cache MR instead of a partial config

2017-01-30 Thread Guenter Roeck
On Mon, Jan 30, 2017 at 06:18:47PM +0100, Alexandre Belloni wrote: > .config is used to cache a part of WDT_MR at probe time and is not used > afterwards. Instead of doing that, actually cache MR and avoid reading it > every time it is modified. > The semantic change here is that the old code

Re: [PATCH 1/2] watchdog: sama5d4: Cache MR instead of a partial config

2017-01-30 Thread Guenter Roeck
On Mon, Jan 30, 2017 at 06:18:47PM +0100, Alexandre Belloni wrote: > .config is used to cache a part of WDT_MR at probe time and is not used > afterwards. Instead of doing that, actually cache MR and avoid reading it > every time it is modified. > The semantic change here is that the old code

[PATCH 1/2] watchdog: sama5d4: Cache MR instead of a partial config

2017-01-30 Thread Alexandre Belloni
.config is used to cache a part of WDT_MR at probe time and is not used afterwards. Instead of doing that, actually cache MR and avoid reading it every time it is modified. Signed-off-by: Alexandre Belloni --- drivers/watchdog/sama5d4_wdt.c | 45

[PATCH 1/2] watchdog: sama5d4: Cache MR instead of a partial config

2017-01-30 Thread Alexandre Belloni
.config is used to cache a part of WDT_MR at probe time and is not used afterwards. Instead of doing that, actually cache MR and avoid reading it every time it is modified. Signed-off-by: Alexandre Belloni --- drivers/watchdog/sama5d4_wdt.c | 45 ++ 1