Re: [PATCH v3 15/28] ARM: dts: qcom: apq8064-cm-qs600: constraint cpufreq regulators

Konrad Dybcio
2023-07-03T11:26:12Z

On 2.07.2023 19:42, 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>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
> .../boot/dts/qcom/qcom-apq8064-cm-qs600.dts | 18 ++++++++++++++++--
> 1 file changed, 16 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064-cm-qs600.dts b/arch/arm/boot/dts/qcom/qcom-apq8064-cm-qs600.dts
> index 6472277d1c6d..ee0090e03fb3 100644
> --- a/arch/arm/boot/dts/qcom/qcom-apq8064-cm-qs600.dts
> +++ b/arch/arm/boot/dts/qcom/qcom-apq8064-cm-qs600.dts
> @@ -106,8 +106,8 @@ pm8921_s1: s1 {
> };
>
> pm8921_s3: s3 {
> - regulator-min-microvolt = <1000000>;
> - regulator-max-microvolt = <1400000>;
> + regulator-min-microvolt = <950000>;
> + regulator-max-microvolt = <1150000>;
> qcom,switch-mode-frequency = <4800000>;
> };
>
> @@ -147,9 +147,23 @@ pm8921_l23: l23 {
> bias-pull-down;
> };
>
> + pm8921_l24: l24 {
> + regulator-min-microvolt = <1050000>;
> + regulator-max-microvolt = <1150000>;
> + bias-pull-down;
> + };
> +
> pm8921_lvs6: lvs6 {
> bias-pull-down;
> };
> +
> + /* HFPLL regulator */
> + pm8921_lvs7: lvs7 {
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> };
> };
>