[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