[0/7,v2] drm/exynos: clean up logs

Message

Inki Dae April 15, 2019, 8:54 a.m. UTC

Just clean up logs of Exynos DRM driver.

What this patch series does is to replace the use of existing DRM_DEBUG/ERROR
macros with DRM_DEV_DEBUG*/ERROR* macros including relevant code cleanup.

Changelog v2:
. Clean up debug/error messages of all Exynos DRM KMS and DMA drivers.
. Clean up vidi driver by adding device pointer to vidi_context.
. Clean up ipp driver by using device pointer instead of drm_device one,
  which makes ipp driver to print out the debug messages with correct
  device name.

Inki Dae (6):
  drm/fimd: use DRM_ERROR instead of DRM_INFO in error case
  drm/exynos: remove unnecessary messages
  drm/exynos: use DRM_DEV_ERROR to print out error message
  drm/exynos: use DRM_DEV_DEBUG* instead of DRM_DEBUG macro
  drm/vidi: replace platform_device pointer with device one
  drm/ipp: clean up debug messages

Seung-Woo Kim (1):
  drm/exynos: g2d: remove style error

 drivers/gpu/drm/exynos/exynos5433_drm_decon.c |  6 +-
 drivers/gpu/drm/exynos/exynos7_drm_decon.c    | 26 +++----
 drivers/gpu/drm/exynos/exynos_dp.c            |  9 ++-
 drivers/gpu/drm/exynos/exynos_drm_dma.c       |  2 +-
 drivers/gpu/drm/exynos/exynos_drm_dpi.c       |  9 ++-
 drivers/gpu/drm/exynos/exynos_drm_dsi.c       |  7 +-
 drivers/gpu/drm/exynos/exynos_drm_fb.c        |  9 ++-
 drivers/gpu/drm/exynos/exynos_drm_fbdev.c     | 26 ++++---
 drivers/gpu/drm/exynos/exynos_drm_fimc.c      | 97 ++++++++++++++-------------
 drivers/gpu/drm/exynos/exynos_drm_fimd.c      | 48 +++++++------
 drivers/gpu/drm/exynos/exynos_drm_g2d.c       | 51 +++++++-------
 drivers/gpu/drm/exynos/exynos_drm_gem.c       | 35 +++++-----
 drivers/gpu/drm/exynos/exynos_drm_gsc.c       | 72 ++++++++++----------
 drivers/gpu/drm/exynos/exynos_drm_ipp.c       | 71 ++++++++++++--------
 drivers/gpu/drm/exynos/exynos_drm_ipp.h       |  9 +--
 drivers/gpu/drm/exynos/exynos_drm_mic.c       | 29 ++++----
 drivers/gpu/drm/exynos/exynos_drm_plane.c     | 15 +++--
 drivers/gpu/drm/exynos/exynos_drm_rotator.c   |  5 +-
 drivers/gpu/drm/exynos/exynos_drm_scaler.c    |  5 +-
 drivers/gpu/drm/exynos/exynos_drm_vidi.c      | 49 ++++++++------
 drivers/gpu/drm/exynos/exynos_hdmi.c          | 75 ++++++++++++---------
 drivers/gpu/drm/exynos/exynos_mixer.c         | 43 +++++++-----
 22 files changed, 400 insertions(+), 298 deletions(-)

Comments

Sam Ravnborg April 15, 2019, 9:13 a.m. UTC | #1

Hi Inki

 
> Inki Dae (6):
>   drm/fimd: use DRM_ERROR instead of DRM_INFO in error case
>   drm/exynos: remove unnecessary messages
>   drm/exynos: use DRM_DEV_ERROR to print out error message
>   drm/exynos: use DRM_DEV_DEBUG* instead of DRM_DEBUG macro
>   drm/vidi: replace platform_device pointer with device one
>   drm/ipp: clean up debug messages

In generel plese prefix all patches with "drm/exynos: " as this
is the driver that is updated.
If a patch is specific to a sub-module in the driver mention
this in the subject.
Examples:
drm/exynos: fix log level in fimd
drm/exynos: use device pointer in vidi
drm/exynos: drop irrelevant logging in ipp

	Sam

Inki Dae April 15, 2019, 11:36 p.m. UTC | #2

Hi Sam,

19. 4. 15. 오후 6:13에 Sam Ravnborg 이(가) 쓴 글:
> Hi Inki
> 
>  
>> Inki Dae (6):
>>   drm/fimd: use DRM_ERROR instead of DRM_INFO in error case
>>   drm/exynos: remove unnecessary messages
>>   drm/exynos: use DRM_DEV_ERROR to print out error message
>>   drm/exynos: use DRM_DEV_DEBUG* instead of DRM_DEBUG macro
>>   drm/vidi: replace platform_device pointer with device one
>>   drm/ipp: clean up debug messages
> 
> In generel plese prefix all patches with "drm/exynos: " as this
> is the driver that is updated.

Oops, sorry. It's my mistake. I didn't check and posted them in a hurry.

> If a patch is specific to a sub-module in the driver mention
> this in the subject.
> Examples:
> drm/exynos: fix log level in fimd
> drm/exynos: use device pointer in vidi> drm/exynos: drop irrelevant logging in ipp

I will send them again.

Thanks,
Inki Dae

> 
> 	Sam
> 
>