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

[AngeloGioacchino Del Regno]

Il 01/09/22 12:30, Krzysztof Kozlowski ha scritto:
> 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.

Right. I like the last one a bit better, as well.

Thanks for the advices!
Angelo