On Thu, 23 May 2024, Michal Wajdeczko <michal.wajdec...@intel.com> wrote: > All drm_device based logging macros, except those related to WARN, > include the [drm] prefix. Fix that. > > [ ] 0000:00:00.0: this is a warning > [ ] 0000:00:00.0: drm_WARN_ON(true) > vs > [ ] 0000:00:00.0: [drm] this is a warning > [ ] 0000:00:00.0: [drm] drm_WARN_ON(true) > > Signed-off-by: Michal Wajdeczko <michal.wajdec...@intel.com>
Reviewed-by: Jani Nikula <jani.nik...@intel.com> > --- > Cc: Maarten Lankhorst <maarten.lankho...@linux.intel.com> > Cc: Maxime Ripard <mrip...@kernel.org> > Cc: Thomas Zimmermann <tzimmerm...@suse.de> > Cc: Jani Nikula <jani.nik...@intel.com> > --- > include/drm/drm_print.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h > index 089950ad8681..112f8830b372 100644 > --- a/include/drm/drm_print.h > +++ b/include/drm/drm_print.h > @@ -632,12 +632,12 @@ void __drm_err(const char *format, ...); > > /* Helper for struct drm_device based WARNs */ > #define drm_WARN(drm, condition, format, arg...) \ > - WARN(condition, "%s %s: " format, \ > + WARN(condition, "%s %s: [drm] " format, \ > dev_driver_string((drm)->dev), \ > dev_name((drm)->dev), ## arg) > > #define drm_WARN_ONCE(drm, condition, format, arg...) > \ > - WARN_ONCE(condition, "%s %s: " format, \ > + WARN_ONCE(condition, "%s %s: [drm] " format, \ > dev_driver_string((drm)->dev), \ > dev_name((drm)->dev), ## arg) -- Jani Nikula, Intel
