Re: [PATCH v2 1/5] dt-bindings: clock: Add i.MX91 clock support

[Krzysztof Kozlowski <krzk@kernel.org>]

On 06/06/2024 01:59, Pengfei Li wrote:
> On Tue, Jun 04, 2024 at 10:03:06AM -0500, Rob Herring wrote:
>> On Wed, May 29, 2024 at 07:26:30PM -0700, Pengfei Li wrote:
>>> i.MX91 has similar Clock Control Module(CCM) design as i.MX93.
>>> Add a new compatible string for i.MX91.
>>>
>>> Signed-off-by: Pengfei Li <pengfei.li_1@nxp.com>
>>> Reviewed-by: Frank Li <Frank.Li@nxp.com>
>>> ---
>>>  Documentation/devicetree/bindings/clock/imx93-clock.yaml | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/clock/imx93-clock.yaml b/Documentation/devicetree/bindings/clock/imx93-clock.yaml
>>> index ccb53c6b96c1..98c0800732ef 100644
>>> --- a/Documentation/devicetree/bindings/clock/imx93-clock.yaml
>>> +++ b/Documentation/devicetree/bindings/clock/imx93-clock.yaml
>>> @@ -16,6 +16,7 @@ description: |
>>>  properties:
>>>    compatible:
>>>      enum:
>>> +      - fsl,imx91-ccm
>>>        - fsl,imx93-ccm
>>
>> Should fallback to fsl,imx93-ccm? Being a superset should be ok because 
>> your DT should never use the non-existent clocks. If not, where is the 
>> driver change?
>>
>> Rob
>>
> 
> Hi Rob Herring,
> 
> Due to the different maintainers of the CCM framework and DTS, I have separated 
> the CCM driver patch and plan to send the DTS patch first before sending the 
> CCM driver patch. If that's possible, could you help merge this patch first? If 
> this is not allowed, I can also send out the CCM driver patch at the same time.

Binding goes with the driver, not DTS.

Best regards,
Krzysztof