diff mbox series

arm64: dts: qcom: msm8996: Reduce vdd_apc voltage

Message ID 20200318054442.3066726-1-bjorn.andersson@linaro.org (mailing list archive)
State Accepted
Commit 1fdae587c13d582c19bcda4207e558f300c63617
Headers show
Series arm64: dts: qcom: msm8996: Reduce vdd_apc voltage | expand

Commit Message

Bjorn Andersson March 18, 2020, 5:44 a.m. UTC
Some msm8996 based devices are unstable when run with VDD_APC of 1.23V,
which is listed as the maximum voltage in "Turbo" mode. Given that the
CPU cluster is not run in "Turbo" mode, reduce this to 0.98V - the
maximum voltage for nominal operation.

Fixes: 7a2a2231ef22 ("arm64: dts: apq8096-db820c: Fix VDD core voltage")
Cc: Loic Poulain <loic.poulain@linaro.org>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
---
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Loic Poulain March 30, 2020, 9:11 a.m. UTC | #1
On Wed, 18 Mar 2020 at 06:46, Bjorn Andersson
<bjorn.andersson@linaro.org> wrote:
>
> Some msm8996 based devices are unstable when run with VDD_APC of 1.23V,
> which is listed as the maximum voltage in "Turbo" mode. Given that the
> CPU cluster is not run in "Turbo" mode, reduce this to 0.98V - the
> maximum voltage for nominal operation.
>
> Fixes: 7a2a2231ef22 ("arm64: dts: apq8096-db820c: Fix VDD core voltage")
> Cc: Loic Poulain <loic.poulain@linaro.org>
> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> ---
>  arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Don't see any problem with that change, tested with and without
cpufreq (db820c).

Tested-by: Loic Poulain <loic.poulain@linaro.org>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
index af87350b5547..4692b7ad16b7 100644
--- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
@@ -658,8 +658,8 @@  s10 {
 	s11 {
 		qcom,saw-leader;
 		regulator-always-on;
-		regulator-min-microvolt = <1230000>;
-		regulator-max-microvolt = <1230000>;
+		regulator-min-microvolt = <980000>;
+		regulator-max-microvolt = <980000>;
 	};
 };