Commit 9187d555 authored by Stephan Gerhold's avatar Stephan Gerhold Committed by Bjorn Andersson

arm64: dts: qcom: msm8939: Fix regulator constraints

The regulator constraints for the MSM8939 devices were originally taken
from Qualcomm's msm-3.10 vendor device tree (for lack of better
documentation). Unfortunately it turns out that Qualcomm's voltages are
slightly off as well and do not match the voltage constraints applied
by the RPM firmware.

This means that we sometimes request a specific voltage but the RPM
firmware actually applies a much lower or higher voltage. This is
particularly critical for pm8916_l11 which is used as SD card VMMC
regulator: The SD card can choose a voltage from the current range of
1.8 - 2.95V. If it chooses to run at 1.8V we pretend that this is fine
but the RPM firmware will still silently end up configuring 2.95V.
This can be easily reproduced with a multimeter or by checking the
SPMI hardware registers of the regulator.

Apply the same change as for MSM8916 in commit 35575082 ("arm64:
dts: qcom: msm8916: Fix regulator constraints") and make the voltages
match the actual "specified range" in the PM8916 Device Specification
which is enforced by the RPM firmware.
Signed-off-by: default avatarStephan Gerhold <stephan@gerhold.net>
Reviewed-by: default avatarBryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: default avatarBjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20230530-msm8939-regulators-v1-5-a3c3ac833567@gerhold.net
parent 8771308c
......@@ -164,13 +164,13 @@ &smd_rpm_regulators {
vdd_l7-supply = <&pm8916_s4>;
pm8916_s3: s3 {
regulator-min-microvolt = <1200000>;
regulator-max-microvolt = <1300000>;
regulator-min-microvolt = <1250000>;
regulator-max-microvolt = <1350000>;
};
pm8916_s4: s4 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <2100000>;
regulator-min-microvolt = <1850000>;
regulator-max-microvolt = <2150000>;
};
/* l1 is fixed to 1225000, but not connected in schematic */
......@@ -201,7 +201,7 @@ pm8916_l7: l7 {
};
pm8916_l8: l8 {
regulator-min-microvolt = <2850000>;
regulator-min-microvolt = <2900000>;
regulator-max-microvolt = <2900000>;
};
......@@ -216,7 +216,7 @@ pm8916_l10: l10 {
};
pm8916_l11: l11 {
regulator-min-microvolt = <1800000>;
regulator-min-microvolt = <2950000>;
regulator-max-microvolt = <2950000>;
};
......
......@@ -64,13 +64,13 @@ &smd_rpm_regulators {
vdd_l7-supply = <&pm8916_s4>;
pm8916_s3: s3 {
regulator-min-microvolt = <1200000>;
regulator-max-microvolt = <1300000>;
regulator-min-microvolt = <1250000>;
regulator-max-microvolt = <1350000>;
};
pm8916_s4: s4 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <2100000>;
regulator-min-microvolt = <1850000>;
regulator-max-microvolt = <2150000>;
};
pm8916_l2: l2 {
......@@ -99,7 +99,7 @@ pm8916_l7: l7 {
};
pm8916_l8: l8 {
regulator-min-microvolt = <2850000>;
regulator-min-microvolt = <2900000>;
regulator-max-microvolt = <2900000>;
};
......@@ -114,7 +114,7 @@ pm8916_l10: l10 {
};
pm8916_l11: l11 {
regulator-min-microvolt = <1800000>;
regulator-min-microvolt = <2950000>;
regulator-max-microvolt = <2950000>;
regulator-system-load = <200000>;
regulator-allow-set-load;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment