++
It's already reviewed, but since this touches shared code, extra acks
are still welcomed
On 17.05.2024 18:34, Michal Wajdeczko wrote:
> We already have some drm printk functions that need to duplicate
> a code to get a similar format of the final result, for example:
>
> [ ] :00:00.0: [drm:foo] bar
> [ ] :00:00.0: [drm] foo bar
> [ ] :00:00.0: [drm] *ERROR* foo
>
> Add a generic __drm_dev_vprintk() function that can format the
> final message like all other existing function do and allows us
> to keep the formatting code in one place.
>
> Above also allows to improve drm_dbg_printer() that today lacks
> of outputing symbolic name of the caller, like drm_dbg() does.
>
> v1: https://patchwork.freedesktop.org/series/133749/
> v2: make it static, keep it simple and use braces (Jani)
>
> Michal Wajdeczko (3):
> drm/print: Add generic drm dev printk function
> drm/print: Improve drm_dbg_printer
> drm/i915: Don't use __func__ as prefix for drm_dbg_printer
>
> drivers/gpu/drm/drm_print.c| 53 --
> drivers/gpu/drm/i915/gt/intel_reset.c | 2 +-
> drivers/gpu/drm/i915/gt/selftest_context.c | 2 +-
> include/drm/drm_print.h| 2 +
> 4 files changed, 34 insertions(+), 25 deletions(-)
>
