On 5/20/26 11:04 AM, Patrick DELAUNAY wrote:
Hi,
On 5/20/26 09:05, Yann Gautier wrote:
On 5/19/26 18:58, Marek Vasut wrote:
On 5/19/26 6:31 PM, Yann Gautier wrote:
No watchdog was enabled for STM32MP13 platform. Add the required
flags to
support it. As done for STM32MP15 (in SCMI config) and STM32MP2x, we
use
the Arm SMC watchdog. The required nodes were already present in Linux
imported DT files (stm32mp13.dtsi & stm32mp135f-dk.dts).
Signed-off-by: Patrick Delaunay <[email protected]>
Signed-off-by: Yann Gautier <[email protected]>
---
Changes in v2:
- update commit message
configs/stm32mp13_defconfig | 3 +++
1 file changed, 3 insertions(+)
diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig
index 620a6da2efe3..cc41585097c7 100644
--- a/configs/stm32mp13_defconfig
+++ b/configs/stm32mp13_defconfig
@@ -113,4 +113,7 @@ CONFIG_USB_GADGET_MANUFACTURER="STMicroelectronics"
CONFIG_USB_GADGET_VENDOR_NUM=0x0483
CONFIG_USB_GADGET_PRODUCT_NUM=0x5720
CONFIG_USB_GADGET_DWC2_OTG=y
+CONFIG_WDT=y
+CONFIG_WDT_STM32MP=y
Why is CONFIG_WDT_STM32MP enabled here , is this the ARM SMC watchdog
driver ?
This is aligned with stm32mp15 config, and no, this one is the STM32
IWDG driver.
I'll check that with Patrick (who originally wrote the patch), as it
seems that this flag could be removed.
I agree with Marek.
Today CONFIG_WDT_STM32MP is not really needed as only arm_wdt is
activated in STM32MP13 device tree for ST boards
Make sure you double-check the stm32mp13xx dhcor too.
If the WDT is supposed to be activated here, it has to be documented in
commit message too .