RE: [PATCH v4 08/14] dt: qcom: Add opp and thermal to the msm8996

[ilialin]

Kryo has single clock per cluster. I define here a shared OPP table per cluster.

> -----Original Message-----
> From: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
> Sent: Friday, May 4, 2018 09:04
> To: Ilia Lin <ilialin@xxxxxxxxxxxxxx>
> Cc: Michael Turquette <mturquette@xxxxxxxxxxxx>; Stephen Boyd
> <sboyd@xxxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>; Mark Rutland
> <mark.rutland@xxxxxxx>; Rafael J. Wysocki <rjw@xxxxxxxxxxxxx>; Liam
> Girdwood <lgirdwood@xxxxxxxxx>; Mark Brown <broonie@xxxxxxxxxx>;
> Andy Gross <andy.gross@xxxxxxxxxx>; David Brown
> <david.brown@xxxxxxxxxx>; Catalin Marinas <catalin.marinas@xxxxxxx>; Will
> Deacon <will.deacon@xxxxxxx>; Linux Clock List <linux-
> clk@xxxxxxxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx; Linux Kernel Mailing List
> <linux-kernel@xxxxxxxxxxxxxxx>; linux-pm@xxxxxxxxxxxxxxx; linux-arm-
> msm@xxxxxxxxxxxxxxx; open list:ARM/QUALCOMM SUPPORT <linux-
> soc@xxxxxxxxxxxxxxx>; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; Nayak,
> Rajendra <rnayak@xxxxxxxxxxxxxx>; Amit Kucheria
> <amit.kucheria@xxxxxxxxxx>; Nicolas Dechesne
> <nicolas.dechesne@xxxxxxxxxx>; celster@xxxxxxxxxxxxxx;
> tfinkel@xxxxxxxxxxxxxx
> Subject: Re: [PATCH v4 08/14] dt: qcom: Add opp and thermal to the
> msm8996
> 
> On 2 April 2018 at 14:46, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> 
> >> +     cluster0_opp: opp_table0 {
> >> +             compatible = "operating-points-v2";
> >> +             opp-shared;
> >
> > Is Kryo like krait where CPUs do DVFS independently ? If yes, then
> > opp-shared thing should be dropped.
> 
> Have you ever replied to this question ? Sorry, but I am not able to find it in
> my inbox :(