Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] tests/kms_atomic: remove zpos < N-planes assert
@ 2023-08-24 16:15 Melissa Wen
  2023-08-24 18:24 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (5 more replies)
  0 siblings, 6 replies; 10+ messages in thread
From: Melissa Wen @ 2023-08-24 16:15 UTC (permalink / raw)
  To: igt-dev, Petri Latvala, Arkadiusz Hiler, Kamil Konieczny,
	Juha-Pekka Heikkila, Bhanuprakash Modem, Ashutosh Dixit
  Cc: Rob Clark, kernel-dev

Generalize plane-immutable-zpos test case by removing not-required
assert. zpos < N-planes is a condition not required by DRM
documentation. Removing this validation doesn't affect the test purpose
on verifying that planes with higher zpos always cover those with lower
zpos. Also, this assert doesn't fit driver's configuration where the
range of plane zpos is just defined by the plane type but its max value
doesn't need to be normalized to the number of planes in a CRTC, for
example, a cursor plane with the highest possible zpos value.

Signed-off-by: Melissa Wen <mwen@igalia.com>
---
 tests/kms_atomic.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c
index 22ddf5633..6986864f3 100644
--- a/tests/kms_atomic.c
+++ b/tests/kms_atomic.c
@@ -458,7 +458,6 @@ plane_immutable_zpos(igt_display_t *display, igt_pipe_t *pipe,
 			continue;
 
 		zpos = igt_plane_get_prop(temp, IGT_PLANE_ZPOS);
-		igt_assert_lt(zpos, n_planes);
 
 		plane_ptr[zpos] = temp;
 	}
-- 
2.40.1

^ permalink raw reply related	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2023-08-29 10:45 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-24 16:15 [igt-dev] [PATCH i-g-t] tests/kms_atomic: remove zpos < N-planes assert Melissa Wen
2023-08-24 18:24 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2023-08-24 18:46 ` [igt-dev] ✓ CI.xeBAT: " Patchwork
2023-08-24 21:26 ` [igt-dev] [PATCH i-g-t] " Rodrigo Siqueira Jordao
2023-08-29 10:44   ` Melissa Wen
2023-08-25  6:33 ` [igt-dev] ✗ Fi.CI.IGT: failure for " Patchwork
2023-08-25 14:55   ` Kamil Konieczny
2023-08-29  4:31     ` Yedireswarapu, SaiX Nandan
2023-08-29  3:58 ` Patchwork
2023-08-29  4:26 ` [igt-dev] ✓ Fi.CI.IGT: success " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox