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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Nov 2, 2022 at 7:03 PM Marek Vasut <marex@xxxxxxx> 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@xxxxxxx>
> ---
> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> Cc: Rob Herring <robh+dt@xxxxxxxxxx>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>
> Cc: Richard Leitner <richard.leitner@xxxxxxxxxxx>
> Cc: linux-usb@xxxxxxxxxxxxxxx
> Cc: devicetree@xxxxxxxxxxxxxxx
> ---
> 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@xxxxxxxxxxx>
> +
> +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



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux