CC the pour soul with the ecovec board. On Wed, Aug 28, 2019 at 9:33 AM Bartosz Golaszewski <brgl@xxxxxxxx> wrote: > pt., 16 sie 2019 o 17:48 Bartosz Golaszewski <brgl@xxxxxxxx> napisał(a): > > czw., 8 sie 2019 o 10:17 Bartosz Golaszewski <brgl@xxxxxxxx> napisał(a): > > > śr., 24 lip 2019 o 10:25 Bartosz Golaszewski <brgl@xxxxxxxx> napisał(a): > > > > From: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx> > > > > > > > > While working on my other series related to gpio-backlight[1] I noticed > > > > that we could simplify the driver if we made the only user of platform > > > > data use GPIO lookups and device properties. This series tries to do > > > > that. > > > > > > > > The first patch adds all necessary data structures to ecovec24. Patch > > > > 2/7 unifies much of the code for both pdata and non-pdata cases. Patches > > > > 3-4/7 remove unused platform data fields. Last three patches contain > > > > additional improvements for the GPIO backlight driver while we're already > > > > modifying it. > > > > > > > > I don't have access to this HW but hopefully this works. Only compile > > > > tested. > > > > > > > > [1] https://lkml.org/lkml/2019/6/25/900 > > > > > > > > v1 -> v2: > > > > - rebased on top of v5.3-rc1 and adjusted to the recent changes from Andy > > > > - added additional two patches with minor improvements > > > > > > > > v2 -> v3: > > > > - in patch 7/7: used initializers to set values for pdata and dev local vars > > > > > > > > Bartosz Golaszewski (7): > > > > sh: ecovec24: add additional properties to the backlight device > > > > backlight: gpio: simplify the platform data handling > > > > sh: ecovec24: don't set unused fields in platform data > > > > backlight: gpio: remove unused fields from platform data > > > > backlight: gpio: remove dev from struct gpio_backlight > > > > backlight: gpio: remove def_value from struct gpio_backlight > > > > backlight: gpio: use a helper variable for &pdev->dev > > > > > > > > arch/sh/boards/mach-ecovec24/setup.c | 33 ++++++-- > > > > drivers/video/backlight/gpio_backlight.c | 82 +++++--------------- > > > > include/linux/platform_data/gpio_backlight.h | 3 - > > > > 3 files changed, 44 insertions(+), 74 deletions(-) > > > > > > > > -- > > > > 2.21.0 > > > > > > > > > > Hi Rich, Yoshinori, > > > > > > can you ack the sh patches in this series? > > > > > > Bart > > > > Ping. > > Hi, > > any chance of getting this reviewed for sh? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
