Hi Ruben,

I am looking to work on a small starter task in DRM and was considering
converting logging in drivers/gpu/drm/gud from dev_info/dev_warn/dev_err
to the drm_*() helpers where a struct drm_device is already (or readily)
available.

Since GUD is a USB-backed DRM driver and can have multiple active
instances, using drm_*() would make log messages instance-aware and
consistent with DRM drivers.

Before preparing a patch series, I wanted to check whether this kind of
conversion is welcome for GUD, and whether there are any parts of the driver
you would prefer to keep using dev_*() instead of drm_*().

If this is fine, I plan to keep the changes mechanical and scoped only to
call sites where drm_device is already in scope, preserving logging
semantics.

Thanks,
Hardik

Reply via email to