[PATCH i-g-t v4 6/8] tests/kms_properties: check colorop properties on active color pipelines
Melissa Wen <[email protected]>
| Newsgroups | org.freedesktop.lists.igt-dev |
|---|---|
| Message-ID | <[email protected]> |
A colorop property can only be changed if its color pipeline is active, i.e. the colorop is in the chain of the color pipeline set in the plane COLOR_PIPELINE property and the plane is active. Set each color pipeline active before checking the properties of its colorops, and restore Bypass afterwards. Signed-off-by: Melissa Wen <[email protected]> --- v2: - detach different changes from a single commit (Chaitanya) v3: - move hunk to the previous commit (Chaitanya) --- tests/kms_properties.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/tests/kms_properties.c b/tests/kms_properties.c index c55a271da..764c77963 100644 --- a/tests/kms_properties.c +++ b/tests/kms_properties.c @@ -271,6 +271,9 @@ static void run_colorop_property_tests(igt_display_t *display, for (i = 0; i < plane->num_color_pipelines; ++i) { /* iterate over all colorops in pipeline*/ colorop = plane->color_pipelines[i]; + igt_plane_set_color_pipeline(plane, colorop); + igt_display_commit2(display, COMMIT_ATOMIC); + while (colorop) { igt_info("Testing colorop properties on %s.#%d.#%d-%s (output: %s)\n", igt_crtc_name(crtc), plane->index, @@ -285,6 +288,8 @@ static void run_colorop_property_tests(igt_display_t *display, } } + igt_plane_set_color_pipeline(plane, NULL); + /* only the fb created above needs to go away here */ if (plane->type != DRM_PLANE_TYPE_PRIMARY) { igt_plane_set_fb(plane, NULL); -- 2.53.0