Re: [PATCH] leds: driver for ab5500 high voltage leds

[Mark Brown <broonie@opensource.wolfsonmicro.com>]

On Wed, Dec 14, 2011 at 07:31:55PM +0530, Shreshtha Kumar SAHU wrote:

> As far as I know some boost converters can be controlled dynamically for
> power optimized operation by PMIC. Yes exactly as you stated, I wanted to
> provide something similar as done to boost converters by exposing current
> control via sysfs. Otherwise this current control from sysfs can be
> completely ignored by the user and simply use brightness for LED control.

Well, there's also devices that don't bother with the PWM and only
implement brightness control via the current limit.

> > It'd seem more natural for the driver to scale both the PWM and current
> > settings with brightness.

> I will try to implement this. In AB5500, PWM provides 1024 steps from 0 to
> full current, and current 16 steps from 2.5mA to 40mA. I need to derive a
> way to optmially change both PWM and current to get requested brigtness in
> range of 0-255 (i.e. LED_OFF to LED_FULL).

> Bur I fear that changing both PWM and current may lead to non linear
> variation and equation needs to be derived to get it correct which will
> add complexity to ab5500_led_brightness_set and hence API brightness.
> In addition if platform sets max current to 10mA then brightness_set()
> will have only 4 steps for current variation and in thus an overhead if
> we combined current and PWM control calculation.

> Please suggest.

It doesn't seem like it should be *too* complex a problem to solve,
it'll depend on how the PWM and the current scale.  If they're both nice
smooth log scales it should be easy enough.