[PATCH] ARM: dts: imx: Add WDT bindings on DH i.MX6 DHSOM

[PATCH] ARM: dts: imx: Add WDT bindings on DH i.MX6 DHSOM

[Marek Vasut]

Add WDT reboot bindings on DH i.MX6 DHSOM to permit the platform
to reboot via WDT in U-Boot. These are custom U-Boot bindings,
hence they are placed in -u-boot.dtsi .

Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Andreas Geisreiter <ageisreiter@dh-electronics.de>
Cc: Christoph Niedermaier <cniedermaier@dh-electronics.com>
Cc: Fabio Estevam <festevam@denx.de>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: u-boot@dh-electronics.com
---
 arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi b/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi
index 190567ab7ba..06ad98bccf6 100644
--- a/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi
+++ b/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi
@@ -8,6 +8,12 @@
 	aliases {
 		eeprom0 = &eeprom0;
 	};
+
+	wdt-reboot {
+		compatible = "wdt-reboot";
+		wdt = <&wdog1>;
+		u-boot,dm-spl;
+	};
 };
 
 &fec {
@@ -25,3 +31,7 @@
 	gpio = <&gpio3 31 GPIO_ACTIVE_HIGH>;
 	enable-active-high;
 };
+
+&wdog1 {
+	u-boot,dm-spl;
+};
-- 
2.39.2

Re: [PATCH] ARM: dts: imx: Add WDT bindings on DH i.MX6 DHSOM

[Fabio Estevam]

On 05/03/2023 17:49, Marek Vasut wrote:
> Add WDT reboot bindings on DH i.MX6 DHSOM to permit the platform
> to reboot via WDT in U-Boot. These are custom U-Boot bindings,
> hence they are placed in -u-boot.dtsi .
> 
> Signed-off-by: Marek Vasut <marex@denx.de>

Reviewed-by: Fabio Estevam <festevam@denx.de>

Re: [PATCH] ARM: dts: imx: Add WDT bindings on DH i.MX6 DHSOM

[Stefano Babic]

Hi Marek,

On 05.03.23 21:49, Marek Vasut wrote:
> Add WDT reboot bindings on DH i.MX6 DHSOM to permit the platform
> to reboot via WDT in U-Boot. These are custom U-Boot bindings,
> hence they are placed in -u-boot.dtsi .
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Andreas Geisreiter <ageisreiter@dh-electronics.de>
> Cc: Christoph Niedermaier <cniedermaier@dh-electronics.com>
> Cc: Fabio Estevam <festevam@denx.de>
> Cc: Peng Fan <peng.fan@nxp.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: u-boot@dh-electronics.com
> ---
>   arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi | 10 ++++++++++
>   1 file changed, 10 insertions(+)
> 
> diff --git a/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi b/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi
> index 190567ab7ba..06ad98bccf6 100644
> --- a/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi
> +++ b/arch/arm/dts/imx6qdl-dhcom-u-boot.dtsi
> @@ -8,6 +8,12 @@
>   	aliases {
>   		eeprom0 = &eeprom0;
>   	};
> +
> +	wdt-reboot {
> +		compatible = "wdt-reboot";
> +		wdt = <&wdog1>;
> +		u-boot,dm-spl;
> +	}

I cannot merge it, u-boot,dm-spl is not allowed anymore (see 
https://source.denx.de/u-boot/custodians/u-boot-imx/-/jobs/604593), 
switched to bootph-pre-ram ?

Same is for the other patch of yours:

http://patchwork.ozlabs.org/project/uboot/patch/20230208113347.131572-1-marex@denx.de/


Best regards,
Stefano
;
>   };
>   
>   &fec {
> @@ -25,3 +31,7 @@
>   	gpio = <&gpio3 31 GPIO_ACTIVE_HIGH>;
>   	enable-active-high;
>   };
> +
> +&wdog1 {
> +	u-boot,dm-spl;
> +};

-- 
=====================================================================
DENX Software Engineering GmbH,        Managing Director: Erika Unter
HRB 165235 Munich,   Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de
=====================================================================