[RFC][PATCH 0/2] drm/panic: Add a drm panic handler

[Javier Martinez Canillas]

Thomas Zimmermann <tzimmermann at suse.de> writes:

Hello Jocelyn and Thomas,

> Hi Jocelyn
>
> Am 05.09.23 um 16:46 schrieb Jocelyn Falempe:

[...]

>> 
>> I'm a bit reluctant to re-use the fbdev code, for a few reasons:
>>   * I want drm_panic to work if CONFIG_FB and CONFIG_DRM_FBDEV_EMULATION 
>> are not set.
>
> The code would live in video/ and be independend from fbdev. It's quite 
> a bit of refactoring, but might be worth it in the long term.
>

FWIW, I agree with Thomas here. The drawing/blitting logic seems to be
useful regardless of fbdev and moving it to video/ makes sense to me,
instead of adding yet another infrastructure to do the same that will
only be used for the DRM panic handler.

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat