2020-09-23 12:55:33

by Oleksij Rempel

[permalink] [raw]
Subject: [PATCH v1] dt-binding: can: mcp25xxfd: documentation fixes

Apply following fixes:
- Use 'interrupts'. (interrupts-extended will automagically be supported
by the tools)
- *-supply is always a single item. So, drop maxItems=1
- add "additionalProperties: false" flag to detect unneeded properties.

Signed-off-by: Oleksij Rempel <[email protected]>
---
.../devicetree/bindings/net/can/microchip,mcp25xxfd.yaml | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/can/microchip,mcp25xxfd.yaml b/Documentation/devicetree/bindings/net/can/microchip,mcp25xxfd.yaml
index aa2cad14d6d7..5beb00a614bf 100644
--- a/Documentation/devicetree/bindings/net/can/microchip,mcp25xxfd.yaml
+++ b/Documentation/devicetree/bindings/net/can/microchip,mcp25xxfd.yaml
@@ -24,7 +24,7 @@ properties:
reg:
maxItems: 1

- interrupts-extended:
+ interrupts:
maxItems: 1

clocks:
@@ -32,11 +32,9 @@ properties:

vdd-supply:
description: Regulator that powers the CAN controller.
- maxItems: 1

xceiver-supply:
description: Regulator that powers the CAN transceiver.
- maxItems: 1

microchip,rx-int-gpios:
description:
@@ -52,9 +50,11 @@ properties:
required:
- compatible
- reg
- - interrupts-extended
+ - interrupts
- clocks

+additionalProperties: false
+
examples:
- |
#include <dt-bindings/gpio/gpio.h>
--
2.28.0


2020-09-29 18:50:34

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH v1] dt-binding: can: mcp25xxfd: documentation fixes

On Wed, 23 Sep 2020 14:53:01 +0200, Oleksij Rempel wrote:
> Apply following fixes:
> - Use 'interrupts'. (interrupts-extended will automagically be supported
> by the tools)
> - *-supply is always a single item. So, drop maxItems=1
> - add "additionalProperties: false" flag to detect unneeded properties.
>
> Signed-off-by: Oleksij Rempel <[email protected]>
> ---
> .../devicetree/bindings/net/can/microchip,mcp25xxfd.yaml | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>

Reviewed-by: Rob Herring <[email protected]>