Re: [PATCH v1 0/5] drm/panel: Use dev_ based logging

2020-08-18 Thread Sam Ravnborg
On Sat, Aug 15, 2020 at 02:54:01PM +0200, Sam Ravnborg wrote:
> The drm/panel drivers uses a mixture of DRM_ and dev_ based logging.
> With this patchset all panel drivers are migrated to use dev_ based
> logging as the DRM_ based logging did not add any extra info.
> 
> Drop the now unused include of drm_print.h.
> 
> With this change new panel drivers will be requires to change to dev_
> based logging - so some of the in-flight panel drivers will need trivial
> updates before they are accepted.
> 
> Patch divided in smaller bites to ease review. There is no dependencies
> between the patches.
> 
> Copied a few people that may have input to the move away from DRM_ based
> logging (Daniel (presumeably on vacation), Jani).
> 
>   Sam
> 
> Sam Ravnborg (5):
>   drm/panel: samsung: Use dev_ based logging
>   drm/panel: leadtek: Use dev_ based logging
>   drm/panel: raydium: Use dev_ based logging
>   drm/panel: sitronix: Use dev_ based logging
>   drm/panel: Use dev_ based logging

Thanks for the reviews from Linus and Guido.
Series is now applied to drm-misc-next.

Sam

> 
>  drivers/gpu/drm/panel/panel-boe-himax8279d.c   | 44 
>  drivers/gpu/drm/panel/panel-elida-kd35t133.c   | 51 +-
>  drivers/gpu/drm/panel/panel-feixin-k101-im2ba02.c  | 19 +++--
>  .../gpu/drm/panel/panel-feiyang-fy07024di26a30d.c  | 21 +++---
>  drivers/gpu/drm/panel/panel-ilitek-ili9322.c   |  3 +-
>  drivers/gpu/drm/panel/panel-innolux-p079zca.c  | 31 +++--
>  drivers/gpu/drm/panel/panel-kingdisplay-kd097d04.c | 33 -
>  drivers/gpu/drm/panel/panel-leadtek-ltk050h3146w.c | 58 ++--
>  drivers/gpu/drm/panel/panel-leadtek-ltk500hd1829.c | 49 +
>  drivers/gpu/drm/panel/panel-novatek-nt35510.c  | 40 ---
>  drivers/gpu/drm/panel/panel-orisetech-otm8009a.c   | 13 ++--
>  drivers/gpu/drm/panel/panel-raydium-rm67191.c  | 33 -
>  drivers/gpu/drm/panel/panel-raydium-rm68200.c  | 18 +++--
>  drivers/gpu/drm/panel/panel-ronbo-rb070d30.c   | 16 ++---
>  drivers/gpu/drm/panel/panel-samsung-ld9040.c   |  3 +-
>  drivers/gpu/drm/panel/panel-samsung-s6d16d0.c  | 23 +++---
>  drivers/gpu/drm/panel/panel-samsung-s6e3ha2.c  |  3 +-
>  drivers/gpu/drm/panel/panel-samsung-s6e63j0x03.c   |  3 +-
>  drivers/gpu/drm/panel/panel-samsung-s6e63m0.c  | 22 +++---
>  drivers/gpu/drm/panel/panel-samsung-s6e8aa0.c  |  3 +-
>  drivers/gpu/drm/panel/panel-sitronix-st7701.c  | 10 ++-
>  drivers/gpu/drm/panel/panel-sitronix-st7703.c  | 61 ++--
>  drivers/gpu/drm/panel/panel-sony-acx424akp.c   | 81 
> --
>  drivers/gpu/drm/panel/panel-tpo-tpg110.c   | 38 +-
>  drivers/gpu/drm/panel/panel-truly-nt35597.c| 63 ++---
>  drivers/gpu/drm/panel/panel-visionox-rm69299.c | 41 ---
>  drivers/gpu/drm/panel/panel-xinpeng-xpp055c272.c   | 51 +-
>  27 files changed, 308 insertions(+), 523 deletions(-)
> 
> 
> 
> ___
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH v1 0/5] drm/panel: Use dev_ based logging

2020-08-15 Thread Guido Günther
Hi,
On Sat, Aug 15, 2020 at 02:54:01PM +0200, Sam Ravnborg wrote:
> The drm/panel drivers uses a mixture of DRM_ and dev_ based logging.
> With this patchset all panel drivers are migrated to use dev_ based
> logging as the DRM_ based logging did not add any extra info.
> 
> Drop the now unused include of drm_print.h.
> 
> With this change new panel drivers will be requires to change to dev_
> based logging - so some of the in-flight panel drivers will need trivial
> updates before they are accepted.
> 
> Patch divided in smaller bites to ease review. There is no dependencies
> between the patches.
> 
> Copied a few people that may have input to the move away from DRM_ based
> logging (Daniel (presumeably on vacation), Jani).

Looks good to me and after applying to next-20200824 i couldn't spot any
DRM_ style logging leftovers:

Reviewed-by: Guido Günther 

Cheers,
 -- Guido

> 
>   Sam
> 
> Sam Ravnborg (5):
>   drm/panel: samsung: Use dev_ based logging
>   drm/panel: leadtek: Use dev_ based logging
>   drm/panel: raydium: Use dev_ based logging
>   drm/panel: sitronix: Use dev_ based logging
>   drm/panel: Use dev_ based logging
> 
>  drivers/gpu/drm/panel/panel-boe-himax8279d.c   | 44 
>  drivers/gpu/drm/panel/panel-elida-kd35t133.c   | 51 +-
>  drivers/gpu/drm/panel/panel-feixin-k101-im2ba02.c  | 19 +++--
>  .../gpu/drm/panel/panel-feiyang-fy07024di26a30d.c  | 21 +++---
>  drivers/gpu/drm/panel/panel-ilitek-ili9322.c   |  3 +-
>  drivers/gpu/drm/panel/panel-innolux-p079zca.c  | 31 +++--
>  drivers/gpu/drm/panel/panel-kingdisplay-kd097d04.c | 33 -
>  drivers/gpu/drm/panel/panel-leadtek-ltk050h3146w.c | 58 ++--
>  drivers/gpu/drm/panel/panel-leadtek-ltk500hd1829.c | 49 +
>  drivers/gpu/drm/panel/panel-novatek-nt35510.c  | 40 ---
>  drivers/gpu/drm/panel/panel-orisetech-otm8009a.c   | 13 ++--
>  drivers/gpu/drm/panel/panel-raydium-rm67191.c  | 33 -
>  drivers/gpu/drm/panel/panel-raydium-rm68200.c  | 18 +++--
>  drivers/gpu/drm/panel/panel-ronbo-rb070d30.c   | 16 ++---
>  drivers/gpu/drm/panel/panel-samsung-ld9040.c   |  3 +-
>  drivers/gpu/drm/panel/panel-samsung-s6d16d0.c  | 23 +++---
>  drivers/gpu/drm/panel/panel-samsung-s6e3ha2.c  |  3 +-
>  drivers/gpu/drm/panel/panel-samsung-s6e63j0x03.c   |  3 +-
>  drivers/gpu/drm/panel/panel-samsung-s6e63m0.c  | 22 +++---
>  drivers/gpu/drm/panel/panel-samsung-s6e8aa0.c  |  3 +-
>  drivers/gpu/drm/panel/panel-sitronix-st7701.c  | 10 ++-
>  drivers/gpu/drm/panel/panel-sitronix-st7703.c  | 61 ++--
>  drivers/gpu/drm/panel/panel-sony-acx424akp.c   | 81 
> --
>  drivers/gpu/drm/panel/panel-tpo-tpg110.c   | 38 +-
>  drivers/gpu/drm/panel/panel-truly-nt35597.c| 63 ++---
>  drivers/gpu/drm/panel/panel-visionox-rm69299.c | 41 ---
>  drivers/gpu/drm/panel/panel-xinpeng-xpp055c272.c   | 51 +-
>  27 files changed, 308 insertions(+), 523 deletions(-)
> 
> 
> 
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel