mbox series

[v2,0/7] drm: debug logging improvements

Message ID cover.1712568037.git.jani.nikula@intel.com (mailing list archive)
Headers show
Series drm: debug logging improvements | expand

Message

Jani Nikula April 8, 2024, 9:23 a.m. UTC 
v2 of [1], dropping drm_mode_print() stuff altogether, and switching to
DRM_MODE_FMT/DRM_MODE_ARG() in a separate patch. Also add a few more drm
device based logging conversion patches, so the last patch makes more sense.

BR,
Jani.

[1] https://patchwork.freedesktop.org/series/130881/

Jani Nikula (7):
  drm/probe-helper: switch to drm device based logging
  drm/modes: switch to drm device based error logging
  drm/sysfs: switch to drm device based logging
  drm/client: switch to drm device based logging, and more
  drm/crtc: switch to drm device based logging
  drm/crtc-helper: switch to drm device based logging and warns
  drm: prefer DRM_MODE_FMT/ARG over drm_mode_debug_printmodeline()

 drivers/gpu/drm/drm_atomic_uapi.c    |   6 +-
 drivers/gpu/drm/drm_client_modeset.c | 129 +++++++++++++++------------
 drivers/gpu/drm/drm_crtc.c           |  38 ++++----
 drivers/gpu/drm/drm_crtc_helper.c    | 100 +++++++++++----------
 drivers/gpu/drm/drm_modes.c          |  40 ++++-----
 drivers/gpu/drm/drm_probe_helper.c   |  39 ++++----
 drivers/gpu/drm/drm_sysfs.c          |  20 ++---
 7 files changed, 193 insertions(+), 179 deletions(-)

Comments

Thomas Zimmermann April 8, 2024, 11:15 a.m. UTC | #1 
Hi

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

for patches 5 to 7.

Best regards
Thomas

Am 08.04.24 um 11:23 schrieb Jani Nikula:
> v2 of [1], dropping drm_mode_print() stuff altogether, and switching to
> DRM_MODE_FMT/DRM_MODE_ARG() in a separate patch. Also add a few more drm
> device based logging conversion patches, so the last patch makes more sense.
>
> BR,
> Jani.
>
> [1] https://patchwork.freedesktop.org/series/130881/
>
> Jani Nikula (7):
>    drm/probe-helper: switch to drm device based logging
>    drm/modes: switch to drm device based error logging
>    drm/sysfs: switch to drm device based logging
>    drm/client: switch to drm device based logging, and more
>    drm/crtc: switch to drm device based logging
>    drm/crtc-helper: switch to drm device based logging and warns
>    drm: prefer DRM_MODE_FMT/ARG over drm_mode_debug_printmodeline()
>
>   drivers/gpu/drm/drm_atomic_uapi.c    |   6 +-
>   drivers/gpu/drm/drm_client_modeset.c | 129 +++++++++++++++------------
>   drivers/gpu/drm/drm_crtc.c           |  38 ++++----
>   drivers/gpu/drm/drm_crtc_helper.c    | 100 +++++++++++----------
>   drivers/gpu/drm/drm_modes.c          |  40 ++++-----
>   drivers/gpu/drm/drm_probe_helper.c   |  39 ++++----
>   drivers/gpu/drm/drm_sysfs.c          |  20 ++---
>   7 files changed, 193 insertions(+), 179 deletions(-)
>
Jani Nikula April 15, 2024, 1:44 p.m. UTC | #2 
On Mon, 08 Apr 2024, Jani Nikula <jani.nikula@intel.com> wrote:
> v2 of [1], dropping drm_mode_print() stuff altogether, and switching to
> DRM_MODE_FMT/DRM_MODE_ARG() in a separate patch. Also add a few more drm
> device based logging conversion patches, so the last patch makes more sense.

Thanks for the reviews, pushed the lot to drm-misc-next.

BR,
Jani.

>
> BR,
> Jani.
>
> [1] https://patchwork.freedesktop.org/series/130881/
>
> Jani Nikula (7):
>   drm/probe-helper: switch to drm device based logging
>   drm/modes: switch to drm device based error logging
>   drm/sysfs: switch to drm device based logging
>   drm/client: switch to drm device based logging, and more
>   drm/crtc: switch to drm device based logging
>   drm/crtc-helper: switch to drm device based logging and warns
>   drm: prefer DRM_MODE_FMT/ARG over drm_mode_debug_printmodeline()
>
>  drivers/gpu/drm/drm_atomic_uapi.c    |   6 +-
>  drivers/gpu/drm/drm_client_modeset.c | 129 +++++++++++++++------------
>  drivers/gpu/drm/drm_crtc.c           |  38 ++++----
>  drivers/gpu/drm/drm_crtc_helper.c    | 100 +++++++++++----------
>  drivers/gpu/drm/drm_modes.c          |  40 ++++-----
>  drivers/gpu/drm/drm_probe_helper.c   |  39 ++++----
>  drivers/gpu/drm/drm_sysfs.c          |  20 ++---
>  7 files changed, 193 insertions(+), 179 deletions(-)