Apart from what Krzysztof already said:
On 12-10-21, 10:51, Krzysztof Kozlowski wrote:
> On 11/10/2021 18:57, Hector Martin wrote:
> > + pcluster_opp: opp-table-1 {
> > + compatible = "operating-points-v2";
> > + opp-shared;
> > +
> > + opp01 {
> > + opp-hz = /bits/ 64 <600000000>;
> > + opp-microvolt = <781000>;
> > + opp-level = <1>;
The opp-level thing wasn't designed to work this way, though it may
work just fine. It was designed as a unique key for power-domains,
which don't have opp-hz. The OPP core currently looks at 3 different
values, which can act as a unique key to identify the OPP. clk-rate,
bandwidth and level.
I think this is the first platform which has both hz and level in the
CPUs OPP table. What exactly is level in this case ?
Again, it may work fine, I just don't know where it may end up
breaking :)
--
viresh
