Re: [PATCH 1/1] dt-bindings: usb: Convert fsl-usb to yaml

[Krzysztof Kozlowski <krzk@kernel.org>]

On 25/06/2024 04:25, Frank Li wrote:
> Convert fsl-usb binding doc to yaml format.
> 
> Additional change:


> -	usb@4000 {
> -		compatible = "fsl,mpc5121-usb2-dr";
> -		reg = <0x4000 0x1000>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -		interrupt-parent = < &ipic >;
> -		interrupts = <44 0x8>;
> -		dr_mode = "otg";
> -		phy_type = "utmi_wide";
> -		fsl,invert-drvvbus;
> -		fsl,invert-pwr-fault;
> -	};
> diff --git a/Documentation/devicetree/bindings/usb/fsl-usb.yaml b/Documentation/devicetree/bindings/usb/fsl-usb.yaml
> new file mode 100644
> index 0000000000000..8b5724e213f09
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/usb/fsl-usb.yaml

fsl,usb.yaml
or: fsl,usb2.yaml

> +
> +maintainers:
> +  - Frank Li <Frank.Li@nxp.com>
> +
> +description: |

Do not need '|' unless you need to preserve formatting.

> +  The device node for a USB controller that is part of a Freescale
> +  SOC is as described in the document "Open Firmware Recommended
> +  Practice: Universal Serial Bus" with the following modifications
> +  and additions.
> +
> +properties:
> +  compatible:
> +    oneOf:
> +      - enum:
> +          - fsl-usb2-mph
> +          - fsl-usb2-dr
> +          - fsl-usb2-dr-v2.2

It cannot be standalone and not-standalone. Cannot be both. Choose one.

> +      - items:
> +          - enum:
> +              - fsl-usb2-dr-v2.2
> +              - fsl-usb2-dr-v2.5
> +          - const: fsl-usb2-dr
> +
> +  phy_type:
> +    $ref: /schemas/types.yaml#/definitions/string
> +    enum: [ulpi, serial, utmi, utmi_wide]
> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    maxItems: 1
> +
> +  port0:
> +    $ref: /schemas/types.yaml#/definitions/flag
> +    description:
> +      Indicates port0 is connected for fsl-usb2-mph compatible controllers.
> +
> +  port1:
> +    $ref: /schemas/types.yaml#/definitions/flag
> +    description:
> +      Indicates port1 is connected for "fsl-usb2-mph" compatible controllers.
> +
> +  fsl,invert-drvvbus:
> +    $ref: /schemas/types.yaml#/definitions/flag
> +    description:
> +      for MPC5121 USB0 only. Indicates the
> +      port power polarity of internal PHY signal DRVVBUS is inverted.
> +
> +  fsl,invert-pwr-fault:
> +    $ref: /schemas/types.yaml#/definitions/flag
> +    description:
> +      for MPC5121 USB0 only. Indicates
> +      the PWR_FAULT signal polarity is inverted.
> +
> +required:
> +  - compatible
> +  - reg
> +  - interrupts
> +  - phy_type

Keep the same order as in "properties:". Preferably this order, so
adjust "properties:" to match "required:".

> +
> +allOf:
> +  - $ref: usb-drd.yaml#
> +


Best regards,
Krzysztof