On Tue, Mar 07, 2017 at 05:27:03PM +0200, ville.syrjala@xxxxxxxxxxxxxxx wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > 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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx