Re: [PATCH v4 71/80] drm/vmwgfx: Run DRM default client setup

[Thomas Zimmermann <tzimmermann@suse.de>]

Hi

Am 09.09.24 um 19:52 schrieb Zack Rusin:
> On Mon, Sep 9, 2024 at 7:37 AM Thomas Zimmermann <tzimmermann@suse.de> wrote:
>> Call drm_client_setup() to run the kernel's default client setup
>> for DRM. Set fbdev_probe in struct drm_driver, so that the client
>> setup can start the common fbdev client.
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
>> Cc: Zack Rusin <zack.rusin@broadcom.com>
>> Cc: Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com>
>> Acked-by: Javier Martinez Canillas <javierm@redhat.com>
> Quick note: I love what you did with drm client and drm fbdev. Thanks
> a lot for that work!

Thanks for the kind words. Noralf deserves his share of praise here, as 
he first implemented the client code. And Javier went through *a lot* of 
patches reviewing them.

Once drm_client_setup() has been merged, I hope that there won't be any 
such large patch series. The rest of the work should be internal rework 
that doesn't touch the whole subsystem.

Best regards
Thomas

>
> Reviewed-by: Zack Rusin <zack.rusin@broadcom.com>
>
> z

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)