[0/3] leds-mt6323: Make driver flexible and cleanups

Message

AngeloGioacchino Del Regno May 16, 2022, 9:42 a.m. UTC

MT6323 is not the only PMIC that has a LEDs controller IP and it was
found that the others do have a compatible register layout, except
for some register offsets.
The logic contained in this driver can be totally reused for other
PMICs as well, so I can't see any reason to keep this specific to
the MT6323 part.

This series brings meaningful platform data to this driver, giving
it flexibility and making it possible and straightforward to add
support for other (older, or newer!) PMICs.

AngeloGioacchino Del Regno (3):
  leds: leds-mt6323: Specify registers and specs in platform data
  leds: leds-mt6323: Open code and drop MT6323_CAL_HW_DUTY macro
  leds: leds-mt6323: Drop MT6323_ prefix from macros and defines

 drivers/leds/leds-mt6323.c | 258 ++++++++++++++++++++++++-------------
 1 file changed, 166 insertions(+), 92 deletions(-)

Comments

AngeloGioacchino Del Regno June 22, 2022, 1:53 p.m. UTC | #1

Il 16/05/22 11:42, AngeloGioacchino Del Regno ha scritto:
> MT6323 is not the only PMIC that has a LEDs controller IP and it was
> found that the others do have a compatible register layout, except
> for some register offsets.
> The logic contained in this driver can be totally reused for other
> PMICs as well, so I can't see any reason to keep this specific to
> the MT6323 part.
> 
> This series brings meaningful platform data to this driver, giving
> it flexibility and making it possible and straightforward to add
> support for other (older, or newer!) PMICs.
> 
> AngeloGioacchino Del Regno (3):
>    leds: leds-mt6323: Specify registers and specs in platform data
>    leds: leds-mt6323: Open code and drop MT6323_CAL_HW_DUTY macro
>    leds: leds-mt6323: Drop MT6323_ prefix from macros and defines
> 
>   drivers/leds/leds-mt6323.c | 258 ++++++++++++++++++++++++-------------
>   1 file changed, 166 insertions(+), 92 deletions(-)
> 

Gentle ping on this series...

Thanks,
Angelo