* [PATCH v2] i-g-t: Adding test case to test background color.
@ 2015-02-28 1:25 Chandra Konduru
2015-03-02 17:53 ` Daniel Vetter
2015-03-30 20:44 ` [PATCH v3] " Chandra Konduru
0 siblings, 2 replies; 7+ messages in thread
From: Chandra Konduru @ 2015-02-28 1:25 UTC (permalink / raw)
To: intel-gfx
From: chandra konduru <chandra.konduru@intel.com>
Adding i-g-t test case to test display crtc background color.
v2:
- Added IGT_TEST_DESCRIPTION() (Thomas Wood)
- Added to .gitignore (Thomas Wood)
- Added additional details to function header (Thomas Wood)
- Simplified igt_main (Thomas Wood)
Signed-off-by: chandra konduru <chandra.konduru@intel.com>
---
lib/igt_kms.c | 61 ++++++++++++
lib/igt_kms.h | 4 +
tests/.gitignore | 1 +
tests/Android.mk | 1 +
tests/Makefile.sources | 1 +
tests/kms_crtc_background_color.c | 201 ++++++++++++++++++++++++++++++++++++++
6 files changed, 269 insertions(+)
create mode 100644 tests/kms_crtc_background_color.c
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index d0c3690..72841ef 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -926,6 +926,22 @@ igt_plane_set_property(igt_plane_t *plane, uint32_t prop_id, uint64_t value)
DRM_MODE_OBJECT_PLANE, prop_id, value);
}
+static bool
+get_crtc_property(int drm_fd, uint32_t crtc_id, const char *name,
+ uint32_t *prop_id /* out */, uint64_t *value /* out */,
+ drmModePropertyPtr *prop /* out */)
+{
+ return kmstest_get_property(drm_fd, crtc_id, DRM_MODE_OBJECT_CRTC,
+ name, prop_id, value, prop);
+}
+
+static void
+igt_crtc_set_property(igt_output_t *output, uint32_t prop_id, uint64_t value)
+{
+ drmModeObjectSetProperty(output->display->drm_fd,
+ output->config.crtc->crtc_id, DRM_MODE_OBJECT_CRTC, prop_id, value);
+}
+
/*
* Walk a plane's property list to determine its type. If we don't
* find a type property, then the kernel doesn't support universal
@@ -1083,6 +1099,7 @@ void igt_display_init(igt_display_t *display, int drm_fd)
igt_assert(display->outputs);
for (i = 0; i < display->n_outputs; i++) {
+ int j;
igt_output_t *output = &display->outputs[i];
/*
@@ -1094,6 +1111,19 @@ void igt_display_init(igt_display_t *display, int drm_fd)
output->display = display;
igt_output_refresh(output);
+
+ for (j = 0; j < display->n_pipes; j++) {
+ uint64_t prop_value;
+ igt_pipe_t *pipe = &display->pipes[j];
+ if (output->config.crtc) {
+ get_crtc_property(display->drm_fd, output->config.crtc->crtc_id,
+ "background_color",
+ &pipe->background_property,
+ &prop_value,
+ NULL);
+ pipe->background = (uint32_t)prop_value;
+ }
+ }
}
drmModeFreePlaneResources(plane_resources);
@@ -1513,6 +1543,13 @@ static int igt_output_commit(igt_output_t *output,
pipe = igt_output_get_driving_pipe(output);
+ if (pipe->background_changed) {
+ igt_crtc_set_property(output, pipe->background_property,
+ pipe->background);
+
+ pipe->background_changed = false;
+ }
+
for (i = 0; i < pipe->n_planes; i++) {
igt_plane_t *plane = &pipe->planes[i];
@@ -1765,6 +1802,30 @@ void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation)
plane->rotation_changed = true;
}
+/**
+ * igt_crtc_set_background:
+ * @pipe: pipe pointer to which background color to be set
+ * @background: background color value in BGR 16bpc
+ *
+ * Sets background color for requested pipe. Color value provided here
+ * will be actually submitted at output commit time via "background_color"
+ * property.
+ * For example to get red as background, set background = 0x00000000FFFF.
+ */
+void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background)
+{
+ igt_display_t *display = pipe->display;
+
+ LOG(display, "%s.%d: crtc_set_background(%lu)\n",
+ kmstest_pipe_name(pipe->pipe),
+ pipe->pipe, background);
+
+ pipe->background = background;
+
+ pipe->background_changed = true;
+}
+
+
void igt_wait_for_vblank(int drm_fd, enum pipe pipe)
{
drmVBlank wait_vbl;
diff --git a/lib/igt_kms.h b/lib/igt_kms.h
index a1483a4..4fada1b 100644
--- a/lib/igt_kms.h
+++ b/lib/igt_kms.h
@@ -206,6 +206,9 @@ struct igt_pipe {
#define IGT_MAX_PLANES 4
int n_planes;
igt_plane_t planes[IGT_MAX_PLANES];
+ uint64_t background; /* Background color MSB BGR 16bpc LSB */
+ uint32_t background_changed : 1;
+ uint32_t background_property;
};
typedef struct {
@@ -251,6 +254,7 @@ void igt_plane_set_position(igt_plane_t *plane, int x, int y);
void igt_plane_set_size(igt_plane_t *plane, int w, int h);
void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation);
+void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background);
void igt_wait_for_vblank(int drm_fd, enum pipe pipe);
diff --git a/tests/.gitignore b/tests/.gitignore
index 88a6405..69d2ba0 100644
--- a/tests/.gitignore
+++ b/tests/.gitignore
@@ -137,6 +137,7 @@ kms_rotation_crc
kms_setmode
kms_sink_crc_basic
kms_universal_plane
+kms_crtc_background_color
multi-tests.txt
pm_lpsp
pm_psr
diff --git a/tests/Android.mk b/tests/Android.mk
index 814b846..96a32b9 100644
--- a/tests/Android.mk
+++ b/tests/Android.mk
@@ -75,6 +75,7 @@ else
kms_rotation_crc \
kms_force_connector \
kms_flip_event_leak \
+ kms_crtc_background_color \
kms_pwrite_crc
IGT_LOCAL_CFLAGS += -DANDROID_HAS_CAIRO=0
endif
diff --git a/tests/Makefile.sources b/tests/Makefile.sources
index 74deec3..4f612e6 100644
--- a/tests/Makefile.sources
+++ b/tests/Makefile.sources
@@ -77,6 +77,7 @@ TESTS_progs_M = \
kms_rotation_crc \
kms_setmode \
kms_universal_plane \
+ kms_crtc_background_color \
pm_lpsp \
pm_rpm \
pm_rps \
diff --git a/tests/kms_crtc_background_color.c b/tests/kms_crtc_background_color.c
new file mode 100644
index 0000000..3b60654
--- /dev/null
+++ b/tests/kms_crtc_background_color.c
@@ -0,0 +1,201 @@
+/*
+ * Copyright © 2013,2014 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ */
+
+#include <math.h>
+
+#include "drmtest.h"
+#include "igt_debugfs.h"
+#include "igt_kms.h"
+#include "igt_core.h"
+
+IGT_TEST_DESCRIPTION("Test crtc background color feature");
+
+typedef struct {
+ int gfx_fd;
+ igt_display_t display;
+ struct igt_fb fb;
+ igt_crc_t ref_crc;
+ igt_pipe_crc_t *pipe_crc;
+} data_t;
+
+#define BLACK 0x000000 /* BGR 8bpc */
+#define CYAN 0xFFFF00 /* BGR 8bpc */
+#define PURPLE 0xFF00FF /* BGR 8bpc */
+#define WHITE 0xFFFFFF /* BGR 8bpc */
+
+#define BLACK64 0x000000000000 /* BGR 16bpc */
+#define CYAN64 0xFFFFFFFF0000 /* BGR 16bpc */
+#define PURPLE64 0xFFFF0000FFFF /* BGR 16bpc */
+#define YELLOW64 0x0000FFFFFFFF /* BGR 16bpc */
+#define WHITE64 0xFFFFFFFFFFFF /* BGR 16bpc */
+#define RED64 0x00000000FFFF /* BGR 16bpc */
+#define GREEN64 0x0000FFFF0000 /* BGR 16bpc */
+#define BLUE64 0xFFFF00000000 /* BGR 16bpc */
+
+static void
+paint_background(data_t *data, struct igt_fb *fb, drmModeModeInfo *mode,
+ uint32_t background, double alpha)
+{
+ cairo_t *cr;
+ int w, h;
+ double r, g, b;
+
+ w = mode->hdisplay;
+ h = mode->vdisplay;
+
+ cr = igt_get_cairo_ctx(data->gfx_fd, &data->fb);
+
+ /* Paint with background color */
+ r = (double) (background & 0xFF) / 255.0;
+ g = (double) ((background & 0xFF00) >> 8) / 255.0;
+ b = (double) ((background & 0xFF0000) >> 16) / 255.0;
+ igt_paint_color_alpha(cr, 0, 0, w, h, r, g, b, alpha);
+
+ cairo_destroy(cr);
+}
+
+static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe,
+ igt_plane_t *plane, int opaque_buffer, int plane_color,
+ uint64_t pipe_background_color)
+{
+ drmModeModeInfo *mode;
+ igt_display_t *display = &data->display;
+ int fb_id;
+ double alpha;
+
+ igt_output_set_pipe(output, pipe);
+
+ /* create the pipe_crc object for this pipe */
+ igt_pipe_crc_free(data->pipe_crc);
+ data->pipe_crc = igt_pipe_crc_new(pipe, INTEL_PIPE_CRC_SOURCE_AUTO);
+
+ mode = igt_output_get_mode(output);
+
+ fb_id = igt_create_fb(data->gfx_fd,
+ mode->hdisplay, mode->vdisplay,
+ DRM_FORMAT_XRGB8888,
+ false, /* tiled */
+ &data->fb);
+ igt_assert(fb_id);
+
+ /* To make FB pixel win with background color, set alpha as full opaque */
+ igt_crtc_set_background(plane->pipe, pipe_background_color);
+ if (opaque_buffer)
+ alpha = 1.0; /* alpha 1 is fully opque */
+ else
+ alpha = 0.0; /* alpha 0 is fully transparent */
+ paint_background(data, &data->fb, mode, plane_color, alpha);
+
+ igt_plane_set_fb(plane, &data->fb);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+}
+
+static void cleanup_crtc(data_t *data, igt_output_t *output, igt_plane_t *plane)
+{
+ igt_display_t *display = &data->display;
+
+ igt_pipe_crc_free(data->pipe_crc);
+ data->pipe_crc = NULL;
+
+ igt_remove_fb(data->gfx_fd, &data->fb);
+
+ igt_crtc_set_background(plane->pipe, BLACK64);
+ igt_plane_set_fb(plane, NULL);
+ igt_output_set_pipe(output, PIPE_ANY);
+
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+}
+
+static void test_crtc_background(data_t *data)
+{
+ igt_display_t *display = &data->display;
+ igt_output_t *output;
+ enum pipe pipe;
+ int valid_tests = 0;
+
+ igt_require(data->display.has_universal_planes);
+
+ for_each_connected_output(display, output) {
+ igt_plane_t *plane;
+
+ pipe = output->config.pipe;
+ igt_output_set_pipe(output, pipe);
+
+ plane = igt_output_get_plane(output, IGT_PLANE_PRIMARY);
+ igt_require(plane->pipe->background_property);
+
+ prepare_crtc(data, output, pipe, plane, 1, PURPLE, BLACK64);
+
+ /* Now set background without using a plane, i.e.,
+ * Disable the plane to let hw background color win blend. */
+ igt_plane_set_fb(plane, NULL);
+ igt_crtc_set_background(plane->pipe, PURPLE64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ /* Try few other background colors */
+ sleep(1);
+ igt_crtc_set_background(plane->pipe, CYAN64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ sleep(1);
+ igt_crtc_set_background(plane->pipe, YELLOW64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ sleep(1);
+ igt_crtc_set_background(plane->pipe, RED64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ sleep(1);
+ igt_crtc_set_background(plane->pipe, GREEN64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ sleep(1);
+ igt_crtc_set_background(plane->pipe, BLUE64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ sleep(1);
+ igt_crtc_set_background(plane->pipe, WHITE64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ valid_tests++;
+ cleanup_crtc(data, output, plane);
+ }
+ igt_require_f(valid_tests, "no valid crtc/connector combinations found\n");
+}
+
+igt_main
+{
+ data_t data = {};
+
+ igt_skip_on_simulation();
+
+ data.gfx_fd = drm_open_any();
+ igt_require_pipe_crc();
+ igt_display_init(&data.display, data.gfx_fd);
+
+ igt_subtest("pipe-background")
+ test_crtc_background(&data);
+
+ igt_display_fini(&data.display);
+}
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2] i-g-t: Adding test case to test background color.
2015-02-28 1:25 [PATCH v2] i-g-t: Adding test case to test background color Chandra Konduru
@ 2015-03-02 17:53 ` Daniel Vetter
2015-03-24 18:51 ` Konduru, Chandra
2015-03-30 20:44 ` [PATCH v3] " Chandra Konduru
1 sibling, 1 reply; 7+ messages in thread
From: Daniel Vetter @ 2015-03-02 17:53 UTC (permalink / raw)
To: Chandra Konduru; +Cc: intel-gfx
On Fri, Feb 27, 2015 at 05:25:52PM -0800, Chandra Konduru wrote:
> From: chandra konduru <chandra.konduru@intel.com>
>
> Adding i-g-t test case to test display crtc background color.
>
> v2:
> - Added IGT_TEST_DESCRIPTION() (Thomas Wood)
> - Added to .gitignore (Thomas Wood)
> - Added additional details to function header (Thomas Wood)
> - Simplified igt_main (Thomas Wood)
>
> Signed-off-by: chandra konduru <chandra.konduru@intel.com>
This test doesn't seem to actually use crc checksums anywhere ...
> ---
> lib/igt_kms.c | 61 ++++++++++++
> lib/igt_kms.h | 4 +
> tests/.gitignore | 1 +
> tests/Android.mk | 1 +
> tests/Makefile.sources | 1 +
> tests/kms_crtc_background_color.c | 201 ++++++++++++++++++++++++++++++++++++++
> 6 files changed, 269 insertions(+)
> create mode 100644 tests/kms_crtc_background_color.c
>
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index d0c3690..72841ef 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -926,6 +926,22 @@ igt_plane_set_property(igt_plane_t *plane, uint32_t prop_id, uint64_t value)
> DRM_MODE_OBJECT_PLANE, prop_id, value);
> }
>
> +static bool
> +get_crtc_property(int drm_fd, uint32_t crtc_id, const char *name,
> + uint32_t *prop_id /* out */, uint64_t *value /* out */,
> + drmModePropertyPtr *prop /* out */)
> +{
> + return kmstest_get_property(drm_fd, crtc_id, DRM_MODE_OBJECT_CRTC,
> + name, prop_id, value, prop);
> +}
> +
> +static void
> +igt_crtc_set_property(igt_output_t *output, uint32_t prop_id, uint64_t value)
> +{
> + drmModeObjectSetProperty(output->display->drm_fd,
> + output->config.crtc->crtc_id, DRM_MODE_OBJECT_CRTC, prop_id, value);
> +}
> +
> /*
> * Walk a plane's property list to determine its type. If we don't
> * find a type property, then the kernel doesn't support universal
> @@ -1083,6 +1099,7 @@ void igt_display_init(igt_display_t *display, int drm_fd)
> igt_assert(display->outputs);
>
> for (i = 0; i < display->n_outputs; i++) {
> + int j;
> igt_output_t *output = &display->outputs[i];
>
> /*
> @@ -1094,6 +1111,19 @@ void igt_display_init(igt_display_t *display, int drm_fd)
> output->display = display;
>
> igt_output_refresh(output);
> +
> + for (j = 0; j < display->n_pipes; j++) {
> + uint64_t prop_value;
> + igt_pipe_t *pipe = &display->pipes[j];
> + if (output->config.crtc) {
> + get_crtc_property(display->drm_fd, output->config.crtc->crtc_id,
> + "background_color",
> + &pipe->background_property,
> + &prop_value,
> + NULL);
> + pipe->background = (uint32_t)prop_value;
> + }
> + }
> }
>
> drmModeFreePlaneResources(plane_resources);
> @@ -1513,6 +1543,13 @@ static int igt_output_commit(igt_output_t *output,
>
> pipe = igt_output_get_driving_pipe(output);
>
> + if (pipe->background_changed) {
> + igt_crtc_set_property(output, pipe->background_property,
> + pipe->background);
> +
> + pipe->background_changed = false;
> + }
> +
> for (i = 0; i < pipe->n_planes; i++) {
> igt_plane_t *plane = &pipe->planes[i];
>
> @@ -1765,6 +1802,30 @@ void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation)
> plane->rotation_changed = true;
> }
>
> +/**
> + * igt_crtc_set_background:
> + * @pipe: pipe pointer to which background color to be set
> + * @background: background color value in BGR 16bpc
> + *
> + * Sets background color for requested pipe. Color value provided here
> + * will be actually submitted at output commit time via "background_color"
> + * property.
> + * For example to get red as background, set background = 0x00000000FFFF.
> + */
> +void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background)
> +{
> + igt_display_t *display = pipe->display;
> +
> + LOG(display, "%s.%d: crtc_set_background(%lu)\n",
> + kmstest_pipe_name(pipe->pipe),
> + pipe->pipe, background);
> +
> + pipe->background = background;
> +
> + pipe->background_changed = true;
> +}
> +
> +
> void igt_wait_for_vblank(int drm_fd, enum pipe pipe)
> {
> drmVBlank wait_vbl;
> diff --git a/lib/igt_kms.h b/lib/igt_kms.h
> index a1483a4..4fada1b 100644
> --- a/lib/igt_kms.h
> +++ b/lib/igt_kms.h
> @@ -206,6 +206,9 @@ struct igt_pipe {
> #define IGT_MAX_PLANES 4
> int n_planes;
> igt_plane_t planes[IGT_MAX_PLANES];
> + uint64_t background; /* Background color MSB BGR 16bpc LSB */
> + uint32_t background_changed : 1;
> + uint32_t background_property;
> };
>
> typedef struct {
> @@ -251,6 +254,7 @@ void igt_plane_set_position(igt_plane_t *plane, int x, int y);
> void igt_plane_set_size(igt_plane_t *plane, int w, int h);
> void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
> void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation);
> +void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background);
>
> void igt_wait_for_vblank(int drm_fd, enum pipe pipe);
>
> diff --git a/tests/.gitignore b/tests/.gitignore
> index 88a6405..69d2ba0 100644
> --- a/tests/.gitignore
> +++ b/tests/.gitignore
> @@ -137,6 +137,7 @@ kms_rotation_crc
> kms_setmode
> kms_sink_crc_basic
> kms_universal_plane
> +kms_crtc_background_color
> multi-tests.txt
> pm_lpsp
> pm_psr
> diff --git a/tests/Android.mk b/tests/Android.mk
> index 814b846..96a32b9 100644
> --- a/tests/Android.mk
> +++ b/tests/Android.mk
> @@ -75,6 +75,7 @@ else
> kms_rotation_crc \
> kms_force_connector \
> kms_flip_event_leak \
> + kms_crtc_background_color \
> kms_pwrite_crc
> IGT_LOCAL_CFLAGS += -DANDROID_HAS_CAIRO=0
> endif
> diff --git a/tests/Makefile.sources b/tests/Makefile.sources
> index 74deec3..4f612e6 100644
> --- a/tests/Makefile.sources
> +++ b/tests/Makefile.sources
> @@ -77,6 +77,7 @@ TESTS_progs_M = \
> kms_rotation_crc \
> kms_setmode \
> kms_universal_plane \
> + kms_crtc_background_color \
> pm_lpsp \
> pm_rpm \
> pm_rps \
> diff --git a/tests/kms_crtc_background_color.c b/tests/kms_crtc_background_color.c
> new file mode 100644
> index 0000000..3b60654
> --- /dev/null
> +++ b/tests/kms_crtc_background_color.c
> @@ -0,0 +1,201 @@
> +/*
> + * Copyright © 2013,2014 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> + * IN THE SOFTWARE.
> + *
> + */
> +
> +#include <math.h>
> +
> +#include "drmtest.h"
> +#include "igt_debugfs.h"
> +#include "igt_kms.h"
> +#include "igt_core.h"
> +
> +IGT_TEST_DESCRIPTION("Test crtc background color feature");
> +
> +typedef struct {
> + int gfx_fd;
> + igt_display_t display;
> + struct igt_fb fb;
> + igt_crc_t ref_crc;
> + igt_pipe_crc_t *pipe_crc;
> +} data_t;
> +
> +#define BLACK 0x000000 /* BGR 8bpc */
> +#define CYAN 0xFFFF00 /* BGR 8bpc */
> +#define PURPLE 0xFF00FF /* BGR 8bpc */
> +#define WHITE 0xFFFFFF /* BGR 8bpc */
> +
> +#define BLACK64 0x000000000000 /* BGR 16bpc */
> +#define CYAN64 0xFFFFFFFF0000 /* BGR 16bpc */
> +#define PURPLE64 0xFFFF0000FFFF /* BGR 16bpc */
> +#define YELLOW64 0x0000FFFFFFFF /* BGR 16bpc */
> +#define WHITE64 0xFFFFFFFFFFFF /* BGR 16bpc */
> +#define RED64 0x00000000FFFF /* BGR 16bpc */
> +#define GREEN64 0x0000FFFF0000 /* BGR 16bpc */
> +#define BLUE64 0xFFFF00000000 /* BGR 16bpc */
> +
> +static void
> +paint_background(data_t *data, struct igt_fb *fb, drmModeModeInfo *mode,
> + uint32_t background, double alpha)
> +{
> + cairo_t *cr;
> + int w, h;
> + double r, g, b;
> +
> + w = mode->hdisplay;
> + h = mode->vdisplay;
> +
> + cr = igt_get_cairo_ctx(data->gfx_fd, &data->fb);
> +
> + /* Paint with background color */
> + r = (double) (background & 0xFF) / 255.0;
> + g = (double) ((background & 0xFF00) >> 8) / 255.0;
> + b = (double) ((background & 0xFF0000) >> 16) / 255.0;
> + igt_paint_color_alpha(cr, 0, 0, w, h, r, g, b, alpha);
> +
> + cairo_destroy(cr);
> +}
> +
> +static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe,
> + igt_plane_t *plane, int opaque_buffer, int plane_color,
> + uint64_t pipe_background_color)
> +{
> + drmModeModeInfo *mode;
> + igt_display_t *display = &data->display;
> + int fb_id;
> + double alpha;
> +
> + igt_output_set_pipe(output, pipe);
> +
> + /* create the pipe_crc object for this pipe */
> + igt_pipe_crc_free(data->pipe_crc);
> + data->pipe_crc = igt_pipe_crc_new(pipe, INTEL_PIPE_CRC_SOURCE_AUTO);
> +
> + mode = igt_output_get_mode(output);
> +
> + fb_id = igt_create_fb(data->gfx_fd,
> + mode->hdisplay, mode->vdisplay,
> + DRM_FORMAT_XRGB8888,
> + false, /* tiled */
> + &data->fb);
> + igt_assert(fb_id);
> +
> + /* To make FB pixel win with background color, set alpha as full opaque */
> + igt_crtc_set_background(plane->pipe, pipe_background_color);
> + if (opaque_buffer)
> + alpha = 1.0; /* alpha 1 is fully opque */
> + else
> + alpha = 0.0; /* alpha 0 is fully transparent */
> + paint_background(data, &data->fb, mode, plane_color, alpha);
> +
> + igt_plane_set_fb(plane, &data->fb);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +}
> +
> +static void cleanup_crtc(data_t *data, igt_output_t *output, igt_plane_t *plane)
> +{
> + igt_display_t *display = &data->display;
> +
> + igt_pipe_crc_free(data->pipe_crc);
> + data->pipe_crc = NULL;
> +
> + igt_remove_fb(data->gfx_fd, &data->fb);
> +
> + igt_crtc_set_background(plane->pipe, BLACK64);
> + igt_plane_set_fb(plane, NULL);
> + igt_output_set_pipe(output, PIPE_ANY);
> +
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +}
> +
> +static void test_crtc_background(data_t *data)
> +{
> + igt_display_t *display = &data->display;
> + igt_output_t *output;
> + enum pipe pipe;
> + int valid_tests = 0;
> +
> + igt_require(data->display.has_universal_planes);
> +
> + for_each_connected_output(display, output) {
> + igt_plane_t *plane;
> +
> + pipe = output->config.pipe;
> + igt_output_set_pipe(output, pipe);
> +
> + plane = igt_output_get_plane(output, IGT_PLANE_PRIMARY);
> + igt_require(plane->pipe->background_property);
> +
> + prepare_crtc(data, output, pipe, plane, 1, PURPLE, BLACK64);
> +
> + /* Now set background without using a plane, i.e.,
> + * Disable the plane to let hw background color win blend. */
> + igt_plane_set_fb(plane, NULL);
> + igt_crtc_set_background(plane->pipe, PURPLE64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + /* Try few other background colors */
> + sleep(1);
> + igt_crtc_set_background(plane->pipe, CYAN64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + sleep(1);
> + igt_crtc_set_background(plane->pipe, YELLOW64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + sleep(1);
> + igt_crtc_set_background(plane->pipe, RED64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + sleep(1);
Please use igt_interactive_debug if you want to make a testcase useful for
visual inspection. But igts _really_ must run fully automated, and that's
possible using crc checksums. I.e. you need to draw reference frames using
software, grab crc checksums and then compare against the same composition
using the hw feature.
Cheers, Daniel
> + igt_crtc_set_background(plane->pipe, GREEN64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + sleep(1);
> + igt_crtc_set_background(plane->pipe, BLUE64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + sleep(1);
> + igt_crtc_set_background(plane->pipe, WHITE64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + valid_tests++;
> + cleanup_crtc(data, output, plane);
> + }
> + igt_require_f(valid_tests, "no valid crtc/connector combinations found\n");
> +}
> +
> +igt_main
> +{
> + data_t data = {};
> +
> + igt_skip_on_simulation();
> +
> + data.gfx_fd = drm_open_any();
> + igt_require_pipe_crc();
> + igt_display_init(&data.display, data.gfx_fd);
> +
> + igt_subtest("pipe-background")
> + test_crtc_background(&data);
> +
> + igt_display_fini(&data.display);
> +}
> --
> 1.9.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] i-g-t: Adding test case to test background color.
2015-03-02 17:53 ` Daniel Vetter
@ 2015-03-24 18:51 ` Konduru, Chandra
2015-03-25 13:48 ` Daniel Vetter
0 siblings, 1 reply; 7+ messages in thread
From: Konduru, Chandra @ 2015-03-24 18:51 UTC (permalink / raw)
To: Daniel Vetter; +Cc: intel-gfx
> -----Original Message-----
> From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of Daniel Vetter
> Sent: Monday, March 02, 2015 9:54 AM
> To: Konduru, Chandra
> Cc: intel-gfx@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH v2] i-g-t: Adding test case to test background
> color.
>
> On Fri, Feb 27, 2015 at 05:25:52PM -0800, Chandra Konduru wrote:
> > From: chandra konduru <chandra.konduru@intel.com>
> >
> > Adding i-g-t test case to test display crtc background color.
> >
> > v2:
> > - Added IGT_TEST_DESCRIPTION() (Thomas Wood)
> > - Added to .gitignore (Thomas Wood)
> > - Added additional details to function header (Thomas Wood)
> > - Simplified igt_main (Thomas Wood)
> >
> > Signed-off-by: chandra konduru <chandra.konduru@intel.com>
>
> This test doesn't seem to actually use crc checksums anywhere ...
>
> > ---
> > lib/igt_kms.c | 61 ++++++++++++
> > lib/igt_kms.h | 4 +
> > tests/.gitignore | 1 +
> > tests/Android.mk | 1 +
> > tests/Makefile.sources | 1 +
> > tests/kms_crtc_background_color.c | 201
> > ++++++++++++++++++++++++++++++++++++++
> > 6 files changed, 269 insertions(+)
> > create mode 100644 tests/kms_crtc_background_color.c
> >
> > diff --git a/lib/igt_kms.c b/lib/igt_kms.c index d0c3690..72841ef
> > 100644
> > --- a/lib/igt_kms.c
> > +++ b/lib/igt_kms.c
> > @@ -926,6 +926,22 @@ igt_plane_set_property(igt_plane_t *plane, uint32_t
> prop_id, uint64_t value)
> > DRM_MODE_OBJECT_PLANE, prop_id, value);
> }
> >
> > +static bool
> > +get_crtc_property(int drm_fd, uint32_t crtc_id, const char *name,
> > + uint32_t *prop_id /* out */, uint64_t *value /* out */,
> > + drmModePropertyPtr *prop /* out */) {
> > + return kmstest_get_property(drm_fd, crtc_id,
> DRM_MODE_OBJECT_CRTC,
> > + name, prop_id, value, prop);
> > +}
> > +
> > +static void
> > +igt_crtc_set_property(igt_output_t *output, uint32_t prop_id,
> > +uint64_t value) {
> > + drmModeObjectSetProperty(output->display->drm_fd,
> > + output->config.crtc->crtc_id, DRM_MODE_OBJECT_CRTC,
> prop_id,
> > +value); }
> > +
> > /*
> > * Walk a plane's property list to determine its type. If we don't
> > * find a type property, then the kernel doesn't support universal @@
> > -1083,6 +1099,7 @@ void igt_display_init(igt_display_t *display, int drm_fd)
> > igt_assert(display->outputs);
> >
> > for (i = 0; i < display->n_outputs; i++) {
> > + int j;
> > igt_output_t *output = &display->outputs[i];
> >
> > /*
> > @@ -1094,6 +1111,19 @@ void igt_display_init(igt_display_t *display, int
> drm_fd)
> > output->display = display;
> >
> > igt_output_refresh(output);
> > +
> > + for (j = 0; j < display->n_pipes; j++) {
> > + uint64_t prop_value;
> > + igt_pipe_t *pipe = &display->pipes[j];
> > + if (output->config.crtc) {
> > + get_crtc_property(display->drm_fd, output-
> >config.crtc->crtc_id,
> > + "background_color",
> > + &pipe-
> >background_property,
> > + &prop_value,
> > + NULL);
> > + pipe->background = (uint32_t)prop_value;
> > + }
> > + }
> > }
> >
> > drmModeFreePlaneResources(plane_resources);
> > @@ -1513,6 +1543,13 @@ static int igt_output_commit(igt_output_t
> > *output,
> >
> > pipe = igt_output_get_driving_pipe(output);
> >
> > + if (pipe->background_changed) {
> > + igt_crtc_set_property(output, pipe->background_property,
> > + pipe->background);
> > +
> > + pipe->background_changed = false;
> > + }
> > +
> > for (i = 0; i < pipe->n_planes; i++) {
> > igt_plane_t *plane = &pipe->planes[i];
> >
> > @@ -1765,6 +1802,30 @@ void igt_plane_set_rotation(igt_plane_t *plane,
> igt_rotation_t rotation)
> > plane->rotation_changed = true;
> > }
> >
> > +/**
> > + * igt_crtc_set_background:
> > + * @pipe: pipe pointer to which background color to be set
> > + * @background: background color value in BGR 16bpc
> > + *
> > + * Sets background color for requested pipe. Color value provided
> > +here
> > + * will be actually submitted at output commit time via "background_color"
> > + * property.
> > + * For example to get red as background, set background = 0x00000000FFFF.
> > + */
> > +void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background) {
> > + igt_display_t *display = pipe->display;
> > +
> > + LOG(display, "%s.%d: crtc_set_background(%lu)\n",
> > + kmstest_pipe_name(pipe->pipe),
> > + pipe->pipe, background);
> > +
> > + pipe->background = background;
> > +
> > + pipe->background_changed = true;
> > +}
> > +
> > +
> > void igt_wait_for_vblank(int drm_fd, enum pipe pipe) {
> > drmVBlank wait_vbl;
> > diff --git a/lib/igt_kms.h b/lib/igt_kms.h index a1483a4..4fada1b
> > 100644
> > --- a/lib/igt_kms.h
> > +++ b/lib/igt_kms.h
> > @@ -206,6 +206,9 @@ struct igt_pipe {
> > #define IGT_MAX_PLANES 4
> > int n_planes;
> > igt_plane_t planes[IGT_MAX_PLANES];
> > + uint64_t background; /* Background color MSB BGR 16bpc LSB */
> > + uint32_t background_changed : 1;
> > + uint32_t background_property;
> > };
> >
> > typedef struct {
> > @@ -251,6 +254,7 @@ void igt_plane_set_position(igt_plane_t *plane,
> > int x, int y); void igt_plane_set_size(igt_plane_t *plane, int w, int
> > h); void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
> > void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t
> > rotation);
> > +void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background);
> >
> > void igt_wait_for_vblank(int drm_fd, enum pipe pipe);
> >
> > diff --git a/tests/.gitignore b/tests/.gitignore index
> > 88a6405..69d2ba0 100644
> > --- a/tests/.gitignore
> > +++ b/tests/.gitignore
> > @@ -137,6 +137,7 @@ kms_rotation_crc
> > kms_setmode
> > kms_sink_crc_basic
> > kms_universal_plane
> > +kms_crtc_background_color
> > multi-tests.txt
> > pm_lpsp
> > pm_psr
> > diff --git a/tests/Android.mk b/tests/Android.mk index
> > 814b846..96a32b9 100644
> > --- a/tests/Android.mk
> > +++ b/tests/Android.mk
> > @@ -75,6 +75,7 @@ else
> > kms_rotation_crc \
> > kms_force_connector \
> > kms_flip_event_leak \
> > + kms_crtc_background_color \
> > kms_pwrite_crc
> > IGT_LOCAL_CFLAGS += -DANDROID_HAS_CAIRO=0 endif diff --git
> > a/tests/Makefile.sources b/tests/Makefile.sources index
> > 74deec3..4f612e6 100644
> > --- a/tests/Makefile.sources
> > +++ b/tests/Makefile.sources
> > @@ -77,6 +77,7 @@ TESTS_progs_M = \
> > kms_rotation_crc \
> > kms_setmode \
> > kms_universal_plane \
> > + kms_crtc_background_color \
> > pm_lpsp \
> > pm_rpm \
> > pm_rps \
> > diff --git a/tests/kms_crtc_background_color.c
> > b/tests/kms_crtc_background_color.c
> > new file mode 100644
> > index 0000000..3b60654
> > --- /dev/null
> > +++ b/tests/kms_crtc_background_color.c
> > @@ -0,0 +1,201 @@
> > +/*
> > + * Copyright (c) 2013,2014 Intel Corporation
> > + *
> > + * Permission is hereby granted, free of charge, to any person
> > +obtaining a
> > + * copy of this software and associated documentation files (the
> > +"Software"),
> > + * to deal in the Software without restriction, including without
> > +limitation
> > + * the rights to use, copy, modify, merge, publish, distribute,
> > +sublicense,
> > + * and/or sell copies of the Software, and to permit persons to whom
> > +the
> > + * Software is furnished to do so, subject to the following conditions:
> > + *
> > + * The above copyright notice and this permission notice (including
> > +the next
> > + * paragraph) shall be included in all copies or substantial portions
> > +of the
> > + * Software.
> > + *
> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> > +EXPRESS OR
> > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> > +MERCHANTABILITY,
> > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO
> EVENT
> > +SHALL
> > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
> DAMAGES
> > +OR OTHER
> > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> > +ARISING
> > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> > +OTHER DEALINGS
> > + * IN THE SOFTWARE.
> > + *
> > + */
> > +
> > +#include <math.h>
> > +
> > +#include "drmtest.h"
> > +#include "igt_debugfs.h"
> > +#include "igt_kms.h"
> > +#include "igt_core.h"
> > +
> > +IGT_TEST_DESCRIPTION("Test crtc background color feature");
> > +
> > +typedef struct {
> > + int gfx_fd;
> > + igt_display_t display;
> > + struct igt_fb fb;
> > + igt_crc_t ref_crc;
> > + igt_pipe_crc_t *pipe_crc;
> > +} data_t;
> > +
> > +#define BLACK 0x000000 /* BGR 8bpc */
> > +#define CYAN 0xFFFF00 /* BGR 8bpc */
> > +#define PURPLE 0xFF00FF /* BGR 8bpc */
> > +#define WHITE 0xFFFFFF /* BGR 8bpc */
> > +
> > +#define BLACK64 0x000000000000 /* BGR 16bpc */
> > +#define CYAN64 0xFFFFFFFF0000 /* BGR 16bpc */
> > +#define PURPLE64 0xFFFF0000FFFF /* BGR 16bpc */
> > +#define YELLOW64 0x0000FFFFFFFF /* BGR 16bpc */
> > +#define WHITE64 0xFFFFFFFFFFFF /* BGR 16bpc */
> > +#define RED64 0x00000000FFFF /* BGR 16bpc */
> > +#define GREEN64 0x0000FFFF0000 /* BGR 16bpc */
> > +#define BLUE64 0xFFFF00000000 /* BGR 16bpc */
> > +
> > +static void
> > +paint_background(data_t *data, struct igt_fb *fb, drmModeModeInfo
> *mode,
> > + uint32_t background, double alpha)
> > +{
> > + cairo_t *cr;
> > + int w, h;
> > + double r, g, b;
> > +
> > + w = mode->hdisplay;
> > + h = mode->vdisplay;
> > +
> > + cr = igt_get_cairo_ctx(data->gfx_fd, &data->fb);
> > +
> > + /* Paint with background color */
> > + r = (double) (background & 0xFF) / 255.0;
> > + g = (double) ((background & 0xFF00) >> 8) / 255.0;
> > + b = (double) ((background & 0xFF0000) >> 16) / 255.0;
> > + igt_paint_color_alpha(cr, 0, 0, w, h, r, g, b, alpha);
> > +
> > + cairo_destroy(cr);
> > +}
> > +
> > +static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe,
> > + igt_plane_t *plane, int opaque_buffer, int plane_color,
> > + uint64_t pipe_background_color)
> > +{
> > + drmModeModeInfo *mode;
> > + igt_display_t *display = &data->display;
> > + int fb_id;
> > + double alpha;
> > +
> > + igt_output_set_pipe(output, pipe);
> > +
> > + /* create the pipe_crc object for this pipe */
> > + igt_pipe_crc_free(data->pipe_crc);
> > + data->pipe_crc = igt_pipe_crc_new(pipe,
> INTEL_PIPE_CRC_SOURCE_AUTO);
> > +
> > + mode = igt_output_get_mode(output);
> > +
> > + fb_id = igt_create_fb(data->gfx_fd,
> > + mode->hdisplay, mode->vdisplay,
> > + DRM_FORMAT_XRGB8888,
> > + false, /* tiled */
> > + &data->fb);
> > + igt_assert(fb_id);
> > +
> > + /* To make FB pixel win with background color, set alpha as full opaque
> */
> > + igt_crtc_set_background(plane->pipe, pipe_background_color);
> > + if (opaque_buffer)
> > + alpha = 1.0; /* alpha 1 is fully opque */
> > + else
> > + alpha = 0.0; /* alpha 0 is fully transparent */
> > + paint_background(data, &data->fb, mode, plane_color, alpha);
> > +
> > + igt_plane_set_fb(plane, &data->fb);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL); }
> > +
> > +static void cleanup_crtc(data_t *data, igt_output_t *output,
> > +igt_plane_t *plane) {
> > + igt_display_t *display = &data->display;
> > +
> > + igt_pipe_crc_free(data->pipe_crc);
> > + data->pipe_crc = NULL;
> > +
> > + igt_remove_fb(data->gfx_fd, &data->fb);
> > +
> > + igt_crtc_set_background(plane->pipe, BLACK64);
> > + igt_plane_set_fb(plane, NULL);
> > + igt_output_set_pipe(output, PIPE_ANY);
> > +
> > + igt_display_commit2(display, COMMIT_UNIVERSAL); }
> > +
> > +static void test_crtc_background(data_t *data) {
> > + igt_display_t *display = &data->display;
> > + igt_output_t *output;
> > + enum pipe pipe;
> > + int valid_tests = 0;
> > +
> > + igt_require(data->display.has_universal_planes);
> > +
> > + for_each_connected_output(display, output) {
> > + igt_plane_t *plane;
> > +
> > + pipe = output->config.pipe;
> > + igt_output_set_pipe(output, pipe);
> > +
> > + plane = igt_output_get_plane(output, IGT_PLANE_PRIMARY);
> > + igt_require(plane->pipe->background_property);
> > +
> > + prepare_crtc(data, output, pipe, plane, 1, PURPLE, BLACK64);
> > +
> > + /* Now set background without using a plane, i.e.,
> > + * Disable the plane to let hw background color win blend. */
> > + igt_plane_set_fb(plane, NULL);
> > + igt_crtc_set_background(plane->pipe, PURPLE64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + /* Try few other background colors */
> > + sleep(1);
> > + igt_crtc_set_background(plane->pipe, CYAN64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + sleep(1);
> > + igt_crtc_set_background(plane->pipe, YELLOW64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + sleep(1);
> > + igt_crtc_set_background(plane->pipe, RED64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + sleep(1);
>
> Please use igt_interactive_debug if you want to make a testcase useful for visual
> inspection. But igts _really_ must run fully automated, and that's possible using
> crc checksums. I.e. you need to draw reference frames using software, grab crc
> checksums and then compare against the same composition using the hw
> feature.
Initially I planned to use crcs, but issue is that when all planes are disabled,
which is required for this test, crcs aren't reliable to depend for comparison.
Will sendout updated patch with sleeps takenout.
By the way, also I tried using igt_interactive_debug() but execution doesn't
wait for keypress in response to calling igt_debug_wait_for_keypress.
I might be missing something here. Is there a sample sequence
to use igt_interactive_debug?
>
> Cheers, Daniel
>
> > + igt_crtc_set_background(plane->pipe, GREEN64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + sleep(1);
> > + igt_crtc_set_background(plane->pipe, BLUE64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + sleep(1);
> > + igt_crtc_set_background(plane->pipe, WHITE64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + valid_tests++;
> > + cleanup_crtc(data, output, plane);
> > + }
> > + igt_require_f(valid_tests, "no valid crtc/connector combinations
> > +found\n"); }
> > +
> > +igt_main
> > +{
> > + data_t data = {};
> > +
> > + igt_skip_on_simulation();
> > +
> > + data.gfx_fd = drm_open_any();
> > + igt_require_pipe_crc();
> > + igt_display_init(&data.display, data.gfx_fd);
> > +
> > + igt_subtest("pipe-background")
> > + test_crtc_background(&data);
> > +
> > + igt_display_fini(&data.display);
> > +}
> > --
> > 1.9.1
> >
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> +41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] i-g-t: Adding test case to test background color.
2015-03-24 18:51 ` Konduru, Chandra
@ 2015-03-25 13:48 ` Daniel Vetter
0 siblings, 0 replies; 7+ messages in thread
From: Daniel Vetter @ 2015-03-25 13:48 UTC (permalink / raw)
To: Konduru, Chandra; +Cc: intel-gfx
On Tue, Mar 24, 2015 at 06:51:43PM +0000, Konduru, Chandra wrote:
> > Please use igt_interactive_debug if you want to make a testcase useful for visual
> > inspection. But igts _really_ must run fully automated, and that's possible using
> > crc checksums. I.e. you need to draw reference frames using software, grab crc
> > checksums and then compare against the same composition using the hw
> > feature.
>
> Initially I planned to use crcs, but issue is that when all planes are disabled,
> which is required for this test, crcs aren't reliable to depend for comparison.
> Will sendout updated patch with sleeps takenout.
Funny that crcs don't work when all planes are disabled, we should still
be scanning out a proper black. And the crcs we use by default are all at
the pipe leve. What exactly falls apart?
> By the way, also I tried using igt_interactive_debug() but execution doesn't
> wait for keypress in response to calling igt_debug_wait_for_keypress.
> I might be missing something here. Is there a sample sequence
> to use igt_interactive_debug?
You need to enable it with --interactive-debug=all The help text for that
functions even explains that ...
Cheers, Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v3] i-g-t: Adding test case to test background color.
2015-02-28 1:25 [PATCH v2] i-g-t: Adding test case to test background color Chandra Konduru
2015-03-02 17:53 ` Daniel Vetter
@ 2015-03-30 20:44 ` Chandra Konduru
2015-04-14 17:02 ` Thomas Wood
1 sibling, 1 reply; 7+ messages in thread
From: Chandra Konduru @ 2015-03-30 20:44 UTC (permalink / raw)
To: intel-gfx
From: chandra konduru <chandra.konduru@intel.com>
Adding i-g-t test case to test display crtc background color.
v2:
- Added IGT_TEST_DESCRIPTION() (Thomas Wood)
- Added to .gitignore (Thomas Wood)
- Added additional details to function header (Thomas Wood)
- Simplified igt_main (Thomas Wood)
v3:
- rebased to latest master (me)
- took sleep calls out (Daniel)
- use new tiled types when calling igt_create_fb (me)
Signed-off-by: chandra konduru <chandra.konduru@intel.com>
---
lib/igt_kms.c | 61 ++++++++++++
lib/igt_kms.h | 4 +
tests/.gitignore | 1 +
tests/Android.mk | 1 +
tests/Makefile.sources | 1 +
tests/kms_crtc_background_color.c | 195 ++++++++++++++++++++++++++++++++++++++
6 files changed, 263 insertions(+)
create mode 100644 tests/kms_crtc_background_color.c
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 9c131f0..9c46e18 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -940,6 +940,22 @@ igt_plane_set_property(igt_plane_t *plane, uint32_t prop_id, uint64_t value)
DRM_MODE_OBJECT_PLANE, prop_id, value);
}
+static bool
+get_crtc_property(int drm_fd, uint32_t crtc_id, const char *name,
+ uint32_t *prop_id /* out */, uint64_t *value /* out */,
+ drmModePropertyPtr *prop /* out */)
+{
+ return kmstest_get_property(drm_fd, crtc_id, DRM_MODE_OBJECT_CRTC,
+ name, prop_id, value, prop);
+}
+
+static void
+igt_crtc_set_property(igt_output_t *output, uint32_t prop_id, uint64_t value)
+{
+ drmModeObjectSetProperty(output->display->drm_fd,
+ output->config.crtc->crtc_id, DRM_MODE_OBJECT_CRTC, prop_id, value);
+}
+
/*
* Walk a plane's property list to determine its type. If we don't
* find a type property, then the kernel doesn't support universal
@@ -1097,6 +1113,7 @@ void igt_display_init(igt_display_t *display, int drm_fd)
igt_assert(display->outputs);
for (i = 0; i < display->n_outputs; i++) {
+ int j;
igt_output_t *output = &display->outputs[i];
/*
@@ -1108,6 +1125,19 @@ void igt_display_init(igt_display_t *display, int drm_fd)
output->display = display;
igt_output_refresh(output);
+
+ for (j = 0; j < display->n_pipes; j++) {
+ uint64_t prop_value;
+ igt_pipe_t *pipe = &display->pipes[j];
+ if (output->config.crtc) {
+ get_crtc_property(display->drm_fd, output->config.crtc->crtc_id,
+ "background_color",
+ &pipe->background_property,
+ &prop_value,
+ NULL);
+ pipe->background = (uint32_t)prop_value;
+ }
+ }
}
drmModeFreePlaneResources(plane_resources);
@@ -1527,6 +1557,13 @@ static int igt_output_commit(igt_output_t *output,
pipe = igt_output_get_driving_pipe(output);
+ if (pipe->background_changed) {
+ igt_crtc_set_property(output, pipe->background_property,
+ pipe->background);
+
+ pipe->background_changed = false;
+ }
+
for (i = 0; i < pipe->n_planes; i++) {
igt_plane_t *plane = &pipe->planes[i];
@@ -1779,6 +1816,30 @@ void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation)
plane->rotation_changed = true;
}
+/**
+ * igt_crtc_set_background:
+ * @pipe: pipe pointer to which background color to be set
+ * @background: background color value in BGR 16bpc
+ *
+ * Sets background color for requested pipe. Color value provided here
+ * will be actually submitted at output commit time via "background_color"
+ * property.
+ * For example to get red as background, set background = 0x00000000FFFF.
+ */
+void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background)
+{
+ igt_display_t *display = pipe->display;
+
+ LOG(display, "%s.%d: crtc_set_background(%lu)\n",
+ kmstest_pipe_name(pipe->pipe),
+ pipe->pipe, background);
+
+ pipe->background = background;
+
+ pipe->background_changed = true;
+}
+
+
void igt_wait_for_vblank(int drm_fd, enum pipe pipe)
{
drmVBlank wait_vbl;
diff --git a/lib/igt_kms.h b/lib/igt_kms.h
index 565df14..84e8456 100644
--- a/lib/igt_kms.h
+++ b/lib/igt_kms.h
@@ -207,6 +207,9 @@ struct igt_pipe {
#define IGT_MAX_PLANES 4
int n_planes;
igt_plane_t planes[IGT_MAX_PLANES];
+ uint64_t background; /* Background color MSB BGR 16bpc LSB */
+ uint32_t background_changed : 1;
+ uint32_t background_property;
};
typedef struct {
@@ -252,6 +255,7 @@ void igt_plane_set_position(igt_plane_t *plane, int x, int y);
void igt_plane_set_size(igt_plane_t *plane, int w, int h);
void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation);
+void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background);
void igt_wait_for_vblank(int drm_fd, enum pipe pipe);
diff --git a/tests/.gitignore b/tests/.gitignore
index 402e062..0f08ce0 100644
--- a/tests/.gitignore
+++ b/tests/.gitignore
@@ -141,6 +141,7 @@ kms_setmode
kms_sink_crc_basic
kms_universal_plane
kms_vblank
+kms_crtc_background_color
multi-tests.txt
pm_lpsp
pm_rc6_residency
diff --git a/tests/Android.mk b/tests/Android.mk
index 814b846..96a32b9 100644
--- a/tests/Android.mk
+++ b/tests/Android.mk
@@ -75,6 +75,7 @@ else
kms_rotation_crc \
kms_force_connector \
kms_flip_event_leak \
+ kms_crtc_background_color \
kms_pwrite_crc
IGT_LOCAL_CFLAGS += -DANDROID_HAS_CAIRO=0
endif
diff --git a/tests/Makefile.sources b/tests/Makefile.sources
index a165978..5235b5f 100644
--- a/tests/Makefile.sources
+++ b/tests/Makefile.sources
@@ -82,6 +82,7 @@ TESTS_progs_M = \
kms_setmode \
kms_universal_plane \
kms_vblank \
+ kms_crtc_background_color \
pm_lpsp \
pm_rpm \
pm_rps \
diff --git a/tests/kms_crtc_background_color.c b/tests/kms_crtc_background_color.c
new file mode 100644
index 0000000..fb26411
--- /dev/null
+++ b/tests/kms_crtc_background_color.c
@@ -0,0 +1,195 @@
+/*
+ * Copyright © 2013,2014 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ */
+
+#include <math.h>
+
+#include "drmtest.h"
+#include "igt_debugfs.h"
+#include "igt_kms.h"
+#include "igt_core.h"
+
+IGT_TEST_DESCRIPTION("Test crtc background color feature");
+
+typedef struct {
+ int gfx_fd;
+ igt_display_t display;
+ struct igt_fb fb;
+ igt_crc_t ref_crc;
+ igt_pipe_crc_t *pipe_crc;
+} data_t;
+
+#define BLACK 0x000000 /* BGR 8bpc */
+#define CYAN 0xFFFF00 /* BGR 8bpc */
+#define PURPLE 0xFF00FF /* BGR 8bpc */
+#define WHITE 0xFFFFFF /* BGR 8bpc */
+
+#define BLACK64 0x000000000000 /* BGR 16bpc */
+#define CYAN64 0xFFFFFFFF0000 /* BGR 16bpc */
+#define PURPLE64 0xFFFF0000FFFF /* BGR 16bpc */
+#define YELLOW64 0x0000FFFFFFFF /* BGR 16bpc */
+#define WHITE64 0xFFFFFFFFFFFF /* BGR 16bpc */
+#define RED64 0x00000000FFFF /* BGR 16bpc */
+#define GREEN64 0x0000FFFF0000 /* BGR 16bpc */
+#define BLUE64 0xFFFF00000000 /* BGR 16bpc */
+
+static void
+paint_background(data_t *data, struct igt_fb *fb, drmModeModeInfo *mode,
+ uint32_t background, double alpha)
+{
+ cairo_t *cr;
+ int w, h;
+ double r, g, b;
+
+ w = mode->hdisplay;
+ h = mode->vdisplay;
+
+ cr = igt_get_cairo_ctx(data->gfx_fd, &data->fb);
+
+ /* Paint with background color */
+ r = (double) (background & 0xFF) / 255.0;
+ g = (double) ((background & 0xFF00) >> 8) / 255.0;
+ b = (double) ((background & 0xFF0000) >> 16) / 255.0;
+ igt_paint_color_alpha(cr, 0, 0, w, h, r, g, b, alpha);
+
+ cairo_destroy(cr);
+}
+
+static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe,
+ igt_plane_t *plane, int opaque_buffer, int plane_color,
+ uint64_t pipe_background_color)
+{
+ drmModeModeInfo *mode;
+ igt_display_t *display = &data->display;
+ int fb_id;
+ double alpha;
+
+ igt_output_set_pipe(output, pipe);
+
+ /* create the pipe_crc object for this pipe */
+ igt_pipe_crc_free(data->pipe_crc);
+ data->pipe_crc = igt_pipe_crc_new(pipe, INTEL_PIPE_CRC_SOURCE_AUTO);
+
+ mode = igt_output_get_mode(output);
+
+ fb_id = igt_create_fb(data->gfx_fd,
+ mode->hdisplay, mode->vdisplay,
+ DRM_FORMAT_XRGB8888,
+ LOCAL_DRM_FORMAT_MOD_NONE, /* tiled */
+ &data->fb);
+ igt_assert(fb_id);
+
+ /* To make FB pixel win with background color, set alpha as full opaque */
+ igt_crtc_set_background(plane->pipe, pipe_background_color);
+ if (opaque_buffer)
+ alpha = 1.0; /* alpha 1 is fully opque */
+ else
+ alpha = 0.0; /* alpha 0 is fully transparent */
+ paint_background(data, &data->fb, mode, plane_color, alpha);
+
+ igt_plane_set_fb(plane, &data->fb);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+}
+
+static void cleanup_crtc(data_t *data, igt_output_t *output, igt_plane_t *plane)
+{
+ igt_display_t *display = &data->display;
+
+ igt_pipe_crc_free(data->pipe_crc);
+ data->pipe_crc = NULL;
+
+ igt_remove_fb(data->gfx_fd, &data->fb);
+
+ igt_crtc_set_background(plane->pipe, BLACK64);
+ igt_plane_set_fb(plane, NULL);
+ igt_output_set_pipe(output, PIPE_ANY);
+
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+}
+
+static void test_crtc_background(data_t *data)
+{
+ igt_display_t *display = &data->display;
+ igt_output_t *output;
+ enum pipe pipe;
+ int valid_tests = 0;
+
+ igt_require(data->display.has_universal_planes);
+
+ for_each_connected_output(display, output) {
+ igt_plane_t *plane;
+
+ pipe = output->config.pipe;
+ igt_output_set_pipe(output, pipe);
+
+ plane = igt_output_get_plane(output, IGT_PLANE_PRIMARY);
+ igt_require(plane->pipe->background_property);
+
+ prepare_crtc(data, output, pipe, plane, 1, PURPLE, BLACK64);
+
+ /* Now set background without using a plane, i.e.,
+ * Disable the plane to let hw background color win blend. */
+ igt_plane_set_fb(plane, NULL);
+ igt_crtc_set_background(plane->pipe, PURPLE64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ /* Try few other background colors */
+ igt_crtc_set_background(plane->pipe, CYAN64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ igt_crtc_set_background(plane->pipe, YELLOW64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ igt_crtc_set_background(plane->pipe, RED64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ igt_crtc_set_background(plane->pipe, GREEN64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ igt_crtc_set_background(plane->pipe, BLUE64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ igt_crtc_set_background(plane->pipe, WHITE64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ valid_tests++;
+ cleanup_crtc(data, output, plane);
+ }
+ igt_require_f(valid_tests, "no valid crtc/connector combinations found\n");
+}
+
+igt_main
+{
+ data_t data = {};
+
+ igt_skip_on_simulation();
+
+ data.gfx_fd = drm_open_any();
+ igt_require_pipe_crc();
+ igt_display_init(&data.display, data.gfx_fd);
+
+ igt_subtest("pipe-background")
+ test_crtc_background(&data);
+
+ igt_display_fini(&data.display);
+}
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v3] i-g-t: Adding test case to test background color.
2015-03-30 20:44 ` [PATCH v3] " Chandra Konduru
@ 2015-04-14 17:02 ` Thomas Wood
2015-04-14 17:24 ` Konduru, Chandra
0 siblings, 1 reply; 7+ messages in thread
From: Thomas Wood @ 2015-04-14 17:02 UTC (permalink / raw)
To: Chandra Konduru; +Cc: Intel Graphics Development
On 30 March 2015 at 21:44, Chandra Konduru <chandra.konduru@intel.com> wrote:
> From: chandra konduru <chandra.konduru@intel.com>
>
> Adding i-g-t test case to test display crtc background color.
>
> v2:
> - Added IGT_TEST_DESCRIPTION() (Thomas Wood)
> - Added to .gitignore (Thomas Wood)
> - Added additional details to function header (Thomas Wood)
> - Simplified igt_main (Thomas Wood)
>
> v3:
> - rebased to latest master (me)
> - took sleep calls out (Daniel)
> - use new tiled types when calling igt_create_fb (me)
I've pushed this patch and the panel fitting and plane scaling test
cases, but I have switched them all to use igt_simple_main and remove
the single subtest, which I think was your intention in v2 of this
patch.
>
> Signed-off-by: chandra konduru <chandra.konduru@intel.com>
> ---
> lib/igt_kms.c | 61 ++++++++++++
> lib/igt_kms.h | 4 +
> tests/.gitignore | 1 +
> tests/Android.mk | 1 +
> tests/Makefile.sources | 1 +
> tests/kms_crtc_background_color.c | 195 ++++++++++++++++++++++++++++++++++++++
> 6 files changed, 263 insertions(+)
> create mode 100644 tests/kms_crtc_background_color.c
>
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index 9c131f0..9c46e18 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -940,6 +940,22 @@ igt_plane_set_property(igt_plane_t *plane, uint32_t prop_id, uint64_t value)
> DRM_MODE_OBJECT_PLANE, prop_id, value);
> }
>
> +static bool
> +get_crtc_property(int drm_fd, uint32_t crtc_id, const char *name,
> + uint32_t *prop_id /* out */, uint64_t *value /* out */,
> + drmModePropertyPtr *prop /* out */)
> +{
> + return kmstest_get_property(drm_fd, crtc_id, DRM_MODE_OBJECT_CRTC,
> + name, prop_id, value, prop);
> +}
> +
> +static void
> +igt_crtc_set_property(igt_output_t *output, uint32_t prop_id, uint64_t value)
> +{
> + drmModeObjectSetProperty(output->display->drm_fd,
> + output->config.crtc->crtc_id, DRM_MODE_OBJECT_CRTC, prop_id, value);
> +}
> +
> /*
> * Walk a plane's property list to determine its type. If we don't
> * find a type property, then the kernel doesn't support universal
> @@ -1097,6 +1113,7 @@ void igt_display_init(igt_display_t *display, int drm_fd)
> igt_assert(display->outputs);
>
> for (i = 0; i < display->n_outputs; i++) {
> + int j;
> igt_output_t *output = &display->outputs[i];
>
> /*
> @@ -1108,6 +1125,19 @@ void igt_display_init(igt_display_t *display, int drm_fd)
> output->display = display;
>
> igt_output_refresh(output);
> +
> + for (j = 0; j < display->n_pipes; j++) {
> + uint64_t prop_value;
> + igt_pipe_t *pipe = &display->pipes[j];
> + if (output->config.crtc) {
> + get_crtc_property(display->drm_fd, output->config.crtc->crtc_id,
> + "background_color",
> + &pipe->background_property,
> + &prop_value,
> + NULL);
> + pipe->background = (uint32_t)prop_value;
> + }
> + }
> }
>
> drmModeFreePlaneResources(plane_resources);
> @@ -1527,6 +1557,13 @@ static int igt_output_commit(igt_output_t *output,
>
> pipe = igt_output_get_driving_pipe(output);
>
> + if (pipe->background_changed) {
> + igt_crtc_set_property(output, pipe->background_property,
> + pipe->background);
> +
> + pipe->background_changed = false;
> + }
> +
> for (i = 0; i < pipe->n_planes; i++) {
> igt_plane_t *plane = &pipe->planes[i];
>
> @@ -1779,6 +1816,30 @@ void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation)
> plane->rotation_changed = true;
> }
>
> +/**
> + * igt_crtc_set_background:
> + * @pipe: pipe pointer to which background color to be set
> + * @background: background color value in BGR 16bpc
> + *
> + * Sets background color for requested pipe. Color value provided here
> + * will be actually submitted at output commit time via "background_color"
> + * property.
> + * For example to get red as background, set background = 0x00000000FFFF.
> + */
> +void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background)
> +{
> + igt_display_t *display = pipe->display;
> +
> + LOG(display, "%s.%d: crtc_set_background(%lu)\n",
> + kmstest_pipe_name(pipe->pipe),
> + pipe->pipe, background);
> +
> + pipe->background = background;
> +
> + pipe->background_changed = true;
> +}
> +
> +
> void igt_wait_for_vblank(int drm_fd, enum pipe pipe)
> {
> drmVBlank wait_vbl;
> diff --git a/lib/igt_kms.h b/lib/igt_kms.h
> index 565df14..84e8456 100644
> --- a/lib/igt_kms.h
> +++ b/lib/igt_kms.h
> @@ -207,6 +207,9 @@ struct igt_pipe {
> #define IGT_MAX_PLANES 4
> int n_planes;
> igt_plane_t planes[IGT_MAX_PLANES];
> + uint64_t background; /* Background color MSB BGR 16bpc LSB */
> + uint32_t background_changed : 1;
> + uint32_t background_property;
> };
>
> typedef struct {
> @@ -252,6 +255,7 @@ void igt_plane_set_position(igt_plane_t *plane, int x, int y);
> void igt_plane_set_size(igt_plane_t *plane, int w, int h);
> void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
> void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation);
> +void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background);
>
> void igt_wait_for_vblank(int drm_fd, enum pipe pipe);
>
> diff --git a/tests/.gitignore b/tests/.gitignore
> index 402e062..0f08ce0 100644
> --- a/tests/.gitignore
> +++ b/tests/.gitignore
> @@ -141,6 +141,7 @@ kms_setmode
> kms_sink_crc_basic
> kms_universal_plane
> kms_vblank
> +kms_crtc_background_color
> multi-tests.txt
> pm_lpsp
> pm_rc6_residency
> diff --git a/tests/Android.mk b/tests/Android.mk
> index 814b846..96a32b9 100644
> --- a/tests/Android.mk
> +++ b/tests/Android.mk
> @@ -75,6 +75,7 @@ else
> kms_rotation_crc \
> kms_force_connector \
> kms_flip_event_leak \
> + kms_crtc_background_color \
> kms_pwrite_crc
> IGT_LOCAL_CFLAGS += -DANDROID_HAS_CAIRO=0
> endif
> diff --git a/tests/Makefile.sources b/tests/Makefile.sources
> index a165978..5235b5f 100644
> --- a/tests/Makefile.sources
> +++ b/tests/Makefile.sources
> @@ -82,6 +82,7 @@ TESTS_progs_M = \
> kms_setmode \
> kms_universal_plane \
> kms_vblank \
> + kms_crtc_background_color \
> pm_lpsp \
> pm_rpm \
> pm_rps \
> diff --git a/tests/kms_crtc_background_color.c b/tests/kms_crtc_background_color.c
> new file mode 100644
> index 0000000..fb26411
> --- /dev/null
> +++ b/tests/kms_crtc_background_color.c
> @@ -0,0 +1,195 @@
> +/*
> + * Copyright © 2013,2014 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> + * IN THE SOFTWARE.
> + *
> + */
> +
> +#include <math.h>
> +
> +#include "drmtest.h"
> +#include "igt_debugfs.h"
> +#include "igt_kms.h"
> +#include "igt_core.h"
> +
> +IGT_TEST_DESCRIPTION("Test crtc background color feature");
> +
> +typedef struct {
> + int gfx_fd;
> + igt_display_t display;
> + struct igt_fb fb;
> + igt_crc_t ref_crc;
> + igt_pipe_crc_t *pipe_crc;
> +} data_t;
> +
> +#define BLACK 0x000000 /* BGR 8bpc */
> +#define CYAN 0xFFFF00 /* BGR 8bpc */
> +#define PURPLE 0xFF00FF /* BGR 8bpc */
> +#define WHITE 0xFFFFFF /* BGR 8bpc */
> +
> +#define BLACK64 0x000000000000 /* BGR 16bpc */
> +#define CYAN64 0xFFFFFFFF0000 /* BGR 16bpc */
> +#define PURPLE64 0xFFFF0000FFFF /* BGR 16bpc */
> +#define YELLOW64 0x0000FFFFFFFF /* BGR 16bpc */
> +#define WHITE64 0xFFFFFFFFFFFF /* BGR 16bpc */
> +#define RED64 0x00000000FFFF /* BGR 16bpc */
> +#define GREEN64 0x0000FFFF0000 /* BGR 16bpc */
> +#define BLUE64 0xFFFF00000000 /* BGR 16bpc */
> +
> +static void
> +paint_background(data_t *data, struct igt_fb *fb, drmModeModeInfo *mode,
> + uint32_t background, double alpha)
> +{
> + cairo_t *cr;
> + int w, h;
> + double r, g, b;
> +
> + w = mode->hdisplay;
> + h = mode->vdisplay;
> +
> + cr = igt_get_cairo_ctx(data->gfx_fd, &data->fb);
> +
> + /* Paint with background color */
> + r = (double) (background & 0xFF) / 255.0;
> + g = (double) ((background & 0xFF00) >> 8) / 255.0;
> + b = (double) ((background & 0xFF0000) >> 16) / 255.0;
> + igt_paint_color_alpha(cr, 0, 0, w, h, r, g, b, alpha);
> +
> + cairo_destroy(cr);
> +}
> +
> +static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe,
> + igt_plane_t *plane, int opaque_buffer, int plane_color,
> + uint64_t pipe_background_color)
> +{
> + drmModeModeInfo *mode;
> + igt_display_t *display = &data->display;
> + int fb_id;
> + double alpha;
> +
> + igt_output_set_pipe(output, pipe);
> +
> + /* create the pipe_crc object for this pipe */
> + igt_pipe_crc_free(data->pipe_crc);
> + data->pipe_crc = igt_pipe_crc_new(pipe, INTEL_PIPE_CRC_SOURCE_AUTO);
> +
> + mode = igt_output_get_mode(output);
> +
> + fb_id = igt_create_fb(data->gfx_fd,
> + mode->hdisplay, mode->vdisplay,
> + DRM_FORMAT_XRGB8888,
> + LOCAL_DRM_FORMAT_MOD_NONE, /* tiled */
> + &data->fb);
> + igt_assert(fb_id);
> +
> + /* To make FB pixel win with background color, set alpha as full opaque */
> + igt_crtc_set_background(plane->pipe, pipe_background_color);
> + if (opaque_buffer)
> + alpha = 1.0; /* alpha 1 is fully opque */
> + else
> + alpha = 0.0; /* alpha 0 is fully transparent */
> + paint_background(data, &data->fb, mode, plane_color, alpha);
> +
> + igt_plane_set_fb(plane, &data->fb);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +}
> +
> +static void cleanup_crtc(data_t *data, igt_output_t *output, igt_plane_t *plane)
> +{
> + igt_display_t *display = &data->display;
> +
> + igt_pipe_crc_free(data->pipe_crc);
> + data->pipe_crc = NULL;
> +
> + igt_remove_fb(data->gfx_fd, &data->fb);
> +
> + igt_crtc_set_background(plane->pipe, BLACK64);
> + igt_plane_set_fb(plane, NULL);
> + igt_output_set_pipe(output, PIPE_ANY);
> +
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +}
> +
> +static void test_crtc_background(data_t *data)
> +{
> + igt_display_t *display = &data->display;
> + igt_output_t *output;
> + enum pipe pipe;
> + int valid_tests = 0;
> +
> + igt_require(data->display.has_universal_planes);
> +
> + for_each_connected_output(display, output) {
> + igt_plane_t *plane;
> +
> + pipe = output->config.pipe;
> + igt_output_set_pipe(output, pipe);
> +
> + plane = igt_output_get_plane(output, IGT_PLANE_PRIMARY);
> + igt_require(plane->pipe->background_property);
> +
> + prepare_crtc(data, output, pipe, plane, 1, PURPLE, BLACK64);
> +
> + /* Now set background without using a plane, i.e.,
> + * Disable the plane to let hw background color win blend. */
> + igt_plane_set_fb(plane, NULL);
> + igt_crtc_set_background(plane->pipe, PURPLE64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + /* Try few other background colors */
> + igt_crtc_set_background(plane->pipe, CYAN64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + igt_crtc_set_background(plane->pipe, YELLOW64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + igt_crtc_set_background(plane->pipe, RED64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + igt_crtc_set_background(plane->pipe, GREEN64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + igt_crtc_set_background(plane->pipe, BLUE64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + igt_crtc_set_background(plane->pipe, WHITE64);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + valid_tests++;
> + cleanup_crtc(data, output, plane);
> + }
> + igt_require_f(valid_tests, "no valid crtc/connector combinations found\n");
> +}
> +
> +igt_main
> +{
> + data_t data = {};
> +
> + igt_skip_on_simulation();
> +
> + data.gfx_fd = drm_open_any();
> + igt_require_pipe_crc();
> + igt_display_init(&data.display, data.gfx_fd);
> +
> + igt_subtest("pipe-background")
> + test_crtc_background(&data);
> +
> + igt_display_fini(&data.display);
> +}
> --
> 1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v3] i-g-t: Adding test case to test background color.
2015-04-14 17:02 ` Thomas Wood
@ 2015-04-14 17:24 ` Konduru, Chandra
0 siblings, 0 replies; 7+ messages in thread
From: Konduru, Chandra @ 2015-04-14 17:24 UTC (permalink / raw)
To: Wood, Thomas; +Cc: Intel Graphics Development
> -----Original Message-----
> From: Thomas Wood [mailto:thomas.wood@intel.com]
> Sent: Tuesday, April 14, 2015 10:02 AM
> To: Konduru, Chandra
> Cc: Intel Graphics Development
> Subject: Re: [Intel-gfx] [PATCH v3] i-g-t: Adding test case to test background
> color.
>
> On 30 March 2015 at 21:44, Chandra Konduru <chandra.konduru@intel.com>
> wrote:
> > From: chandra konduru <chandra.konduru@intel.com>
> >
> > Adding i-g-t test case to test display crtc background color.
> >
> > v2:
> > - Added IGT_TEST_DESCRIPTION() (Thomas Wood)
> > - Added to .gitignore (Thomas Wood)
> > - Added additional details to function header (Thomas Wood)
> > - Simplified igt_main (Thomas Wood)
> >
> > v3:
> > - rebased to latest master (me)
> > - took sleep calls out (Daniel)
> > - use new tiled types when calling igt_create_fb (me)
>
> I've pushed this patch and the panel fitting and plane scaling test cases, but I
> have switched them all to use igt_simple_main and remove the single subtest,
> which I think was your intention in v2 of this patch.
That should be ok.
Thanks for merging.
>
>
> >
> > Signed-off-by: chandra konduru <chandra.konduru@intel.com>
> > ---
> > lib/igt_kms.c | 61 ++++++++++++
> > lib/igt_kms.h | 4 +
> > tests/.gitignore | 1 +
> > tests/Android.mk | 1 +
> > tests/Makefile.sources | 1 +
> > tests/kms_crtc_background_color.c | 195
> > ++++++++++++++++++++++++++++++++++++++
> > 6 files changed, 263 insertions(+)
> > create mode 100644 tests/kms_crtc_background_color.c
> >
> > diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 9c131f0..9c46e18
> > 100644
> > --- a/lib/igt_kms.c
> > +++ b/lib/igt_kms.c
> > @@ -940,6 +940,22 @@ igt_plane_set_property(igt_plane_t *plane, uint32_t
> prop_id, uint64_t value)
> > DRM_MODE_OBJECT_PLANE, prop_id,
> > value); }
> >
> > +static bool
> > +get_crtc_property(int drm_fd, uint32_t crtc_id, const char *name,
> > + uint32_t *prop_id /* out */, uint64_t *value /* out */,
> > + drmModePropertyPtr *prop /* out */) {
> > + return kmstest_get_property(drm_fd, crtc_id,
> DRM_MODE_OBJECT_CRTC,
> > + name, prop_id, value, prop); }
> > +
> > +static void
> > +igt_crtc_set_property(igt_output_t *output, uint32_t prop_id,
> > +uint64_t value) {
> > + drmModeObjectSetProperty(output->display->drm_fd,
> > + output->config.crtc->crtc_id, DRM_MODE_OBJECT_CRTC,
> > +prop_id, value); }
> > +
> > /*
> > * Walk a plane's property list to determine its type. If we don't
> > * find a type property, then the kernel doesn't support universal @@
> > -1097,6 +1113,7 @@ void igt_display_init(igt_display_t *display, int drm_fd)
> > igt_assert(display->outputs);
> >
> > for (i = 0; i < display->n_outputs; i++) {
> > + int j;
> > igt_output_t *output = &display->outputs[i];
> >
> > /*
> > @@ -1108,6 +1125,19 @@ void igt_display_init(igt_display_t *display, int
> drm_fd)
> > output->display = display;
> >
> > igt_output_refresh(output);
> > +
> > + for (j = 0; j < display->n_pipes; j++) {
> > + uint64_t prop_value;
> > + igt_pipe_t *pipe = &display->pipes[j];
> > + if (output->config.crtc) {
> > + get_crtc_property(display->drm_fd, output->config.crtc-
> >crtc_id,
> > + "background_color",
> > + &pipe->background_property,
> > + &prop_value,
> > + NULL);
> > + pipe->background = (uint32_t)prop_value;
> > + }
> > + }
> > }
> >
> > drmModeFreePlaneResources(plane_resources);
> > @@ -1527,6 +1557,13 @@ static int igt_output_commit(igt_output_t
> > *output,
> >
> > pipe = igt_output_get_driving_pipe(output);
> >
> > + if (pipe->background_changed) {
> > + igt_crtc_set_property(output, pipe->background_property,
> > + pipe->background);
> > +
> > + pipe->background_changed = false;
> > + }
> > +
> > for (i = 0; i < pipe->n_planes; i++) {
> > igt_plane_t *plane = &pipe->planes[i];
> >
> > @@ -1779,6 +1816,30 @@ void igt_plane_set_rotation(igt_plane_t *plane,
> igt_rotation_t rotation)
> > plane->rotation_changed = true; }
> >
> > +/**
> > + * igt_crtc_set_background:
> > + * @pipe: pipe pointer to which background color to be set
> > + * @background: background color value in BGR 16bpc
> > + *
> > + * Sets background color for requested pipe. Color value provided
> > +here
> > + * will be actually submitted at output commit time via "background_color"
> > + * property.
> > + * For example to get red as background, set background = 0x00000000FFFF.
> > + */
> > +void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background) {
> > + igt_display_t *display = pipe->display;
> > +
> > + LOG(display, "%s.%d: crtc_set_background(%lu)\n",
> > + kmstest_pipe_name(pipe->pipe),
> > + pipe->pipe, background);
> > +
> > + pipe->background = background;
> > +
> > + pipe->background_changed = true; }
> > +
> > +
> > void igt_wait_for_vblank(int drm_fd, enum pipe pipe) {
> > drmVBlank wait_vbl;
> > diff --git a/lib/igt_kms.h b/lib/igt_kms.h index 565df14..84e8456
> > 100644
> > --- a/lib/igt_kms.h
> > +++ b/lib/igt_kms.h
> > @@ -207,6 +207,9 @@ struct igt_pipe {
> > #define IGT_MAX_PLANES 4
> > int n_planes;
> > igt_plane_t planes[IGT_MAX_PLANES];
> > + uint64_t background; /* Background color MSB BGR 16bpc LSB */
> > + uint32_t background_changed : 1;
> > + uint32_t background_property;
> > };
> >
> > typedef struct {
> > @@ -252,6 +255,7 @@ void igt_plane_set_position(igt_plane_t *plane,
> > int x, int y); void igt_plane_set_size(igt_plane_t *plane, int w, int
> > h); void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
> > void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t
> > rotation);
> > +void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background);
> >
> > void igt_wait_for_vblank(int drm_fd, enum pipe pipe);
> >
> > diff --git a/tests/.gitignore b/tests/.gitignore index
> > 402e062..0f08ce0 100644
> > --- a/tests/.gitignore
> > +++ b/tests/.gitignore
> > @@ -141,6 +141,7 @@ kms_setmode
> > kms_sink_crc_basic
> > kms_universal_plane
> > kms_vblank
> > +kms_crtc_background_color
> > multi-tests.txt
> > pm_lpsp
> > pm_rc6_residency
> > diff --git a/tests/Android.mk b/tests/Android.mk index
> > 814b846..96a32b9 100644
> > --- a/tests/Android.mk
> > +++ b/tests/Android.mk
> > @@ -75,6 +75,7 @@ else
> > kms_rotation_crc \
> > kms_force_connector \
> > kms_flip_event_leak \
> > + kms_crtc_background_color \
> > kms_pwrite_crc
> > IGT_LOCAL_CFLAGS += -DANDROID_HAS_CAIRO=0 endif diff --git
> > a/tests/Makefile.sources b/tests/Makefile.sources index
> > a165978..5235b5f 100644
> > --- a/tests/Makefile.sources
> > +++ b/tests/Makefile.sources
> > @@ -82,6 +82,7 @@ TESTS_progs_M = \
> > kms_setmode \
> > kms_universal_plane \
> > kms_vblank \
> > + kms_crtc_background_color \
> > pm_lpsp \
> > pm_rpm \
> > pm_rps \
> > diff --git a/tests/kms_crtc_background_color.c
> > b/tests/kms_crtc_background_color.c
> > new file mode 100644
> > index 0000000..fb26411
> > --- /dev/null
> > +++ b/tests/kms_crtc_background_color.c
> > @@ -0,0 +1,195 @@
> > +/*
> > + * Copyright © 2013,2014 Intel Corporation
> > + *
> > + * Permission is hereby granted, free of charge, to any person
> > +obtaining a
> > + * copy of this software and associated documentation files (the
> > +"Software"),
> > + * to deal in the Software without restriction, including without
> > +limitation
> > + * the rights to use, copy, modify, merge, publish, distribute,
> > +sublicense,
> > + * and/or sell copies of the Software, and to permit persons to whom
> > +the
> > + * Software is furnished to do so, subject to the following conditions:
> > + *
> > + * The above copyright notice and this permission notice (including
> > +the next
> > + * paragraph) shall be included in all copies or substantial portions
> > +of the
> > + * Software.
> > + *
> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> > +EXPRESS OR
> > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> > +MERCHANTABILITY,
> > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO
> EVENT
> > +SHALL
> > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
> DAMAGES
> > +OR OTHER
> > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> > +ARISING
> > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> > +OTHER DEALINGS
> > + * IN THE SOFTWARE.
> > + *
> > + */
> > +
> > +#include <math.h>
> > +
> > +#include "drmtest.h"
> > +#include "igt_debugfs.h"
> > +#include "igt_kms.h"
> > +#include "igt_core.h"
> > +
> > +IGT_TEST_DESCRIPTION("Test crtc background color feature");
> > +
> > +typedef struct {
> > + int gfx_fd;
> > + igt_display_t display;
> > + struct igt_fb fb;
> > + igt_crc_t ref_crc;
> > + igt_pipe_crc_t *pipe_crc;
> > +} data_t;
> > +
> > +#define BLACK 0x000000 /* BGR 8bpc */
> > +#define CYAN 0xFFFF00 /* BGR 8bpc */
> > +#define PURPLE 0xFF00FF /* BGR 8bpc */
> > +#define WHITE 0xFFFFFF /* BGR 8bpc */
> > +
> > +#define BLACK64 0x000000000000 /* BGR 16bpc */
> > +#define CYAN64 0xFFFFFFFF0000 /* BGR 16bpc */
> > +#define PURPLE64 0xFFFF0000FFFF /* BGR 16bpc */
> > +#define YELLOW64 0x0000FFFFFFFF /* BGR 16bpc */
> > +#define WHITE64 0xFFFFFFFFFFFF /* BGR 16bpc */
> > +#define RED64 0x00000000FFFF /* BGR 16bpc */
> > +#define GREEN64 0x0000FFFF0000 /* BGR 16bpc */
> > +#define BLUE64 0xFFFF00000000 /* BGR 16bpc */
> > +
> > +static void
> > +paint_background(data_t *data, struct igt_fb *fb, drmModeModeInfo
> *mode,
> > + uint32_t background, double alpha) {
> > + cairo_t *cr;
> > + int w, h;
> > + double r, g, b;
> > +
> > + w = mode->hdisplay;
> > + h = mode->vdisplay;
> > +
> > + cr = igt_get_cairo_ctx(data->gfx_fd, &data->fb);
> > +
> > + /* Paint with background color */
> > + r = (double) (background & 0xFF) / 255.0;
> > + g = (double) ((background & 0xFF00) >> 8) / 255.0;
> > + b = (double) ((background & 0xFF0000) >> 16) / 255.0;
> > + igt_paint_color_alpha(cr, 0, 0, w, h, r, g, b, alpha);
> > +
> > + cairo_destroy(cr);
> > +}
> > +
> > +static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe,
> > + igt_plane_t *plane, int opaque_buffer, int plane_color,
> > + uint64_t pipe_background_color) {
> > + drmModeModeInfo *mode;
> > + igt_display_t *display = &data->display;
> > + int fb_id;
> > + double alpha;
> > +
> > + igt_output_set_pipe(output, pipe);
> > +
> > + /* create the pipe_crc object for this pipe */
> > + igt_pipe_crc_free(data->pipe_crc);
> > + data->pipe_crc = igt_pipe_crc_new(pipe,
> > + INTEL_PIPE_CRC_SOURCE_AUTO);
> > +
> > + mode = igt_output_get_mode(output);
> > +
> > + fb_id = igt_create_fb(data->gfx_fd,
> > + mode->hdisplay, mode->vdisplay,
> > + DRM_FORMAT_XRGB8888,
> > + LOCAL_DRM_FORMAT_MOD_NONE, /* tiled */
> > + &data->fb);
> > + igt_assert(fb_id);
> > +
> > + /* To make FB pixel win with background color, set alpha as full opaque
> */
> > + igt_crtc_set_background(plane->pipe, pipe_background_color);
> > + if (opaque_buffer)
> > + alpha = 1.0; /* alpha 1 is fully opque */
> > + else
> > + alpha = 0.0; /* alpha 0 is fully transparent */
> > + paint_background(data, &data->fb, mode, plane_color, alpha);
> > +
> > + igt_plane_set_fb(plane, &data->fb);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL); }
> > +
> > +static void cleanup_crtc(data_t *data, igt_output_t *output,
> > +igt_plane_t *plane) {
> > + igt_display_t *display = &data->display;
> > +
> > + igt_pipe_crc_free(data->pipe_crc);
> > + data->pipe_crc = NULL;
> > +
> > + igt_remove_fb(data->gfx_fd, &data->fb);
> > +
> > + igt_crtc_set_background(plane->pipe, BLACK64);
> > + igt_plane_set_fb(plane, NULL);
> > + igt_output_set_pipe(output, PIPE_ANY);
> > +
> > + igt_display_commit2(display, COMMIT_UNIVERSAL); }
> > +
> > +static void test_crtc_background(data_t *data) {
> > + igt_display_t *display = &data->display;
> > + igt_output_t *output;
> > + enum pipe pipe;
> > + int valid_tests = 0;
> > +
> > + igt_require(data->display.has_universal_planes);
> > +
> > + for_each_connected_output(display, output) {
> > + igt_plane_t *plane;
> > +
> > + pipe = output->config.pipe;
> > + igt_output_set_pipe(output, pipe);
> > +
> > + plane = igt_output_get_plane(output, IGT_PLANE_PRIMARY);
> > + igt_require(plane->pipe->background_property);
> > +
> > + prepare_crtc(data, output, pipe, plane, 1, PURPLE,
> > + BLACK64);
> > +
> > + /* Now set background without using a plane, i.e.,
> > + * Disable the plane to let hw background color win blend. */
> > + igt_plane_set_fb(plane, NULL);
> > + igt_crtc_set_background(plane->pipe, PURPLE64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + /* Try few other background colors */
> > + igt_crtc_set_background(plane->pipe, CYAN64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + igt_crtc_set_background(plane->pipe, YELLOW64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + igt_crtc_set_background(plane->pipe, RED64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + igt_crtc_set_background(plane->pipe, GREEN64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + igt_crtc_set_background(plane->pipe, BLUE64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + igt_crtc_set_background(plane->pipe, WHITE64);
> > + igt_display_commit2(display, COMMIT_UNIVERSAL);
> > +
> > + valid_tests++;
> > + cleanup_crtc(data, output, plane);
> > + }
> > + igt_require_f(valid_tests, "no valid crtc/connector
> > +combinations found\n"); }
> > +
> > +igt_main
> > +{
> > + data_t data = {};
> > +
> > + igt_skip_on_simulation();
> > +
> > + data.gfx_fd = drm_open_any();
> > + igt_require_pipe_crc();
> > + igt_display_init(&data.display, data.gfx_fd);
> > +
> > + igt_subtest("pipe-background")
> > + test_crtc_background(&data);
> > +
> > + igt_display_fini(&data.display); }
> > --
> > 1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-04-14 17:24 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-28 1:25 [PATCH v2] i-g-t: Adding test case to test background color Chandra Konduru
2015-03-02 17:53 ` Daniel Vetter
2015-03-24 18:51 ` Konduru, Chandra
2015-03-25 13:48 ` Daniel Vetter
2015-03-30 20:44 ` [PATCH v3] " Chandra Konduru
2015-04-14 17:02 ` Thomas Wood
2015-04-14 17:24 ` Konduru, Chandra
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.