drm/i915/mtl: Wa_22011802037: don't complain about missing regs on MTL
authorJohn Harrison <John.C.Harrison@Intel.com>
Tue, 24 Jan 2023 23:11:11 +0000 (15:11 -0800)
committerUmesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Tue, 31 Jan 2023 23:17:30 +0000 (15:17 -0800)
Wa_22011802037 requires waiting for an engine-specific register to
clear. A missing entry for GSC engine in the register table is flagged
as a drm_err. The drm_err was originally intended to catch missing
register entries for newer engines, however, it was later found that the
WA is only required for 'legacy' engines. So just drop the drm_err.

Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
Reviewed-by: Andrzej Hajda <andrzej.hajda@intel.com>
Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230124231111.1786429-1-umesh.nerlige.ramappa@intel.com
drivers/gpu/drm/i915/gt/intel_engine_cs.c

index 9f703f2..d4e29da 100644 (file)
@@ -1584,11 +1584,8 @@ static u32 __cs_pending_mi_force_wakes(struct intel_engine_cs *engine)
        };
        u32 val;
 
-       if (!_reg[engine->id].reg) {
-               drm_err(&engine->i915->drm,
-                       "MSG IDLE undefined for engine id %u\n", engine->id);
+       if (!_reg[engine->id].reg)
                return 0;
-       }
 
        val = intel_uncore_read(engine->uncore, _reg[engine->id]);