From: Nemesa Garg <nemesa.garg@intel.com>
To: intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org
Cc: chaitanya.kumar.borah@intel.com, Nemesa Garg <nemesa.garg@intel.com>
Subject: [PATCH 5/5] drm/i915/cursor: Allow joiner cursor fast path update
Date: Thu, 27 Aug 2026 22:12:03 +0530 [thread overview]
Message-ID: <20260827164203.2371794-6-nemesa.garg@intel.com> (raw)
In-Reply-To: <20260827164203.2371794-1-nemesa.garg@intel.com>
The legacy cursor path forced all joiner updates to the
slow path by checking joiner_pipes.
Drop the condition so that joiner cursor updates can use
the fast path.
v4: Trimmed to only the joiner_pipes gate drop. [Chaitanya]
Assisted-by: Claude:claude-sonnet-4.6
Signed-off-by: Nemesa Garg <nemesa.garg@intel.com>
Reviewed-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
---
drivers/gpu/drm/i915/display/intel_cursor.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_cursor.c b/drivers/gpu/drm/i915/display/intel_cursor.c
index ec20319834f9..c3ac138f78b4 100644
--- a/drivers/gpu/drm/i915/display/intel_cursor.c
+++ b/drivers/gpu/drm/i915/display/intel_cursor.c
@@ -914,13 +914,10 @@ intel_legacy_cursor_update(struct drm_plane *_plane,
* PSR2 selective fetch also requires the slow path as
* PSR2 plane and transcoder registers can only be updated during
* vblank.
- *
- * FIXME joiner fastpath would be good
*/
if (!crtc_state->hw.active ||
intel_crtc_needs_modeset(crtc_state) ||
- intel_crtc_needs_fastset(crtc_state) ||
- crtc_state->joiner_pipes)
+ intel_crtc_needs_fastset(crtc_state))
goto slow;
/*
--
2.25.1
next prev parent reply other threads:[~2026-08-27 16:44 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-27 16:41 [PATCH 0/5] Enable joiner cursor fast updates Nemesa Garg
2026-08-27 16:41 ` [PATCH 1/5] drm/i915/cursor: Check joiner cursor commit status Nemesa Garg
2026-08-27 16:42 ` [PATCH 2/5] drm/i915/cursor: Add helper to update cursor plane Nemesa Garg
2026-08-27 16:42 ` [PATCH v7 3/5] drm/i915/cursor: Handle secondary cursor state Nemesa Garg
2026-08-28 10:26 ` Borah, Chaitanya Kumar
2026-08-28 11:21 ` Garg, Nemesa
2026-08-27 16:42 ` [PATCH 4/5] drm/i915/cursor: Program secondary cursor planes Nemesa Garg
2026-08-27 16:42 ` Nemesa Garg [this message]
2026-08-27 16:51 ` ✗ CI.checkpatch: warning for Enable joiner cursor fast updates (rev7) Patchwork
2026-08-27 16:52 ` ✓ CI.KUnit: success " Patchwork
2026-08-27 17:47 ` ✗ Xe.CI.BAT: failure " Patchwork
2026-08-28 3:27 ` Garg, Nemesa
2026-08-27 18:25 ` ✓ i915.CI.BAT: success " Patchwork
2026-08-27 19:01 ` ✗ Xe.CI.FULL: failure " Patchwork
2026-08-28 3:28 ` Garg, Nemesa
2026-08-27 23:05 ` ✗ i915.CI.Full: " Patchwork
2026-08-28 12:23 ` ✓ i915.CI.Full: success " Patchwork
-- strict thread matches above, loose matches on Subject: below --
2026-09-01 7:28 [PATCH 0/5] Enable joiner cursor fast updates Nemesa Garg
2026-09-01 7:28 ` [PATCH 5/5] drm/i915/cursor: Allow joiner cursor fast path update Nemesa Garg
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=20260827164203.2371794-6-nemesa.garg@intel.com \
--to=nemesa.garg@intel.com \
--cc=chaitanya.kumar.borah@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
/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.