2023-01-12 07:59:12

by Johan Hovold

[permalink] [raw]
Subject: [PATCH] arm64: dts: qcom: sc8280xp-crd: allow vreg_l3b to be disabled

The vreg_l3b supply is used by the eDP, UFS and USB1 PHYs which are now
described by the devicetree so that the regulator no longer needs to be
marked always-on.

Signed-off-by: Johan Hovold <[email protected]>
---
arch/arm64/boot/dts/qcom/sc8280xp-crd.dts | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
index db12d8678861..e5e75cc2c670 100644
--- a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
+++ b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
@@ -150,7 +150,6 @@ vreg_l3b: ldo3 {
regulator-max-microvolt = <1200000>;
regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
regulator-boot-on;
- regulator-always-on;
};

vreg_l4b: ldo4 {
--
2.38.2


2023-01-12 15:59:55

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: sc8280xp-crd: allow vreg_l3b to be disabled



On 12.01.2023 08:45, Johan Hovold wrote:
> The vreg_l3b supply is used by the eDP, UFS and USB1 PHYs which are now
> described by the devicetree so that the regulator no longer needs to be
> marked always-on.
>
> Signed-off-by: Johan Hovold <[email protected]>
> ---
Reviewed-by: Konrad Dybcio <[email protected]>

Konrad
> arch/arm64/boot/dts/qcom/sc8280xp-crd.dts | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
> index db12d8678861..e5e75cc2c670 100644
> --- a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
> +++ b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
> @@ -150,7 +150,6 @@ vreg_l3b: ldo3 {
> regulator-max-microvolt = <1200000>;
> regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> regulator-boot-on;
> - regulator-always-on;
> };
>
> vreg_l4b: ldo4 {

2023-01-12 16:30:50

by Bjorn Andersson

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: sc8280xp-crd: allow vreg_l3b to be disabled

On Thu, 12 Jan 2023 08:45:03 +0100, Johan Hovold wrote:
> The vreg_l3b supply is used by the eDP, UFS and USB1 PHYs which are now
> described by the devicetree so that the regulator no longer needs to be
> marked always-on.
>
>

Applied, thanks!

[1/1] arm64: dts: qcom: sc8280xp-crd: allow vreg_l3b to be disabled
commit: e15a815884925177f252a0adfe3b93dece6a75a8

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