On 05/06/2015 08:02 PM, Konduru, Chandra wrote:
diff --git a/lib/igt_kms.c b/lib/igt_kms.c index b5ba273..0665d70 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -1765,9 +1765,7 @@ void igt_plane_set_fb(igt_plane_t *plane, struct igt_fb *fb) plane->fb = fb; /* hack to keep tests working that don't call igt_plane_set_size() */ if (fb) { - /* set default plane pos/size as fb size */ - plane->crtc_x = 0; - plane->crtc_y = 0;Reason for doing this is to make existing kms tests to run without modification. Otherwise every existing test has to explicitly call igt_plane_set_position to make sure it works. Can you make sure removing above 2 lines doesn't break existing tests?
Hm, before your patch tests could set a plane position, change the fb, and plane position would remain the same. After your patch changing the fb resets the plane position. My patch reverts this behaviour to old one.
So I am not sure in what cases resetting plane position on fb changes helps and which tests?
Which tests do you think should be run? Regards, Tvrtko _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
