[PATCH v2 2/3] configs: stm32mp13: activate watchdog

[Yann Gautier]

On 5/20/26 14:25, Marek Vasut wrote:
> 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 <patrick.delaunay at foss.st.com>
>>>>> Signed-off-by: Yann Gautier <yann.gautier at foss.st.com>
>>>>>
>>>>> ---
>>>>>
>>>>> 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 .
I've updated the commit message in v3:
https://patchwork.ozlabs.org/project/uboot/patch/20260520160715.175032-3-yann.gautier@foss.st.com/

For stm32mp13xx dhcor, the Linux DT need to be updated, as well as 
stm32mp13_dhcor_defconfig.


Best regards,
Yann