Skip to content

drm/i915/gvt: further reduce dependency on display

Jani Nikula intel-gfx intel-xe 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 6 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4drm/i915/display: move enum aux_ch to intel_display_limits.hdrivers/gpu/drm/i915/display/intel_display.h drivers/gpu/drm/i915/display/intel_display_limits.h drivers/gpu/drm/i915/intel_gvt_mmio_table.c+27 -272 replies
2/4drm/i915/gvt: drop dependency on display struct dplldrivers/gpu/drm/i915/gvt/handlers.c+14 -191 reply
3/4drm/i915/gvt: drop dependency on intel_display_types.hdrivers/gpu/drm/i915/gvt/handlers.c+4 -12 replies
4/4drm/i915/gvt: drop unnecessary include of intel_display.hdrivers/gpu/drm/i915/gvt/display.c+0 -11 reply

Reviews trailers per patch

All 4 patches have a Reviewed-by or Acked-by.

PatchSuraj KandpalAnkit Nautiyal
1/4R-b R-b
2/4R-b
3/4R-b
4/4R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/4] drm/i915/gvt: further reduce dependency on display

GVT still includes a handful of core display headers, and uses struct dpll. Migrate away from them, reducing the include "surface" to avoid even accidental usages of display functions or types.

With this, GVT should only use 1) register macros, 2) some enums, 3) the handful of APIs to support register macros.

BR,
Jani.

Jani Nikula (4):
drm/i915/display: move enum aux_ch to intel_display_limits.h drm/i915/gvt: drop dependency on display struct dpll drm/i915/gvt: drop dependency on intel_display_types.h drm/i915/gvt: drop unnecessary include of intel_display.h

 drivers/gpu/drm/i915/display/intel_display.h  | 26 -------------
 .../drm/i915/display/intel_display_limits.h   | 26 +++++++++++++
 drivers/gpu/drm/i915/gvt/display.c            |  1 -
 drivers/gpu/drm/i915/gvt/handlers.c           | 38 +++++++++----------
 drivers/gpu/drm/i915/intel_gvt_mmio_table.c   |  2 +-
 5 files changed, 45 insertions(+), 48 deletions(-)
Signature · 1 line
2.47.3