Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t v3 0/3] Put "Run type" highest in documentation hierarchy
@ 2024-04-24  8:05 Katarzyna Piecielska
  2024-04-24  8:05 ` [PATCH i-g-t v3 1/3] tests/intel: Put "Run type" highest in Xe " Katarzyna Piecielska
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Katarzyna Piecielska @ 2024-04-24  8:05 UTC (permalink / raw)
  To: igt-dev
  Cc: Katarzyna Piecielska, Kamil Konieczny, Jari Tahvanainen,
	Mauro Carvalho Chehab

Let's put "Run type" at highest place in documentation hierarchy. It
will be reflected in html generated documentation.

v2: Make that change in Xe, i915 and KMS (Mauro)
v3: move only required fields under 'Run type'

Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska@intel.com>
Cc: Kamil Konieczny <kamil.konieczny@intel.com>
Cc: Jari Tahvanainen <jari.tahvanainen@intel.com>
Cc: Mauro Carvalho Chehab <mauro.chehab@linux.intel.com>

Katarzyna Piecielska (3):
  tests/intel: Put "Run type" highest in Xe documentation hierarchy
  tests/intel: Put "Run type" highest in KMS documentation hierarchy
  tests/intel: Put "Run type" highest in i915 documentation hierarchy

 tests/intel/i915_test_config.json | 66 +++++++++++++++----------------
 tests/intel/kms_test_config.json  | 56 +++++++++++++-------------
 tests/intel/xe_test_config.json   | 66 +++++++++++++++----------------
 3 files changed, 94 insertions(+), 94 deletions(-)

-- 
2.34.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH i-g-t v3 1/3] tests/intel: Put "Run type" highest in Xe documentation hierarchy
  2024-04-24  8:05 [PATCH i-g-t v3 0/3] Put "Run type" highest in documentation hierarchy Katarzyna Piecielska
@ 2024-04-24  8:05 ` Katarzyna Piecielska
  2024-04-24 15:47   ` Kamil Konieczny
  2024-04-24  8:05 ` [PATCH i-g-t v3 2/3] tests/intel: Put "Run type" highest in KMS " Katarzyna Piecielska
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 6+ messages in thread
From: Katarzyna Piecielska @ 2024-04-24  8:05 UTC (permalink / raw)
  To: igt-dev
  Cc: Katarzyna Piecielska, Kamil Konieczny, Jari Tahvanainen,
	Mauro Carvalho Chehab

Let's put "Run type" at highest place in Xe documentation hierarchy. It
will be reflected in html generated documentation.

Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska@intel.com>
Cc: Kamil Konieczny <kamil.konieczny@intel.com>
Cc: Jari Tahvanainen <jari.tahvanainen@intel.com>
Cc: Mauro Carvalho Chehab <mauro.chehab@linux.intel.com>
---
 tests/intel/xe_test_config.json | 66 ++++++++++++++++-----------------
 1 file changed, 33 insertions(+), 33 deletions(-)

diff --git a/tests/intel/xe_test_config.json b/tests/intel/xe_test_config.json
index 22b43ffac..f88883414 100644
--- a/tests/intel/xe_test_config.json
+++ b/tests/intel/xe_test_config.json
@@ -4,51 +4,51 @@
     "drivers": [ "xe" ],
     "files": [ "xe_*.c", "../core_hotunplug.c", "../sriov_basic.c", "intel_hwmon.c" ],
     "fields": {
-        "Category": {
+        "Run type": {
             "_properties_": {
                 "mandatory": true,
-                "description": "Contains the major group for the tested functionality, being hardware, software or firmware"
+                "description": "Defines what category of testlist it belongs",
+                "default-testlist": "FULL",
+                "update-from-file": {
+                    "exclude-type": "regex-ignorecase",
+                    "append-value-if-not-excluded": "Xe FULL",
+                    "include": [
+                        { "Xe BAT": "../intel-ci/xe-fast-feedback.testlist" }
+                    ],
+                    "exclude": [
+                        { "Xe BAT, Xe FULL": "../intel-ci/xe.blocklist.txt" }
+                    ]
+                },
+                "order": [
+                    "boot",
+                    "__all__",
+                    "kunit"
+                ]
             },
-            "Mega feature": {
+            "Category": {
                 "_properties_": {
-                    "description": "Contains the mega feature for end to end use case, e. g. the 'sellable' feature."
+                    "mandatory": true,
+                    "description": "Contains the major group for the tested functionality, being hardware, software or firmware"
                 },
-                "Sub-category": {
+                "Mega feature": {
                     "_properties_": {
-                        "mandatory": true,
-                        "description": "Contains the the technical feature/functionality"
+                        "description": "Contains the mega feature for end to end use case, e. g. the 'sellable' feature."
                     },
-                    "Functionality": {
+                    "Sub-category": {
                         "_properties_": {
                             "mandatory": true,
-                            "description": "Groups page table tests on buckets containg more detailed functionality"
+                            "description": "Contains the technical feature/functionality"
                         },
-                        "Feature": {
+                        "Functionality": {
                             "_properties_": {
-                                "description": "Describes the lowest level feature bucket"
-                            }
-                        }
-                    },
-                    "Run type": {
-                        "_properties_": {
-                            "mandatory": true,
-                            "description": "Defines what category of testlist it belongs",
-                            "default-testlist": "FULL",
-                            "update-from-file": {
-                                "exclude-type": "regex-ignorecase",
-                                "append-value-if-not-excluded": "Xe FULL",
-                                "include": [
-                                    { "Xe BAT": "../intel-ci/xe-fast-feedback.testlist" }
-                                ],
-                                "exclude": [
-                                    { "Xe BAT, Xe FULL": "../intel-ci/xe.blocklist.txt" }
-                                ]
+                                "mandatory": true,
+                                "description": "Groups page table tests on buckets containg more detailed functionality"
                             },
-                            "order": [
-                                "boot",
-                                "__all__",
-                                "kunit"
-                            ]
+                            "Feature": {
+                                "_properties_": {
+                                    "description": "Describes the lowest level feature bucket"
+                                }
+                            }
                         }
                     }
                 }
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH i-g-t v3 2/3] tests/intel: Put "Run type" highest in KMS documentation hierarchy
  2024-04-24  8:05 [PATCH i-g-t v3 0/3] Put "Run type" highest in documentation hierarchy Katarzyna Piecielska
  2024-04-24  8:05 ` [PATCH i-g-t v3 1/3] tests/intel: Put "Run type" highest in Xe " Katarzyna Piecielska
@ 2024-04-24  8:05 ` Katarzyna Piecielska
  2024-04-24  8:05 ` [PATCH i-g-t v3 3/3] tests/intel: Put "Run type" highest in i915 " Katarzyna Piecielska
  2024-04-24  9:08 ` [PATCH i-g-t v3 0/3] Put "Run type" highest in " Mauro Carvalho Chehab
  3 siblings, 0 replies; 6+ messages in thread
From: Katarzyna Piecielska @ 2024-04-24  8:05 UTC (permalink / raw)
  To: igt-dev
  Cc: Katarzyna Piecielska, Kamil Konieczny, Jari Tahvanainen,
	Mauro Carvalho Chehab

Let's put "Run type" at highest place in KMS documentation hierarchy. It
will be reflected in html generated documentation.

v2: move only required fields under 'Run type'

Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska@intel.com>
Cc: Kamil Konieczny <kamil.konieczny@intel.com>
Cc: Jari Tahvanainen <jari.tahvanainen@intel.com>
Cc: Mauro Carvalho Chehab <mauro.chehab@linux.intel.com>
---
 tests/intel/kms_test_config.json | 56 ++++++++++++++++----------------
 1 file changed, 28 insertions(+), 28 deletions(-)

diff --git a/tests/intel/kms_test_config.json b/tests/intel/kms_test_config.json
index f3f6bfad6..7c37c831b 100644
--- a/tests/intel/kms_test_config.json
+++ b/tests/intel/kms_test_config.json
@@ -6,39 +6,39 @@
     "exclude_files": [ "../chamelium/kms_chamelium_helper.c", "../kms_color_helper.c",
                        "kms_dsc_helper.c" ],
     "fields": {
-        "Category": {
+        "Run type": {
             "_properties_": {
-                "description": "Contains the major group for the tested functionality 'Display'"
-            },
-            "Mega feature": {
-                "_properties_": {
-                    "description": "Contains the mega feature for end to end use case, e.g. the 'PSR' feature."
-                },
-                "Functionality": {
-                    "_properties_": {
-                        "description": "Groups tests on buckets containg more detailed functionality"
-                    }
-                }
-            },
-            "Run type": {
-                "_properties_": {
-                    "description": "Defines what category of testlist it belongs",
-                    "update-from-file": {
-                        "append-value-if-not-excluded": "Xe FULL, i915 FULL",
-                        "exclude-type": "regex-ignorecase",
-                        "include": [
-                            { "i915 BAT": "../intel-ci/fast-feedback.testlist" },
-                            { "i915 BAT chamelium": "../intel-ci/fast-feedback-chamelium-only.testlist" },
-                            { "i915 chamelium": "../intel-ci/chamelium-only.testlist" },
+                "description": "Defines what category of testlist it belongs",
+                "update-from-file": {
+                    "append-value-if-not-excluded": "Xe FULL, i915 FULL",
+                    "exclude-type": "regex-ignorecase",
+                    "include": [
+                        { "i915 BAT": "../intel-ci/fast-feedback.testlist" },
+                        { "i915 BAT chamelium": "../intel-ci/fast-feedback-chamelium-only.testlist" },
+                        { "i915 chamelium": "../intel-ci/chamelium-only.testlist" },
 
-                            { "Xe BAT": "../intel-ci/xe-fast-feedback.testlist" },
-                            { "Xe BAT chamelium": "../intel-ci/xe-fast-feedback-chamelium-only.testlist" }
-                        ],
-                        "exclude": [
-                            { "i915 BAT, i915 BAT chamelium, i915 chamelium, i915 FULL": "../intel-ci/blacklist.txt" },
+                        { "Xe BAT": "../intel-ci/xe-fast-feedback.testlist" },
+                        { "Xe BAT chamelium": "../intel-ci/xe-fast-feedback-chamelium-only.testlist" }
+                    ],
+                    "exclude": [
+                        { "i915 BAT, i915 BAT chamelium, i915 chamelium, i915 FULL": "../intel-ci/blacklist.txt" },
                             { "Xe BAT, Xe BAT chamelium, Xe FULL": "../intel-ci/xe.blocklist.txt" }
                         ]
                     }
+                },
+            "Category": {
+                "_properties_": {
+                    "description": "Contains the major group for the tested functionality 'Display'"
+                },
+                "Mega feature": {
+                    "_properties_": {
+                        "description": "Contains the mega feature for end to end use case, e.g. the 'PSR' feature."
+                    },
+                    "Functionality": {
+                        "_properties_": {
+                            "description": "Groups tests on buckets containg more detailed functionality"
+                        }
+                    }
                 }
             }
         },
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH i-g-t v3 3/3] tests/intel: Put "Run type" highest in i915 documentation hierarchy
  2024-04-24  8:05 [PATCH i-g-t v3 0/3] Put "Run type" highest in documentation hierarchy Katarzyna Piecielska
  2024-04-24  8:05 ` [PATCH i-g-t v3 1/3] tests/intel: Put "Run type" highest in Xe " Katarzyna Piecielska
  2024-04-24  8:05 ` [PATCH i-g-t v3 2/3] tests/intel: Put "Run type" highest in KMS " Katarzyna Piecielska
@ 2024-04-24  8:05 ` Katarzyna Piecielska
  2024-04-24  9:08 ` [PATCH i-g-t v3 0/3] Put "Run type" highest in " Mauro Carvalho Chehab
  3 siblings, 0 replies; 6+ messages in thread
From: Katarzyna Piecielska @ 2024-04-24  8:05 UTC (permalink / raw)
  To: igt-dev
  Cc: Katarzyna Piecielska, Kamil Konieczny, Jari Tahvanainen,
	Mauro Carvalho Chehab

Let's put "Run type" at highest place in i915 documentation hierarchy. It
will be reflected in html generated documentation.

v2: move only required fields under 'Run type'

Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska@intel.com>
Cc: Kamil Konieczny <kamil.konieczny@intel.com>
Cc: Jari Tahvanainen <jari.tahvanainen@intel.com>
Cc: Mauro Carvalho Chehab <mauro.chehab@linux.intel.com>
---
 tests/intel/i915_test_config.json | 66 +++++++++++++++----------------
 1 file changed, 33 insertions(+), 33 deletions(-)

diff --git a/tests/intel/i915_test_config.json b/tests/intel/i915_test_config.json
index 61638a4f1..f51ae2a55 100644
--- a/tests/intel/i915_test_config.json
+++ b/tests/intel/i915_test_config.json
@@ -16,51 +16,51 @@
     "exclude_files": [ "kms_*", "gem_concurrent_all.c", "gem_concurrent_blit.c",
                        "xe_*" ],
     "fields": {
-        "Category": {
+        "Run type": {
             "_properties_": {
                 "mandatory": true,
-                "description": "Contains the major group for the tested functionality, being hardware, software or firmware"
+                "description": "Defines what category of testlist it belongs",
+                "default-testlist": "FULL",
+                "update-from-file": {
+                    "exclude-type": "regex-ignorecase",
+                    "append-value-if-not-excluded": "i915 FULL",
+                    "include": [
+                        { "i915 BAT": "../intel-ci/fast-feedback.testlist" }
+                    ],
+                    "exclude": [
+                        { "all": "../intel-ci/blacklist.txt" }
+                    ]
+                },
+                "order": [
+                    "boot",
+                    "__all__",
+                    "kunit"
+                ]
             },
-            "Mega feature": {
+            "Category": {
                 "_properties_": {
-                    "description": "Contains the mega feature for end to end use case, e. g. the 'sellable' feature."
+                    "mandatory": true,
+                    "description": "Contains the major group for the tested functionality, being hardware, software or firmware"
                 },
-                "Sub-category": {
+                "Mega feature": {
                     "_properties_": {
-                        "mandatory": true,
-                        "description": "Contains the technical feature/functionality"
+                        "description": "Contains the mega feature for end to end use case, e. g. the 'sellable' feature."
                     },
-                    "Functionality": {
+                    "Sub-category": {
                         "_properties_": {
                             "mandatory": true,
-                            "description": "Groups page table tests on buckets containg more detailed functionality"
+                            "description": "Contains the technical feature/functionality"
                         },
-                        "Feature": {
+                        "Functionality": {
                             "_properties_": {
-                                "description": "Describes the lowest level feature bucket"
-                            }
-                        }
-                    },
-                    "Run type": {
-                        "_properties_": {
-                            "mandatory": true,
-                            "description": "Defines what category of testlist it belongs",
-                            "default-testlist": "FULL",
-                            "update-from-file": {
-                                "exclude-type": "regex-ignorecase",
-                                "append-value-if-not-excluded": "i915 FULL",
-                                "include": [
-                                    { "i915 BAT": "../intel-ci/fast-feedback.testlist" }
-                                ],
-                                "exclude": [
-                                    { "all": "../intel-ci/blacklist.txt" }
-                                ]
+                                "mandatory": true,
+                                "description": "Groups page table tests on buckets containg more detailed functionality"
                             },
-                            "order": [
-                                "boot",
-                                "__all__",
-                                "kunit"
-                            ]
+                            "Feature": {
+                                "_properties_": {
+                                    "description": "Describes the lowest level feature bucket"
+                                }
+                            }
                         }
                     }
                 }
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH i-g-t v3 0/3] Put "Run type" highest in documentation hierarchy
  2024-04-24  8:05 [PATCH i-g-t v3 0/3] Put "Run type" highest in documentation hierarchy Katarzyna Piecielska
                   ` (2 preceding siblings ...)
  2024-04-24  8:05 ` [PATCH i-g-t v3 3/3] tests/intel: Put "Run type" highest in i915 " Katarzyna Piecielska
@ 2024-04-24  9:08 ` Mauro Carvalho Chehab
  3 siblings, 0 replies; 6+ messages in thread
From: Mauro Carvalho Chehab @ 2024-04-24  9:08 UTC (permalink / raw)
  To: Katarzyna Piecielska; +Cc: igt-dev, Kamil Konieczny, Jari Tahvanainen

On Wed, 24 Apr 2024 08:05:30 +0000
Katarzyna Piecielska <katarzyna.piecielska@intel.com> wrote:

> Let's put "Run type" at highest place in documentation hierarchy. It
> will be reflected in html generated documentation.
> 
> v2: Make that change in Xe, i915 and KMS (Mauro)
> v3: move only required fields under 'Run type'
> 
> Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska@intel.com>
> Cc: Kamil Konieczny <kamil.konieczny@intel.com>
> Cc: Jari Tahvanainen <jari.tahvanainen@intel.com>
> Cc: Mauro Carvalho Chehab <mauro.chehab@linux.intel.com>
> 
> Katarzyna Piecielska (3):
>   tests/intel: Put "Run type" highest in Xe documentation hierarchy
>   tests/intel: Put "Run type" highest in KMS documentation hierarchy
>   tests/intel: Put "Run type" highest in i915 documentation hierarchy
> 
>  tests/intel/i915_test_config.json | 66 +++++++++++++++----------------
>  tests/intel/kms_test_config.json  | 56 +++++++++++++-------------
>  tests/intel/xe_test_config.json   | 66 +++++++++++++++----------------
>  3 files changed, 94 insertions(+), 94 deletions(-)
> 

The entire series LGTM.

Acked-by: Mauro Carvalho Chehab <mchehab@kernel.org>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH i-g-t v3 1/3] tests/intel: Put "Run type" highest in Xe documentation hierarchy
  2024-04-24  8:05 ` [PATCH i-g-t v3 1/3] tests/intel: Put "Run type" highest in Xe " Katarzyna Piecielska
@ 2024-04-24 15:47   ` Kamil Konieczny
  0 siblings, 0 replies; 6+ messages in thread
From: Kamil Konieczny @ 2024-04-24 15:47 UTC (permalink / raw)
  To: igt-dev; +Cc: Katarzyna Piecielska, Jari Tahvanainen, Mauro Carvalho Chehab

Hi Katarzyna,
On 2024-04-24 at 08:05:31 +0000, Katarzyna Piecielska wrote:
> Let's put "Run type" at highest place in Xe documentation hierarchy. It
> will be reflected in html generated documentation.
> 
> Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska@intel.com>
> Cc: Kamil Konieczny <kamil.konieczny@intel.com>
> Cc: Jari Tahvanainen <jari.tahvanainen@intel.com>
> Cc: Mauro Carvalho Chehab <mauro.chehab@linux.intel.com>

Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

> ---
>  tests/intel/xe_test_config.json | 66 ++++++++++++++++-----------------
>  1 file changed, 33 insertions(+), 33 deletions(-)
> 
> diff --git a/tests/intel/xe_test_config.json b/tests/intel/xe_test_config.json
> index 22b43ffac..f88883414 100644
> --- a/tests/intel/xe_test_config.json
> +++ b/tests/intel/xe_test_config.json
> @@ -4,51 +4,51 @@
>      "drivers": [ "xe" ],
>      "files": [ "xe_*.c", "../core_hotunplug.c", "../sriov_basic.c", "intel_hwmon.c" ],
>      "fields": {
> -        "Category": {
> +        "Run type": {
>              "_properties_": {
>                  "mandatory": true,
> -                "description": "Contains the major group for the tested functionality, being hardware, software or firmware"
> +                "description": "Defines what category of testlist it belongs",
> +                "default-testlist": "FULL",
> +                "update-from-file": {
> +                    "exclude-type": "regex-ignorecase",
> +                    "append-value-if-not-excluded": "Xe FULL",
> +                    "include": [
> +                        { "Xe BAT": "../intel-ci/xe-fast-feedback.testlist" }
> +                    ],
> +                    "exclude": [
> +                        { "Xe BAT, Xe FULL": "../intel-ci/xe.blocklist.txt" }
> +                    ]
> +                },
> +                "order": [
> +                    "boot",
> +                    "__all__",
> +                    "kunit"
> +                ]
>              },
> -            "Mega feature": {
> +            "Category": {
>                  "_properties_": {
> -                    "description": "Contains the mega feature for end to end use case, e. g. the 'sellable' feature."
> +                    "mandatory": true,
> +                    "description": "Contains the major group for the tested functionality, being hardware, software or firmware"
>                  },
> -                "Sub-category": {
> +                "Mega feature": {
>                      "_properties_": {
> -                        "mandatory": true,
> -                        "description": "Contains the the technical feature/functionality"
> +                        "description": "Contains the mega feature for end to end use case, e. g. the 'sellable' feature."
>                      },
> -                    "Functionality": {
> +                    "Sub-category": {
>                          "_properties_": {
>                              "mandatory": true,
> -                            "description": "Groups page table tests on buckets containg more detailed functionality"
> +                            "description": "Contains the technical feature/functionality"
>                          },
> -                        "Feature": {
> +                        "Functionality": {
>                              "_properties_": {
> -                                "description": "Describes the lowest level feature bucket"
> -                            }
> -                        }
> -                    },
> -                    "Run type": {
> -                        "_properties_": {
> -                            "mandatory": true,
> -                            "description": "Defines what category of testlist it belongs",
> -                            "default-testlist": "FULL",
> -                            "update-from-file": {
> -                                "exclude-type": "regex-ignorecase",
> -                                "append-value-if-not-excluded": "Xe FULL",
> -                                "include": [
> -                                    { "Xe BAT": "../intel-ci/xe-fast-feedback.testlist" }
> -                                ],
> -                                "exclude": [
> -                                    { "Xe BAT, Xe FULL": "../intel-ci/xe.blocklist.txt" }
> -                                ]
> +                                "mandatory": true,
> +                                "description": "Groups page table tests on buckets containg more detailed functionality"
>                              },
> -                            "order": [
> -                                "boot",
> -                                "__all__",
> -                                "kunit"
> -                            ]
> +                            "Feature": {
> +                                "_properties_": {
> +                                    "description": "Describes the lowest level feature bucket"
> +                                }
> +                            }
>                          }
>                      }
>                  }
> -- 
> 2.34.1
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2024-04-24 15:48 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-24  8:05 [PATCH i-g-t v3 0/3] Put "Run type" highest in documentation hierarchy Katarzyna Piecielska
2024-04-24  8:05 ` [PATCH i-g-t v3 1/3] tests/intel: Put "Run type" highest in Xe " Katarzyna Piecielska
2024-04-24 15:47   ` Kamil Konieczny
2024-04-24  8:05 ` [PATCH i-g-t v3 2/3] tests/intel: Put "Run type" highest in KMS " Katarzyna Piecielska
2024-04-24  8:05 ` [PATCH i-g-t v3 3/3] tests/intel: Put "Run type" highest in i915 " Katarzyna Piecielska
2024-04-24  9:08 ` [PATCH i-g-t v3 0/3] Put "Run type" highest in " Mauro Carvalho Chehab

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox