diff options
| author | Andrew Jeffery <andrew@aj.id.au> | 2017-09-20 08:30:17 +0300 |
|---|---|---|
| committer | Joel Stanley <joel@jms.id.au> | 2017-11-28 07:37:41 +0300 |
| commit | f23fe499af5507338272d37521a06faad552da90 (patch) | |
| tree | 05b0cec9ace1f44a4cda83becf6872942d475784 /usr | |
| parent | b0bd684bcc2e66318b50882ab534e0c2e2c8b3d8 (diff) | |
| download | linux-f23fe499af5507338272d37521a06faad552da90.tar.xz | |
watchdog: aspeed: Retain watchdog enabled state
An unintended post-condition of probe() is that the watchdog is
disabled. This behaviour was introduced by an unnecessary write to the
control register to configure the hardware based on the devicetree. The
write is unnecessary because the cached control value that is
manipulated by the code parsing the devicetree is eventually written by
aspeed_wdt_enable(), which is when we care how the control register
should be configured.
Remove the write to restore expected behaviour.
Fixes: b7f0b8ad25f3 ("drivers/watchdog: ASPEED reference dev tree properties for config")
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Reviewed-by: Joel Stanley <joel@jms.id.au>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
(cherry picked from commit 6bd639385f61ff918e82e67fd8b246e6e7f486c3)
Signed-off-by: Joel Stanley <joel@jms.id.au>
Diffstat (limited to 'usr')
0 files changed, 0 insertions, 0 deletions