On Fri, Sep 01, 2017 at 07:15:55AM +0200, Maarten Lankhorst wrote: > Op 31-08-17 om 20:50 schreef Ville Syrjälä: > > On Wed, Aug 23, 2017 at 06:22:23PM +0300, ville.syrjala@xxxxxxxxxxxxxxx wrote: > >> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > >> > >> Eliminate plane->state and crtc->state usage from > >> intel_plane_atomic_check_with_state() and its callers. Instead pass the > >> proper states in or dig them up from the top level atomic state. > >> > >> Note that intel_plane_atomic_check_with_state() itself isn't allowed to > >> use the top level atomic state as there is none when it gets called from > >> the legacy cursor short circuit path. > >> > >> v2: Rename some variables for easier comprehension (Maarten) > >> > >> Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> > >> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > This patch would still need some review love... In the meantime I > > pushed the first three patches from the series. Thanks for the reviews > > thus far. > I love it. > > Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> Thank you. Pushed the rest of the series to dinq. -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx