[Intel-gfx] [PATCH 04/10] drm/i915: Clean up cursor junk from intel_crtc

Chris Wilson chris at chris-wilson.co.uk
Tue Mar 7 22:33:50 UTC 2017


On Tue, Mar 07, 2017 at 05:27:03PM +0200, ville.syrjala at linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> Move cursor_base, cursor_cntl, and cursor_size from intel_crtc
> into intel_plane so that we don't need the crtc for cursor stuff
> so much.
> 
> Also entirely nuke cursor_addr which IMO doesn't provide any benefit
> since it's not actually used by the cursor code itself. I'm not 100%
> sure what the SKL+ DDB is code is after by looking at cursor_addr so
> I just make it do its checks unconditionally. If that's not correct
> then we should likely replace it with somehting like
> plane_state->visible.

Looks ok to me, but I'm going to bow out of checking atomic state and
friends.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list