[PATCH 0/6] drm: debug logging improvements
Switch a handful of places over to drm device based logging and WARNs, unify connector and crtc logging, etc. Jani Nikula (6): drm/modes: add drm_mode_print() to dump mode in drm_printer drm/probe-helper: switch to drm device based logging drm/modes: switch drm_mode_prune_invalid() to use struct drm_printer 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 drivers/gpu/drm/drm_client_modeset.c | 129 +++ drivers/gpu/drm/drm_modes.c | 51 +++ drivers/gpu/drm/drm_probe_helper.c | 41 - drivers/gpu/drm/drm_sysfs.c | 21 +++-- include/drm/drm_modes.h | 2 + 5 files changed, 137 insertions(+), 107 deletions(-) -- 2.39.2
Re: [PATCH 0/6] drm: debug logging improvements
On Thu, 07 Mar 2024, Jani Nikula wrote: > Switch a handful of places over to drm device based logging and WARNs, > unify connector and crtc logging, etc. Ping for review, please. :) > > Jani Nikula (6): > drm/modes: add drm_mode_print() to dump mode in drm_printer > drm/probe-helper: switch to drm device based logging > drm/modes: switch drm_mode_prune_invalid() to use struct drm_printer > 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 > > drivers/gpu/drm/drm_client_modeset.c | 129 +++ > drivers/gpu/drm/drm_modes.c | 51 +++ > drivers/gpu/drm/drm_probe_helper.c | 41 - > drivers/gpu/drm/drm_sysfs.c | 21 +++-- > include/drm/drm_modes.h | 2 + > 5 files changed, 137 insertions(+), 107 deletions(-) -- Jani Nikula, Intel
Re: [PATCH 0/6] drm: debug logging improvements
Hi, with your Oops on [5/6] fixed and my comment on [1/6] considered: Reviewed-by: Thomas Zimmermann Best regards Thomas Am 07.03.24 um 21:39 schrieb Jani Nikula: Switch a handful of places over to drm device based logging and WARNs, unify connector and crtc logging, etc. Jani Nikula (6): drm/modes: add drm_mode_print() to dump mode in drm_printer drm/probe-helper: switch to drm device based logging drm/modes: switch drm_mode_prune_invalid() to use struct drm_printer 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 drivers/gpu/drm/drm_client_modeset.c | 129 +++ drivers/gpu/drm/drm_modes.c | 51 +++ drivers/gpu/drm/drm_probe_helper.c | 41 - drivers/gpu/drm/drm_sysfs.c | 21 +++-- include/drm/drm_modes.h | 2 + 5 files changed, 137 insertions(+), 107 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)
Re: [PATCH 0/6] drm: debug logging improvements
On Fri, 05 Apr 2024, Thomas Zimmermann wrote: > Hi, > > with your Oops on [5/6] fixed and my comment on [1/6] considered: > > Reviewed-by: Thomas Zimmermann Thanks! I ended up dropping a couple of patches based on Ville's comment about mode logging, and added a few more in v2. I tried to preserve your R-b where I think I didn't change much or nothing at all. BR, Jani. > > Best regards > Thomas > > > Am 07.03.24 um 21:39 schrieb Jani Nikula: >> Switch a handful of places over to drm device based logging and WARNs, >> unify connector and crtc logging, etc. >> >> Jani Nikula (6): >>drm/modes: add drm_mode_print() to dump mode in drm_printer >>drm/probe-helper: switch to drm device based logging >>drm/modes: switch drm_mode_prune_invalid() to use struct drm_printer >>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 >> >> drivers/gpu/drm/drm_client_modeset.c | 129 +++ >> drivers/gpu/drm/drm_modes.c | 51 +++ >> drivers/gpu/drm/drm_probe_helper.c | 41 - >> drivers/gpu/drm/drm_sysfs.c | 21 +++-- >> include/drm/drm_modes.h | 2 + >> 5 files changed, 137 insertions(+), 107 deletions(-) >> -- Jani Nikula, Intel