Re: [PATCH 6/6] drm/i915/cursor: Allow joiner cursor fast path update

["Borah, Chaitanya Kumar" <chaitanya.kumar.borah@intel.com>]

On 8/21/2026 10:17 AM, Nemesa Garg wrote:
> 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]
> 

LGTM
Reviewed-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>

> Assisted-by: Claude:claude-sonnet-4.6
> Signed-off-by: Nemesa Garg <nemesa.garg@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 dedb6f87359c..fbf0c9e143a1 100644
> --- a/drivers/gpu/drm/i915/display/intel_cursor.c
> +++ b/drivers/gpu/drm/i915/display/intel_cursor.c
> @@ -909,13 +909,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;
>   
>   	/*