Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757929AbaGOGjp (ORCPT ); Tue, 15 Jul 2014 02:39:45 -0400 Received: from mail-qg0-f50.google.com ([209.85.192.50]:56446 "EHLO mail-qg0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757689AbaGOGjl (ORCPT ); Tue, 15 Jul 2014 02:39:41 -0400 MIME-Version: 1.0 In-Reply-To: <20140714150749.GH26465@leverpostej> References: <1405340169-2820-1-git-send-email-harinik@xilinx.com> <1405340169-2820-2-git-send-email-harinik@xilinx.com> <20140714150749.GH26465@leverpostej> Date: Tue, 15 Jul 2014 12:09:40 +0530 Message-ID: Subject: Re: [PATCH v3 2/2] devicetree: Add Cadence WDT devicetree bindings documentation From: Harini Katakam To: Mark Rutland Cc: "wim@iguana.be" , "grant.likely@linaro.org" , "robh+dt@kernel.org" , Pawel Moll , "ijc+devicetree@hellion.org.uk" , "galak@codeaurora.org" , "rob@landley.net" , "michals@xilinx.com" , "linux-watchdog@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-doc@vger.kernel.org" , "harini.katakam@xilinx.com" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mark, On Mon, Jul 14, 2014 at 8:37 PM, Mark Rutland wrote: > On Mon, Jul 14, 2014 at 01:16:09PM +0100, Harini Katakam wrote: >> Add cadence-wdt bindings documentation. >> >> Signed-off-by: Harini Katakam >> --- >> >> v3 changes: >> - Change reset property type and improve description. >> - Improve description of clocks and interrupts. >> - Use watchdog@ in the example. >> - Use only cdns compatible string for now. >> >> v2: >> No changes >> >> --- >> .../devicetree/bindings/watchdog/cadence-wdt.txt | 27 ++++++++++++++++++++ >> 1 file changed, 27 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/watchdog/cadence-wdt.txt >> >> diff --git a/Documentation/devicetree/bindings/watchdog/cadence-wdt.txt b/Documentation/devicetree/bindings/watchdog/cadence-wdt.txt >> new file mode 100644 >> index 0000000..ab23e38 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/watchdog/cadence-wdt.txt >> @@ -0,0 +1,27 @@ >> +Zynq Watchdog Device Tree Bindings >> +------------------------------------------- >> + >> +Required properties: >> +- compatible : Should be "cdns,wdt-r1p2". >> +- clocks : Input clock specifier. This should be the ref clk. > > This wording makes it sound like the watchdog block has more than one > clock input. Does it? It doesn't. It has one APB clock - named pclk. > >> +- reg : Physical base address and size of WDT registers map. >> +- interrupts : Property with a value describing the interrupt >> + number. This interrupt is used for indication >> + when the watchdog times out. > > Just say "the watchdog timeout interrupt", or (better) use the name of > the interrupt from the documentation. OK. Yeah I'll do that. The interrupt name is wd_irq. > >> +- interrupt-parent : Must be core interrupt controller. >> + >> +Optional properties >> +- reset : If this property exists, then a reset is done >> + when watchdog times out. > > That's a bit of an ambiguous name (too easy to misconstrue as a reset > device reference). Do any other watchdogs have similar properties? I could change it to "reset-on-timeout" if that better. >From the documentation of other drivers, there seems to be a reset-type property in atmel. Dint find any other reset related properties. Regards, Harini -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/