| Message ID | 20220823112920.352563-5-jouni.hogander@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Fixes for damage clips handling | expand |
> -----Original Message----- > From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of Jouni > Högander > Sent: Tuesday, August 23, 2022 2:29 PM > To: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org > Subject: [Intel-gfx] [PATCH v2 4/4] drm/tests: Set also mock plane src_x, src_y, > src_w and src_h > > We need to set also src_x, src_y, src_w and src_h for the mock plane. > After fix for drm_atomic_helper_damage_iter_init we are using these when > iterating damage_clips. > Reviewed-by: Mika Kahola <mika.kahola@intel.com> > Signed-off-by: Jouni Högander <jouni.hogander@intel.com> > --- > drivers/gpu/drm/tests/drm_damage_helper_test.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/gpu/drm/tests/drm_damage_helper_test.c > b/drivers/gpu/drm/tests/drm_damage_helper_test.c > index bf250bd08d7e..c608ae06f0e3 100644 > --- a/drivers/gpu/drm/tests/drm_damage_helper_test.c > +++ b/drivers/gpu/drm/tests/drm_damage_helper_test.c > @@ -59,6 +59,11 @@ static int drm_damage_helper_init(struct kunit *test) > static void set_plane_src(struct drm_plane_state *state, int x1, int y1, int x2, > int y2) > { > + state->src_x = x1; > + state->src_y = y1; > + state->src_w = x2 - x1; > + state->src_h = y2 - y1; > + > state->src.x1 = x1; > state->src.y1 = y1; > state->src.x2 = x2; > -- > 2.34.1
On 8/23/22 08:29, Jouni Högander wrote: > We need to set also src_x, src_y, src_w and src_h for the mock plane. > After fix for drm_atomic_helper_damage_iter_init we are using these > when iterating damage_clips. > > Signed-off-by: Jouni Högander <jouni.hogander@intel.com> Tested-by: Maíra Canal <mairacanal@riseup.net> Best Regards, - Maíra Canal > --- > drivers/gpu/drm/tests/drm_damage_helper_test.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/gpu/drm/tests/drm_damage_helper_test.c b/drivers/gpu/drm/tests/drm_damage_helper_test.c > index bf250bd08d7e..c608ae06f0e3 100644 > --- a/drivers/gpu/drm/tests/drm_damage_helper_test.c > +++ b/drivers/gpu/drm/tests/drm_damage_helper_test.c > @@ -59,6 +59,11 @@ static int drm_damage_helper_init(struct kunit *test) > static void set_plane_src(struct drm_plane_state *state, int x1, int y1, int x2, > int y2) > { > + state->src_x = x1; > + state->src_y = y1; > + state->src_w = x2 - x1; > + state->src_h = y2 - y1; > + > state->src.x1 = x1; > state->src.y1 = y1; > state->src.x2 = x2;
diff --git a/drivers/gpu/drm/tests/drm_damage_helper_test.c b/drivers/gpu/drm/tests/drm_damage_helper_test.c index bf250bd08d7e..c608ae06f0e3 100644 --- a/drivers/gpu/drm/tests/drm_damage_helper_test.c +++ b/drivers/gpu/drm/tests/drm_damage_helper_test.c @@ -59,6 +59,11 @@ static int drm_damage_helper_init(struct kunit *test) static void set_plane_src(struct drm_plane_state *state, int x1, int y1, int x2, int y2) { + state->src_x = x1; + state->src_y = y1; + state->src_w = x2 - x1; + state->src_h = y2 - y1; + state->src.x1 = x1; state->src.y1 = y1; state->src.x2 = x2;
We need to set also src_x, src_y, src_w and src_h for the mock plane. After fix for drm_atomic_helper_damage_iter_init we are using these when iterating damage_clips. Signed-off-by: Jouni Högander <jouni.hogander@intel.com> --- drivers/gpu/drm/tests/drm_damage_helper_test.c | 5 +++++ 1 file changed, 5 insertions(+)