Re: [PATCH 2/4] dt-bindings: cpufreq: mediatek: add mt8186 cpufreq dt-bindings

From: Jia-Wei Chang
Date: Fri Apr 01 2022 - 09:32:46 EST


On Thu, 2022-03-24 at 11:35 +0100, Krzysztof Kozlowski wrote:
> On 24/03/2022 10:42, Jia-Wei Chang wrote:
> > On Mon, 2022-03-07 at 19:59 +0100, Krzysztof Kozlowski wrote:
> > > On 07/03/2022 13:21, Tim Chang wrote:
> > > > 1. add cci property.
> > > > 2. add example of MT8186.
> > >
> > > One logical change at a time. Are these related? Why entirely new
> > > example just for "cci" node? Maybe this should be part of
> > > existing
> > > example?
> >
> > Yes, the cci property is required in some SoC, e.g. mt8183 and
> > mt8186,
> > because cpu and cci share the same power supplies.
>
> I asked why this cannot be part of existing example.

I misunderstood that.
I will update the complete example in the next version.

>
> > I will update the commit message and add an example of mt8186 to
> > present usage of cci.
>
> You added the example here, didn't you?

Yes, I did add it here.

>
> Best regards,
> Krzysztof