diff mbox series

[RFC] dt-bindings: usb: usb251xb: Convert to YAML schema

Message ID 20221103000336.164507-1-marex@denx.de (mailing list archive)
State Superseded
Headers show
Series [RFC] dt-bindings: usb: usb251xb: Convert to YAML schema | expand

Commit Message

Marek Vasut Nov. 3, 2022, 12:03 a.m. UTC
Convert the usb251xb hub DT bindings from text to yaml schema so it is
possible to validate DTs against the schema.

Adjust the example to describe two different hubs at different I2C bus
addresses, to avoid I2C address collission in the example.

Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Richard Leitner <richard.leitner@skidata.com>
Cc: linux-usb@vger.kernel.org
Cc: devicetree@vger.kernel.org
---
NOTE: I keep getting this warning:
      Documentation/devicetree/bindings/usb/usb251xb.example.dtb: usb2514b@2d: vendor-id: b'\x00\x00' is not of type 'integer'
      I suspect this has something to do with the /bits/ 16 in the
      example. Can you provide some hint on what this is about ?
---
 .../devicetree/bindings/usb/usb251xb.txt      |  89 ------
 .../devicetree/bindings/usb/usb251xb.yaml     | 264 ++++++++++++++++++
 2 files changed, 264 insertions(+), 89 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/usb/usb251xb.txt
 create mode 100644 Documentation/devicetree/bindings/usb/usb251xb.yaml

Comments

Rob Herring Nov. 3, 2022, 3:28 a.m. UTC | #1
On Wed, Nov 2, 2022 at 7:03 PM Marek Vasut <marex@denx.de> wrote:
>
> Convert the usb251xb hub DT bindings from text to yaml schema so it is
> possible to validate DTs against the schema.
>
> Adjust the example to describe two different hubs at different I2C bus
> addresses, to avoid I2C address collission in the example.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: Richard Leitner <richard.leitner@skidata.com>
> Cc: linux-usb@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> ---
> NOTE: I keep getting this warning:
>       Documentation/devicetree/bindings/usb/usb251xb.example.dtb: usb2514b@2d: vendor-id: b'\x00\x00' is not of type 'integer'
>       I suspect this has something to do with the /bits/ 16 in the
>       example. Can you provide some hint on what this is about ?
> ---
>  .../devicetree/bindings/usb/usb251xb.txt      |  89 ------
>  .../devicetree/bindings/usb/usb251xb.yaml     | 264 ++++++++++++++++++
>  2 files changed, 264 insertions(+), 89 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/usb/usb251xb.txt
>  create mode 100644 Documentation/devicetree/bindings/usb/usb251xb.yaml
>
> diff --git a/Documentation/devicetree/bindings/usb/usb251xb.txt b/Documentation/devicetree/bindings/usb/usb251xb.txt
> deleted file mode 100644
> index 1a934eab175ee..0000000000000
> --- a/Documentation/devicetree/bindings/usb/usb251xb.txt
> +++ /dev/null
> @@ -1,89 +0,0 @@
> -Microchip USB 2.0 Hi-Speed Hub Controller
> -
> -The device node for the configuration of a Microchip USB251x/xBi USB 2.0
> -Hi-Speed Controller.
> -
> -Required properties :
> - - compatible : Should be "microchip,usb251xb" or one of the specific types:
> -       "microchip,usb2512b", "microchip,usb2512bi", "microchip,usb2513b",
> -       "microchip,usb2513bi", "microchip,usb2514b", "microchip,usb2514bi",
> -       "microchip,usb2517", "microchip,usb2517i", "microchip,usb2422"
> - - reg : I2C address on the selected bus (default is <0x2C>)
> -
> -Optional properties :
> - - reset-gpios : Should specify the gpio for hub reset
> - - vdd-supply : Should specify the phandle to the regulator supplying vdd
> - - skip-config : Skip Hub configuration, but only send the USB-Attach command
> - - vendor-id : Set USB Vendor ID of the hub (16 bit, default is 0x0424)
> - - product-id : Set USB Product ID of the hub (16 bit, default depends on type)
> - - device-id : Set USB Device ID of the hub (16 bit, default is 0x0bb3)
> - - language-id : Set USB Language ID (16 bit, default is 0x0000)
> - - manufacturer : Set USB Manufacturer string (max 31 characters long)
> - - product : Set USB Product string (max 31 characters long)
> - - serial : Set USB Serial string (max 31 characters long)
> - - {bus,self}-powered : selects between self- and bus-powered operation
> -       (boolean, default is self-powered)
> - - disable-hi-speed : disable USB Hi-Speed support (boolean)
> - - {multi,single}-tt : selects between multi- and single-transaction-translator
> -       (boolean, default is multi-tt)
> - - disable-eop : disable End of Packet generation in full-speed mode (boolean)
> - - {ganged,individual}-sensing : select over-current sense type in self-powered
> -       mode (boolean, default is individual)
> - - {ganged,individual}-port-switching : select port power switching mode
> -       (boolean, default is individual)
> - - dynamic-power-switching : enable auto-switching from self- to bus-powered
> -       operation if the local power source is removed or unavailable (boolean)
> - - oc-delay-us : Delay time (in microseconds) for filtering the over-current
> -       sense inputs. Valid values are 100, 4000, 8000 (default) and 16000. If
> -       an invalid value is given, the default is used instead.
> - - compound-device : indicate the hub is part of a compound device (boolean)
> - - port-mapping-mode : enable port mapping mode (boolean)
> - - led-{usb,speed}-mode : led usb/speed indication mode selection
> -       (boolean, default is speed mode)
> - - string-support : enable string descriptor support (required for manufacturer,
> -       product and serial string configuration)
> - - non-removable-ports : Should specify the ports which have a non-removable
> -       device connected.
> - - sp-disabled-ports : Specifies the ports which will be self-power disabled
> - - bp-disabled-ports : Specifies the ports which will be bus-power disabled
> - - sp-max-total-current-microamp: Specifies max current consumed by the hub
> -       from VBUS when operating in self-powered hub. It includes the hub
> -       silicon along with all associated circuitry including a permanently
> -       attached peripheral (range: 0 - 100000 uA, default 1000 uA)
> - - bp-max-total-current-microamp: Specifies max current consumed by the hub
> -       from VBUS when operating in self-powered hub. It includes the hub
> -       silicon along with all associated circuitry including a permanently
> -       attached peripheral (range: 0 - 510000 uA, default 100000 uA)
> - - sp-max-removable-current-microamp: Specifies max current consumed by the hub
> -       from VBUS when operating in self-powered hub. It includes the hub
> -       silicon along with all associated circuitry excluding a permanently
> -       attached peripheral (range: 0 - 100000 uA, default 1000 uA)
> - - bp-max-removable-current-microamp: Specifies max current consumed by the hub
> -       from VBUS when operating in self-powered hub. It includes the hub
> -       silicon along with all associated circuitry excluding a permanently
> -       attached peripheral (range: 0 - 510000 uA, default 100000 uA)
> - - power-on-time-ms : Specifies the time it takes from the time the host
> -       initiates the power-on sequence to a port until the port has adequate
> -       power. The value is given in ms in a 0 - 510 range (default is 100ms).
> - - swap-dx-lanes : Specifies the ports which will swap the differential-pair
> -       (D+/D-), default is not-swapped.
> -
> -Examples:
> -       usb2512b@2c {
> -               compatible = "microchip,usb2512b";
> -               reg = <0x2c>;
> -               reset-gpios = <&gpio1 4 GPIO_ACTIVE_LOW>;
> -       };
> -
> -       usb2514b@2c {
> -               compatible = "microchip,usb2514b";
> -               reg = <0x2c>;
> -               vendor-id = /bits/ 16 <0x0000>;
> -               product-id = /bits/ 16 <0x0000>;
> -               string-support;
> -               manufacturer = "Foo";
> -               product = "Foo-Bar";
> -               serial = "1234567890A";
> -               /* correct misplaced usb connectors on port 1,2 */
> -               swap-dx-lanes = <1 2>;
> -       };
> diff --git a/Documentation/devicetree/bindings/usb/usb251xb.yaml b/Documentation/devicetree/bindings/usb/usb251xb.yaml
> new file mode 100644
> index 0000000000000..d644af5bbb9f6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/usb/usb251xb.yaml
> @@ -0,0 +1,264 @@
> +# SPDX-License-Identifier: GPL-2.0
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/usb/usb251xb.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Microchip USB 2.0 Hi-Speed Hub Controller
> +
> +maintainers:
> +  - Richard Leitner <richard.leitner@skidata.com>
> +
> +properties:
> +  compatible:
> +    enum:
> +      - microchip,usb2422
> +      - microchip,usb2512b
> +      - microchip,usb2512bi
> +      - microchip,usb2513b
> +      - microchip,usb2513bi
> +      - microchip,usb2514b
> +      - microchip,usb2514bi
> +      - microchip,usb2517
> +      - microchip,usb2517i
> +      - microchip,usb251xb
> +
> +  reg:
> +    maxItems: 1
> +
> +  reset-gpios:
> +    description: |
> +      Should specify the gpio for hub reset
> +
> +  vdd-supply:
> +    description: |
> +      Should specify the phandle to the regulator supplying vdd
> +
> +  skip-config:
> +    $ref: /schemas/types.yaml#/definitions/flag
> +    description: |
> +      Skip Hub configuration, but only send the USB-Attach command
> +
> +  vendor-id:
> +    $ref: /schemas/types.yaml#/definitions/uint16-item

'uint16-item' is not a type. Your choices are 'uint16', 'uint16-array'
or 'uint16-matrix'.

> +    description: |
> +      Set USB Vendor ID of the hub (16 bit, default is 0x0424)

default: 0x424
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/usb/usb251xb.txt b/Documentation/devicetree/bindings/usb/usb251xb.txt
deleted file mode 100644
index 1a934eab175ee..0000000000000
--- a/Documentation/devicetree/bindings/usb/usb251xb.txt
+++ /dev/null
@@ -1,89 +0,0 @@ 
-Microchip USB 2.0 Hi-Speed Hub Controller
-
-The device node for the configuration of a Microchip USB251x/xBi USB 2.0
-Hi-Speed Controller.
-
-Required properties :
- - compatible : Should be "microchip,usb251xb" or one of the specific types:
-	"microchip,usb2512b", "microchip,usb2512bi", "microchip,usb2513b",
-	"microchip,usb2513bi", "microchip,usb2514b", "microchip,usb2514bi",
-	"microchip,usb2517", "microchip,usb2517i", "microchip,usb2422"
- - reg : I2C address on the selected bus (default is <0x2C>)
-
-Optional properties :
- - reset-gpios : Should specify the gpio for hub reset
- - vdd-supply : Should specify the phandle to the regulator supplying vdd
- - skip-config : Skip Hub configuration, but only send the USB-Attach command
- - vendor-id : Set USB Vendor ID of the hub (16 bit, default is 0x0424)
- - product-id : Set USB Product ID of the hub (16 bit, default depends on type)
- - device-id : Set USB Device ID of the hub (16 bit, default is 0x0bb3)
- - language-id : Set USB Language ID (16 bit, default is 0x0000)
- - manufacturer : Set USB Manufacturer string (max 31 characters long)
- - product : Set USB Product string (max 31 characters long)
- - serial : Set USB Serial string (max 31 characters long)
- - {bus,self}-powered : selects between self- and bus-powered operation
-	(boolean, default is self-powered)
- - disable-hi-speed : disable USB Hi-Speed support (boolean)
- - {multi,single}-tt : selects between multi- and single-transaction-translator
-	(boolean, default is multi-tt)
- - disable-eop : disable End of Packet generation in full-speed mode (boolean)
- - {ganged,individual}-sensing : select over-current sense type in self-powered
-	mode (boolean, default is individual)
- - {ganged,individual}-port-switching : select port power switching mode
-	(boolean, default is individual)
- - dynamic-power-switching : enable auto-switching from self- to bus-powered
-	operation if the local power source is removed or unavailable (boolean)
- - oc-delay-us : Delay time (in microseconds) for filtering the over-current
-	sense inputs. Valid values are 100, 4000, 8000 (default) and 16000. If
-	an invalid value is given, the default is used instead.
- - compound-device : indicate the hub is part of a compound device (boolean)
- - port-mapping-mode : enable port mapping mode (boolean)
- - led-{usb,speed}-mode : led usb/speed indication mode selection
-	(boolean, default is speed mode)
- - string-support : enable string descriptor support (required for manufacturer,
-	product and serial string configuration)
- - non-removable-ports : Should specify the ports which have a non-removable
-	device connected.
- - sp-disabled-ports : Specifies the ports which will be self-power disabled
- - bp-disabled-ports : Specifies the ports which will be bus-power disabled
- - sp-max-total-current-microamp: Specifies max current consumed by the hub
-	from VBUS when operating in self-powered hub. It includes the hub
-	silicon along with all associated circuitry including a permanently
-	attached peripheral (range: 0 - 100000 uA, default 1000 uA)
- - bp-max-total-current-microamp: Specifies max current consumed by the hub
-	from VBUS when operating in self-powered hub. It includes the hub
-	silicon along with all associated circuitry including a permanently
-	attached peripheral (range: 0 - 510000 uA, default 100000 uA)
- - sp-max-removable-current-microamp: Specifies max current consumed by the hub
-	from VBUS when operating in self-powered hub. It includes the hub
-	silicon along with all associated circuitry excluding a permanently
-	attached peripheral (range: 0 - 100000 uA, default 1000 uA)
- - bp-max-removable-current-microamp: Specifies max current consumed by the hub
-	from VBUS when operating in self-powered hub. It includes the hub
-	silicon along with all associated circuitry excluding a permanently
-	attached peripheral (range: 0 - 510000 uA, default 100000 uA)
- - power-on-time-ms : Specifies the time it takes from the time the host
-	initiates the power-on sequence to a port until the port has adequate
-	power. The value is given in ms in a 0 - 510 range (default is 100ms).
- - swap-dx-lanes : Specifies the ports which will swap the differential-pair
-	(D+/D-), default is not-swapped.
-
-Examples:
-	usb2512b@2c {
-		compatible = "microchip,usb2512b";
-		reg = <0x2c>;
-		reset-gpios = <&gpio1 4 GPIO_ACTIVE_LOW>;
-	};
-
-	usb2514b@2c {
-		compatible = "microchip,usb2514b";
-		reg = <0x2c>;
-		vendor-id = /bits/ 16 <0x0000>;
-		product-id = /bits/ 16 <0x0000>;
-		string-support;
-		manufacturer = "Foo";
-		product = "Foo-Bar";
-		serial = "1234567890A";
-		/* correct misplaced usb connectors on port 1,2 */
-		swap-dx-lanes = <1 2>;
-	};
diff --git a/Documentation/devicetree/bindings/usb/usb251xb.yaml b/Documentation/devicetree/bindings/usb/usb251xb.yaml
new file mode 100644
index 0000000000000..d644af5bbb9f6
--- /dev/null
+++ b/Documentation/devicetree/bindings/usb/usb251xb.yaml
@@ -0,0 +1,264 @@ 
+# SPDX-License-Identifier: GPL-2.0
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/usb/usb251xb.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Microchip USB 2.0 Hi-Speed Hub Controller
+
+maintainers:
+  - Richard Leitner <richard.leitner@skidata.com>
+
+properties:
+  compatible:
+    enum:
+      - microchip,usb2422
+      - microchip,usb2512b
+      - microchip,usb2512bi
+      - microchip,usb2513b
+      - microchip,usb2513bi
+      - microchip,usb2514b
+      - microchip,usb2514bi
+      - microchip,usb2517
+      - microchip,usb2517i
+      - microchip,usb251xb
+
+  reg:
+    maxItems: 1
+
+  reset-gpios:
+    description: |
+      Should specify the gpio for hub reset
+
+  vdd-supply:
+    description: |
+      Should specify the phandle to the regulator supplying vdd
+
+  skip-config:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      Skip Hub configuration, but only send the USB-Attach command
+
+  vendor-id:
+    $ref: /schemas/types.yaml#/definitions/uint16-item
+    description: |
+      Set USB Vendor ID of the hub (16 bit, default is 0x0424)
+
+  product-id:
+    $ref: /schemas/types.yaml#/definitions/uint16-item
+    description: |
+      Set USB Product ID of the hub (16 bit, default depends on type)
+
+  device-id:
+    $ref: /schemas/types.yaml#/definitions/uint16-item
+    description: |
+      Set USB Device ID of the hub (16 bit, default is 0x0bb3)
+
+  language-id:
+    $ref: /schemas/types.yaml#/definitions/uint16-item
+    description: |
+      Set USB Language ID (16 bit, default is 0x0000)
+
+  manufacturer:
+    $ref: /schemas/types.yaml#/definitions/string
+    description: |
+      Set USB Manufacturer string (max 31 characters long)
+
+  product:
+    $ref: /schemas/types.yaml#/definitions/string
+    description: |
+      Set USB Product string (max 31 characters long)
+
+  serial:
+    $ref: /schemas/types.yaml#/definitions/string
+    description: |
+      Set USB Serial string (max 31 characters long)
+
+  bus-powered:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      selects between self- and bus-powered operation
+      (boolean, default is self-powered)
+
+  self-powered:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      selects between self- and bus-powered operation
+      (boolean, default is self-powered)
+
+  disable-hi-speed:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      disable USB Hi-Speed support (boolean)
+
+  multi-tt:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      selects between multi- and single-transaction-translator
+      (boolean, default is multi-tt)
+
+  single-tt:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      selects between multi- and single-transaction-translator
+      (boolean, default is multi-tt)
+
+  disable-eop:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      disable End of Packet generation in full-speed mode (boolean)
+
+  ganged-sensing:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      select over-current sense type in self-powered mode
+      (boolean, default is individual)
+
+  individual-sensing:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      select over-current sense type in self-powered mode
+      (boolean, default is individual)
+
+  ganged-port-switching:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      select port power switching mode (boolean, default is individual)
+
+  individual-port-switching:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      select port power switching mode (boolean, default is individual)
+
+  dynamic-power-switching:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      enable auto-switching from self- to bus-powered operation if the
+      local power source is removed or unavailable (boolean)
+
+  oc-delay-us:
+    enum: [100, 4000, 8000, 16000]
+    description: |
+      Delay time (in microseconds) for filtering the over-current sense
+      inputs. Valid values are 100, 4000, 8000 (default) and 16000. If
+      an invalid value is given, the default is used instead.
+
+  compound-device:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      indicate the hub is part of a compound device (boolean)
+
+  port-mapping-mode:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      enable port mapping mode (boolean)
+
+  led-usb-mode:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      led usb/speed indication mode selection (boolean, default is speed mode)
+
+  led-speed-mode:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      led usb/speed indication mode selection (boolean, default is speed mode)
+
+  string-support:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description: |
+      enable string descriptor support (required for manufacturer, product
+      and serial string configuration)
+
+  non-removable-ports:
+    $ref: /schemas/types.yaml#/definitions/uint8-array
+    description: |
+      Should specify the ports which have a non-removable device connected.
+
+  sp-disabled-ports:
+    $ref: /schemas/types.yaml#/definitions/uint8-array
+    description: |
+      Specifies the ports which will be self-power disabled
+
+  bp-disabled-ports:
+    $ref: /schemas/types.yaml#/definitions/uint8-array
+    description: |
+      Specifies the ports which will be bus-power disabled
+
+  sp-max-total-current-microamp:
+    maximum: 100000
+    description: |
+      Specifies max current consumed by the hub from VBUS when
+      operating in self-powered hub. It includes the hub silicon
+      along with all associated circuitry including a permanently
+      attached peripheral (range: 0 - 100000 uA, default 1000 uA)
+
+  bp-max-total-current-microamp:
+    maximum: 510000
+    description: |
+      Specifies max current consumed by the hub from VBUS when
+      operating in self-powered hub. It includes the hub silicon
+      along with all associated circuitry including a permanently
+      attached peripheral (range: 0 - 510000 uA, default 100000 uA)
+
+  sp-max-removable-current-microamp:
+    maximum: 100000
+    description: |
+      Specifies max current consumed by the hub from VBUS when
+      operating in self-powered hub. It includes the hub silicon
+      along with all associated circuitry excluding a permanently
+      attached peripheral (range: 0 - 100000 uA, default 1000 uA)
+
+  bp-max-removable-current-microamp:
+    maximum: 510000
+    description: |
+      Specifies max current consumed by the hub from VBUS when
+      operating in self-powered hub. It includes the hub silicon
+      along with all associated circuitry excluding a permanently
+      attached peripheral (range: 0 - 510000 uA, default 100000 uA)
+
+  power-on-time-ms:
+    maximum: 510
+    description: |
+      Specifies the time it takes from the time the host initiates the
+      power-on sequence to a port until the port has adequate power.
+      The value is given in ms in a 0 - 510 range (default is 100ms).
+
+  swap-dx-lanes:
+    $ref: /schemas/types.yaml#/definitions/uint8-array
+    description: |
+      Specifies the ports which will swap the differential-pair (D+/D-),
+      default is not-swapped.
+
+unevaluatedProperties: false
+
+required:
+  - compatible
+  - reg
+
+examples:
+  - |
+    #include <dt-bindings/gpio/gpio.h>
+
+    i2c {
+      #address-cells = <1>;
+      #size-cells = <0>;
+
+      usb2512b@2c {
+        compatible = "microchip,usb2512b";
+        reg = <0x2c>;
+        reset-gpios = <&gpio1 4 GPIO_ACTIVE_LOW>;
+      };
+
+      usb2514b@2d {
+        compatible = "microchip,usb2514b";
+        reg = <0x2d>;
+        vendor-id = /bits/ 16 <0x0000>;
+        product-id = /bits/ 16 <0x0000>;
+        string-support;
+        manufacturer = "Foo";
+        product = "Foo-Bar";
+        serial = "1234567890A";
+        /* correct misplaced usb connectors on port 1,2 */
+        swap-dx-lanes = <1 2>;
+      };
+    };