]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/xe: Remove xe_uc_init_hwconfig()
authorMaarten Lankhorst <dev@lankhorst.se>
Thu, 19 Jun 2025 10:49:08 +0000 (12:49 +0200)
committerMaarten Lankhorst <dev@lankhorst.se>
Thu, 26 Jun 2025 20:11:35 +0000 (22:11 +0200)
This function is called immediately after xe_uc_init(), so just put it
there instead.

Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://lore.kernel.org/r/20250619104858.418440-22-dev@lankhorst.se
Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
drivers/gpu/drm/xe/xe_gt.c
drivers/gpu/drm/xe/xe_uc.c
drivers/gpu/drm/xe/xe_uc.h

index 3942f1ac253e7530e1a66691d7b28dea04ec4eec..14761849390d8586ce9cb3c6f4f530a33b188660 100644 (file)
@@ -466,10 +466,6 @@ static int gt_init_with_gt_forcewake(struct xe_gt *gt)
        if (err)
                goto err_force_wake;
 
-       err = xe_uc_init_hwconfig(&gt->uc);
-       if (err)
-               goto err_force_wake;
-
        xe_gt_topology_init(gt);
        xe_gt_mcr_init(gt);
        xe_gt_enable_host_l2_vram(gt);
index 5badba6f85fa5221271694360aa7bb6d691e13dd..52f0b0ecf5a9d546e69e9e6455300989369ec9ae 100644 (file)
@@ -72,15 +72,17 @@ int xe_uc_init(struct xe_uc *uc)
        if (!xe_device_uc_enabled(uc_to_xe(uc)))
                return 0;
 
-       if (IS_SRIOV_VF(uc_to_xe(uc)))
-               return 0;
+       if (!IS_SRIOV_VF(uc_to_xe(uc))) {
+               ret = xe_wopcm_init(&uc->wopcm);
+               if (ret)
+                       goto err;
+       }
 
-       ret = xe_wopcm_init(&uc->wopcm);
+       ret = xe_guc_min_load_for_hwconfig(&uc->guc);
        if (ret)
                goto err;
 
        return 0;
-
 err:
        xe_gt_err(uc_to_gt(uc), "Failed to initialize uC (%pe)\n", ERR_PTR(ret));
        return ret;
@@ -142,27 +144,6 @@ int xe_uc_sanitize_reset(struct xe_uc *uc)
        return uc_reset(uc);
 }
 
-/**
- * xe_uc_init_hwconfig - minimally init Uc, read and parse hwconfig
- * @uc: The UC object
- *
- * Return: 0 on success, negative error code on error.
- */
-int xe_uc_init_hwconfig(struct xe_uc *uc)
-{
-       int ret;
-
-       /* GuC submission not enabled, nothing to do */
-       if (!xe_device_uc_enabled(uc_to_xe(uc)))
-               return 0;
-
-       ret = xe_guc_min_load_for_hwconfig(&uc->guc);
-       if (ret)
-               return ret;
-
-       return 0;
-}
-
 static int vf_uc_init_hw(struct xe_uc *uc)
 {
        int err;
index b573b7731f62eb9c198340744c4d960800962f0a..c13d705fe02c4ae94f00d5ff24c6b6fa4fb0d223 100644 (file)
@@ -10,7 +10,6 @@ struct xe_uc;
 
 int xe_uc_init_noalloc(struct xe_uc *uc);
 int xe_uc_init(struct xe_uc *uc);
-int xe_uc_init_hwconfig(struct xe_uc *uc);
 int xe_uc_init_post_hwconfig(struct xe_uc *uc);
 int xe_uc_init_hw(struct xe_uc *uc);
 int xe_uc_fini_hw(struct xe_uc *uc);