From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751795AbcBLPXN (ORCPT ); Fri, 12 Feb 2016 10:23:13 -0500 Received: from mail.kernel.org ([198.145.29.136]:42709 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750963AbcBLPXL (ORCPT ); Fri, 12 Feb 2016 10:23:11 -0500 Date: Fri, 12 Feb 2016 09:23:06 -0600 From: Rob Herring To: Anup Patel Cc: Catalin Marinas , Will Deacon , Device Tree , Linux ARM Kernel , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Florian Fainelli , Yendapally Reddy Dhananjaya Reddy , Ray Jui , Scott Branden , Vikram Prakash , Linux Kernel , BCM Kernel Feedback Subject: Re: [PATCH 4/6] dt-bindings: watchdog: Add ARM SP805 DT bindings Message-ID: <20160212152306.GA29673@rob-hp-laptop> References: <1455084651-29325-1-git-send-email-anup.patel@broadcom.com> <1455084651-29325-5-git-send-email-anup.patel@broadcom.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1455084651-29325-5-git-send-email-anup.patel@broadcom.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 10, 2016 at 11:40:49AM +0530, Anup Patel wrote: > The ARM SP805 DT node is already present in various DTS files. > This patch adds missing DT bindings documentation for ARM SP805. > > Signed-off-by: Anup Patel > Reviewed-by: Ray Jui > Reviewed-by: Vikram Prakash > Reviewed-by: Scott Branden > --- > .../devicetree/bindings/watchdog/arm,sp805.txt | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > create mode 100644 Documentation/devicetree/bindings/watchdog/arm,sp805.txt > > diff --git a/Documentation/devicetree/bindings/watchdog/arm,sp805.txt b/Documentation/devicetree/bindings/watchdog/arm,sp805.txt > new file mode 100644 > index 0000000..6ebb77d > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/arm,sp805.txt > @@ -0,0 +1,17 @@ > +ARM AMBA Primecell SP805 Watchdog > + > +Required properties: > +- compatible: Should be "arm,sp805" & "arm,primecell" > +- reg: Should contain location and length for watchdog timer register. > +- interrupts: Should contain the list of watchdog timer interrupts. > +- clocks: clocks driving the watchdog timer hardware. This list should be 2 > + clocks. With 2 clocks, the order is wdogclk clock, apb_pclk. > + > +Example: > + wdt@66090000 { This should be watchdog@... Otherwise: Acked-by: Rob Herring From mboxrd@z Thu Jan 1 00:00:00 1970 From: robh@kernel.org (Rob Herring) Date: Fri, 12 Feb 2016 09:23:06 -0600 Subject: [PATCH 4/6] dt-bindings: watchdog: Add ARM SP805 DT bindings In-Reply-To: <1455084651-29325-5-git-send-email-anup.patel@broadcom.com> References: <1455084651-29325-1-git-send-email-anup.patel@broadcom.com> <1455084651-29325-5-git-send-email-anup.patel@broadcom.com> Message-ID: <20160212152306.GA29673@rob-hp-laptop> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Feb 10, 2016 at 11:40:49AM +0530, Anup Patel wrote: > The ARM SP805 DT node is already present in various DTS files. > This patch adds missing DT bindings documentation for ARM SP805. > > Signed-off-by: Anup Patel > Reviewed-by: Ray Jui > Reviewed-by: Vikram Prakash > Reviewed-by: Scott Branden > --- > .../devicetree/bindings/watchdog/arm,sp805.txt | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > create mode 100644 Documentation/devicetree/bindings/watchdog/arm,sp805.txt > > diff --git a/Documentation/devicetree/bindings/watchdog/arm,sp805.txt b/Documentation/devicetree/bindings/watchdog/arm,sp805.txt > new file mode 100644 > index 0000000..6ebb77d > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/arm,sp805.txt > @@ -0,0 +1,17 @@ > +ARM AMBA Primecell SP805 Watchdog > + > +Required properties: > +- compatible: Should be "arm,sp805" & "arm,primecell" > +- reg: Should contain location and length for watchdog timer register. > +- interrupts: Should contain the list of watchdog timer interrupts. > +- clocks: clocks driving the watchdog timer hardware. This list should be 2 > + clocks. With 2 clocks, the order is wdogclk clock, apb_pclk. > + > +Example: > + wdt at 66090000 { This should be watchdog at ... Otherwise: Acked-by: Rob Herring