diff mbox

[v2] i-g-t: Adding test case to test background color.

Message ID 1425086752-5558-1-git-send-email-chandra.konduru@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Chandra Konduru Feb. 28, 2015, 1:25 a.m. UTC
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

Comments

Daniel Vetter March 2, 2015, 5:53 p.m. UTC | #1
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
Chandra Konduru March 24, 2015, 6:51 p.m. UTC | #2
> -----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
Daniel Vetter March 25, 2015, 1:48 p.m. UTC | #3
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
diff mbox

Patch

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);
+}