[PATCH i-g-t 10/24] tests/intel: Introduce Flat-ccs subcategory

Mauro Carvalho Chehab mauro.chehab at linux.intel.com
Mon Mar 4 15:16:51 UTC 2024


On Mon,  4 Mar 2024 11:16:03 +0000
Katarzyna Piecielska <katarzyna.piecielska at intel.com> wrote:

> Let's introduce flat-ccs subcategory.
> 
> Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska at intel.com>
> Cc: Mauro Carvalho Chehab <mauro.chehab at linux.intel.com>
> Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> ---
>  tests/intel/gem_ccs.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/tests/intel/gem_ccs.c b/tests/intel/gem_ccs.c
> index 3442ecfd3..e6257aeac 100644
> --- a/tests/intel/gem_ccs.c
> +++ b/tests/intel/gem_ccs.c
> @@ -20,6 +20,9 @@
>   * TEST: gem ccs
>   * Description: Exercise gen12 blitter with and without flatccs compression
>   * Feature: flat_ccs_mapping
> + * Sub-category: Flat-ccs
> + * Category: Desktop \ Server
> + * Functionality: block-copy

LGTM.

A minor nitpick: on some places like "Desktop \ Server" you're using the
left/reverse backslash, while on others, you're using the right one. 

I would standardize it to always use "/".

I guess this can be solved on a separate series. So, for this one:

Reviewed-by: Mauro Carvalho Chehab <mchehab at kernel.org>

>   * Run type: FULL
>   *
>   * SUBTEST: block-copy-compressed


More information about the igt-dev mailing list