[Intel-xe] [PATCH 0/7] Encapsulate display

Lucas De Marchi lucas.demarchi at intel.com
Thu Mar 2 01:34:04 UTC 2023


Since the addition of display, the entire driver is built with
additional CFLAGS to allow the i915 types/headers to be used.  This has
already been problematic as the i915_reg_t and i915_mcr_reg_t were
conflated.

Completely removing this dependency is still not achieved as it would
mean making the structs opaque. However since now there is support for
compiling without display, with a few more ifdefs it's possible to make
sure the driver continues to work without those types/headers. So, move
the display-related initialization inside xe_display.c and remove the
CFLAGS when display is disabled.

Lucas De Marchi (7):
  drm/xe: Fix typo persitent->persistent
  drm/xe/display: Fix return code for deferred probing
  drm/xe/display: Rename and clarify xe_display_enable()
  drm/xe/display: Move display sw init to xe_display.c
  drm/xe/display: Move device info initialization to display
  drm/xe/display: Rename GEN13_DISPLAY
  drm/xe: Remove i915 header dependency when building without display

 drivers/gpu/drm/xe/Makefile          |   6 -
 drivers/gpu/drm/xe/xe_device.c       |  75 ++++-------
 drivers/gpu/drm/xe/xe_device.h       |   6 +-
 drivers/gpu/drm/xe/xe_device_types.h |  17 ++-
 drivers/gpu/drm/xe/xe_display.c      | 183 ++++++++++++++++++++++++++-
 drivers/gpu/drm/xe/xe_display.h      |  25 ++--
 drivers/gpu/drm/xe/xe_engine.c       |   6 +-
 drivers/gpu/drm/xe/xe_engine_types.h |   6 +-
 drivers/gpu/drm/xe/xe_execlist.c     |   2 +-
 drivers/gpu/drm/xe/xe_guc_submit.c   |   2 +-
 drivers/gpu/drm/xe/xe_pci.c          |  90 +------------
 11 files changed, 252 insertions(+), 166 deletions(-)

-- 
2.39.0



More information about the Intel-xe mailing list