lkml.org 
[lkml]   [2020]   [Nov]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2] leds: lm3697: Rename struct into more appropriate name
Hi!

> > From the perspective of Linux you see the two control banks as 2 LED
> >class devices (because you are setting brightness for control banks,
> >not for the LED strips).
>
> The way Marek explains it is correct and the way I wrote the driver
> intially.  There is no direct control of the LEDs only controlling the 2
> banks.
>
> As an example a device can put LED string 1 and 2 on a single bank to
> control the backlight for a display and put LED string 3 on a different bank
> to control the backlight of a keyboard. Like in the Droid and Droid 4
> devices.  2 strings illuminate the display backlight and 1 string
> illuminates the keyboard the display backlight can have a independent
> brightness then the keyboard.
>
> To me the name of the structure does not impose any functional changes just
> semantic changes.  And it just makes it a bit more difficult to back port
> functional fixes as this patch would be made mandatory for cherry picking. 
> But I do not get many requests to back port this driver so it maybe be a
> moot point.

Ok, sorry for the confusion, and .. I believe the code can stay as-is.

Bank is single entity Linux controls, and it does not need to know how
many pins are really controlled on the hardware level.

It will be confusing one way or another.

Best regards,
Pavel
--
http://www.livejournal.com/~pavelmachek
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2020-11-25 12:26    [W:0.368 / U:0.000 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site