Re: [PATCH 3/9] MIPS: DTS: CI20: Add parent supplies to ACT8600 regulators
From: H. Nikolaus Schaller
Date: Thu Jun 15 2023 - 03:20:16 EST
> Am 04.06.2023 um 16:56 schrieb Paul Cercueil <paul@xxxxxxxxxxxxxxx>:
>
> Provide parent regulators to the ACT8600 regulators that need one.
>
> Signed-off-by: Paul Cercueil <paul@xxxxxxxxxxxxxxx>
> ---
> arch/mips/boot/dts/ingenic/ci20.dts | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/arch/mips/boot/dts/ingenic/ci20.dts b/arch/mips/boot/dts/ingenic/ci20.dts
> index 5361606c5e13..662796acda41 100644
> --- a/arch/mips/boot/dts/ingenic/ci20.dts
> +++ b/arch/mips/boot/dts/ingenic/ci20.dts
> @@ -242,16 +242,19 @@ regulators {
> vddcore: DCDC1 {
> regulator-min-microvolt = <1100000>;
> regulator-max-microvolt = <1100000>;
> + vp1-supply = <&vcc_33v>;
where is vcc_33v defined?
Unless [7/9] "MIPS: DTS: CI20: Enable support for WiFi / Bluetooth" is applied?
This way the patches in between can't be bisected.
> regulator-always-on;
> };
> vddmem: DCDC2 {
> regulator-min-microvolt = <1500000>;
> regulator-max-microvolt = <1500000>;
> + vp2-supply = <&vcc_33v>;
> regulator-always-on;
> };
> vcc_33: DCDC3 {
> regulator-min-microvolt = <3300000>;
> regulator-max-microvolt = <3300000>;
> + vp3-supply = <&vcc_33v>;
> regulator-always-on;
> };
> vcc_50: SUDCDC_REG4 {
> @@ -262,21 +265,25 @@ vcc_50: SUDCDC_REG4 {
> vcc_25: LDO5 {
> regulator-min-microvolt = <2500000>;
> regulator-max-microvolt = <2500000>;
> + inl-supply = <&vcc_33v>;
> regulator-always-on;
> };
> wifi_io: LDO6 {
> regulator-min-microvolt = <2500000>;
> regulator-max-microvolt = <2500000>;
> + inl-supply = <&vcc_33v>;
> regulator-always-on;
> };
> cim_io_28: LDO7 {
> regulator-min-microvolt = <2800000>;
> regulator-max-microvolt = <2800000>;
> + inl-supply = <&vcc_33v>;
> regulator-always-on;
> };
> cim_io_15: LDO8 {
> regulator-min-microvolt = <1500000>;
> regulator-max-microvolt = <1500000>;
> + inl-supply = <&vcc_33v>;
> regulator-always-on;
> };
> vrtc_18: LDO_REG9 {
> --
> 2.39.2
>