Re: [PATCH v3 11/13] arm64: dts: st: add iwdg1 and iwdg2 support to stm32mp231

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] arm64: dts: st: undocumented access-controllers in st,stm32-iwdg
  binding
--

commit 93c31545a492246bcc374e2c2022c94ad8a0467c
Author: Antonio Borneo <antonio.borneo@foss.st.com>

arm64: dts: st: add iwdg1 and iwdg2 support to stm32mp231

This commit adds the iwdg1 and iwdg2 watchdog device tree nodes to
stm32mp231.dtsi.

> diff --git a/arch/arm64/boot/dts/st/stm32mp231.dtsi b/arch/arm64/boot/dts/st/stm32mp231.dtsi
> index fefbf6f4f1537..db19429896eb3 100644
> --- a/arch/arm64/boot/dts/st/stm32mp231.dtsi
> +++ b/arch/arm64/boot/dts/st/stm32mp231.dtsi
> @@ -724,6 +724,26 @@ rng: rng@42020000 {
>  				status = "disabled";
>  			};
>  
> +			iwdg1: watchdog@44010000 {
> +				compatible = "st,stm32mp1-iwdg";
> +				reg = <0x44010000 0x400>;
> +				interrupts = <GIC_SPI 2 IRQ_TYPE_LEVEL_HIGH>;
> +				clocks = <&rcc CK_BUS_IWDG1>, <&scmi_clk CK_SCMI_LSI>;
> +				clock-names = "pclk", "lsi";
> +				access-controllers = <&rifsc 98>;
> +				status = "disabled";
> +			};
> +
> +			iwdg2: watchdog@44020000 {
> +				compatible = "st,stm32mp1-iwdg";
> +				reg = <0x44020000 0x400>;
> +				interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>;
> +				clocks = <&rcc CK_BUS_IWDG2>, <&scmi_clk CK_SCMI_LSI>;
> +				clock-names = "pclk", "lsi";
> +				access-controllers = <&rifsc 99>;
> +				status = "disabled";
> +			};

[Severity: Low]
Does adding access-controllers to iwdg1 and iwdg2 cause dtbs_check to fail
against Documentation/devicetree/bindings/watchdog/st,stm32-iwdg.yaml?
That schema sets unevaluatedProperties: false and only references
watchdog.yaml, neither of which documents access-controllers.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261106163429.618392-1-antonio.borneo@foss.st.com?part=11