From mboxrd@z Thu Jan 1 00:00:00 1970 From: sergei.shtylyov@cogentembedded.com (Sergei Shtylyov) Date: Sun, 7 Feb 2016 00:37:39 +0300 Subject: [PATCH 4/5] ARM: dts: NSP: Add SP805 Support to DT In-Reply-To: References: <1454712204-30419-1-git-send-email-jonmason@broadcom.com> <1454712204-30419-5-git-send-email-jonmason@broadcom.com> <56B5EBCB.4080305@cogentembedded.com> Message-ID: <56B667A3.8010200@cogentembedded.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 > > --- > 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 at 39000 { > > > Should be "watchdog at 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH 4/5] ARM: dts: NSP: Add SP805 Support to DT Date: Sun, 7 Feb 2016 00:37:39 +0300 Message-ID: <56B667A3.8010200@cogentembedded.com> References: <1454712204-30419-1-git-send-email-jonmason@broadcom.com> <1454712204-30419-5-git-send-email-jonmason@broadcom.com> <56B5EBCB.4080305@cogentembedded.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Scott Branden , Jon Mason Cc: Jon Mason , Florian Fainelli , devicetree@vger.kernel.org, bcm-kernel-feedback-list@broadcom.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org 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 > > --- > 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