[PATCH DRM] drm: msm: Use DRM_DEV_* instead of dev_*
Rob Clark
robdclark at gmail.com
Tue Oct 23 18:46:37 UTC 2018
On Tue, Oct 23, 2018 at 12:41 PM Sean Paul <sean at poorly.run> wrote:
>
> On Mon, Oct 22, 2018 at 08:22:57PM +0100, Chris Wilson wrote:
> > Quoting Mamta Shukla (2018-10-20 18:49:26)
> > > Use DRM_DEV_INFO/ERROR/WARN instead of dev_info/err/debug to generate
> > > drm-formatted specific log messages so that it will be easy to
> > > differentiate in case of multiple instances of driver.
> >
> > Do you not think this is a bit backwards? Both dev_* and DRM_DEV_*
> > include the device id so that multiple instances can be distinguished.
> > What DRM_DEV_* adds is the function name, which for user facing messages
> > should not be required given a self-encapsulating informative message.
>
> "given a self-encapsulating informative message." being the key here. I'm not
> sure about i915, but most other drivers have vague/duplicated log messages which
> rely on the function name for differentiation.
>
agreed about it being nice to have the fxn name in the message.. even
if it doesn't mean anything to the user, it often means something to
the person they send an kernel log to..
Thanks Matma, I've pulled this into msm-next
BR,
-R
More information about the dri-devel
mailing list