From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Long Subject: Re: [PATCH v7 1/2] dt-bindings: watchdog: Add Spreadtrum watchdog documentation Date: Tue, 28 Nov 2017 14:13:04 +0800 Message-ID: <20171128061304.GA26779@spreadtrum.com> References: <61f88b7f9c1ac8a2d4bd7dbab672ad913bd2707d.1511835645.git.eric.long@spreadtrum.com> <445d1641-de20-860c-5448-4b1533426441@roeck-us.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: Content-Disposition: inline In-Reply-To: <445d1641-de20-860c-5448-4b1533426441@roeck-us.net> Sender: linux-kernel-owner@vger.kernel.org To: Guenter Roeck , eric.long@spreadtrum.com Cc: Rob Herring , Mark Rutland , baolin.wang@linaro.org, linux-watchdog@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, huitao.yue@spreadtrum.com List-Id: devicetree@vger.kernel.org Hi Guenter, Thanks again for you review. I am very glad that this patch has been accepted. Best Regards, Eric Long On Mon, Nov 27, 2017 at 08:09:18PM -0800, Guenter Roeck wrote: > On 11/27/2017 06:18 PM, Eric Long wrote: > >This patch adds the documentation for Spreadtrum watchdog driver. > > > >Signed-off-by: Eric Long > >Acked-by: Rob Herring > > The patch series is in my tree, and will ultimately find its way into Wim's tree. > No need to resend it anymore. > > In general, if a patch is available in > git://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git watchdog-next > there is no need to resend it. > > Thanks, > Guenter > > >--- > >Changes since v6: > >- No updates. > > > >Changes since v5: > >- No updates. > > > >Changes since v4: > >- No updates. > > > >Changes since v3: > >- No updates. > > > >Changes since v2: > >- Add acked tag from Rob. > > > >Changes since v1: > >- No updates. > >--- > > .../devicetree/bindings/watchdog/sprd-wdt.txt | 19 +++++++++++++++++++ > > 1 file changed, 19 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/watchdog/sprd-wdt.txt > > > >diff --git a/Documentation/devicetree/bindings/watchdog/sprd-wdt.txt b/Documentation/devicetree/bindings/watchdog/sprd-wdt.txt > >new file mode 100644 > >index 0000000..aeaf3e0 > >--- /dev/null > >+++ b/Documentation/devicetree/bindings/watchdog/sprd-wdt.txt > >@@ -0,0 +1,19 @@ > >+Spreadtrum SoCs Watchdog timer > >+ > >+Required properties: > >+- compatible : Should be "sprd,sp9860-wdt". > >+- reg : Specifies base physical address and size of the registers. > >+- interrupts : Exactly one interrupt specifier. > >+- timeout-sec : Contain the default watchdog timeout in seconds. > >+- clock-names : Contain the input clock names. > >+- clocks : Phandles to input clocks. > >+ > >+Example: > >+ watchdog: watchdog@40310000 { > >+ compatible = "sprd,sp9860-wdt"; > >+ reg = <0 0x40310000 0 0x1000>; > >+ interrupts = ; > >+ timeout-sec = <12>; > >+ clock-names = "enable", "rtc_enable"; > >+ clocks = <&clk_aon_apb_gates1 8>, <&clk_aon_apb_rtc_gates 9>; > >+ }; > > >