On Tue, Mar 07, 2017 at 05:27:08PM +0200, ville.syrjala@xxxxxxxxxxxxxxx wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > The cursor code currently ignores fb->pitches[0] (except when creating > the fb itself), and just uses the cursor_width*4 as the stride. Let's > make sure fb->pitches[0] actually matches what we expect it to be. > > We can also relax the stride vs. cursor width relationship on 845/865 > since the stride is programmed separately. The only constraint is that > width*cpp doesn't exceed the stride, and that's already been checked > by the core since it makes sure the entire plane fits within the fb. > > We can also drop the bo size check as that's already checked when > we create the fb. That is the fb is guaranteed to fit within the bo. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
