On Thu, 23 May 2024, Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> 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.wajdeczko@xxxxxxxxx> Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> > --- > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> > Cc: Maxime Ripard <mripard@xxxxxxxxxx> > Cc: Thomas Zimmermann <tzimmermann@xxxxxxx> > Cc: Jani Nikula <jani.nikula@xxxxxxxxx> > --- > 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
