[PATCH i-g-t v2 2/4] scripts/igt_doc.py: ignore "all" when producing testlists

Kamil Konieczny kamil.konieczny at linux.intel.com
Mon Feb 26 10:58:52 UTC 2024


Hi Mauro,
On 2024-02-22 at 12:45:26 +0100, Mauro Carvalho Chehab wrote:
> From: Mauro Carvalho Chehab <mchehab at kernel.org>
> 
> The meaning of "all" is an special case for blocklists, meant to be
------------------------- ^^
s/an/a/

> used only when some block lists are to be applied everywhere. It is
> not a testlist itself.
> 
> So, exclude it as a different kind of GPU, as the default case is
> already covered.
> 
> Signed-off-by: Mauro Carvalho Chehab <mchehab at kernel.org>
> ---
>  scripts/igt_doc.py | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/scripts/igt_doc.py b/scripts/igt_doc.py
> index 7404e3d0934c..7985cdd5ac5e 100755
> --- a/scripts/igt_doc.py
> +++ b/scripts/igt_doc.py
> @@ -229,6 +229,10 @@ class IntelciTestlist:
>              for gpu, names in gpus.items():
>                  gpu = re.sub(r"[\W_]+", "-", gpu).lower()
>  
> +                # "all" is used used only as a default value for unlisted GPUs
----------------------------- ^^^^ ^^^^
Repeated word "used",
s/used used/used/

With that fixed:

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> +                if gpu == "all":
> +                    continue
> +
>                  dname = os.path.join(driver_path, gpu)
>                  try:
>                      os.makedirs(dname)
> -- 
> 2.43.2
> 


More information about the igt-dev mailing list