diff mbox series

[V9,1/4] dt-bindings: watchdog: add i.MX system controller watchdog

Message ID 1553134884-1720-2-git-send-email-Anson.Huang@nxp.com (mailing list archive)
State Accepted
Headers show
Series Add i.MX8QXP system controller watchdog | expand

Commit Message

Anson Huang March 21, 2019, 2:26 a.m. UTC
Add i.MX system controller watchdog binding doc.

Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
---
Changes since V8:
	- add optional property "timeout-sec" according to driver change.
---
 .../bindings/watchdog/fsl-imx-sc-wdt.txt           | 24 ++++++++++++++++++++++
 1 file changed, 24 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/watchdog/fsl-imx-sc-wdt.txt

Comments

Rob Herring (Arm) March 25, 2019, 8:37 p.m. UTC | #1
On Thu, Mar 21, 2019 at 02:26:38AM +0000, Anson Huang wrote:
> Add i.MX system controller watchdog binding doc.
> 
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> ---
> Changes since V8:
> 	- add optional property "timeout-sec" according to driver change.
> ---
>  .../bindings/watchdog/fsl-imx-sc-wdt.txt           | 24 ++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/watchdog/fsl-imx-sc-wdt.txt

I still think what's in ATF should be discoverable, but:

Acked-by: Rob Herring <robh@kernel.org>
Anson Huang March 29, 2019, 7:44 a.m. UTC | #2
Hi, Guenter/Shawn
	Since DT has been acked, would you pick up this patch series if no additional comments?

Best Regards!
Anson Huang

> -----Original Message-----
> From: Rob Herring [mailto:robh@kernel.org]
> Sent: 2019年3月26日 4:37
> To: Anson Huang <anson.huang@nxp.com>
> Cc: wim@linux-watchdog.org; linux@roeck-us.net; mark.rutland@arm.com;
> shawnguo@kernel.org; s.hauer@pengutronix.de; kernel@pengutronix.de;
> festevam@gmail.com; catalin.marinas@arm.com; will.deacon@arm.com;
> Aisheng Dong <aisheng.dong@nxp.com>; Daniel Baluta
> <daniel.baluta@nxp.com>; horms+renesas@verge.net.au; heiko@sntech.de;
> maxime.ripard@bootlin.com; Andy Gross <andy.gross@linaro.org>;
> jagan@amarulasolutions.com; bjorn.andersson@linaro.org;
> enric.balletbo@collabora.com; ezequiel@collabora.com;
> stefan.wahren@i2se.com; marc.w.gonzalez@free.fr; olof@lixom.net; linux-
> watchdog@vger.kernel.org; devicetree@vger.kernel.org; linux-arm-
> kernel@lists.infradead.org; linux-kernel@vger.kernel.org; dl-linux-imx
> <linux-imx@nxp.com>
> Subject: Re: [PATCH V9 1/4] dt-bindings: watchdog: add i.MX system
> controller watchdog
> 
> On Thu, Mar 21, 2019 at 02:26:38AM +0000, Anson Huang wrote:
> > Add i.MX system controller watchdog binding doc.
> >
> > Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> > ---
> > Changes since V8:
> > 	- add optional property "timeout-sec" according to driver change.
> > ---
> >  .../bindings/watchdog/fsl-imx-sc-wdt.txt           | 24
> ++++++++++++++++++++++
> >  1 file changed, 24 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/watchdog/fsl-
> imx-sc-wdt.txt
> 
> I still think what's in ATF should be discoverable, but:
> 
> Acked-by: Rob Herring <robh@kernel.org>
Guenter Roeck March 29, 2019, 4:49 p.m. UTC | #3
On Fri, Mar 29, 2019 at 07:44:21AM +0000, Anson Huang wrote:
> Hi, Guenter/Shawn
> 	Since DT has been acked, would you pick up this patch series if no additional comments?
> 

I added patch 1 and 2 of the series to my watchdog-next tree.
Wim usually picks it up from there.

Guenter

> Best Regards!
> Anson Huang
> 
> > -----Original Message-----
> > From: Rob Herring [mailto:robh@kernel.org]
> > Sent: 2019年3月26日 4:37
> > To: Anson Huang <anson.huang@nxp.com>
> > Cc: wim@linux-watchdog.org; linux@roeck-us.net; mark.rutland@arm.com;
> > shawnguo@kernel.org; s.hauer@pengutronix.de; kernel@pengutronix.de;
> > festevam@gmail.com; catalin.marinas@arm.com; will.deacon@arm.com;
> > Aisheng Dong <aisheng.dong@nxp.com>; Daniel Baluta
> > <daniel.baluta@nxp.com>; horms+renesas@verge.net.au; heiko@sntech.de;
> > maxime.ripard@bootlin.com; Andy Gross <andy.gross@linaro.org>;
> > jagan@amarulasolutions.com; bjorn.andersson@linaro.org;
> > enric.balletbo@collabora.com; ezequiel@collabora.com;
> > stefan.wahren@i2se.com; marc.w.gonzalez@free.fr; olof@lixom.net; linux-
> > watchdog@vger.kernel.org; devicetree@vger.kernel.org; linux-arm-
> > kernel@lists.infradead.org; linux-kernel@vger.kernel.org; dl-linux-imx
> > <linux-imx@nxp.com>
> > Subject: Re: [PATCH V9 1/4] dt-bindings: watchdog: add i.MX system
> > controller watchdog
> > 
> > On Thu, Mar 21, 2019 at 02:26:38AM +0000, Anson Huang wrote:
> > > Add i.MX system controller watchdog binding doc.
> > >
> > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> > > ---
> > > Changes since V8:
> > > 	- add optional property "timeout-sec" according to driver change.
> > > ---
> > >  .../bindings/watchdog/fsl-imx-sc-wdt.txt           | 24
> > ++++++++++++++++++++++
> > >  1 file changed, 24 insertions(+)
> > >  create mode 100644 Documentation/devicetree/bindings/watchdog/fsl-
> > imx-sc-wdt.txt
> > 
> > I still think what's in ATF should be discoverable, but:
> > 
> > Acked-by: Rob Herring <robh@kernel.org>
Shawn Guo April 3, 2019, 8:53 a.m. UTC | #4
On Fri, Mar 29, 2019 at 07:44:21AM +0000, Anson Huang wrote:
> Hi, Guenter/Shawn
> 	Since DT has been acked, would you pick up this patch series if no additional comments?
> 

Applied patch #3 and #4, thanks.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/watchdog/fsl-imx-sc-wdt.txt b/Documentation/devicetree/bindings/watchdog/fsl-imx-sc-wdt.txt
new file mode 100644
index 0000000..02b87e9
--- /dev/null
+++ b/Documentation/devicetree/bindings/watchdog/fsl-imx-sc-wdt.txt
@@ -0,0 +1,24 @@ 
+* Freescale i.MX System Controller Watchdog
+
+i.MX system controller watchdog is for i.MX SoCs with system controller inside,
+the watchdog is managed by system controller, users can ONLY communicate with
+system controller from secure mode for watchdog operations, so Linux i.MX system
+controller watchdog driver will call ARM SMC API and trap into ARM-Trusted-Firmware
+for watchdog operations, ARM-Trusted-Firmware is running at secure EL3 mode and
+it will request system controller to execute the watchdog operation passed from
+Linux kernel.
+
+Required properties:
+- compatible:	Should be :
+		"fsl,imx8qxp-sc-wdt"
+		followed by "fsl,imx-sc-wdt";
+
+Optional properties:
+- timeout-sec : Contains the watchdog timeout in seconds.
+
+Examples:
+
+watchdog {
+	compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt";
+	timeout-sec = <60>;
+};