[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