Re: [PATCH v2 0/7] drm: debug logging improvements

[Thomas Zimmermann <tzimmermann@suse.de>]

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(-)
>

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)