Cleanup by removing unneeded quotes from refs and add maxItems to reset-gpios and fix the required list. Fixes: 31360c28dfdd ("dt-bindings: usb: Add binding for Via lab VL817 hub controller") Signed-off-by: Anand Moon <linux.amoon@xxxxxxxxx> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> --- v5: added Rev by Krzysztof fix the subject as suggested by Krzysztof. v4: Fix the subject and patch description. --- .../devicetree/bindings/usb/vialab,vl817.yaml | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/Documentation/devicetree/bindings/usb/vialab,vl817.yaml b/Documentation/devicetree/bindings/usb/vialab,vl817.yaml index 5f9771e22058..23a13e1d5c7a 100644 --- a/Documentation/devicetree/bindings/usb/vialab,vl817.yaml +++ b/Documentation/devicetree/bindings/usb/vialab,vl817.yaml @@ -14,29 +14,32 @@ allOf: properties: compatible: - items: - - enum: - - usb2109,2817 - - usb2109,817 + enum: + - usb2109,2817 + - usb2109,817 reg: true reset-gpios: - description: GPIO controlling the RESET# pin. + maxItems: 1 + description: + GPIO controlling the RESET# pin. vdd-supply: description: phandle to the regulator that provides power to the hub. peer-hub: - $ref: '/schemas/types.yaml#/definitions/phandle' + $ref: /schemas/types.yaml#/definitions/phandle description: phandle to the peer hub on the controller. required: - - peer-hub - compatible - reg + - reset-gpios + - vdd-supply + - peer-hub additionalProperties: false @@ -45,7 +48,6 @@ examples: #include <dt-bindings/gpio/gpio.h> usb { - dr_mode = "host"; #address-cells = <1>; #size-cells = <0>; -- 2.38.1