drm/xe/guc: Remove special handling for PVC A*
authorLucas De Marchi <lucas.demarchi@intel.com>
Thu, 4 May 2023 07:32:46 +0000 (00:32 -0700)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Tue, 19 Dec 2023 23:33:14 +0000 (18:33 -0500)
The rest of the driver doesn't really support PVC before B0 stepping.
Drop the special handling in xe_guc.c.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://lore.kernel.org/r/20230504073250.1436293-4-lucas.demarchi@intel.com
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/xe/xe_guc.c

index 89d20faced19dd51f815902f27156a26a398521c..62b4fcf84acfa9bdd689998f21dfb11d2fb231ae 100644 (file)
@@ -143,8 +143,7 @@ static u32 guc_ctl_wa_flags(struct xe_guc *guc)
 
        /* Wa_16011759253 */
        /* Wa_22011383443 */
-       if (IS_SUBPLATFORM_STEP(xe, XE_DG2, XE_SUBPLATFORM_DG2_G10, STEP_A0, STEP_B0) ||
-           IS_PLATFORM_STEP(xe, XE_PVC, STEP_A0, STEP_B0))
+       if (IS_SUBPLATFORM_STEP(xe, XE_DG2, XE_SUBPLATFORM_DG2_G10, STEP_A0, STEP_B0))
                flags |= GUC_WA_GAM_CREDITS;
 
        /* Wa_14014475959 */
@@ -164,11 +163,8 @@ static u32 guc_ctl_wa_flags(struct xe_guc *guc)
 
        /*
         * Wa_2201180203
-        * GUC_WA_PRE_PARSER causes media workload hang for PVC A0 and PCIe
-        * errors. Disable this for PVC A0 steppings.
         */
-       if (GRAPHICS_VER(xe) <= 12 &&
-           !IS_PLATFORM_STEP(xe, XE_PVC, STEP_A0, STEP_B0))
+       if (GRAPHICS_VER(xe) <= 12)
                flags |= GUC_WA_PRE_PARSER;
 
        /* Wa_16011777198 */
@@ -180,9 +176,6 @@ static u32 guc_ctl_wa_flags(struct xe_guc *guc)
        /*
         * Wa_22012727170
         * Wa_22012727685
-        *
-        * This WA is applicable to PVC CT A0, but causes media regressions. 
-        * Drop the WA for PVC.
         */
        if (IS_SUBPLATFORM_STEP(xe, XE_DG2, XE_SUBPLATFORM_DG2_G10, STEP_A0, STEP_C0) ||
            IS_SUBPLATFORM_STEP(xe, XE_DG2, XE_SUBPLATFORM_DG2_G11, STEP_A0,
@@ -194,10 +187,9 @@ static u32 guc_ctl_wa_flags(struct xe_guc *guc)
                flags |= GUC_WA_RCS_REGS_IN_CCS_REGS_LIST;
 
        /* Wa_1509372804 */
-       if (IS_PLATFORM_STEP(xe, XE_PVC, STEP_A0, STEP_C0))
+       if (IS_PLATFORM_STEP(xe, XE_PVC, STEP_B0, STEP_C0))
                flags |= GUC_WA_RENDER_RST_RC6_EXIT;
 
-
        return flags;
 }