[Intel-gfx] [PATCH] kms_universal_plane: subtest for yuv pixel format in primary plane

Kumar, Mahesh mahesh1.kumar at intel.com
Mon Jul 20 08:00:41 PDT 2015


This test commit YUV framebuffer in primay plane. I'm using empty FB,
because this fulfills the purpose of testing YUV.

Increase IGT_MAX_PLANES to 5 and add enum for IGT_PLANE_4, as in Broxton
we have 4Plane+1cursor plane.

Signed-off-by: Kumar, Mahesh <mahesh1.kumar at intel.com>
---
 lib/igt_fb.c                |  4 +++
 lib/igt_kms.c               |  1 +
 lib/igt_kms.h               |  3 +-
 tests/kms_universal_plane.c | 82 +++++++++++++++++++++++++++++++++++++++++++++
 4 files changed, 89 insertions(+), 1 deletion(-)

diff --git a/lib/igt_fb.c b/lib/igt_fb.c
index 134dbd2..e570bfa 100644
--- a/lib/igt_fb.c
+++ b/lib/igt_fb.c
@@ -66,6 +66,10 @@ static struct format_desc_struct {
 	DF(XRGB8888,	RGB24,		32, 24),
 	DF(XRGB2101010,	RGB30,		32, 30),
 	DF(ARGB8888,	ARGB32,		32, 32),
+	DF(YUYV,	INVALID,	16, 16),
+	DF(YVYU,	INVALID,	16, 16),
+	DF(UYVY,	INVALID,	16, 16),
+	DF(VYUY,	INVALID,	16, 16),
 };
 #undef DF
 
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 0bb16b4..781ffa5 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -213,6 +213,7 @@ const char *kmstest_plane_name(enum igt_plane plane)
 		[IGT_PLANE_1] = "plane1",
 		[IGT_PLANE_2] = "plane2",
 		[IGT_PLANE_3] = "plane3",
+		[IGT_PLANE_4] = "plane4",
 		[IGT_PLANE_CURSOR] = "cursor",
 	};
 
diff --git a/lib/igt_kms.h b/lib/igt_kms.h
index 09c08aa..14c8b28 100644
--- a/lib/igt_kms.h
+++ b/lib/igt_kms.h
@@ -53,6 +53,7 @@ enum igt_plane {
         IGT_PLANE_PRIMARY = IGT_PLANE_1,
         IGT_PLANE_2,
         IGT_PLANE_3,
+        IGT_PLANE_4,
         IGT_PLANE_CURSOR,
 };
 
@@ -205,7 +206,7 @@ struct igt_pipe {
 	igt_display_t *display;
 	enum pipe pipe;
 	bool enabled;
-#define IGT_MAX_PLANES	4
+#define IGT_MAX_PLANES	5
 	int n_planes;
 	igt_plane_t planes[IGT_MAX_PLANES];
 	uint64_t background; /* Background color MSB BGR 16bpc LSB */
diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c
index 635cc79..ed4b134 100644
--- a/tests/kms_universal_plane.c
+++ b/tests/kms_universal_plane.c
@@ -57,6 +57,12 @@ typedef struct {
 	struct igt_fb red_fb, blue_fb;
 } pageflip_test_t;
 
+typedef struct {
+	data_t *data;
+	struct igt_fb fullsize_fb, undersize_fb;
+} primary_yuv_test_t;
+
+
 static void
 functional_test_init(functional_test_t *test, igt_output_t *output, enum pipe pipe)
 {
@@ -430,6 +436,77 @@ sanity_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output)
 }
 
 static void
+primary_yuv_test_init(primary_yuv_test_t *test, igt_output_t *output, enum pipe pipe)
+{
+	data_t *data = test->data;
+	drmModeModeInfo *mode;
+
+	igt_output_set_pipe(output, pipe);
+
+	mode = igt_output_get_mode(output);
+	igt_create_fb(data->drm_fd, mode->hdisplay, mode->vdisplay,
+			    DRM_FORMAT_YUYV,
+			    LOCAL_DRM_FORMAT_MOD_NONE,
+			    &test->fullsize_fb);
+	igt_create_fb(data->drm_fd, 300, 300,
+			    DRM_FORMAT_YVYU,
+			    LOCAL_DRM_FORMAT_MOD_NONE,
+			    &test->undersize_fb);
+}
+
+static void
+primary_yuv_test_fini(primary_yuv_test_t *test, igt_output_t *output)
+{
+	igt_remove_fb(test->data->drm_fd, &test->fullsize_fb);
+	igt_remove_fb(test->data->drm_fd, &test->undersize_fb);
+
+	igt_output_set_pipe(output, PIPE_ANY);
+	igt_display_commit2(&test->data->display, COMMIT_LEGACY);
+}
+
+/*
+ * YUV pixel format test for primary plane
+ * Display Full frame in primay then 300x300 frame
+ */
+
+static void
+primary_yuv_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output)
+{
+	primary_yuv_test_t test = { .data = data };
+	igt_plane_t *primary;
+
+	igt_skip_on(pipe >= data->display.n_pipes);
+
+	igt_output_set_pipe(output, pipe);
+
+	primary_yuv_test_init(&test, output, pipe);
+
+	primary = igt_output_get_plane(output, IGT_PLANE_PRIMARY);
+
+	/* Use legacy API to set a mode with a Fullsize FB */
+	igt_plane_set_fb(primary, &test.fullsize_fb);
+	igt_display_commit2(&data->display, COMMIT_LEGACY);
+
+	/* Disable the primary plane */
+	igt_plane_set_fb(primary, NULL);
+	igt_display_commit2(&data->display, COMMIT_UNIVERSAL);
+
+	/* Use Universal API to set a mode with a Fullsize FB */
+	igt_plane_set_fb(primary, &test.fullsize_fb);
+	igt_display_commit2(&data->display, COMMIT_UNIVERSAL);
+
+	/* Use Universal API to set a mode with a Undersize FB */
+	igt_plane_set_fb(primary, &test.undersize_fb);
+	igt_display_commit2(&data->display, COMMIT_UNIVERSAL);
+
+	/* Disable the primary plane */
+	igt_plane_set_fb(primary, NULL);
+
+	primary_yuv_test_fini(&test, output);
+
+}
+
+static void
 pageflip_test_init(pageflip_test_t *test, igt_output_t *output, enum pipe pipe)
 {
 	data_t *data = test->data;
@@ -663,6 +740,11 @@ run_tests_for_pipe(data_t *data, enum pipe pipe)
 		      kmstest_pipe_name(pipe))
 		for_each_connected_output(&data->display, output)
 			cursor_leak_test_pipe(data, pipe, output);
+
+	igt_subtest_f("primary-plane-pipe-%s-yuv",
+		      kmstest_pipe_name(pipe))
+		for_each_connected_output(&data->display, output)
+			primary_yuv_test_pipe(data, pipe, output);
 }
 
 static data_t data;
-- 
1.9.1



More information about the Intel-gfx mailing list