| Message ID | 20190215115013.11098-6-stefan@olimex.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | leds: Add AXP20X CHGLED | expand |
On Fri 2019-02-15 13:50:10, Stefan Mavrodiev wrote: > Teres-I and A64-OLinuXino commes with populated LED connected > to AXP803. So to be used as battery indication, pass the LED control > to the charger itself in mode 0 ( FULL while charging, OFF no battery > or completed). > > Signed-off-by: Stefan Mavrodiev <stefan@olimex.com> > --- > arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts | 6 ++++++ > arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts | 6 ++++++ > 2 files changed, 12 insertions(+) > > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts > index f7a4bccaa5d4..7d6930319fba 100644 > --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts > +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts > @@ -177,6 +177,12 @@ > > #include "axp803.dtsi" > > +&axp_led { > + label = "a64-olinuxino:yellow:chgled"; There's little chance userspace will recognize this led. Can you make it "platform:yellow:charging" and fix it for other boards, too? Pavel
On 2/15/19 8:49 PM, Pavel Machek wrote: > On Fri 2019-02-15 13:50:10, Stefan Mavrodiev wrote: >> Teres-I and A64-OLinuXino commes with populated LED connected >> to AXP803. So to be used as battery indication, pass the LED control >> to the charger itself in mode 0 ( FULL while charging, OFF no battery >> or completed). >> >> Signed-off-by: Stefan Mavrodiev <stefan@olimex.com> >> --- >> arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts | 6 ++++++ >> arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts | 6 ++++++ >> 2 files changed, 12 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts >> index f7a4bccaa5d4..7d6930319fba 100644 >> --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts >> +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts >> @@ -177,6 +177,12 @@ >> >> #include "axp803.dtsi" >> >> +&axp_led { >> + label = "a64-olinuxino:yellow:chgled"; > There's little chance userspace will recognize this led. > > Can you make it "platform:yellow:charging" and fix it for other > boards, too? Sure. Best regards, Stefan Mavrodiev > > Pavel
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts index f7a4bccaa5d4..7d6930319fba 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts @@ -177,6 +177,12 @@ #include "axp803.dtsi" +&axp_led { + label = "a64-olinuxino:yellow:chgled"; + status = "okay"; + x-powers,charger-mode = <0>; +}; + ®_aldo1 { regulator-always-on; regulator-min-microvolt = <2800000>; diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts index c455b24dd079..cdffdd4e4561 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts @@ -145,6 +145,12 @@ #include "axp803.dtsi" +&axp_led { + label = "teres-i:yellow:chgled"; + status = "okay"; + x-powers,charger-mode = <0>; +}; + ®_aldo1 { regulator-always-on; regulator-min-microvolt = <2800000>;
Teres-I and A64-OLinuXino commes with populated LED connected to AXP803. So to be used as battery indication, pass the LED control to the charger itself in mode 0 ( FULL while charging, OFF no battery or completed). Signed-off-by: Stefan Mavrodiev <stefan@olimex.com> --- arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts | 6 ++++++ arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts | 6 ++++++ 2 files changed, 12 insertions(+)