Re: [PATCH v7 12/12] led: bd71828: Support LED outputs on ROHM BD71828 PMIC

Pavel Machek
2019-12-21T19:48:07Z

[-- Attachment #1: Type: text/plain, Size: 1168 bytes --]

Hi!

> ROHM BD71828 power management IC has two LED outputs for charge status
> and button pressing indications. The LED outputs can also be forced
> by SW so add driver allowing to use these LEDs for other indications
> as well.
>
> Leds are controlled by SW using 'Force ON' bits. Please note the
> constrains mentioned in data-sheet:
> 1. If one LED is forced ON - then also the other LED is forced.
> => You can't use SW control to force ON one LED and allow HW
> to control the other.
> 2. You can't force both LEDs OFF. If the FORCE bit for both LED's is
> zero, then LEDs are controlled by HW and indicate button/charger
> states as explained in data-sheet.

That's really quite sad, is it?

All the effort and all we got is ... one working LED. Because hardware
does not allow you to control both LEDs...

...and we don't even have support selecting if the LED should be sw or
hw controlled in the mainline, yet...

Best regards,
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]