* [PATCH V4] arm64: dts: qcom: qcm6490-idp: Allow UFS regulators load/mode setting
@ 2024-10-17 12:28 Rakesh Kota
2024-12-26 18:26 ` Bjorn Andersson
0 siblings, 1 reply; 2+ messages in thread
From: Rakesh Kota @ 2024-10-17 12:28 UTC (permalink / raw)
To: dmitry.baryshkov, andersson, konradybcio, robh, krzk+dt, conor+dt,
linux-arm-msm, devicetree, linux-kernel
Cc: quic_kamalw, quic_jprakash, quic_kotarake
The UFS driver expects to be able to set load (and by extension, mode)
on its supply regulators. Add the necessary properties to make that
possible.
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: Rakesh Kota <quic_kotarake@quicinc.com>
---
Changes from V3:
- Add missing change history for V1 and V2.
- Add missing Reviewed-by tag.
Changes from V2:
- Add missing semicolon (;) after regulator-allow-set-load prop for
ldo9 regulator.
Changes from V1:
- Dropped the removing Min and Max Voltage change as suggested by
Dmitry.
- Link to V1: https://lore.kernel.org/all/20241004080110.4150476-1-quic_kotarake@quicinc.com
---
arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
index 5f3d4807ac43..bfb1cdc238cc 100644
--- a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
+++ b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
@@ -258,6 +258,8 @@ vreg_l6b_1p2: ldo6 {
regulator-name = "vreg_l6b_1p2";
regulator-min-microvolt = <1140000>;
regulator-max-microvolt = <1260000>;
+ regulator-allow-set-load;
+ regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM RPMH_REGULATOR_MODE_HPM>;
regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
};
@@ -265,6 +267,8 @@ vreg_l7b_2p952: ldo7 {
regulator-name = "vreg_l7b_2p952";
regulator-min-microvolt = <2400000>;
regulator-max-microvolt = <3544000>;
+ regulator-allow-set-load;
+ regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM RPMH_REGULATOR_MODE_HPM>;
regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
};
@@ -279,6 +283,8 @@ vreg_l9b_1p2: ldo9 {
regulator-name = "vreg_l9b_1p2";
regulator-min-microvolt = <1200000>;
regulator-max-microvolt = <1304000>;
+ regulator-allow-set-load;
+ regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM RPMH_REGULATOR_MODE_HPM>;
regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
};
@@ -467,6 +473,8 @@ vreg_l10c_0p88: ldo10 {
regulator-name = "vreg_l10c_0p88";
regulator-min-microvolt = <720000>;
regulator-max-microvolt = <1050000>;
+ regulator-allow-set-load;
+ regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM RPMH_REGULATOR_MODE_HPM>;
regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH V4] arm64: dts: qcom: qcm6490-idp: Allow UFS regulators load/mode setting
2024-10-17 12:28 [PATCH V4] arm64: dts: qcom: qcm6490-idp: Allow UFS regulators load/mode setting Rakesh Kota
@ 2024-12-26 18:26 ` Bjorn Andersson
0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Andersson @ 2024-12-26 18:26 UTC (permalink / raw)
To: dmitry.baryshkov, konradybcio, robh, krzk+dt, conor+dt,
linux-arm-msm, devicetree, linux-kernel, Rakesh Kota
Cc: quic_kamalw, quic_jprakash
On Thu, 17 Oct 2024 17:58:58 +0530, Rakesh Kota wrote:
> The UFS driver expects to be able to set load (and by extension, mode)
> on its supply regulators. Add the necessary properties to make that
> possible.
>
>
Applied, thanks!
[1/1] arm64: dts: qcom: qcm6490-idp: Allow UFS regulators load/mode setting
commit: abc0c29f5e49f9190b2bb987cd894b5be9cb7469
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-12-26 18:27 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-17 12:28 [PATCH V4] arm64: dts: qcom: qcm6490-idp: Allow UFS regulators load/mode setting Rakesh Kota
2024-12-26 18:26 ` Bjorn Andersson
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).