drm/i915/uncore: Replace gen8 write functions with general fwtable
authorMatt Roper <matthew.d.roper@intel.com>
Fri, 10 Sep 2021 20:10:27 +0000 (13:10 -0700)
committerMatt Roper <matthew.d.roper@intel.com>
Wed, 22 Sep 2021 00:11:06 +0000 (17:11 -0700)
Now that we have both a standard forcewake table (albeit a single-entry
table) and the shadow table stored in the uncore, we can drop the
gen8-specific write handlers in favor of the general fwtable version.

Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210910201030.3436066-4-matthew.d.roper@intel.com
drivers/gpu/drm/i915/intel_uncore.c

index a733bf6f6519aa9d669dd55a7db4e96f0786f7b1..f3c77f617458651397ef69491f06c13edde34310 100644 (file)
@@ -1043,16 +1043,6 @@ gen6_reg_write_fw_domains(struct intel_uncore *uncore, i915_reg_t reg)
        return FORCEWAKE_RENDER;
 }
 
-#define __gen8_reg_write_fw_domains(uncore, offset) \
-({ \
-       enum forcewake_domains __fwd; \
-       if (NEEDS_FORCE_WAKE(offset) && !is_shadowed(uncore, offset)) \
-               __fwd = FORCEWAKE_RENDER; \
-       else \
-               __fwd = 0; \
-       __fwd; \
-})
-
 static const struct intel_forcewake_range __gen6_fw_ranges[] = {
        GEN_FW_RANGE(0x0, 0x3ffff, FORCEWAKE_RENDER),
 };
@@ -1707,7 +1697,6 @@ __gen_write(func, 32)
 __gen_reg_write_funcs(gen12_fwtable);
 __gen_reg_write_funcs(gen11_fwtable);
 __gen_reg_write_funcs(fwtable);
-__gen_reg_write_funcs(gen8);
 
 #undef __gen_reg_write_funcs
 #undef GEN6_WRITE_FOOTER
@@ -2117,7 +2106,7 @@ static int uncore_forcewake_init(struct intel_uncore *uncore)
        } else if (GRAPHICS_VER(i915) == 8) {
                ASSIGN_FW_DOMAINS_TABLE(uncore, __gen6_fw_ranges);
                ASSIGN_SHADOW_TABLE(uncore, gen8_shadowed_regs);
-               ASSIGN_WRITE_MMIO_VFUNCS(uncore, gen8);
+               ASSIGN_WRITE_MMIO_VFUNCS(uncore, fwtable);
                ASSIGN_READ_MMIO_VFUNCS(uncore, fwtable);
        } else if (IS_VALLEYVIEW(i915)) {
                ASSIGN_FW_DOMAINS_TABLE(uncore, __vlv_fw_ranges);