Re: [PATCH v3 18/23] drm/qxl: remove dead qxl fbdev emulation code

From: Noralf TrÃnnes
Date: Fri Jan 25 2019 - 13:44:31 EST




Den 25.01.2019 19.10, skrev Sam Ravnborg:
> Hi Noralf.
>
>>> Lovely diffstat, thanks to the new generic fbdev emulation.
>>>
>>> drm/qxl/Makefile | 2
>>> drm/qxl/qxl_draw.c | 232 ----------------------------------------
>>> drm/qxl/qxl_drv.h | 21 ---
>>> drm/qxl/qxl_fb.c | 300 -----------------------------------------------------
>>>
>>> Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx>
>>> ---
>>> drivers/gpu/drm/qxl/qxl_drv.h | 21 ---
>>> drivers/gpu/drm/qxl/qxl_draw.c | 232 -------------------------------
>>> drivers/gpu/drm/qxl/qxl_fb.c | 300 -----------------------------------------
>>> drivers/gpu/drm/qxl/Makefile | 2 +-
>>> 4 files changed, 1 insertion(+), 554 deletions(-)
>>> delete mode 100644 drivers/gpu/drm/qxl/qxl_fb.c
>>>
>>
>> Nice!
>>
>> Acked-by: Noralf TrÃnnes <noralf@xxxxxxxxxxx>
> It must be a very satisfying experience to see such benefits
> of the many hours of works you have put into the genric
> fbdev emulation code.

Indeed and having more users of the code increases the chance of
detecting any bugs lurking underneath the surface.

> Well done, and indeed a nice patch from Gerd.
>
> Sam
>