linux-kernel - Re: [linux-sunxi] Re: [PATCH 0/4] Add support for charger LED for AXP813 and TBS A711 Tablet

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1464324b-2b69-c20e-4af8-f32ae5451ea6@olimex.com>
Date:   Mon, 24 Feb 2020 08:31:06 +0200
From:   Stefan Mavrodiev <stefan@...mex.com>
To:     linux-sunxi@...glegroups.com,
        Jacek Anaszewski <jacek.anaszewski@...il.com>,
        Pavel Machek <pavel@....cz>, Dan Murphy <dmurphy@...com>,
        Rob Herring <robh+dt@...nel.org>, Chen-Yu Tsai <wens@...e.org>,
        Maxime Ripard <mripard@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Lee Jones <lee.jones@...aro.org>, linux-leds@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org
Subject: Re: [linux-sunxi] Re: [PATCH 0/4] Add support for charger LED for
 AXP813 and TBS A711 Tablet


On 2/23/20 3:35 PM, Ondřej Jirman wrote:
> On Sun, Feb 23, 2020 at 02:27:30PM +0100, megous hlavni wrote:
>> On Sun, Feb 23, 2020 at 02:14:31PM +0100, megous hlavni wrote:
>>> The tablet has a charger LED exposed on the top. This LED is controlled
>>> by AXP813 PMIC. Add support for enabling the LED and using it either
>>> for charging indication (handled by PMIC automatically) or for other uses
>>> via user control.
>> Dang, I just noticed someone sent a similar driver recently, although I had this
>> one prepared for quite some time (since 2017) in my tree. I guess I should have
>> sent it earlier.
>>
>> Please ignore.
> Though the meaning of "recently" is a bit relative. The other work was sent in
> a year ago. Here's a reference:
>
>    https://lore.kernel.org/patchwork/cover/1042764/

Hi,

I'm the author of the 'other' work. I don't know the full story here, 
but I don't
mind someone else submitting this patch as his.

When I submitted the last patch, there was the proposal to use the 
ledtrig-pattern instead
of sysfs entries. Also AXP209 has inverted CTRL bit.

Please read the the 'other' discussion.

Best regards,
Stefan Mavrodiev

>
>> regards,
>> 	o.
>>
>>
>>> Please take a look.
>>>
>>> thank you and regards,
>>>    Ondrej Jirman
>>>
>>> Ondrej Jirman (4):
>>>    dt-bindings: leds: Add a binding for AXP813 charger led
>>>    leds: axp20x: Support charger LED on AXP20x like PMICs
>>>    ARM: dts: axp813: Add charger LED
>>>    ARM: dts: sun8i-a83t-tbs-a711: Enable charging LED
>>>
>>>   .../devicetree/bindings/leds/leds-axp20x.yaml |  24 ++
>>>   arch/arm/boot/dts/axp81x.dtsi                 |   5 +
>>>   arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts     |   4 +
>>>   drivers/leds/Kconfig                          |   7 +
>>>   drivers/leds/Makefile                         |   1 +
>>>   drivers/leds/leds-axp20x.c                    | 240 ++++++++++++++++++
>>>   drivers/mfd/axp20x.c                          |   3 +
>>>   7 files changed, 284 insertions(+)
>>>   create mode 100644 Documentation/devicetree/bindings/leds/leds-axp20x.yaml
>>>   create mode 100644 drivers/leds/leds-axp20x.c
>>>
>>> -- 
>>> 2.25.1
>>>