[PATCH 2/3] drm/i915/display: use drm_print_hex_dump() for crtc state dump
Jani Nikula
jani.nikula at intel.com
Thu Dec 5 13:32:08 UTC 2024
On Thu, 05 Dec 2024, Andi Shyti <andi.shyti at linux.intel.com> wrote:
> Hi Jani,
>
> On Thu, Dec 05, 2024 at 11:49:34AM +0200, Jani Nikula wrote:
>> Use the drm_printer based printer to get the device specific printing of
>> the hex dump.
>>
>> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
>
> ...
>
>> -static void
>> -intel_dump_buffer(const char *prefix, const u8 *buf, size_t len)
>> -{
>> - if (!drm_debug_enabled(DRM_UT_KMS))
>> - return;
>
> We lose this check now, anyway,
That now depends on the drm_printer, as it should.
Moreover, intel_crtc_state_dump() already has that check, so this is
completely redundant.
> Reviewed-by: Andi Shyti <andi.shyti at linux.intel.com>
Thanks,
Jani.
>
> Thanks,
> Andi
--
Jani Nikula, Intel
More information about the Intel-gfx
mailing list