diff mbox

[4/5] ARM: dts: NSP: Add SP805 Support to DT

Message ID 1454712204-30419-5-git-send-email-jonmason@broadcom.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jon Mason Feb. 5, 2016, 10:43 p.m. UTC
Add support for the ARM SP805 Watchdog timer to the Northstar Plus
device tree.

Signed-off-by: Jon Mason <jonmason@broadcom.com>
---
 arch/arm/boot/dts/bcm-nsp.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Sergei Shtylyov Feb. 6, 2016, 12:49 p.m. UTC | #1
Hello.

On 2/6/2016 1:43 AM, Jon Mason wrote:

> Add support for the ARM SP805 Watchdog timer to the Northstar Plus
> device tree.
>
> Signed-off-by: Jon Mason <jonmason@broadcom.com>
> ---
>   arch/arm/boot/dts/bcm-nsp.dtsi | 8 ++++++++
>   1 file changed, 8 insertions(+)
>
> diff --git a/arch/arm/boot/dts/bcm-nsp.dtsi b/arch/arm/boot/dts/bcm-nsp.dtsi
> index 415a0a6..0e8cb6e 100644
> --- a/arch/arm/boot/dts/bcm-nsp.dtsi
> +++ b/arch/arm/boot/dts/bcm-nsp.dtsi
> @@ -203,6 +203,14 @@
>   			clock-frequency = <100000>;
>   		};
>
> +		wdt@39000 {

    Should be "watchdog@39000", according to ePAPR.

[...]

MBR, Sergei
Sergei Shtylyov Feb. 6, 2016, 9:37 p.m. UTC | #2
On 02/06/2016 11:47 PM, Scott Branden wrote:

>             Add support for the ARM SP805 Watchdog timer to the Northstar Plus
>             device tree.
>
>             Signed-off-by: Jon Mason <jonmason@broadcom.com
>             <mailto:jonmason@broadcom.com>>
>             ---
>                arch/arm/boot/dts/bcm-nsp.dtsi | 8 ++++++++
>                1 file changed, 8 insertions(+)
>
>             diff --git a/arch/arm/boot/dts/bcm-nsp.dtsi
>             b/arch/arm/boot/dts/bcm-nsp.dtsi
>             index 415a0a6..0e8cb6e 100644
>             --- a/arch/arm/boot/dts/bcm-nsp.dtsi
>             +++ b/arch/arm/boot/dts/bcm-nsp.dtsi
>             @@ -203,6 +203,14 @@
>                                      clock-frequency = <100000>;
>                              };
>
>             +               wdt@39000 {
>
>
>             Should be "watchdog@39000", according to ePAPR.
>
>
>     Thanks, I'll fix it and send it right out.

> I didn't know ePAPR was a requirement for a linux dtb?  We haven't done this
> with the wtd in other dts files.  Is this described somewhere?

    Nobody reads the docs. Like this:

http://www.devicetree.org/Device_Tree_Usage

    That page refers to the ePAPR 1.0 (not the most recent).

MBR, Sergei
diff mbox

Patch

diff --git a/arch/arm/boot/dts/bcm-nsp.dtsi b/arch/arm/boot/dts/bcm-nsp.dtsi
index 415a0a6..0e8cb6e 100644
--- a/arch/arm/boot/dts/bcm-nsp.dtsi
+++ b/arch/arm/boot/dts/bcm-nsp.dtsi
@@ -203,6 +203,14 @@ 
 			clock-frequency = <100000>;
 		};
 
+		wdt@39000 {
+			compatible = "arm,sp805", "arm,primecell";
+			reg = <0x39000 0x1000>;
+			interrupts = <GIC_SPI 126 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&iprocslow>, <&iprocslow>;
+			clock-names = "wdogclk", "apb_pclk";
+		};
+
 		lcpll0: lcpll0@3f100 {
 			#clock-cells = <1>;
 			compatible = "brcm,nsp-lcpll0";