Re: [PATCH 3/3] arm64: dts: qcom: ipq5018: enable the CPUFreq support

[Gokul Sriram P]

> > > > >    #include <dt-bindings/interrupt-controller/arm-gic.h>
> > > > >    #include <dt-bindings/clock/qcom,gcc-ipq5018.h>
> > > > >    #include <dt-bindings/reset/qcom,gcc-ipq5018.h>
> > > > > +#include <dt-bindings/clock/qcom,apss-ipq.h>
> > > > c is before r.
 Sure, will update.

[...]
> > > > > +   cpu_opp_table: opp-table-cpu {
> > > > > +           compatible = "operating-points-v2";
> > > > > +           opp-shared;
> > > > > +
> > > > > +           opp-1008000000 {
> > > > > +                   opp-hz = /bits/ 64 <1008000000>;
> > > > > +                   opp-microvolt = <1100000>;
> > > > > +                   clock-latency-ns = <200000>;
> > > > And the rest of OPPs?
> > > Hi Krzysztof,
> > > IPQ5018 only supports running at 1.1GHz, but its running at 800MHz
> > > by default from the bootloader so there is only one OPP.
> > Isn't this contradictory? If it is running at 800 initially, then it
> > supports running at 800...
> I can only guess that it's not validated at 800MHz.
As per the h/w design team, there is negligible power or thermal benefit 
by lowering to 800MHz clock.
Hence, 800MHz opp wasn't included here.
> Regards,
> Robert
> > Best regards,
> > Krzysztof