watchdog: pm8916_wdt: Avoid read of write-only PET register
authorStephan Gerhold <stephan.gerhold@kernkonzept.com>
Wed, 29 Jun 2022 08:48:14 +0000 (10:48 +0200)
committerWim Van Sebroeck <wim@linux-watchdog.org>
Sat, 23 Jul 2022 12:21:27 +0000 (14:21 +0200)
commit4e5b406514fce00b16b7fc8cc5a6c9ec51518b0a
tree06e7542e27942743f846e6153e436fcee52b71e6
parent5f02d033f3a512397e2304c3cd0c0d69471e5338
watchdog: pm8916_wdt: Avoid read of write-only PET register

PMIC_WD_RESET_PET is a write-only register that is used to ping
the watchdog. It does not make sense to use read-modify-write
for it: a register read will never return anything but zero.
(And actually even if it did we would still want to write again
to ensure the watchdog is pinged.)

Reduce the overhead for the watchdog ping slightly by using
regmap_write() directly instead.

Signed-off-by: Stephan Gerhold <stephan.gerhold@kernkonzept.com>
Reviewed-by: Loic Poulain <loic.poulain@linaro.org>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Link: https://lore.kernel.org/r/20220629084816.125515-2-stephan.gerhold@kernkonzept.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@linux-watchdog.org>
drivers/watchdog/pm8916_wdt.c