[PATCH i-g-t 10/24] tests/intel: Introduce Flat-ccs subcategory
Piecielska, Katarzyna
katarzyna.piecielska at intel.com
Thu Mar 7 09:50:02 UTC 2024
> 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 "/".
[Katarzyna] Looking at i915 for DII I see only 'Desktop \ Server'. If we want to standardize that - let's do that in separate commit.
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