diff mbox

[i-g-t,v2,14/33] tests/kms_legacy_colorkey: Add support for dynamic number of planes

Message ID 20170124233400.5053-15-robert.foss@collabora.com (mailing list archive)
State New, archived
Headers show

Commit Message

Robert Foss Jan. 24, 2017, 11:33 p.m. UTC 
Add changes reflecting the new support for dynamic number of planes per pipe.

Signed-off-by: Robert Foss <robert.foss@collabora.com>
---
 tests/kms_legacy_colorkey.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Mika Kahola Jan. 25, 2017, 12:31 p.m. UTC | #1 
Reviewed-by: Mika Kahola <mika.kahola@intel.com>

On Tue, 2017-01-24 at 18:33 -0500, Robert Foss wrote:
> Add changes reflecting the new support for dynamic number of planes
> per pipe.
> 
> Signed-off-by: Robert Foss <robert.foss@collabora.com>
> ---
>  tests/kms_legacy_colorkey.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/kms_legacy_colorkey.c
> b/tests/kms_legacy_colorkey.c
> index 25f98aad..150520ce 100644
> --- a/tests/kms_legacy_colorkey.c
> +++ b/tests/kms_legacy_colorkey.c
> @@ -55,8 +55,10 @@ igt_simple_main
>  
>  	for_each_pipe(&display, p) {
>  		for_each_plane_on_pipe(&display, p, plane) {
> +			bool is_valid = (plane->type ==
> DRM_PLANE_TYPE_PRIMARY ||
> +			                 plane->type ==
> DRM_PLANE_TYPE_CURSOR);
>  			test_plane(plane->drm_plane->plane_id,
> -				   (plane->is_cursor || plane-
> >is_primary) ? -ENOENT : 0);
> +				   is_valid ? -ENOENT : 0);
>  
>  			max_id = max(max_id, plane->drm_plane-
> >plane_id);
>  		}
diff mbox

Patch

diff --git a/tests/kms_legacy_colorkey.c b/tests/kms_legacy_colorkey.c
index 25f98aad..150520ce 100644
--- a/tests/kms_legacy_colorkey.c
+++ b/tests/kms_legacy_colorkey.c
@@ -55,8 +55,10 @@  igt_simple_main
 
 	for_each_pipe(&display, p) {
 		for_each_plane_on_pipe(&display, p, plane) {
+			bool is_valid = (plane->type == DRM_PLANE_TYPE_PRIMARY ||
+			                 plane->type == DRM_PLANE_TYPE_CURSOR);
 			test_plane(plane->drm_plane->plane_id,
-				   (plane->is_cursor || plane->is_primary) ? -ENOENT : 0);
+				   is_valid ? -ENOENT : 0);
 
 			max_id = max(max_id, plane->drm_plane->plane_id);
 		}