On Mon, Apr 04, 2016 at 06:47:25PM +0300, Marius Vlad wrote: > igt_atomic_prepare_plane_commit() assumes that the framebuffer is always > set-up. > > Signed-off-by: Marius Vlad <[email protected]> > --- > lib/igt_kms.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/igt_kms.c b/lib/igt_kms.c > index 82257a6..30c5b7e 100644 > --- a/lib/igt_kms.c > +++ b/lib/igt_kms.c > @@ -1585,7 +1585,7 @@ igt_atomic_prepare_plane_commit(igt_plane_t *plane, > igt_output_t *output, > igt_atomic_populate_plane_req(req, plane, IGT_PLANE_FB_ID, > fb_id); > } > > - if (plane->position_changed || plane->size_changed) { > + if ((plane->position_changed || plane->size_changed) && fb_id != 0) { > uint32_t src_x = IGT_FIXED(plane->fb->src_x, 0); /* src_x */ > uint32_t src_y = IGT_FIXED(plane->fb->src_y, 0); /* src_y */ > uint32_t src_w = IGT_FIXED(plane->fb->src_w, 0); /* src_w */
Someone should really move the src coordinates from the fb into the plane. -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
