2023-12-20 11:00:51

by Komal Bajaj

Subject: [PATCH 0/2] Correcting the voltage setting for vph_pwr

Correcting the voltage setting for vph_pwr for qcm6490-idp and
qcs6490-rb3gen2 boards.

Komal Bajaj (2):
arm64: dts: qcom: qcm6490-idp: Correct the voltage setting for vph_pwr
arm64: dts: qcom: qcs6490-rb3gen2: Correct the voltage setting for
vph_pwr

arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 4 ++--
arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)

--
2.42.0

2023-12-20 11:01:09

by Komal Bajaj

Subject: [PATCH 1/2] arm64: dts: qcom: qcm6490-idp: Correct the voltage setting for vph_pwr

Min and max voltages for vph_pwr should be same, otherwise rpmh
will not probe, so correcting the min and max voltages for vph_pwr.

Fixes: 9af6a9f32ad0 ("arm64: dts: qcom: Add base qcm6490 idp board dts")
Signed-off-by: Komal Bajaj <[email protected]>
---
arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
index 03e97e27d16d..74f5a43d53db 100644
--- a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
+++ b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
@@ -123,8 +123,8 @@ debug_vm_mem: debug-vm@d0600000 {
vph_pwr: vph-pwr-regulator {
compatible = "regulator-fixed";
regulator-name = "vph_pwr";
- regulator-min-microvolt = <2500000>;
- regulator-max-microvolt = <4350000>;
+ regulator-min-microvolt = <3700000>;
+ regulator-max-microvolt = <3700000>;
};
};

--
2.42.0

2023-12-20 11:01:27

by Komal Bajaj

Subject: [PATCH 2/2] arm64: dts: qcom: qcs6490-rb3gen2: Correct the voltage setting for vph_pwr

Min and max voltages for vph_pwr should be same, otherwise rpmh
will not probe, so correcting the min and max voltages for vph_pwr.

Fixes: 04cf333afc75 ("arm64: dts: qcom: Add base qcs6490-rb3gen2 board dts")
Signed-off-by: Komal Bajaj <[email protected]>
---
arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
index 8bb7d13d85f6..ae1632182d7c 100644
--- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
+++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
@@ -124,8 +124,8 @@ debug_vm_mem: debug-vm@d0600000 {
vph_pwr: vph-pwr-regulator {
compatible = "regulator-fixed";
regulator-name = "vph_pwr";
- regulator-min-microvolt = <2500000>;
- regulator-max-microvolt = <4350000>;
+ regulator-min-microvolt = <3700000>;
+ regulator-max-microvolt = <3700000>;
};
};

--
2.42.0

2023-12-20 12:14:03

by Konrad Dybcio

Subject: Re: [PATCH 1/2] arm64: dts: qcom: qcm6490-idp: Correct the voltage setting for vph_pwr

On 20.12.2023 12:00, Komal Bajaj wrote:
> Min and max voltages for vph_pwr should be same, otherwise rpmh
> will not probe, so correcting the min and max voltages for vph_pwr.
>
> Fixes: 9af6a9f32ad0 ("arm64: dts: qcom: Add base qcm6490 idp board dts")
> Signed-off-by: Komal Bajaj <[email protected]>
> ---
2.5V is just a bit low :D

Reviewed-by: Konrad Dybcio <[email protected]>

Konrad

2023-12-20 12:14:55

by Konrad Dybcio

Subject: Re: [PATCH 2/2] arm64: dts: qcom: qcs6490-rb3gen2: Correct the voltage setting for vph_pwr

On 20.12.2023 12:00, Komal Bajaj wrote:
> Min and max voltages for vph_pwr should be same, otherwise rpmh
> will not probe, so correcting the min and max voltages for vph_pwr.
>
> Fixes: 04cf333afc75 ("arm64: dts: qcom: Add base qcs6490-rb3gen2 board dts")
> Signed-off-by: Komal Bajaj <[email protected]>
> ---
Reviewed-by: Konrad Dybcio <[email protected]>

Konrad

2024-02-07 11:52:31

by Komal Bajaj

Subject: Re: [PATCH 0/2] Correcting the voltage setting for vph_pwr

On 12/20/2023 4:30 PM, Komal Bajaj wrote:
> Correcting the voltage setting for vph_pwr for qcm6490-idp and
> qcs6490-rb3gen2 boards.
>
Can you please apply these two patches.

Thanks
Komal

2024-02-09 17:58:25

by Bjorn Andersson

Subject: Re: (subset) [PATCH 0/2] Correcting the voltage setting for vph_pwr

On Wed, 20 Dec 2023 16:30:13 +0530, Komal Bajaj wrote:
> Correcting the voltage setting for vph_pwr for qcm6490-idp and
> qcs6490-rb3gen2 boards.
>
> Komal Bajaj (2):
> arm64: dts: qcom: qcm6490-idp: Correct the voltage setting for vph_pwr
> arm64: dts: qcom: qcs6490-rb3gen2: Correct the voltage setting for
> vph_pwr
>
> [...]

Applied, thanks!

[1/2] arm64: dts: qcom: qcm6490-idp: Correct the voltage setting for vph_pwr
commit: aa56130e88de50773f84de4039c7de81ab783744
[2/2] arm64: dts: qcom: qcs6490-rb3gen2: Correct the voltage setting for vph_pwr
commit: 05f439c0e64b877c1f9cc7f0bed894b6df45d43d

Best regards,
--
Bjorn Andersson <[email protected]>