Re: [PATCH RESEND V2 1/4] dt-bindings: fsl: scu: add watchdog binding

From: Guenter Roeck
Date: Sat Feb 23 2019 - 12:08:17 EST


On 2/22/19 11:52 AM, Rob Herring wrote:
On Mon, Feb 18, 2019 at 06:53:48AM +0000, Anson Huang wrote:
Add i.MX8QXP system controller watchdog binding.

Signed-off-by: Anson Huang <Anson.Huang@xxxxxxx>
---
Changes since V1:
- update dts node name to "watchdog";
---
Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt | 10 ++++++++++
1 file changed, 10 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt b/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
index 4b79751..f388ec6 100644
--- a/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
+++ b/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
@@ -136,6 +136,12 @@ Required properties:
resource id for thermal driver to get temperature via
SCU IPC.
+Watchdog bindings based on SCU Message Protocol
+------------------------------------------------------------
+
+Required properties:
+- compatible: should be "fsl,imx8qxp-sc-wdt";
+
Example (imx8qxp):
-------------
lsio_mu1: mailbox@5d1c0000 {
@@ -188,6 +194,10 @@ firmware {
tsens-num = <1>;
#thermal-sensor-cells = <1>;
};
+
+ watchdog: watchdog {
+ compatible = "fsl,imx8qxp-sc-wdt";

As-is, there's no reason for this to be in DT. The parent node's driver
can instantiate the wdog.


As the driver is currently written, you are correct, since it doesn't accept
watchdog timeout configuration through devicetree.

Question is if that is intended. Is it ?

Thanks,
Guenter