Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

ARM: dts: axp803/axp81x: Drop GPIO LDO pinctrl nodes

The "ldo-io0" and "ldo-io1" regulators are enabled/disabled by toggling
the pinmux between two functions. This happens in the regulator driver.
Setting the pinmux to "ldo" in the DT is inappropriate because it would
enable the regulator before the driver has a chance to set the correct
initial voltage.

Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>
Link: https://lore.kernel.org/r/20220916042751.47906-2-samuel@sholland.org
Signed-off-by: Jernej Skrabec <jernej.skrabec@gmail.com>

authored by

Samuel Holland and committed by
Jernej Skrabec
26737f55 9abf2313

-24
-14
arch/arm/boot/dts/axp81x.dtsi
··· 62 62 compatible = "x-powers,axp813-gpio"; 63 63 gpio-controller; 64 64 #gpio-cells = <2>; 65 - 66 - gpio0_ldo: gpio0-ldo-pin { 67 - pins = "GPIO0"; 68 - function = "ldo"; 69 - }; 70 - 71 - gpio1_ldo: gpio1-ldo-pin { 72 - pins = "GPIO1"; 73 - function = "ldo"; 74 - }; 75 65 }; 76 66 77 67 battery_power_supply: battery-power { ··· 134 144 }; 135 145 136 146 reg_ldo_io0: ldo-io0 { 137 - pinctrl-names = "default"; 138 - pinctrl-0 = <&gpio0_ldo>; 139 147 /* Disable by default to avoid conflicts with GPIO */ 140 148 status = "disabled"; 141 149 }; 142 150 143 151 reg_ldo_io1: ldo-io1 { 144 - pinctrl-names = "default"; 145 - pinctrl-0 = <&gpio1_ldo>; 146 152 /* Disable by default to avoid conflicts with GPIO */ 147 153 status = "disabled"; 148 154 };
-10
arch/arm64/boot/dts/allwinner/axp803.dtsi
··· 25 25 compatible = "x-powers,axp803-gpio", "x-powers,axp813-gpio"; 26 26 gpio-controller; 27 27 #gpio-cells = <2>; 28 - 29 - gpio0_ldo: gpio0-ldo-pin { 30 - pins = "GPIO0"; 31 - function = "ldo"; 32 - }; 33 - 34 - gpio1_ldo: gpio1-ldo-pin { 35 - pins = "GPIO1"; 36 - function = "ldo"; 37 - }; 38 28 }; 39 29 40 30 battery_power_supply: battery-power {