[PATCH i-g-t v4 3/8] tests/kms_properties: don't check colorop if no plane color pipeline prop

Melissa Wen <[email protected]>
Newsgroups org.freedesktop.lists.igt-dev
Message-ID <[email protected]>
Don't check the colorop properties on planes that do not support them
(for example, on AMD driver cursor planes). If the COLOR_PIPELINE
property of the plane is not available, the plane does not support
colorop pipelines and there is no point in checking the colorop
properties in this case.

Reviewed-by: Alex Hung <[email protected]>
Signed-off-by: Melissa Wen <[email protected]>
---

v2:
- detach different changes from a single commit (Chaitanya)
v3:
- grammar fixes (Alex H.)
---
 tests/kms_properties.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tests/kms_properties.c b/tests/kms_properties.c
index 93bf1ea26..2b4cb152b 100644
--- a/tests/kms_properties.c
+++ b/tests/kms_properties.c
@@ -247,6 +247,10 @@ static void run_colorop_property_tests(igt_display_t *display,
 		     &fb);
 
 	for_each_plane_on_crtc(crtc, plane) {
+		/* skip planes that don't support color pipelines */
+		if (!igt_plane_has_prop(plane, IGT_PLANE_COLOR_PIPELINE))
+			continue;
+
 		igt_info("Testing colorop properties on plane %s.#%d-%s (output: %s)\n",
 			 igt_crtc_name(crtc), plane->index,
 			 kmstest_plane_type_name(plane->type), output->name);
-- 
2.53.0
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.