[PATCH 83/86] drm/{i915,xe}: Run DRM default client setup

Jani Nikula jani.nikula at linux.intel.com
Fri Aug 16 13:18:27 UTC 2024


On Fri, 16 Aug 2024, Thomas Zimmermann <tzimmermann at suse.de> wrote:
> diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
> index 206328387150..7a28396abb25 100644
> --- a/drivers/gpu/drm/xe/xe_device.c
> +++ b/drivers/gpu/drm/xe/xe_device.c
> @@ -17,6 +17,8 @@
>  #include <drm/drm_print.h>
>  #include <drm/xe_drm.h>
>  
> +#include "intel_fbdev.h"
> +
>  #include "display/xe_display.h"
>  #include "instructions/xe_gpu_commands.h"
>  #include "regs/xe_gt_regs.h"
> @@ -267,6 +269,7 @@ static struct drm_driver driver = {
>  
>  	.dumb_create = xe_bo_dumb_create,
>  	.dumb_map_offset = drm_gem_ttm_dumb_map_offset,
> +	INTEL_FBDEV_DRIVER_OPS,
>  #ifdef CONFIG_PROC_FS
>  	.show_fdinfo = xe_drm_client_fdinfo,
>  #endif

Basically xe_device.c should have close to zero idea about display
details, and should not include intel_fbdev.h directly.

There's a xe_display_driver_set_hooks() call that is the right place to
set the driver->fbdev_probe hook.

It's a bit of a bummer in the sense that this prevents struct drm_driver
from being const, but that's how it already is for xe.


BR,
Jani.


-- 
Jani Nikula, Intel


More information about the amd-gfx mailing list