* [PATCH v2] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting
@ 2024-05-24 8:22 Komal Bajaj
2024-05-24 9:29 ` Dmitry Baryshkov
0 siblings, 1 reply; 4+ messages in thread
From: Komal Bajaj @ 2024-05-24 8:22 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley
Cc: Melody Olvera, linux-arm-msm, devicetree, linux-kernel, Komal Bajaj
While adding the USB support, it was found that the configuration
for regulator smps5 was incorrectly set. Upon cross verifying for
all the regulators, found that smps4, smps6 and smps8 are also
incorrectly configured. This patch fixes the same.
In particular -
- smps4 is 1.574V min and 2.04V max
- smps5 is 1.2V min and 1.4V max
- smps6 is 0.382V min and 1.12V max
- smps8 is fixed at 0.752V
Fixes: d1f2cfe2f669 ("arm64: dts: qcom: Add base QDU1000/QRU1000 IDP DTs")
Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com>
---
Changes in v2-
* Updated the commit message as suggested by Krzysztof
* Link to v1: https://lore.kernel.org/linux-arm-msm/20240514131038.28036-1-quic_kbajaj@quicinc.com/
---
arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 16 ++++++++--------
arch/arm64/boot/dts/qcom/qru1000-idp.dts | 16 ++++++++--------
2 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
index 6e129dc123ed..89b84fb0f70a 100644
--- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
+++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
@@ -96,20 +96,20 @@ vreg_s3a_1p05: smps3 {
vreg_s4a_1p8: smps4 {
regulator-name = "vreg_s4a_1p8";
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <1800000>;
+ regulator-min-microvolt = <1574000>;
+ regulator-max-microvolt = <2040000>;
};
vreg_s5a_2p0: smps5 {
regulator-name = "vreg_s5a_2p0";
- regulator-min-microvolt = <1904000>;
- regulator-max-microvolt = <2000000>;
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1400000>;
};
vreg_s6a_0p9: smps6 {
regulator-name = "vreg_s6a_0p9";
- regulator-min-microvolt = <920000>;
- regulator-max-microvolt = <1128000>;
+ regulator-min-microvolt = <382000>;
+ regulator-max-microvolt = <1120000>;
};
vreg_s7a_1p2: smps7 {
@@ -120,8 +120,8 @@ vreg_s7a_1p2: smps7 {
vreg_s8a_1p3: smps8 {
regulator-name = "vreg_s8a_1p3";
- regulator-min-microvolt = <1352000>;
- regulator-max-microvolt = <1352000>;
+ regulator-min-microvolt = <752000>;
+ regulator-max-microvolt = <752000>;
};
vreg_l1a_0p91: ldo1 {
diff --git a/arch/arm64/boot/dts/qcom/qru1000-idp.dts b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
index 2a862c83309e..258483af065b 100644
--- a/arch/arm64/boot/dts/qcom/qru1000-idp.dts
+++ b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
@@ -96,20 +96,20 @@ vreg_s3a_1p05: smps3 {
vreg_s4a_1p8: smps4 {
regulator-name = "vreg_s4a_1p8";
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <1800000>;
+ regulator-min-microvolt = <1574000>;
+ regulator-max-microvolt = <2040000>;
};
vreg_s5a_2p0: smps5 {
regulator-name = "vreg_s5a_2p0";
- regulator-min-microvolt = <1904000>;
- regulator-max-microvolt = <2000000>;
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1400000>;
};
vreg_s6a_0p9: smps6 {
regulator-name = "vreg_s6a_0p9";
- regulator-min-microvolt = <920000>;
- regulator-max-microvolt = <1128000>;
+ regulator-min-microvolt = <382000>;
+ regulator-max-microvolt = <1120000>;
};
vreg_s7a_1p2: smps7 {
@@ -120,8 +120,8 @@ vreg_s7a_1p2: smps7 {
vreg_s8a_1p3: smps8 {
regulator-name = "vreg_s8a_1p3";
- regulator-min-microvolt = <1352000>;
- regulator-max-microvolt = <1352000>;
+ regulator-min-microvolt = <752000>;
+ regulator-max-microvolt = <752000>;
};
vreg_l1a_0p91: ldo1 {
--
2.42.0
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting
2024-05-24 8:22 [PATCH v2] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting Komal Bajaj
@ 2024-05-24 9:29 ` Dmitry Baryshkov
2024-06-13 11:43 ` Komal Bajaj
0 siblings, 1 reply; 4+ messages in thread
From: Dmitry Baryshkov @ 2024-05-24 9:29 UTC (permalink / raw)
To: Komal Bajaj
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Melody Olvera, linux-arm-msm, devicetree,
linux-kernel
On Fri, 24 May 2024 at 11:23, Komal Bajaj <quic_kbajaj@quicinc.com> wrote:
>
> While adding the USB support, it was found that the configuration
> for regulator smps5 was incorrectly set. Upon cross verifying for
> all the regulators, found that smps4, smps6 and smps8 are also
> incorrectly configured. This patch fixes the same.
Nit: see Documentation/process/submitting-patches.rst, "This patch..."
> In particular -
> - smps4 is 1.574V min and 2.04V max
> - smps5 is 1.2V min and 1.4V max
> - smps6 is 0.382V min and 1.12V max
Just for my understanding, will anything further constraint these values?
> - smps8 is fixed at 0.752V
>
> Fixes: d1f2cfe2f669 ("arm64: dts: qcom: Add base QDU1000/QRU1000 IDP DTs")
> Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com>
> ---
> Changes in v2-
> * Updated the commit message as suggested by Krzysztof
> * Link to v1: https://lore.kernel.org/linux-arm-msm/20240514131038.28036-1-quic_kbajaj@quicinc.com/
> ---
>
> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 16 ++++++++--------
> arch/arm64/boot/dts/qcom/qru1000-idp.dts | 16 ++++++++--------
> 2 files changed, 16 insertions(+), 16 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
> index 6e129dc123ed..89b84fb0f70a 100644
> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
> @@ -96,20 +96,20 @@ vreg_s3a_1p05: smps3 {
>
> vreg_s4a_1p8: smps4 {
> regulator-name = "vreg_s4a_1p8";
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <1800000>;
> + regulator-min-microvolt = <1574000>;
> + regulator-max-microvolt = <2040000>;
> };
>
> vreg_s5a_2p0: smps5 {
> regulator-name = "vreg_s5a_2p0";
> - regulator-min-microvolt = <1904000>;
> - regulator-max-microvolt = <2000000>;
> + regulator-min-microvolt = <1200000>;
> + regulator-max-microvolt = <1400000>;
> };
>
> vreg_s6a_0p9: smps6 {
> regulator-name = "vreg_s6a_0p9";
> - regulator-min-microvolt = <920000>;
> - regulator-max-microvolt = <1128000>;
> + regulator-min-microvolt = <382000>;
> + regulator-max-microvolt = <1120000>;
> };
>
> vreg_s7a_1p2: smps7 {
> @@ -120,8 +120,8 @@ vreg_s7a_1p2: smps7 {
>
> vreg_s8a_1p3: smps8 {
> regulator-name = "vreg_s8a_1p3";
> - regulator-min-microvolt = <1352000>;
> - regulator-max-microvolt = <1352000>;
> + regulator-min-microvolt = <752000>;
> + regulator-max-microvolt = <752000>;
> };
>
> vreg_l1a_0p91: ldo1 {
> diff --git a/arch/arm64/boot/dts/qcom/qru1000-idp.dts b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
> index 2a862c83309e..258483af065b 100644
> --- a/arch/arm64/boot/dts/qcom/qru1000-idp.dts
> +++ b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
> @@ -96,20 +96,20 @@ vreg_s3a_1p05: smps3 {
>
> vreg_s4a_1p8: smps4 {
> regulator-name = "vreg_s4a_1p8";
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <1800000>;
> + regulator-min-microvolt = <1574000>;
> + regulator-max-microvolt = <2040000>;
> };
>
> vreg_s5a_2p0: smps5 {
> regulator-name = "vreg_s5a_2p0";
> - regulator-min-microvolt = <1904000>;
> - regulator-max-microvolt = <2000000>;
> + regulator-min-microvolt = <1200000>;
> + regulator-max-microvolt = <1400000>;
> };
>
> vreg_s6a_0p9: smps6 {
> regulator-name = "vreg_s6a_0p9";
> - regulator-min-microvolt = <920000>;
> - regulator-max-microvolt = <1128000>;
> + regulator-min-microvolt = <382000>;
> + regulator-max-microvolt = <1120000>;
> };
>
> vreg_s7a_1p2: smps7 {
> @@ -120,8 +120,8 @@ vreg_s7a_1p2: smps7 {
>
> vreg_s8a_1p3: smps8 {
> regulator-name = "vreg_s8a_1p3";
> - regulator-min-microvolt = <1352000>;
> - regulator-max-microvolt = <1352000>;
> + regulator-min-microvolt = <752000>;
> + regulator-max-microvolt = <752000>;
> };
>
> vreg_l1a_0p91: ldo1 {
> --
> 2.42.0
>
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting
2024-05-24 9:29 ` Dmitry Baryshkov
@ 2024-06-13 11:43 ` Komal Bajaj
2024-06-13 11:51 ` Dmitry Baryshkov
0 siblings, 1 reply; 4+ messages in thread
From: Komal Bajaj @ 2024-06-13 11:43 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Melody Olvera, linux-arm-msm, devicetree,
linux-kernel
On 5/24/2024 2:59 PM, Dmitry Baryshkov wrote:
> On Fri, 24 May 2024 at 11:23, Komal Bajaj <quic_kbajaj@quicinc.com> wrote:
>>
>> While adding the USB support, it was found that the configuration
>> for regulator smps5 was incorrectly set. Upon cross verifying for
>> all the regulators, found that smps4, smps6 and smps8 are also
>> incorrectly configured. This patch fixes the same.
>
> Nit: see Documentation/process/submitting-patches.rst, "This patch..."
Will update the commit message.
>
>> In particular -
>> - smps4 is 1.574V min and 2.04V max
>> - smps5 is 1.2V min and 1.4V max
>> - smps6 is 0.382V min and 1.12V max
>
> Just for my understanding, will anything further constraint these values?
As far as i know, this is going to be final.
currently, usb device is not working without this min/max change.
Thanks
Komal
>
>> - smps8 is fixed at 0.752V
>>
>> Fixes: d1f2cfe2f669 ("arm64: dts: qcom: Add base QDU1000/QRU1000 IDP DTs")
>> Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com>
>> ---
>> Changes in v2-
>> * Updated the commit message as suggested by Krzysztof
>> * Link to v1: https://lore.kernel.org/linux-arm-msm/20240514131038.28036-1-quic_kbajaj@quicinc.com/
>> ---
>>
>> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 16 ++++++++--------
>> arch/arm64/boot/dts/qcom/qru1000-idp.dts | 16 ++++++++--------
>> 2 files changed, 16 insertions(+), 16 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
>> index 6e129dc123ed..89b84fb0f70a 100644
>> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
>> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
>> @@ -96,20 +96,20 @@ vreg_s3a_1p05: smps3 {
>>
>> vreg_s4a_1p8: smps4 {
>> regulator-name = "vreg_s4a_1p8";
>> - regulator-min-microvolt = <1800000>;
>> - regulator-max-microvolt = <1800000>;
>> + regulator-min-microvolt = <1574000>;
>> + regulator-max-microvolt = <2040000>;
>> };
>>
>> vreg_s5a_2p0: smps5 {
>> regulator-name = "vreg_s5a_2p0";
>> - regulator-min-microvolt = <1904000>;
>> - regulator-max-microvolt = <2000000>;
>> + regulator-min-microvolt = <1200000>;
>> + regulator-max-microvolt = <1400000>;
>> };
>>
>> vreg_s6a_0p9: smps6 {
>> regulator-name = "vreg_s6a_0p9";
>> - regulator-min-microvolt = <920000>;
>> - regulator-max-microvolt = <1128000>;
>> + regulator-min-microvolt = <382000>;
>> + regulator-max-microvolt = <1120000>;
>> };
>>
>> vreg_s7a_1p2: smps7 {
>> @@ -120,8 +120,8 @@ vreg_s7a_1p2: smps7 {
>>
>> vreg_s8a_1p3: smps8 {
>> regulator-name = "vreg_s8a_1p3";
>> - regulator-min-microvolt = <1352000>;
>> - regulator-max-microvolt = <1352000>;
>> + regulator-min-microvolt = <752000>;
>> + regulator-max-microvolt = <752000>;
>> };
>>
>> vreg_l1a_0p91: ldo1 {
>> diff --git a/arch/arm64/boot/dts/qcom/qru1000-idp.dts b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
>> index 2a862c83309e..258483af065b 100644
>> --- a/arch/arm64/boot/dts/qcom/qru1000-idp.dts
>> +++ b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
>> @@ -96,20 +96,20 @@ vreg_s3a_1p05: smps3 {
>>
>> vreg_s4a_1p8: smps4 {
>> regulator-name = "vreg_s4a_1p8";
>> - regulator-min-microvolt = <1800000>;
>> - regulator-max-microvolt = <1800000>;
>> + regulator-min-microvolt = <1574000>;
>> + regulator-max-microvolt = <2040000>;
>> };
>>
>> vreg_s5a_2p0: smps5 {
>> regulator-name = "vreg_s5a_2p0";
>> - regulator-min-microvolt = <1904000>;
>> - regulator-max-microvolt = <2000000>;
>> + regulator-min-microvolt = <1200000>;
>> + regulator-max-microvolt = <1400000>;
>> };
>>
>> vreg_s6a_0p9: smps6 {
>> regulator-name = "vreg_s6a_0p9";
>> - regulator-min-microvolt = <920000>;
>> - regulator-max-microvolt = <1128000>;
>> + regulator-min-microvolt = <382000>;
>> + regulator-max-microvolt = <1120000>;
>> };
>>
>> vreg_s7a_1p2: smps7 {
>> @@ -120,8 +120,8 @@ vreg_s7a_1p2: smps7 {
>>
>> vreg_s8a_1p3: smps8 {
>> regulator-name = "vreg_s8a_1p3";
>> - regulator-min-microvolt = <1352000>;
>> - regulator-max-microvolt = <1352000>;
>> + regulator-min-microvolt = <752000>;
>> + regulator-max-microvolt = <752000>;
>> };
>>
>> vreg_l1a_0p91: ldo1 {
>> --
>> 2.42.0
>>
>>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting
2024-06-13 11:43 ` Komal Bajaj
@ 2024-06-13 11:51 ` Dmitry Baryshkov
0 siblings, 0 replies; 4+ messages in thread
From: Dmitry Baryshkov @ 2024-06-13 11:51 UTC (permalink / raw)
To: Komal Bajaj
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Melody Olvera, linux-arm-msm, devicetree,
linux-kernel
On Thu, 13 Jun 2024 at 14:43, Komal Bajaj <quic_kbajaj@quicinc.com> wrote:
>
>
>
> On 5/24/2024 2:59 PM, Dmitry Baryshkov wrote:
> > On Fri, 24 May 2024 at 11:23, Komal Bajaj <quic_kbajaj@quicinc.com> wrote:
> >>
> >> While adding the USB support, it was found that the configuration
> >> for regulator smps5 was incorrectly set. Upon cross verifying for
> >> all the regulators, found that smps4, smps6 and smps8 are also
> >> incorrectly configured. This patch fixes the same.
> >
> > Nit: see Documentation/process/submitting-patches.rst, "This patch..."
>
> Will update the commit message.
>
> >
> >> In particular -
> >> - smps4 is 1.574V min and 2.04V max
> >> - smps5 is 1.2V min and 1.4V max
> >> - smps6 is 0.382V min and 1.12V max
> >
> > Just for my understanding, will anything further constraint these values?
>
> As far as i know, this is going to be final.
Ack
> currently, usb device is not working without this min/max change.
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-06-13 11:51 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-05-24 8:22 [PATCH v2] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting Komal Bajaj
2024-05-24 9:29 ` Dmitry Baryshkov
2024-06-13 11:43 ` Komal Bajaj
2024-06-13 11:51 ` Dmitry Baryshkov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®