diff options
| author | Michal Wajdeczko <michal.wajdeczko@intel.com> | 2025-08-29 19:19:19 +0200 |
|---|---|---|
| committer | Michal Wajdeczko <michal.wajdeczko@intel.com> | 2025-09-05 12:57:26 +0200 |
| commit | ddbe5aecea846fd8f7510540e720919d5d781bd9 (patch) | |
| tree | 61142d5775c40acfc3f7d261c63f35c3ed124198 /drivers/gpu | |
| parent | drm/xe/kunit: Introduce xe_pci_fake_data_desc() (diff) | |
| download | linux-ddbe5aecea846fd8f7510540e720919d5d781bd9.tar.gz linux-ddbe5aecea846fd8f7510540e720919d5d781bd9.zip | |
drm/xe/kunit: Drop custom struct platform_test_case
Custom struct platform_test_case definition in xe_wa is now almost
identical to generic struct xe_pci_fake_data defintiion except the
.name member, which could be generated by xe_pci_fake_data_desc().
Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://lore.kernel.org/r/20250829171922.572-5-michal.wajdeczko@intel.com
Diffstat (limited to 'drivers/gpu')
| -rw-r--r-- | drivers/gpu/drm/xe/tests/xe_wa_test.c | 31 |
1 files changed, 4 insertions, 27 deletions
diff --git a/drivers/gpu/drm/xe/tests/xe_wa_test.c b/drivers/gpu/drm/xe/tests/xe_wa_test.c index a0de55f09267..7bf34fd0d189 100644 --- a/drivers/gpu/drm/xe/tests/xe_wa_test.c +++ b/drivers/gpu/drm/xe/tests/xe_wa_test.c @@ -15,18 +15,8 @@ #include "xe_tuning.h" #include "xe_wa.h" -struct platform_test_case { - const char *name; - enum xe_platform platform; - enum xe_subplatform subplatform; - u32 graphics_verx100; - u32 media_verx100; - struct xe_step_info step; -}; - #define PLATFORM_CASE(platform__, graphics_step__) \ { \ - .name = #platform__ " (" #graphics_step__ ")", \ .platform = XE_ ## platform__, \ .subplatform = XE_SUBPLATFORM_NONE, \ .step = { .graphics = STEP_ ## graphics_step__ } \ @@ -35,7 +25,6 @@ struct platform_test_case { #define SUBPLATFORM_CASE(platform__, subplatform__, graphics_step__) \ { \ - .name = #platform__ "_" #subplatform__ " (" #graphics_step__ ")", \ .platform = XE_ ## platform__, \ .subplatform = XE_SUBPLATFORM_ ## platform__ ## _ ## subplatform__, \ .step = { .graphics = STEP_ ## graphics_step__ } \ @@ -44,7 +33,6 @@ struct platform_test_case { #define GMDID_CASE(platform__, graphics_verx100__, graphics_step__, \ media_verx100__, media_step__) \ { \ - .name = #platform__ " (g:" #graphics_step__ ", m:" #media_step__ ")",\ .platform = XE_ ## platform__, \ .subplatform = XE_SUBPLATFORM_NONE, \ .graphics_verx100 = graphics_verx100__, \ @@ -53,7 +41,7 @@ struct platform_test_case { .media = STEP_ ## media_step__ } \ } -static const struct platform_test_case cases[] = { +static const struct xe_pci_fake_data cases[] = { PLATFORM_CASE(TIGERLAKE, B0), PLATFORM_CASE(DG1, A0), PLATFORM_CASE(DG1, B0), @@ -78,23 +66,12 @@ static const struct platform_test_case cases[] = { GMDID_CASE(PANTHERLAKE, 3000, A0, 3000, A0), }; -static void platform_desc(const struct platform_test_case *t, char *desc) -{ - strscpy(desc, t->name, KUNIT_PARAM_DESC_SIZE); -} - -KUNIT_ARRAY_PARAM(platform, cases, platform_desc); +KUNIT_ARRAY_PARAM(platform, cases, xe_pci_fake_data_desc); static int xe_wa_test_init(struct kunit *test) { - const struct platform_test_case *param = test->param_value; - struct xe_pci_fake_data data = { - .platform = param->platform, - .subplatform = param->subplatform, - .graphics_verx100 = param->graphics_verx100, - .media_verx100 = param->media_verx100, - .step = param->step, - }; + const struct xe_pci_fake_data *param = test->param_value; + struct xe_pci_fake_data data = *param; struct xe_device *xe; struct device *dev; int ret; |