mbox series

[00/18] backlight: Constify lcd_ops

Message ID 20240414-video-backlight-lcd-ops-v1-0-9b37fcbf546a@kernel.org
Headers show
Series backlight: Constify lcd_ops | expand

Message

Krzysztof Kozlowski April 14, 2024, 4:35 p.m. UTC
Hi,

Dependencies
============
All further patches depend on the first patch.  Therefore everything
could go via backlight tree (please ack) or via cross-tree pulls. Or
whatever maintainer choose, just coordinate this with backlight.

Best regards,
Krzysztof

---
Krzysztof Kozlowski (18):
      backlight: Constify lcd_ops
      backlight: ams369fg06: Constify lcd_ops
      backlight: corgi_lcd: Constify lcd_ops
      backlight: hx8357: Constify lcd_ops
      backlight: ili922x: Constify lcd_ops
      backlight: ili9320: Constify lcd_ops
      backlight: jornada720_lcd: Constify lcd_ops
      backlight: l4f00242t03: Constify lcd_ops
      backlight: lms283gf05: Constify lcd_ops
      backlight: lms501kf03: Constify lcd_ops
      backlight: ltv350qv: Constify lcd_ops
      backlight: otm3225a: Constify lcd_ops
      backlight: platform_lcd: Constify lcd_ops
      backlight: tdo24m: Constify lcd_ops
      HID: picoLCD: Constify lcd_ops
      fbdev: clps711x: Constify lcd_ops
      fbdev: imx: Constify lcd_ops
      fbdev: omap: lcd_ams_delta: Constify lcd_ops

 drivers/hid/hid-picolcd_lcd.c            | 2 +-
 drivers/video/backlight/ams369fg06.c     | 2 +-
 drivers/video/backlight/corgi_lcd.c      | 2 +-
 drivers/video/backlight/hx8357.c         | 2 +-
 drivers/video/backlight/ili922x.c        | 2 +-
 drivers/video/backlight/ili9320.c        | 2 +-
 drivers/video/backlight/jornada720_lcd.c | 2 +-
 drivers/video/backlight/l4f00242t03.c    | 2 +-
 drivers/video/backlight/lcd.c            | 4 ++--
 drivers/video/backlight/lms283gf05.c     | 2 +-
 drivers/video/backlight/lms501kf03.c     | 2 +-
 drivers/video/backlight/ltv350qv.c       | 2 +-
 drivers/video/backlight/otm3225a.c       | 2 +-
 drivers/video/backlight/platform_lcd.c   | 2 +-
 drivers/video/backlight/tdo24m.c         | 2 +-
 drivers/video/fbdev/clps711x-fb.c        | 2 +-
 drivers/video/fbdev/imxfb.c              | 2 +-
 drivers/video/fbdev/omap/lcd_ams_delta.c | 2 +-
 include/linux/lcd.h                      | 6 +++---
 19 files changed, 22 insertions(+), 22 deletions(-)
---
base-commit: 9ed46da14b9b9b2ad4edb3b0c545b6dbe5c00d39
change-id: 20240414-video-backlight-lcd-ops-276d8439ffb8

Best regards,

Comments

Thomas Zimmermann April 15, 2024, 6:56 a.m. UTC | #1
Hi

for patches 16, 17 and 18:

Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>

Best regards
Thomas

Am 14.04.24 um 18:35 schrieb Krzysztof Kozlowski:
> Hi,
>
> Dependencies
> ============
> All further patches depend on the first patch.  Therefore everything
> could go via backlight tree (please ack) or via cross-tree pulls. Or
> whatever maintainer choose, just coordinate this with backlight.
>
> Best regards,
> Krzysztof
>
> ---
> Krzysztof Kozlowski (18):
>        backlight: Constify lcd_ops
>        backlight: ams369fg06: Constify lcd_ops
>        backlight: corgi_lcd: Constify lcd_ops
>        backlight: hx8357: Constify lcd_ops
>        backlight: ili922x: Constify lcd_ops
>        backlight: ili9320: Constify lcd_ops
>        backlight: jornada720_lcd: Constify lcd_ops
>        backlight: l4f00242t03: Constify lcd_ops
>        backlight: lms283gf05: Constify lcd_ops
>        backlight: lms501kf03: Constify lcd_ops
>        backlight: ltv350qv: Constify lcd_ops
>        backlight: otm3225a: Constify lcd_ops
>        backlight: platform_lcd: Constify lcd_ops
>        backlight: tdo24m: Constify lcd_ops
>        HID: picoLCD: Constify lcd_ops
>        fbdev: clps711x: Constify lcd_ops
>        fbdev: imx: Constify lcd_ops
>        fbdev: omap: lcd_ams_delta: Constify lcd_ops
>
>   drivers/hid/hid-picolcd_lcd.c            | 2 +-
>   drivers/video/backlight/ams369fg06.c     | 2 +-
>   drivers/video/backlight/corgi_lcd.c      | 2 +-
>   drivers/video/backlight/hx8357.c         | 2 +-
>   drivers/video/backlight/ili922x.c        | 2 +-
>   drivers/video/backlight/ili9320.c        | 2 +-
>   drivers/video/backlight/jornada720_lcd.c | 2 +-
>   drivers/video/backlight/l4f00242t03.c    | 2 +-
>   drivers/video/backlight/lcd.c            | 4 ++--
>   drivers/video/backlight/lms283gf05.c     | 2 +-
>   drivers/video/backlight/lms501kf03.c     | 2 +-
>   drivers/video/backlight/ltv350qv.c       | 2 +-
>   drivers/video/backlight/otm3225a.c       | 2 +-
>   drivers/video/backlight/platform_lcd.c   | 2 +-
>   drivers/video/backlight/tdo24m.c         | 2 +-
>   drivers/video/fbdev/clps711x-fb.c        | 2 +-
>   drivers/video/fbdev/imxfb.c              | 2 +-
>   drivers/video/fbdev/omap/lcd_ams_delta.c | 2 +-
>   include/linux/lcd.h                      | 6 +++---
>   19 files changed, 22 insertions(+), 22 deletions(-)
> ---
> base-commit: 9ed46da14b9b9b2ad4edb3b0c545b6dbe5c00d39
> change-id: 20240414-video-backlight-lcd-ops-276d8439ffb8
>
> Best regards,
Krzysztof Kozlowski April 15, 2024, 7:48 a.m. UTC | #2
On 15/04/2024 08:56, Thomas Zimmermann wrote:
> Hi
> 
> for patches 16, 17 and 18:
> 
> Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>

This does not work like this. Toolset will apply your review tag for
EVERYTHING. You must provide tag under each individual patch.

Best regards,
Krzysztof
Daniel Thompson April 15, 2024, 12:58 p.m. UTC | #3
On Sun, Apr 14, 2024 at 06:35:58PM +0200, Krzysztof Kozlowski wrote:
> Hi,
>
> Dependencies
> ============
> All further patches depend on the first patch.  Therefore everything
> could go via backlight tree (please ack) or via cross-tree pulls. Or
> whatever maintainer choose, just coordinate this with backlight.

Thanks for the tidy up.

I've added my Reviewed-by: to all the backlight patches (for Lee) and
I'm happy with the other patches too... but I didn't want my R-b on the
HID and fbdev patches to be confused for an ack.


Daniel.


> ---
> Krzysztof Kozlowski (18):
>       backlight: Constify lcd_ops
>       backlight: ams369fg06: Constify lcd_ops
>       backlight: corgi_lcd: Constify lcd_ops
>       backlight: hx8357: Constify lcd_ops
>       backlight: ili922x: Constify lcd_ops
>       backlight: ili9320: Constify lcd_ops
>       backlight: jornada720_lcd: Constify lcd_ops
>       backlight: l4f00242t03: Constify lcd_ops
>       backlight: lms283gf05: Constify lcd_ops
>       backlight: lms501kf03: Constify lcd_ops
>       backlight: ltv350qv: Constify lcd_ops
>       backlight: otm3225a: Constify lcd_ops
>       backlight: platform_lcd: Constify lcd_ops
>       backlight: tdo24m: Constify lcd_ops
>       HID: picoLCD: Constify lcd_ops
>       fbdev: clps711x: Constify lcd_ops
>       fbdev: imx: Constify lcd_ops
>       fbdev: omap: lcd_ams_delta: Constify lcd_ops
>
>  drivers/hid/hid-picolcd_lcd.c            | 2 +-
>  drivers/video/backlight/ams369fg06.c     | 2 +-
>  drivers/video/backlight/corgi_lcd.c      | 2 +-
>  drivers/video/backlight/hx8357.c         | 2 +-
>  drivers/video/backlight/ili922x.c        | 2 +-
>  drivers/video/backlight/ili9320.c        | 2 +-
>  drivers/video/backlight/jornada720_lcd.c | 2 +-
>  drivers/video/backlight/l4f00242t03.c    | 2 +-
>  drivers/video/backlight/lcd.c            | 4 ++--
>  drivers/video/backlight/lms283gf05.c     | 2 +-
>  drivers/video/backlight/lms501kf03.c     | 2 +-
>  drivers/video/backlight/ltv350qv.c       | 2 +-
>  drivers/video/backlight/otm3225a.c       | 2 +-
>  drivers/video/backlight/platform_lcd.c   | 2 +-
>  drivers/video/backlight/tdo24m.c         | 2 +-
>  drivers/video/fbdev/clps711x-fb.c        | 2 +-
>  drivers/video/fbdev/imxfb.c              | 2 +-
>  drivers/video/fbdev/omap/lcd_ams_delta.c | 2 +-
>  include/linux/lcd.h                      | 6 +++---
>  19 files changed, 22 insertions(+), 22 deletions(-)
> ---
> base-commit: 9ed46da14b9b9b2ad4edb3b0c545b6dbe5c00d39
> change-id: 20240414-video-backlight-lcd-ops-276d8439ffb8
>
> Best regards,
> --
> Krzysztof Kozlowski <krzk@kernel.org>
>
Lee Jones April 16, 2024, 12:17 p.m. UTC | #4
On Mon, 15 Apr 2024, Krzysztof Kozlowski wrote:

> On 15/04/2024 08:56, Thomas Zimmermann wrote:
> > Hi
> > 
> > for patches 16, 17 and 18:
> > 
> > Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
> 
> This does not work like this. Toolset will apply your review tag for
> EVERYTHING. You must provide tag under each individual patch.

And for that to not happen now, you have to resubmit the set. :)