Re: [PATCH 2/4] dt-bindings: arm: mediatek: Add new bindings of MediaTek frequency hopping

From: Krzysztof Kozlowski
Date: Thu Sep 01 2022 - 06:30:42 EST


On 01/09/2022 13:22, AngeloGioacchino Del Regno wrote:
>> That's simply not a proper
>> hardware description, so again:
>>
>> 1. If this is separate device (as you indicated), then it needs
>> expressing the dependencies and uses of other device resources.
>
> Agreed. In this case, what about...
>
> mediatek,hopping-ssc-percents = <&provider CLK_SOMEPLL 3>;
>
> or would it be better to specify the clocks in a separated property?
>
> clocks = <&provider CLK_SOMEPLL>, <&provider CLK_SOME_OTHER_PLL>;
> mediatek,hopping-ssc-percents = <3>, <5>;
>

I propose the last one - using standard clocks property and a matching
table.

Best regards,
Krzysztof