* [PATCH] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting
@ 2024-05-14 13:10 Komal Bajaj
2024-05-14 13:27 ` Krzysztof Kozlowski
0 siblings, 1 reply; 5+ messages in thread
From: Komal Bajaj @ 2024-05-14 13:10 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
Fixing the regulator voltages for qdu/qru1000 idp boards.
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>
---
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 related [flat|nested] 5+ messages in thread* Re: [PATCH] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting
2024-05-14 13:10 [PATCH] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting Komal Bajaj
@ 2024-05-14 13:27 ` Krzysztof Kozlowski
2024-05-16 5:52 ` Komal Bajaj
0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-14 13:27 UTC (permalink / raw)
To: Komal Bajaj, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Cc: Melody Olvera, linux-arm-msm, devicetree, linux-kernel
On 14/05/2024 15:10, Komal Bajaj wrote:
> Fixing the regulator voltages for qdu/qru1000 idp boards.
> 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
Wait, why? This looks, at least partially, you are changing from fixed
voltage choice to full range, without clear explanation.
> - 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>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting
2024-05-14 13:27 ` Krzysztof Kozlowski
@ 2024-05-16 5:52 ` Komal Bajaj
2024-05-17 8:50 ` Krzysztof Kozlowski
0 siblings, 1 reply; 5+ messages in thread
From: Komal Bajaj @ 2024-05-16 5:52 UTC (permalink / raw)
To: Krzysztof Kozlowski, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Cc: Melody Olvera, linux-arm-msm, devicetree, linux-kernel
On 5/14/2024 6:57 PM, Krzysztof Kozlowski wrote:
> On 14/05/2024 15:10, Komal Bajaj wrote:
>> Fixing the regulator voltages for qdu/qru1000 idp boards.
>> 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
>
> Wait, why? This looks, at least partially, you are changing from fixed
> voltage choice to full range, without clear explanation.
>
When we started using one of these regulators for USB enablement as sent
in the patch series [1], we saw a sudden reboot.
After adding more debug logs, came to know the configuration for smps5
was incorrect.
Therefore, cross verified the configurations for all the regulators and
got to know that these are incorrectly configured.
This fixes some manual errors introduced in the initial patch (mentioned
in fixes tag).
[1]
https://lore.kernel.org/linux-arm-msm/20240502090326.21489-1-quic_kbajaj@quicinc.com/
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>
>
> Best regards,
> Krzysztof
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting
2024-05-16 5:52 ` Komal Bajaj
@ 2024-05-17 8:50 ` Krzysztof Kozlowski
2024-05-20 12:19 ` Komal Bajaj
0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-17 8:50 UTC (permalink / raw)
To: Komal Bajaj, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Cc: Melody Olvera, linux-arm-msm, devicetree, linux-kernel
On 16/05/2024 07:52, Komal Bajaj wrote:
>
>
> On 5/14/2024 6:57 PM, Krzysztof Kozlowski wrote:
>> On 14/05/2024 15:10, Komal Bajaj wrote:
>>> Fixing the regulator voltages for qdu/qru1000 idp boards.
>>> 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
>>
>> Wait, why? This looks, at least partially, you are changing from fixed
>> voltage choice to full range, without clear explanation.
>>
>
> When we started using one of these regulators for USB enablement as sent
> in the patch series [1], we saw a sudden reboot.
> After adding more debug logs, came to know the configuration for smps5
> was incorrect.
>
> Therefore, cross verified the configurations for all the regulators and
> got to know that these are incorrectly configured.
> This fixes some manual errors introduced in the initial patch (mentioned
> in fixes tag).
>
> [1]
> https://lore.kernel.org/linux-arm-msm/20240502090326.21489-1-quic_kbajaj@quicinc.com/
All this should be explained in commit msg.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting
2024-05-17 8:50 ` Krzysztof Kozlowski
@ 2024-05-20 12:19 ` Komal Bajaj
0 siblings, 0 replies; 5+ messages in thread
From: Komal Bajaj @ 2024-05-20 12:19 UTC (permalink / raw)
To: Krzysztof Kozlowski, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Cc: Melody Olvera, linux-arm-msm, devicetree, linux-kernel
On 5/17/2024 2:20 PM, Krzysztof Kozlowski wrote:
> On 16/05/2024 07:52, Komal Bajaj wrote:
>>
>>
>> On 5/14/2024 6:57 PM, Krzysztof Kozlowski wrote:
>>> On 14/05/2024 15:10, Komal Bajaj wrote:
>>>> Fixing the regulator voltages for qdu/qru1000 idp boards.
>>>> 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
>>>
>>> Wait, why? This looks, at least partially, you are changing from fixed
>>> voltage choice to full range, without clear explanation.
>>>
>>
>> When we started using one of these regulators for USB enablement as sent
>> in the patch series [1], we saw a sudden reboot.
>> After adding more debug logs, came to know the configuration for smps5
>> was incorrect.
>>
>> Therefore, cross verified the configurations for all the regulators and
>> got to know that these are incorrectly configured.
>> This fixes some manual errors introduced in the initial patch (mentioned
>> in fixes tag).
>>
>> [1]
>> https://lore.kernel.org/linux-arm-msm/20240502090326.21489-1-quic_kbajaj@quicinc.com/
>
> All this should be explained in commit msg.
Got it. Will add all this info into the commit message of the next version.
Thanks
Komal
>
> Best regards,
> Krzysztof
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-05-20 12:19 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-14 13:10 [PATCH] arm64: dts: qcom: qdu/qru1000-idp: Fix the voltage setting Komal Bajaj
2024-05-14 13:27 ` Krzysztof Kozlowski
2024-05-16 5:52 ` Komal Bajaj
2024-05-17 8:50 ` Krzysztof Kozlowski
2024-05-20 12:19 ` Komal Bajaj
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).