| Message ID | 20190813124744.32614-3-mripard@kernel.org |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | [1/5] dt-bindings: mfd: Convert Allwinner GPADC bindings to a schema | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/dt-meta-schema | fail | build log |
On Tue, Aug 13, 2019 at 6:47 AM Maxime Ripard <mripard@kernel.org> wrote: > > From: Maxime Ripard <maxime.ripard@bootlin.com> > > The Allwinner watchdog has an interrupt, either shared or dedicated > depending on the SoC, that has been described in some DT, but not all of > them. > > The binding is also completely missing that description. Let's add that > property to be consistent. I'm fine with fixing errors like this in the conversion patch. > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> > --- > .../bindings/watchdog/allwinner,sun4i-a10-wdt.yaml | 5 +++++ > 1 file changed, 5 insertions(+) Either way: Reviewed-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml b/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml index 93df27ec1664..4ace89f129b4 100644 --- a/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml +++ b/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml @@ -28,6 +28,9 @@ properties: reg: maxItems: 1 + interrupts: + maxItems: 1 + timeout-sec: $ref: /schemas/types.yaml#/definitions/uint32 description: @@ -36,6 +39,7 @@ properties: required: - compatible - reg + - interrupts additionalProperties: false @@ -44,6 +48,7 @@ examples: wdt: watchdog@1c20c90 { compatible = "allwinner,sun4i-a10-wdt"; reg = <0x01c20c90 0x10>; + interrupts = <24>; timeout-sec = <10>; };