| Message ID | f49033de078aff9112fb8fa6fa4331e16998edf2.1411667144.git.joshc@codeaurora.org (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Headers | show |
On Thu, Sep 25, 2014 at 12:48:52PM -0500, Josh Cartwright wrote: > The Qualcomm Krait Processor Sub-system (KPSS) contains one or more > instances of the WDT. Provide documentation on how to describe these in > the device tree. > > Signed-off-by: Josh Cartwright <joshc@codeaurora.org> > --- > .../devicetree/bindings/watchdog/qcom-wdt.txt | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > create mode 100644 Documentation/devicetree/bindings/watchdog/qcom-wdt.txt > > diff --git a/Documentation/devicetree/bindings/watchdog/qcom-wdt.txt b/Documentation/devicetree/bindings/watchdog/qcom-wdt.txt > new file mode 100644 > index 0000000..c75566e > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/qcom-wdt.txt > @@ -0,0 +1,22 @@ > +Qualcomm Krait Processor Sub-system (KPSS) Watchdog > +--------------------------------------------------- > + > +Required properties : > +- compatible : shall contain only one of the following: > + > + "qcom,kpss-wdt-msm8960" > + "qcom,kpss-wdt-apq8064" > + "qcom,kpss-wdt-ipq8064" > + > +- reg : shall contain base register location and length > +- clocks : shall contain the input clock > +- timeout-sec : shall contain the default watchdog timeout in seconds, > + if unset, the default timeout is 30 seconds Hi Josh, timeout-sec is optional, not mandatory. Thanks, Guenter > + > +Example: > + watchdog@208a038 { > + compatible = "qcom,kpss-wdt-ipq8064"; > + reg = <0x0208a038 0x40>; > + clocks = <&sleep_clk>; > + timeout-sec = <10>; > + }; > -- > Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, > hosted by The Linux Foundation > -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, Sep 25, 2014 at 11:43:14AM -0700, Guenter Roeck wrote: > On Thu, Sep 25, 2014 at 12:48:52PM -0500, Josh Cartwright wrote: [..] > > +- timeout-sec : shall contain the default watchdog timeout in seconds, > > + if unset, the default timeout is 30 seconds > > Hi Josh, > > timeout-sec is optional, not mandatory. Indeed, I made this change in v2, but didn't reflect it in the document. Good catch. Will fix. Josh
diff --git a/Documentation/devicetree/bindings/watchdog/qcom-wdt.txt b/Documentation/devicetree/bindings/watchdog/qcom-wdt.txt new file mode 100644 index 0000000..c75566e --- /dev/null +++ b/Documentation/devicetree/bindings/watchdog/qcom-wdt.txt @@ -0,0 +1,22 @@ +Qualcomm Krait Processor Sub-system (KPSS) Watchdog +--------------------------------------------------- + +Required properties : +- compatible : shall contain only one of the following: + + "qcom,kpss-wdt-msm8960" + "qcom,kpss-wdt-apq8064" + "qcom,kpss-wdt-ipq8064" + +- reg : shall contain base register location and length +- clocks : shall contain the input clock +- timeout-sec : shall contain the default watchdog timeout in seconds, + if unset, the default timeout is 30 seconds + +Example: + watchdog@208a038 { + compatible = "qcom,kpss-wdt-ipq8064"; + reg = <0x0208a038 0x40>; + clocks = <&sleep_clk>; + timeout-sec = <10>; + };
The Qualcomm Krait Processor Sub-system (KPSS) contains one or more instances of the WDT. Provide documentation on how to describe these in the device tree. Signed-off-by: Josh Cartwright <joshc@codeaurora.org> --- .../devicetree/bindings/watchdog/qcom-wdt.txt | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) create mode 100644 Documentation/devicetree/bindings/watchdog/qcom-wdt.txt