watchdog: qcom-wdt: Drop read check on write-only WDT_EN register
authorBalaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>
Tue, 1 Jul 2025 06:57:38 +0000 (12:27 +0530)
committerCasey Connolly <casey.connolly@linaro.org>
Mon, 14 Jul 2025 13:35:24 +0000 (15:35 +0200)
commit9cd3118e4aa693caed6ee3fe8ef1e0b91f004484
treec7dafbc03bf92a6e368d06b6e0e65175d3097de4
parentea166b1db189255669d2b24b118ed38aaae76a0f
watchdog: qcom-wdt: Drop read check on write-only WDT_EN register

On some Qualcomm platforms, such as Dragonwing boards, the WDT_EN
register is write-only. Reading it back after enabling the watchdog
can return invalid data or cause unexpected behavior.

In particular, the check:
  if (readl(wdt_addr(wdt, WDT_EN)) != 1)
may fail even though the watchdog is correctly enabled and running.
This leads to misleading error messages and unnecessary failures.

Removing the read check ensures compatibility and avoids false
negatives on platforms where WDT_EN is not readable.

This work builds upon this previous submission:
https://lore.kernel.org/u-boot/20250625094607.1348494-1-gopinath.sekar@oss.qualcomm.com/

Signed-off-by: Balaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>
Reviewed-by: Stefan Roese <sr@denx.de>
Link: https://lore.kernel.org/r/20250701065738.1644669-1-balaji.selvanathan@oss.qualcomm.com
Signed-off-by: Casey Connolly <casey.connolly@linaro.org>
drivers/watchdog/qcom-wdt.c