Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Jani Nikula <jani.nikula@intel.com>
Cc: intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org
Subject: Re: [PATCH 0/9] drm/i915/display: conversions to struct intel_display
Date: Tue, 11 Mar 2025 21:13:32 +0200	[thread overview]
Message-ID: <Z9CLXIAsbXirre3c@intel.com> (raw)
In-Reply-To: <cover.1741715981.git.jani.nikula@intel.com>

On Tue, Mar 11, 2025 at 08:00:34PM +0200, Jani Nikula wrote:
> Primarily convert all hotplug and irq code to struct intel_display, and
> then some.
> 
> Jani Nikula (9):
>   drm/i915/color: prefer display->platform.<platform> checks
>   drm/i915/connector: convert intel_connector.c to struct intel_display
>   drm/i915/hotplug: convert intel_hotplug.[ch] to struct intel_display
>   drm/i915/hotplug: convert hotplug debugfs to struct intel_display
>   drm/i915/hotplug: convert hotplug irq handling to intel_de_*()
>   drm/i915/hotplug: convert intel_hotplug_irq.[ch] to struct
>     intel_display
>   drm/i915/irq: convert intel_display_irq.[ch] interfaces to struct
>     intel_display
>   drm/i915/irq: convert rest of intel_display_irq.[ch] to struct
>     intel_display
>   drm/i915/display: rename I915_HAS_HOTPLUG() to HAS_HOTPLUG

Looked reasonable to me. Series is
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

> 
>  drivers/gpu/drm/i915/display/i9xx_plane.c     |  24 +-
>  drivers/gpu/drm/i915/display/intel_color.c    |  16 +-
>  .../gpu/drm/i915/display/intel_connector.c    |  45 +-
>  drivers/gpu/drm/i915/display/intel_crt.c      |  10 +-
>  .../drm/i915/display/intel_display_debugfs.c  |   2 +-
>  .../drm/i915/display/intel_display_device.h   |   2 +-
>  .../drm/i915/display/intel_display_driver.c   |  16 +-
>  .../gpu/drm/i915/display/intel_display_irq.c  | 742 +++++++++---------
>  .../gpu/drm/i915/display/intel_display_irq.h  |  75 +-
>  .../i915/display/intel_display_power_well.c   |  18 +-
>  .../drm/i915/display/intel_display_reset.c    |   4 +-
>  drivers/gpu/drm/i915/display/intel_dp.c       |   2 +-
>  .../drm/i915/display/intel_fifo_underrun.c    |  27 +-
>  drivers/gpu/drm/i915/display/intel_hotplug.c  | 348 ++++----
>  drivers/gpu/drm/i915/display/intel_hotplug.h  |  26 +-
>  .../gpu/drm/i915/display/intel_hotplug_irq.c  | 623 +++++++--------
>  .../gpu/drm/i915/display/intel_hotplug_irq.h  |  28 +-
>  drivers/gpu/drm/i915/display/intel_pipe_crc.c |   3 +-
>  drivers/gpu/drm/i915/display/intel_sdvo.c     |   2 +-
>  drivers/gpu/drm/i915/display/intel_tv.c       |   4 +-
>  .../drm/i915/display/skl_universal_plane.c    |   6 +-
>  drivers/gpu/drm/i915/gt/intel_rps.c           |   6 +-
>  drivers/gpu/drm/i915/i915_driver.c            |  14 +-
>  drivers/gpu/drm/i915/i915_irq.c               | 123 +--
>  drivers/gpu/drm/xe/display/xe_display.c       |  38 +-
>  25 files changed, 1118 insertions(+), 1086 deletions(-)
> 
> -- 
> 2.39.5

-- 
Ville Syrjälä
Intel

  parent reply	other threads:[~2025-03-11 19:13 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-03-11 18:00 [PATCH 0/9] drm/i915/display: conversions to struct intel_display Jani Nikula
2025-03-11 18:00 ` [PATCH 1/9] drm/i915/color: prefer display->platform.<platform> checks Jani Nikula
2025-03-12  5:01   ` Shankar, Uma
2025-03-11 18:00 ` [PATCH 2/9] drm/i915/connector: convert intel_connector.c to struct intel_display Jani Nikula
2025-03-12  5:06   ` Shankar, Uma
2025-03-11 18:00 ` [PATCH 3/9] drm/i915/hotplug: convert intel_hotplug.[ch] " Jani Nikula
2025-03-12  5:13   ` Shankar, Uma
2025-03-11 18:00 ` [PATCH 4/9] drm/i915/hotplug: convert hotplug debugfs " Jani Nikula
2025-03-12  5:17   ` Shankar, Uma
2025-03-11 18:00 ` [PATCH 5/9] drm/i915/hotplug: convert hotplug irq handling to intel_de_*() Jani Nikula
2025-03-12  5:23   ` Shankar, Uma
2025-03-11 18:00 ` [PATCH 6/9] drm/i915/hotplug: convert intel_hotplug_irq.[ch] to struct intel_display Jani Nikula
2025-03-12  5:33   ` Shankar, Uma
2025-03-11 18:00 ` [PATCH 7/9] drm/i915/irq: convert intel_display_irq.[ch] interfaces " Jani Nikula
2025-03-11 19:15   ` Ville Syrjälä
2025-03-12  9:52     ` Jani Nikula
2025-03-12 10:06       ` Ville Syrjälä
2025-03-12  5:47   ` Shankar, Uma
2025-03-11 18:00 ` [PATCH 8/9] drm/i915/irq: convert rest of intel_display_irq.[ch] " Jani Nikula
2025-03-12  5:53   ` Shankar, Uma
2025-03-11 18:00 ` [PATCH 9/9] drm/i915/display: rename I915_HAS_HOTPLUG() to HAS_HOTPLUG Jani Nikula
2025-03-12  6:39   ` Shankar, Uma
2025-03-11 19:13 ` Ville Syrjälä [this message]
2025-03-11 20:19 ` ✓ CI.Patch_applied: success for drm/i915/display: conversions to struct intel_display (rev2) Patchwork
2025-03-11 20:20 ` ✗ CI.checkpatch: warning " Patchwork
2025-03-11 20:21 ` ✓ CI.KUnit: success " Patchwork
2025-03-11 20:38 ` ✓ CI.Build: " Patchwork
2025-03-11 20:40 ` ✓ CI.Hooks: " Patchwork
2025-03-11 20:42 ` ✓ CI.checksparse: " Patchwork
2025-03-11 21:06 ` ✓ Xe.CI.BAT: " Patchwork
2025-03-12 12:03 ` ✗ 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=Z9CLXIAsbXirre3c@intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=intel-xe@lists.freedesktop.org \
    --cc=jani.nikula@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox