[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20151109161328.GA27217@rob-hp-laptop>
Date: Mon, 9 Nov 2015 10:13:28 -0600
From: Rob Herring <robh@...nel.org>
To: fu.wei@...aro.org
Cc: linaro-acpi@...ts.linaro.org, linux-watchdog@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-doc@...r.kernel.org, tekkamanninja@...il.com, arnd@...db.de,
linux@...ck-us.net, vgandhi@...eaurora.org, wim@...ana.be,
jcm@...hat.com, leo.duran@....com, corbet@....net,
mark.rutland@....com, catalin.marinas@....com, will.deacon@....com,
rjw@...ysocki.net, dyoung@...hat.com, panand@...hat.com,
Suravee.Suthikulpanit@....com, robherring2@...il.com
Subject: Re: [PATCH v9 1/5] Documentation: add sbsa-gwdt driver documentation
On Mon, Nov 09, 2015 at 03:31:15PM +0800, fu.wei@...aro.org wrote:
> From: Fu Wei <fu.wei@...aro.org>
>
> The sbsa-gwdt.txt documentation in devicetree/bindings/watchdog is for
> introducing SBSA(Server Base System Architecture) Generic Watchdog
> device node info into FDT.
>
> Also add sbsa-gwdt introduction in watchdog-parameters.txt
>
> Acked-by: Arnd Bergmann <arnd@...db.de>
> Signed-off-by: Fu Wei <fu.wei@...aro.org>
Acked-by: Rob Herring <robh@...nel.org>
> ---
> .../devicetree/bindings/watchdog/sbsa-gwdt.txt | 34 ++++++++++++++++++++++
> Documentation/watchdog/watchdog-parameters.txt | 6 ++++
> 2 files changed, 40 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/watchdog/sbsa-gwdt.txt b/Documentation/devicetree/bindings/watchdog/sbsa-gwdt.txt
> new file mode 100644
> index 0000000..2da63db
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/watchdog/sbsa-gwdt.txt
> @@ -0,0 +1,34 @@
> +* SBSA (Server Base System Architecture) Generic Watchdog
> +
> +The SBSA Generic Watchdog Timer is used to force a reset of the system
> +after two stages of timeout have elapsed. A detailed definition of the
> +watchdog timer can be found in the ARM document: ARM-DEN-0029 - Server
> +Base System Architecture (SBSA)
> +
> +Required properties:
> +- compatible: Should at least contain "arm,sbsa-gwdt".
> +
> +- reg: Each entry specifies the base physical address of a register frame
> + and the length of that frame; currently, two frames must be defined,
> + in this order:
> + 1: Watchdog control frame;
> + 2: Refresh frame.
> +
> +- interrupts: Should contain the Watchdog Signal 0 (WS0) SPI (Shared
> + Peripheral Interrupt) number of SBSA Generic Watchdog.
> +
> +Optional properties
> +- timeout-sec: Watchdog timeout and pre-timeout values.
> + If used, at least first timeout value (in seconds) must be provided.
> + A second optional value (in seconds) may also be provided and
> + will be used as the pre-timeout value, if it is given.
> +
> +Example for FVP Foundation Model v8:
> +
> +watchdog@...40000 {
> + compatible = "arm,sbsa-gwdt";
> + reg = <0x0 0x2a440000 0 0x1000>,
> + <0x0 0x2a450000 0 0x1000>;
> + interrupts = <0 27 4>;
> + timeout-sec = <60 30>;
> +};
> diff --git a/Documentation/watchdog/watchdog-parameters.txt b/Documentation/watchdog/watchdog-parameters.txt
> index 9f9ec9f..e62c8c4 100644
> --- a/Documentation/watchdog/watchdog-parameters.txt
> +++ b/Documentation/watchdog/watchdog-parameters.txt
> @@ -284,6 +284,12 @@ sbc_fitpc2_wdt:
> margin: Watchdog margin in seconds (default 60s)
> nowayout: Watchdog cannot be stopped once started
> -------------------------------------------------
> +sbsa_gwdt:
> +timeout: Watchdog timeout in seconds. (default 60s)
> +pretimeout: Watchdog pretimeout in seconds. (default 30s)
> +nowayout: Watchdog cannot be stopped once started
> + (default=kernel config parameter)
> +-------------------------------------------------
> sc1200wdt:
> isapnp: When set to 0 driver ISA PnP support will be disabled (default=1)
> io: io port
> --
> 2.5.0
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists