[PATCH] ARM: socfpga: dts: add watchdog0+1

[PATCH] ARM: socfpga: dts: add watchdog0+1

[Steffen Trumtrar]

The SoCFPGA has two watchdog timers. Add them to the dtsi.

Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
---
 arch/arm/boot/dts/socfpga.dtsi | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi
index 56fc214..ee10fa6 100644
--- a/arch/arm/boot/dts/socfpga.dtsi
+++ b/arch/arm/boot/dts/socfpga.dtsi
@@ -549,6 +549,22 @@
 			reg = <0xffd05000 0x1000>;
 		};
 
+		watchdog0: wd at ffd02000 {
+			compatible = "snps,dw-wdt";
+			reg = <0xffd02000 0x1000>;
+			interrupts = <0 171 0>;
+			clocks = <&osc1>;
+			status = "disabled";
+		};
+
+		watchdog1: wd at ffd04000 {
+			compatible = "snps,dw-wdt";
+			reg = <0xffd03000 0x1000>;
+			interrupts = <0 172 0>;
+			clocks = <&osc1>;
+			status = "disabled";
+		};
+
 		sysmgr: sysmgr at ffd08000 {
 			compatible = "altr,sys-mgr", "syscon";
 			reg = <0xffd08000 0x4000>;
-- 
2.0.0.rc0

[PATCH] ARM: socfpga: dts: add watchdog0+1

[Sergei Shtylyov]

Hello.

On 22-05-2014 14:19, Steffen Trumtrar wrote:

> The SoCFPGA has two watchdog timers. Add them to the dtsi.

> Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> ---
>   arch/arm/boot/dts/socfpga.dtsi | 16 ++++++++++++++++
>   1 file changed, 16 insertions(+)

> diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi
> index 56fc214..ee10fa6 100644
> --- a/arch/arm/boot/dts/socfpga.dtsi
> +++ b/arch/arm/boot/dts/socfpga.dtsi
> @@ -549,6 +549,22 @@
>   			reg = <0xffd05000 0x1000>;
>   		};
>
> +		watchdog0: wd at ffd02000 {

    The node name should be "watchdog", according to section 2.2.2 of the 
ePPAR standard [1].


> +		watchdog1: wd at ffd04000 {

    Same here.

[1] http://www.power.org/resources/downloads/Power_ePAPR_APPROVED_v1.0.pdf

WBR, Sergei

[PATCH] ARM: socfpga: dts: add watchdog0+1

[Sergei Shtylyov]

Hello.

On 22-05-2014 14:19, Steffen Trumtrar wrote:

> The SoCFPGA has two watchdog timers. Add them to the dtsi.

> Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> ---
>   arch/arm/boot/dts/socfpga.dtsi | 16 ++++++++++++++++
>   1 file changed, 16 insertions(+)

> diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi
> index 56fc214..ee10fa6 100644
> --- a/arch/arm/boot/dts/socfpga.dtsi
> +++ b/arch/arm/boot/dts/socfpga.dtsi
> @@ -549,6 +549,22 @@
[...]
> +		watchdog1: wd at ffd04000 {

    Oh, and the unit-address part of the name should match the "reg" property.

> +			compatible = "snps,dw-wdt";
> +			reg = <0xffd03000 0x1000>;
> +			interrupts = <0 172 0>;
> +			clocks = <&osc1>;
> +			status = "disabled";
> +		};
> +

WBR, Sergei

[PATCH] ARM: socfpga: dts: add watchdog0+1

[Steffen Trumtrar]

On Thu, May 22, 2014 at 04:48:16PM +0400, Sergei Shtylyov wrote:
> Hello.
> 
> On 22-05-2014 14:19, Steffen Trumtrar wrote:
> 
> >The SoCFPGA has two watchdog timers. Add them to the dtsi.
> 
> >Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> >---
> >  arch/arm/boot/dts/socfpga.dtsi | 16 ++++++++++++++++
> >  1 file changed, 16 insertions(+)
> 
> >diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi
> >index 56fc214..ee10fa6 100644
> >--- a/arch/arm/boot/dts/socfpga.dtsi
> >+++ b/arch/arm/boot/dts/socfpga.dtsi
> >@@ -549,6 +549,22 @@
> [...]
> >+		watchdog1: wd at ffd04000 {
> 
>    Oh, and the unit-address part of the name should match the "reg" property.
> 

Oops. Of course.

> >+			compatible = "snps,dw-wdt";
> >+			reg = <0xffd03000 0x1000>;
> >+			interrupts = <0 172 0>;
> >+			clocks = <&osc1>;
> >+			status = "disabled";
> >+		};
> >+

Thanks,
Steffen

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |