mbox series

[leds,0/7] leds: pca963x cleanup

Message ID 20200920002500.5851-1-marek.behun@nic.cz
Headers show
Series leds: pca963x cleanup | expand

Message

Marek BehĂșn Sept. 20, 2020, 12:24 a.m. UTC
Hi Pavel,

this is a cleanup of pca963x LED driver, in spirit of previous
patches I sent recently.

This series should apply on your for-next.

Marek

Cc: Peter Meerwald <p.meerwald@bct-electronic.com>
Cc: Ricardo Ribalda <ribalda@kernel.org>
Cc: Zahari Petkov <zahari@balena.io>

Marek BehĂșn (7):
  leds: pca963x: cosmetic: use helper variables, better indentation
  leds: pca963x: use devres LED registering function
  leds: pca963x: cosmetic: rename variables
  leds: pca963x: cosmetic: rename variables
  leds: pca963x: use flexible array
  leds: pca963x: register LEDs immediately after parsing, get rid of
    platdata
  leds: pca963x: use struct led_init_data when registering

 drivers/leds/leds-pca963x.c                | 399 +++++++++------------
 include/linux/platform_data/leds-pca963x.h |  35 --
 2 files changed, 168 insertions(+), 266 deletions(-)
 delete mode 100644 include/linux/platform_data/leds-pca963x.h


base-commit: a0e550dc351ab5fabe8ea86e45b974494a0a6bf8

Comments

Pavel Machek Sept. 22, 2020, 5:12 p.m. UTC | #1
Hi!

> this is a cleanup of pca963x LED driver, in spirit of previous
> patches I sent recently.
> 
> This series should apply on your for-next.

I like the series. Looks good on quick review. Someone testing it
would be very welcome...

Best regards,
									Pavel
Pavel Machek Sept. 24, 2020, 12:16 p.m. UTC | #2
Hi!

> Instead of doing two allocations, allocate only once, by utilizing
> flexible array members.

Thanks, I have applied 1-5 of the series. I would not mind getting
some testing here.

Best regards,
							Pavel