* [PATCH] arm64: dts: qcom: msm8996: Reduce vdd_apc voltage
@ 2020-03-18 5:44 Bjorn Andersson
2020-03-30 9:11 ` Loic Poulain
0 siblings, 1 reply; 2+ messages in thread
From: Bjorn Andersson @ 2020-03-18 5:44 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Loic Poulain
Cc: Rob Herring, linux-arm-msm, devicetree, linux-kernel
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(-)
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>;
};
};
--
2.24.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] arm64: dts: qcom: msm8996: Reduce vdd_apc voltage
2020-03-18 5:44 [PATCH] arm64: dts: qcom: msm8996: Reduce vdd_apc voltage Bjorn Andersson
@ 2020-03-30 9:11 ` Loic Poulain
0 siblings, 0 replies; 2+ messages in thread
From: Loic Poulain @ 2020-03-30 9:11 UTC (permalink / raw)
To: Bjorn Andersson
Cc: Andy Gross, Rob Herring, linux-arm-msm, devicetree, linux-kernel
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>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-03-30 9:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-18 5:44 [PATCH] arm64: dts: qcom: msm8996: Reduce vdd_apc voltage Bjorn Andersson
2020-03-30 9:11 ` Loic Poulain
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).