Re: [PATCH v5 9/9] drm/selftests: Add tests for drm_format_info* helpers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Oct 19, 2018 at 05:29:15PM +0200, Daniel Vetter wrote:
> On Fri, Oct 19, 2018 at 11:57:52AM +0100, Alexandru Gheorghe wrote:
> > Add selftests for the following newly added functions:
> >  - drm_format_info_block_width
> >  - drm_format_info_block_height
> >  - drm_format_info_min_pitch
> > 
> > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@xxxxxxx>
> > ---
> >  drivers/gpu/drm/selftests/Makefile            |   3 +-
> >  .../gpu/drm/selftests/drm_modeset_selftests.h |   3 +
> >  drivers/gpu/drm/selftests/test-drm_format.c   | 290 ++++++++++++++++++
> >  .../drm/selftests/test-drm_modeset_common.h   |   3 +
> >  4 files changed, 298 insertions(+), 1 deletion(-)
> >  create mode 100644 drivers/gpu/drm/selftests/test-drm_format.c
> > 
> > diff --git a/drivers/gpu/drm/selftests/Makefile b/drivers/gpu/drm/selftests/Makefile
> > index 7e6581921da0..07b4f88b422a 100644
> > --- a/drivers/gpu/drm/selftests/Makefile
> > +++ b/drivers/gpu/drm/selftests/Makefile
> > @@ -1,3 +1,4 @@
> > -test-drm_modeset-y := test-drm_modeset_common.o test-drm_plane_helper.o
> > +test-drm_modeset-y := test-drm_modeset_common.o test-drm_plane_helper.o \
> > +                      test-drm_format.o
> >  
> >  obj-$(CONFIG_DRM_DEBUG_SELFTEST) += test-drm_mm.o test-drm_modeset.o
> > diff --git a/drivers/gpu/drm/selftests/drm_modeset_selftests.h b/drivers/gpu/drm/selftests/drm_modeset_selftests.h
> > index 9771290ed228..4e203ac8c134 100644
> > --- a/drivers/gpu/drm/selftests/drm_modeset_selftests.h
> > +++ b/drivers/gpu/drm/selftests/drm_modeset_selftests.h
> > @@ -7,3 +7,6 @@
> >   * Tests are executed in order by igt/drm_selftests_helper
> >   */
> >  selftest(check_plane_state, igt_check_plane_state)
> > +selftest(check_drm_format_block_width, igt_check_drm_format_block_width)
> > +selftest(check_drm_format_block_height, igt_check_drm_format_block_height)
> > +selftest(check_drm_format_min_pitch, igt_check_drm_format_min_pitch)
> > diff --git a/drivers/gpu/drm/selftests/test-drm_format.c b/drivers/gpu/drm/selftests/test-drm_format.c
> > new file mode 100644
> > index 000000000000..5abfd5e28d7d
> > --- /dev/null
> > +++ b/drivers/gpu/drm/selftests/test-drm_format.c
> > @@ -0,0 +1,290 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Test cases for the drm_format functions
> > + */
> > +
> > +#define pr_fmt(fmt) "drm_format: " fmt
> > +
> > +#include <linux/errno.h>
> > +#include <linux/kernel.h>
> > +
> > +#include <drm/drm_fourcc.h>
> > +
> > +#include "test-drm_modeset_common.h"
> > +
> > +int igt_check_drm_format_block_width(void *ignored)
> > +{
> > +	const struct drm_format_info *info = NULL;
> > +
> > +	/* Test invalid arguments */
> > +	FAIL_ON(drm_format_info_block_width(info, 0) != 0);
> > +	FAIL_ON(drm_format_info_block_width(info, -1) != 0);
> > +	FAIL_ON(drm_format_info_block_width(info, 1) != 0);
> > +
> > +	/* Test 1 plane format */
> > +	info = drm_format_info(DRM_FORMAT_XRGB4444);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_block_width(info, 0) != 1);
> > +	FAIL_ON(drm_format_info_block_width(info, 1) != 0);
> > +	FAIL_ON(drm_format_info_block_width(info, -1) != 0);
> > +
> > +	/* Test 2 planes format */
> > +	info = drm_format_info(DRM_FORMAT_NV12);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_block_width(info, 0) != 1);
> > +	FAIL_ON(drm_format_info_block_width(info, 1) != 1);
> > +	FAIL_ON(drm_format_info_block_width(info, 2) != 0);
> > +	FAIL_ON(drm_format_info_block_width(info, -1) != 0);
> > +
> > +	/* Test 3 planes format */
> > +	info = drm_format_info(DRM_FORMAT_YUV422);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_block_width(info, 0) != 1);
> > +	FAIL_ON(drm_format_info_block_width(info, 1) != 1);
> > +	FAIL_ON(drm_format_info_block_width(info, 2) != 1);
> > +	FAIL_ON(drm_format_info_block_width(info, 3) != 0);
> > +	FAIL_ON(drm_format_info_block_width(info, -1) != 0);
> > +
> > +	/* Test a tiled format */
> > +	info = drm_format_info(DRM_FORMAT_X0L0);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_block_width(info, 0) != 2);
> > +	FAIL_ON(drm_format_info_block_width(info, 1) != 0);
> > +	FAIL_ON(drm_format_info_block_width(info, -1) != 0);
> > +
> > +	return 0;
> > +}
> > +
> > +int igt_check_drm_format_block_height(void *ignored)
> > +{
> > +	const struct drm_format_info *info = NULL;
> > +
> > +	/* Test invalid arguments */
> > +	FAIL_ON(drm_format_info_block_height(info, 0) != 0);
> > +	FAIL_ON(drm_format_info_block_height(info, -1) != 0);
> > +	FAIL_ON(drm_format_info_block_height(info, 1) != 0);
> > +
> > +	/* Test 1 plane format */
> > +	info = drm_format_info(DRM_FORMAT_XRGB4444);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_block_height(info, 0) != 1);
> > +	FAIL_ON(drm_format_info_block_height(info, 1) != 0);
> > +	FAIL_ON(drm_format_info_block_height(info, -1) != 0);
> > +
> > +	/* Test 2 planes format */
> > +	info = drm_format_info(DRM_FORMAT_NV12);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_block_height(info, 0) != 1);
> > +	FAIL_ON(drm_format_info_block_height(info, 1) != 1);
> > +	FAIL_ON(drm_format_info_block_height(info, 2) != 0);
> > +	FAIL_ON(drm_format_info_block_height(info, -1) != 0);
> > +
> > +	/* Test 3 planes format */
> > +	info = drm_format_info(DRM_FORMAT_YUV422);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_block_height(info, 0) != 1);
> > +	FAIL_ON(drm_format_info_block_height(info, 1) != 1);
> > +	FAIL_ON(drm_format_info_block_height(info, 2) != 1);
> > +	FAIL_ON(drm_format_info_block_height(info, 3) != 0);
> > +	FAIL_ON(drm_format_info_block_height(info, -1) != 0);
> > +
> > +	/* Test a tiled format */
> > +	info = drm_format_info(DRM_FORMAT_X0L0);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_block_height(info, 0) != 2);
> > +	FAIL_ON(drm_format_info_block_height(info, 1) != 0);
> > +	FAIL_ON(drm_format_info_block_height(info, -1) != 0);
> > +
> > +	return 0;
> > +}
> > +
> > +int igt_check_drm_format_min_pitch(void *ignored)
> > +{
> > +	const struct drm_format_info *info = NULL;
> > +
> > +	/* Test invalid arguments */
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, -1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 0) != 0);
> > +
> > +	/* Test 1 plane 8 bits per pixel format */
> > +	info = drm_format_info(DRM_FORMAT_RGB332);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, -1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 0) != 0);
> > +
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1) != 1);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 2) != 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 640) != 640);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1024) != 1024);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1920) != 1920);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 4096) != 4096);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 671) != 671);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX) !=
> > +			(uint64_t)UINT_MAX);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, (UINT_MAX - 1)) !=
> > +			(uint64_t)(UINT_MAX - 1));
> > +
> > +	/* Test 1 plane 16 bits per pixel format */
> > +	info = drm_format_info(DRM_FORMAT_XRGB4444);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, -1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 0) != 0);
> > +
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1) != 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 2) != 4);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 640) != 1280);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1024) != 2048);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1920) != 3840);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 4096) != 8192);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 671) != 1342);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX) !=
> > +			(uint64_t)UINT_MAX * 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, (UINT_MAX - 1)) !=
> > +			(uint64_t)(UINT_MAX - 1) * 2);
> > +
> > +	/* Test 1 plane 24 bits per pixel format */
> > +	info = drm_format_info(DRM_FORMAT_RGB888);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, -1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 0) != 0);
> > +
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1) != 3);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 2) != 6);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 640) != 1920);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1024) != 3072);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1920) != 5760);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 4096) != 12288);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 671) != 2013);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX) !=
> > +			(uint64_t)UINT_MAX * 3);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX - 1) !=
> > +			(uint64_t)(UINT_MAX - 1) * 3);
> > +
> > +	/* Test 1 plane 32 bits per pixel format */
> > +	info = drm_format_info(DRM_FORMAT_ABGR8888);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, -1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 0) != 0);
> > +
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1) != 4);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 2) != 8);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 640) != 2560);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1024) != 4096);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1920) != 7680);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 4096) != 16384);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 671) != 2684);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX) !=
> > +			(uint64_t)UINT_MAX * 4);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX - 1) !=
> > +			(uint64_t)(UINT_MAX - 1) * 4);
> > +
> > +	/* Test 2 planes format */
> > +	info = drm_format_info(DRM_FORMAT_NV12);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, -1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, 0) != 0);
> > +
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1) != 1);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 1) != 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 2) != 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 1) != 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 640) != 640);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 320) != 640);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1024) != 1024);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 512) != 1024);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1920) != 1920);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 960) != 1920);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 4096) != 4096);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 2048) != 4096);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 671) != 671);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 336) != 672);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX) !=
> > +			(uint64_t)UINT_MAX);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, UINT_MAX / 2 + 1) !=
> > +			(uint64_t)UINT_MAX + 1);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, (UINT_MAX - 1)) !=
> > +			(uint64_t)(UINT_MAX - 1));
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, (UINT_MAX - 1) /  2) !=
> > +			(uint64_t)(UINT_MAX - 1));
> > +
> > +	/* Test 3 planes 8 bits per pixel format */
> > +	info = drm_format_info(DRM_FORMAT_YUV422);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, -1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 3, 0) != 0);
> > +
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1) != 1);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 1) != 1);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, 1) != 1);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 2) != 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 2) != 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, 2) != 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 640) != 640);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 320) != 320);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, 320) != 320);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1024) != 1024);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 512) != 512);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, 512) != 512);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1920) != 1920);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 960) != 960);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, 960) != 960);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 4096) != 4096);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 2048) != 2048);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, 2048) != 2048);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 671) != 671);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 336) != 336);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, 336) != 336);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX) !=
> > +			(uint64_t)UINT_MAX);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, UINT_MAX / 2 + 1) !=
> > +			(uint64_t)UINT_MAX / 2 + 1);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, UINT_MAX / 2 + 1) !=
> > +			(uint64_t)UINT_MAX / 2 + 1);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, (UINT_MAX - 1) / 2) !=
> > +			(uint64_t)(UINT_MAX - 1) / 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, (UINT_MAX - 1) / 2) !=
> > +			(uint64_t)(UINT_MAX - 1) / 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 2, (UINT_MAX - 1) / 2) !=
> > +			(uint64_t)(UINT_MAX - 1) / 2);
> > +
> > +	/* Test tiled format */
> > +	info = drm_format_info(DRM_FORMAT_X0L2);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, -1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 0) != 0);
> > +
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1) != 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 2) != 4);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 640) != 1280);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1024) != 2048);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1920) != 3840);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 4096) != 8192);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 671) != 1342);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX) !=
> > +			(uint64_t)UINT_MAX * 2);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX - 1) !=
> > +			(uint64_t)(UINT_MAX - 1) * 2);
> > +
> > +	/* Test format with cpp/char_per_block 0 */
> > +	info = drm_format_info(DRM_FORMAT_VUY101010);
> > +	FAIL_ON(!info);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, -1, 0) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 1, 0) != 0);
> > +
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, 1) != 0);
> > +	FAIL_ON(drm_format_info_min_pitch(info, 0, UINT_MAX) != 0);
> > +
> > +	return 0;
> > +}
> 
> Comprehensive, but also fairly boring. Anyway, it's a start, so:

Unit tests usually are :)

> 
> Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx>

Can I also get your reviewed-by on the patches that adds this functions ?

> 
> What I had in mind is maybe not quite so low-level unit testing, but
> exercising the entire framebuffer_create machinery. The recently
> drm_client_framebuffer_create essentially gives you that interface for
> modules (but I guess we could also EXPORT_SYMBOL_FOR_TESTS_ONLY,
> conditional on drm selftests being enabled, directly on
> drm_internal_framebuffer_create).

Yeah, that's exactly what I thought I started looking at it, but I got
discouraged, by the lack of mocking, so I decided to start small.

> 
> This needs some fairly minimal mocking of a drm_device with a bunch of
> dummy functions (throw them into test-drm_modeset_common.c if you feel
> like). Then go through a pile of valid and invalid combinations and make
> sure your ->fb_create driver callback is only called when when it should
> be (and not for anything invalid). I think that would yield the much more
> interesting testing ...

This combined with EXPORT_SYMBOL_FOR_TESTS_ONLY should be fairly
simple, but it still feels like a drop in the ocean, I can't even
test the other functions that patch [1] touches like:
drm_gem_fb_create_with_funcs and drm_fb_cma_get_gem_addr.

[1] https://lists.freedesktop.org/archives/dri-devel/2018-October/193671.html

> 
> That will also pave the ground for some rather more serious addfb testing
> in the future. And eventually even more serious testing on e.g. atomic
> helpers.
> 
> Anyway, if you feel like, would be awesome to follow up a bit in that
> direction.
> 
> Cheers, Daniel
> 
> > diff --git a/drivers/gpu/drm/selftests/test-drm_modeset_common.h b/drivers/gpu/drm/selftests/test-drm_modeset_common.h
> > index b0065a2eb067..592a6581b189 100644
> > --- a/drivers/gpu/drm/selftests/test-drm_modeset_common.h
> > +++ b/drivers/gpu/drm/selftests/test-drm_modeset_common.h
> > @@ -14,5 +14,8 @@
> >  #define FAIL_ON(x) FAIL((x), "%s", "FAIL_ON(" __stringify(x) ")\n")
> >  
> >  int igt_check_plane_state(void *ignored);
> > +int igt_check_drm_format_block_width(void *ignored);
> > +int igt_check_drm_format_block_height(void *ignored);
> > +int igt_check_drm_format_min_pitch(void *ignored);
> >  
> >  #endif
> > -- 
> > 2.18.0
> > 
> 
> -- 
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch

-- 
Cheers,
Alex G
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux