All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org
Cc: jani.nikula@intel.com, lucas.demarchi@intel.com, rodrigo.vivi@intel.com
Subject: [PATCH 0/3] drm/i915 & drm/xe: allocate struct intel_display dynamically
Date: Wed, 21 May 2025 13:21:45 +0300	[thread overview]
Message-ID: <cover.1747822630.git.jani.nikula@intel.com> (raw)

Allocate struct intel_display dynamically in
intel_display_device_probe(), and make i915->display and xe->display
opaque pointers to most of the non-display parts of the drivers.

The guts of struct intel_display is still accessed directly by some
non-display code, but with the explicit include of the
intel_display_core.h and other display headers, so it's much more
limited than before.

This should also give us a nice boost in incremental builds, as the
entire xe and i915 drivers don't need to be rebuilt when display headers
change.

BR,
Jani.


Jani Nikula (3):
  drm/i915/display: allocate struct intel_display dynamically
  drm/i915: stop including display/intel_display_{core,limits}.h from
    i915_drv.h
  drm/xe: stop including intel_display_{core,device}.h from
    xe_device_types.h

 drivers/gpu/drm/i915/display/intel_bios.c         |  1 +
 .../drm/i915/display/intel_display_conversion.c   |  1 +
 .../gpu/drm/i915/display/intel_display_device.c   | 10 +++++++++-
 .../gpu/drm/i915/display/intel_display_driver.c   |  1 +
 .../gpu/drm/i915/display/intel_display_reset.c    |  1 +
 drivers/gpu/drm/i915/display/intel_display_rpm.c  |  1 +
 drivers/gpu/drm/i915/display/intel_display_rps.c  |  1 +
 drivers/gpu/drm/i915/display/intel_dpt.c          |  1 +
 drivers/gpu/drm/i915/display/intel_encoder.c      |  2 +-
 drivers/gpu/drm/i915/display/intel_fb.c           |  1 +
 drivers/gpu/drm/i915/display/intel_fb_bo.c        |  1 +
 drivers/gpu/drm/i915/display/intel_fb_pin.c       |  1 +
 drivers/gpu/drm/i915/display/intel_fbdev_fb.c     |  1 +
 drivers/gpu/drm/i915/display/intel_hotplug.c      |  1 +
 drivers/gpu/drm/i915/display/intel_opregion.c     |  1 +
 .../gpu/drm/i915/display/intel_plane_initial.c    |  1 +
 .../drm/i915/gem/selftests/i915_gem_client_blt.c  |  1 +
 drivers/gpu/drm/i915/gt/intel_gt_pm.c             |  2 ++
 drivers/gpu/drm/i915/gvt/cmd_parser.c             |  1 +
 drivers/gpu/drm/i915/gvt/display.c                |  1 +
 drivers/gpu/drm/i915/gvt/edid.c                   |  1 +
 drivers/gpu/drm/i915/gvt/fb_decoder.c             |  1 +
 drivers/gpu/drm/i915/gvt/handlers.c               |  1 +
 drivers/gpu/drm/i915/i915_driver.c                | 15 +++++++++------
 drivers/gpu/drm/i915/i915_drv.h                   |  8 +++-----
 drivers/gpu/drm/i915/i915_irq.c                   |  1 +
 drivers/gpu/drm/i915/i915_switcheroo.c            |  2 ++
 drivers/gpu/drm/i915/intel_clock_gating.c         |  1 +
 drivers/gpu/drm/i915/intel_gvt_mmio_table.c       |  1 +
 drivers/gpu/drm/i915/intel_uncore.c               |  2 ++
 drivers/gpu/drm/i915/selftests/mock_gem_device.c  | 13 ++++++++++---
 drivers/gpu/drm/i915/soc/intel_dram.c             |  2 ++
 drivers/gpu/drm/i915/soc/intel_gmch.c             |  2 ++
 drivers/gpu/drm/xe/display/intel_fbdev_fb.c       |  1 +
 drivers/gpu/drm/xe/display/xe_display.c           | 12 ++++++++----
 drivers/gpu/drm/xe/display/xe_display_rpm.c       |  1 +
 drivers/gpu/drm/xe/display/xe_display_wa.c        |  2 +-
 drivers/gpu/drm/xe/display/xe_fb_pin.c            |  1 +
 drivers/gpu/drm/xe/display/xe_plane_initial.c     |  1 +
 drivers/gpu/drm/xe/display/xe_tdf.c               |  4 ++--
 drivers/gpu/drm/xe/xe_device_types.h              |  7 +------
 41 files changed, 81 insertions(+), 29 deletions(-)

-- 
2.39.5


             reply	other threads:[~2025-05-21 10:21 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-21 10:21 Jani Nikula [this message]
2025-05-21 10:21 ` [PATCH 1/3] drm/i915/display: allocate struct intel_display dynamically Jani Nikula
2025-05-21 13:31   ` Rodrigo Vivi
2025-05-21 15:33   ` Lucas De Marchi
2025-05-21 10:21 ` [PATCH 2/3] drm/i915: stop including display/intel_display_{core, limits}.h from i915_drv.h Jani Nikula
2025-05-21 10:21 ` [PATCH 3/3] drm/xe: stop including intel_display_{core, device}.h from xe_device_types.h Jani Nikula
2025-05-21 16:35   ` Lucas De Marchi
2025-05-21 12:18 ` ✓ CI.Patch_applied: success for drm/i915 & drm/xe: allocate struct intel_display dynamically Patchwork
2025-05-21 12:18 ` ✓ CI.checkpatch: " Patchwork
2025-05-21 12:20 ` ✓ CI.KUnit: " Patchwork
2025-05-21 12:30 ` ✓ CI.Build: " Patchwork
2025-05-21 12:33 ` ✓ CI.Hooks: " Patchwork
2025-05-21 12:34 ` ✗ CI.checksparse: warning " Patchwork
2025-05-21 13:04 ` ✗ Xe.CI.BAT: failure " Patchwork
2025-05-21 13:30 ` ✗ Fi.CI.CHECKPATCH: warning " Patchwork
2025-05-21 13:48 ` ✓ i915.CI.BAT: success " Patchwork
2025-05-21 15:49 ` ✓ i915.CI.Full: " Patchwork
2025-05-22  2:15 ` ✗ Xe.CI.Full: failure " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1747822630.git.jani.nikula@intel.com \
    --to=jani.nikula@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=intel-xe@lists.freedesktop.org \
    --cc=lucas.demarchi@intel.com \
    --cc=rodrigo.vivi@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.