Re: [PATCH 3/5] max44000: Support controlling LED current output

2016-04-10 Thread Jonathan Cameron
On 07/04/16 17:21, Crestez Dan Leonard wrote: > This is exposed as an output channel with "led" as an extend_name. > > Other sensors also have support for controlling an external LED. It's > not clear that simply exposing an undecorated output channel is the > correct approach. Agreed that this

Re: [PATCH 3/5] max44000: Support controlling LED current output

2016-04-10 Thread Jonathan Cameron
On 07/04/16 17:21, Crestez Dan Leonard wrote: > This is exposed as an output channel with "led" as an extend_name. > > Other sensors also have support for controlling an external LED. It's > not clear that simply exposing an undecorated output channel is the > correct approach. Agreed that this

[PATCH 3/5] max44000: Support controlling LED current output

2016-04-07 Thread Crestez Dan Leonard
This is exposed as an output channel with "led" as an extend_name. Other sensors also have support for controlling an external LED. It's not clear that simply exposing an undecorated output channel is the correct approach. Signed-off-by: Crestez Dan Leonard ---

[PATCH 3/5] max44000: Support controlling LED current output

2016-04-07 Thread Crestez Dan Leonard
This is exposed as an output channel with "led" as an extend_name. Other sensors also have support for controlling an external LED. It's not clear that simply exposing an undecorated output channel is the correct approach. Signed-off-by: Crestez Dan Leonard --- drivers/iio/light/max44000.c |