| Message ID | 20170214094112.27426-18-quentin.schulz@free-electrons.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | add support for AXP20X and AXP22X power supply drivers | expand |
On Tue, Feb 14, 2017 at 5:41 PM, Quentin Schulz <quentin.schulz@free-electrons.com> wrote: > The Sinlinx SinA33 has an AXP223 PMIC and a battery connector, thus, we > enable the battery power supply subnode in its Device Tree. > > Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com> > Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com> Battery charger enabled without any description of the associated battery... In this case I think the driver should use the lowest charge current to be safe? Acked-by: Chen-Yu Tsai <wens@csie.org>
On 21/02/2017 05:50, Chen-Yu Tsai wrote: > On Tue, Feb 14, 2017 at 5:41 PM, Quentin Schulz > <quentin.schulz@free-electrons.com> wrote: >> The Sinlinx SinA33 has an AXP223 PMIC and a battery connector, thus, we >> enable the battery power supply subnode in its Device Tree. >> >> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com> >> Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com> > > Battery charger enabled without any description of the associated battery... > In this case I think the driver should use the lowest charge current to be > safe? > Seems fair. Quentin -- Quentin Schulz, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
diff --git a/arch/arm/boot/dts/sun8i-a33-sinlinx-sina33.dts b/arch/arm/boot/dts/sun8i-a33-sinlinx-sina33.dts index bf53408..2fe9299 100644 --- a/arch/arm/boot/dts/sun8i-a33-sinlinx-sina33.dts +++ b/arch/arm/boot/dts/sun8i-a33-sinlinx-sina33.dts @@ -151,6 +151,10 @@ status = "okay"; }; +&battery_power_supply { + status = "okay"; +}; + ®_aldo1 { regulator-always-on; regulator-min-microvolt = <3000000>;