[EXT] Re: [PATCH v7 04/11] dt-bindings: usb: ci-hdrc-usb2-imx: move imx parts to dedicated schema

Krzysztof Kozlowski krzysztof.kozlowski at linaro.org
Fri Mar 8 00:38:02 PST 2024


On 08/03/2024 09:30, Xu Yang wrote:
>>> +          - enum:
>>> +              - fsl,imx8mm-usb
>>> +              - fsl,imx8mn-usb
>>> +          - const: fsl,imx7d-usb
>>> +          - const: fsl,imx27-usb
>>> +      - items:
>>> +          - enum:
>>> +              - fsl,imx6sll-usb
>>> +              - fsl,imx7ulp-usb
>>> +          - const: fsl,imx6ul-usb
>>> +          - const: fsl,imx27-usb
>>> +
>>> +  clocks:
>>> +    minItems: 1
>>> +    maxItems: 3
>>
>>
>> No need to, it's already in common. Drop clocks.
> 
> No. I cannot remove clocks property in this yaml since the usb node may use assigned-clocks
> and assigned-clock-parents. Then dtschema will have warning like below patch:
> https://lore.kernel.org/all/20230508071837.68552-1-krzysztof.kozlowski@linaro.org/

This was a year ago, so please check if it is still the case. With
latest dtschema. If it is still the case, then indeed keep clocks here.

Best regards,
Krzysztof




More information about the linux-arm-kernel mailing list