[igt-dev] [PATCH i-g-t 3/3] tests/gem_render_copy: Added skip for I915_TILING_Y and I915_TILING_Yf on DG2

[Zbigniew Kempczyński]

On Wed, Mar 02, 2022 at 01:12:53PM +0100, Zbigniew Kempczyński wrote:
> From: Arjun Melkaveri <arjun.melkaveri at intel.com>
> 
> Added skip for I915_TILING_Y and I915_TILING_Yf on DG2, as those tiling
> formats are not supported.

At the moment it is ok, but we need to add ccs support in the future.

Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>

--
Zbigniew

> 
> Signed-off-by: Arjun Melkaveri <arjun.melkaveri at intel.com>
> Cc: Sai Gowtham <sai.gowtham.ch at intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> ---
>  tests/i915/gem_render_copy.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/tests/i915/gem_render_copy.c b/tests/i915/gem_render_copy.c
> index d485e3415a..3da29ec4e3 100644
> --- a/tests/i915/gem_render_copy.c
> +++ b/tests/i915/gem_render_copy.c
> @@ -852,6 +852,12 @@ igt_main_args("dac", NULL, help_str, opt_handler, NULL)
>  			 force_vebox_dst_copy ? "vebox-" : "",
>  			 dst_mode);
>  		igt_subtest_with_dynamic(name) {
> +			igt_skip_on(IS_DG2(data.devid) &&
> +				    ((t->src_tiling == I915_TILING_Y) ||
> +				     (t->src_tiling == I915_TILING_Yf) ||
> +				     (t->dst_tiling == I915_TILING_Y) ||
> +				     (t->dst_tiling == I915_TILING_Yf)));
> +
>  			igt_require_f(data.vebox_copy || !vebox_copy_used,
>  				      "no vebox-copy function\n");
>  			for_each_combination(region_set, 1, set) {
> -- 
> 2.32.0
>