| Message ID | 20230625202547.174647-22-dmitry.baryshkov@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | ARM: qcom: apq8064: support CPU frequency scaling | expand |
On 25.06.2023 22:25, Dmitry Baryshkov wrote: > Add additional constraints to the CPUfreq-related regulators, it is > better be safe than sorry there. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- same comment as in p20 Acked-by: Konrad Dybcio <konrad.dybcio@linaro.org> Konrad > .../arm/boot/dts/qcom/qcom-apq8064-ifc6410.dts | 18 ++++++++++++++++-- > 1 file changed, 16 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064-ifc6410.dts b/arch/arm/boot/dts/qcom/qcom-apq8064-ifc6410.dts > index 96307550523a..ad3cd45362df 100644 > --- a/arch/arm/boot/dts/qcom/qcom-apq8064-ifc6410.dts > +++ b/arch/arm/boot/dts/qcom/qcom-apq8064-ifc6410.dts > @@ -215,8 +215,8 @@ s1 { > }; > > s3 { > - regulator-min-microvolt = <1000000>; > - regulator-max-microvolt = <1400000>; > + regulator-min-microvolt = <950000>; > + regulator-max-microvolt = <1150000>; > qcom,switch-mode-frequency = <4800000>; > }; > > @@ -262,6 +262,12 @@ l23 { > bias-pull-down; > }; > > + l24 { > + regulator-min-microvolt = <1050000>; > + regulator-max-microvolt = <1150000>; > + bias-pull-down; > + }; > + > lvs1 { > bias-pull-down; > }; > @@ -269,6 +275,14 @@ lvs1 { > lvs6 { > bias-pull-down; > }; > + > + /* HFPLL regulator */ > + lvs7 { > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-boot-on; > + regulator-always-on; > + }; > }; > }; >
diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064-ifc6410.dts b/arch/arm/boot/dts/qcom/qcom-apq8064-ifc6410.dts index 96307550523a..ad3cd45362df 100644 --- a/arch/arm/boot/dts/qcom/qcom-apq8064-ifc6410.dts +++ b/arch/arm/boot/dts/qcom/qcom-apq8064-ifc6410.dts @@ -215,8 +215,8 @@ s1 { }; s3 { - regulator-min-microvolt = <1000000>; - regulator-max-microvolt = <1400000>; + regulator-min-microvolt = <950000>; + regulator-max-microvolt = <1150000>; qcom,switch-mode-frequency = <4800000>; }; @@ -262,6 +262,12 @@ l23 { bias-pull-down; }; + l24 { + regulator-min-microvolt = <1050000>; + regulator-max-microvolt = <1150000>; + bias-pull-down; + }; + lvs1 { bias-pull-down; }; @@ -269,6 +275,14 @@ lvs1 { lvs6 { bias-pull-down; }; + + /* HFPLL regulator */ + lvs7 { + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-boot-on; + regulator-always-on; + }; }; };
Add additional constraints to the CPUfreq-related regulators, it is better be safe than sorry there. Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- .../arm/boot/dts/qcom/qcom-apq8064-ifc6410.dts | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-)