drm/i915/gt: Cleanup interface for MCR operations
authorMatt Roper <matthew.d.roper@intel.com>
Wed, 15 Jun 2022 00:10:19 +0000 (17:10 -0700)
committerMatt Roper <matthew.d.roper@intel.com>
Fri, 17 Jun 2022 15:05:40 +0000 (08:05 -0700)
Let's replace the assortment of intel_gt_* and intel_uncore_* functions
that operate on MCR registers with a cleaner set of interfaces:

  * intel_gt_mcr_read -- unicast read from specific instance
  * intel_gt_mcr_read_any[_fw] -- unicast read from any non-terminated
    instance
  * intel_gt_mcr_unicast_write -- unicast write to specific instance
  * intel_gt_mcr_multicast_write[_fw] -- multicast write to all instances

We'll also replace the historic "slice" and "subslice" terminology with
"group" and "instance" to match the documentation for more recent
platforms; these days MCR steering applies to more types of replication
than just slice/subslice.

v2:
 - Reference the new kerneldoc from i915.rst.  (Jani)
 - Tweak the wording of the documentation for a couple functions to
   clarify the difference between "_fw" and non-"_fw" forms.

v3:
 - s/read/write/ to fix copy-paste mistake in a couple comments.
   (Harish)

Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Acked-by: Jani Nikula <jani.nikula@linux.intel.com>
Reviewed-by: Harish Chegondi <harish.chegondi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220615001019.1821989-3-matthew.d.roper@intel.com
Documentation/gpu/i915.rst
drivers/gpu/drm/i915/gem/i915_gem_stolen.c
drivers/gpu/drm/i915/gt/intel_engine_cs.c
drivers/gpu/drm/i915/gt/intel_gt_debugfs.c
drivers/gpu/drm/i915/gt/intel_gt_mcr.c
drivers/gpu/drm/i915/gt/intel_gt_mcr.h
drivers/gpu/drm/i915/gt/intel_region_lmem.c
drivers/gpu/drm/i915/gt/intel_workarounds.c
drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c

index 54060cd..4e59db1 100644 (file)
@@ -246,6 +246,18 @@ Display State Buffer
 .. kernel-doc:: drivers/gpu/drm/i915/display/intel_dsb.c
    :internal:
 
+GT Programming
+==============
+
+Multicast/Replicated (MCR) Registers
+------------------------------------
+
+.. kernel-doc:: drivers/gpu/drm/i915/gt/intel_gt_mcr.c
+   :doc: GT Multicast/Replicated (MCR) Register Support
+
+.. kernel-doc:: drivers/gpu/drm/i915/gt/intel_gt_mcr.c
+   :internal:
+
 Memory Management and Command Submission
 ========================================
 
index da30503..fa54823 100644 (file)
@@ -835,7 +835,7 @@ i915_gem_stolen_lmem_setup(struct drm_i915_private *i915, u16 type,
        } else {
                resource_size_t lmem_range;
 
-               lmem_range = intel_gt_read_register(&i915->gt0, XEHPSDV_TILE0_ADDR_RANGE) & 0xFFFF;
+               lmem_range = intel_gt_mcr_read_any(&i915->gt0, XEHPSDV_TILE0_ADDR_RANGE) & 0xFFFF;
                lmem_size = lmem_range >> XEHPSDV_TILE_LMEM_RANGE_SHIFT;
                lmem_size *= SZ_1G;
        }
index 244af1b..136cc44 100644 (file)
@@ -1428,14 +1428,6 @@ void intel_engine_cancel_stop_cs(struct intel_engine_cs *engine)
        ENGINE_WRITE_FW(engine, RING_MI_MODE, _MASKED_BIT_DISABLE(STOP_RING));
 }
 
-static u32
-read_subslice_reg(const struct intel_engine_cs *engine,
-                 int slice, int subslice, i915_reg_t reg)
-{
-       return intel_uncore_read_with_mcr_steering(engine->uncore, reg,
-                                                  slice, subslice);
-}
-
 /* NB: please notice the memset */
 void intel_engine_get_instdone(const struct intel_engine_cs *engine,
                               struct intel_instdone *instdone)
@@ -1469,28 +1461,33 @@ void intel_engine_get_instdone(const struct intel_engine_cs *engine,
                if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 50)) {
                        for_each_instdone_gslice_dss_xehp(i915, sseu, iter, slice, subslice) {
                                instdone->sampler[slice][subslice] =
-                                       read_subslice_reg(engine, slice, subslice,
-                                                         GEN7_SAMPLER_INSTDONE);
+                                       intel_gt_mcr_read(engine->gt,
+                                                         GEN7_SAMPLER_INSTDONE,
+                                                         slice, subslice);
                                instdone->row[slice][subslice] =
-                                       read_subslice_reg(engine, slice, subslice,
-                                                         GEN7_ROW_INSTDONE);
+                                       intel_gt_mcr_read(engine->gt,
+                                                         GEN7_ROW_INSTDONE,
+                                                         slice, subslice);
                        }
                } else {
                        for_each_instdone_slice_subslice(i915, sseu, slice, subslice) {
                                instdone->sampler[slice][subslice] =
-                                       read_subslice_reg(engine, slice, subslice,
-                                                         GEN7_SAMPLER_INSTDONE);
+                                       intel_gt_mcr_read(engine->gt,
+                                                         GEN7_SAMPLER_INSTDONE,
+                                                         slice, subslice);
                                instdone->row[slice][subslice] =
-                                       read_subslice_reg(engine, slice, subslice,
-                                                         GEN7_ROW_INSTDONE);
+                                       intel_gt_mcr_read(engine->gt,
+                                                         GEN7_ROW_INSTDONE,
+                                                         slice, subslice);
                        }
                }
 
                if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 55)) {
                        for_each_instdone_gslice_dss_xehp(i915, sseu, iter, slice, subslice)
                                instdone->geom_svg[slice][subslice] =
-                                       read_subslice_reg(engine, slice, subslice,
-                                                         XEHPG_INSTDONE_GEOM_SVG);
+                                       intel_gt_mcr_read(engine->gt,
+                                                         XEHPG_INSTDONE_GEOM_SVG,
+                                                         slice, subslice);
                }
        } else if (GRAPHICS_VER(i915) >= 7) {
                instdone->instdone =
index ea07f2b..dd53641 100644 (file)
@@ -65,7 +65,7 @@ static int steering_show(struct seq_file *m, void *data)
        struct drm_printer p = drm_seq_file_printer(m);
        struct intel_gt *gt = m->private;
 
-       intel_gt_report_steering(&p, gt, true);
+       intel_gt_mcr_report_steering(&p, gt, true);
 
        return 0;
 }
index 1279a1f..777025d 100644 (file)
@@ -134,23 +134,22 @@ void intel_gt_mcr_init(struct intel_gt *gt)
        }
 }
 
-/**
- * uncore_rw_with_mcr_steering_fw - Access a register after programming
- *                                 the MCR selector register.
+/*
+ * rw_with_mcr_steering_fw - Access a register with specific MCR steering
  * @uncore: pointer to struct intel_uncore
  * @reg: register being accessed
  * @rw_flag: FW_REG_READ for read access or FW_REG_WRITE for write access
- * @slice: slice number (ignored for multi-cast write)
- * @subslice: sub-slice number (ignored for multi-cast write)
+ * @group: group number (documented as "sliceid" on older platforms)
+ * @instance: instance number (documented as "subsliceid" on older platforms)
  * @value: register value to be written (ignored for read)
  *
  * Return: 0 for write access. register value for read access.
  *
  * Caller needs to make sure the relevant forcewake wells are up.
  */
-static u32 uncore_rw_with_mcr_steering_fw(struct intel_uncore *uncore,
-                                         i915_reg_t reg, u8 rw_flag,
-                                         int slice, int subslice, u32 value)
+static u32 rw_with_mcr_steering_fw(struct intel_uncore *uncore,
+                                  i915_reg_t reg, u8 rw_flag,
+                                  int group, int instance, u32 value)
 {
        u32 mcr_mask, mcr_ss, mcr, old_mcr, val = 0;
 
@@ -158,7 +157,7 @@ static u32 uncore_rw_with_mcr_steering_fw(struct intel_uncore *uncore,
 
        if (GRAPHICS_VER(uncore->i915) >= 11) {
                mcr_mask = GEN11_MCR_SLICE_MASK | GEN11_MCR_SUBSLICE_MASK;
-               mcr_ss = GEN11_MCR_SLICE(slice) | GEN11_MCR_SUBSLICE(subslice);
+               mcr_ss = GEN11_MCR_SLICE(group) | GEN11_MCR_SUBSLICE(instance);
 
                /*
                 * Wa_22013088509
@@ -176,7 +175,7 @@ static u32 uncore_rw_with_mcr_steering_fw(struct intel_uncore *uncore,
                        mcr_mask |= GEN11_MCR_MULTICAST;
        } else {
                mcr_mask = GEN8_MCR_SLICE_MASK | GEN8_MCR_SUBSLICE_MASK;
-               mcr_ss = GEN8_MCR_SLICE(slice) | GEN8_MCR_SUBSLICE(subslice);
+               mcr_ss = GEN8_MCR_SLICE(group) | GEN8_MCR_SUBSLICE(instance);
        }
 
        old_mcr = mcr = intel_uncore_read_fw(uncore, GEN8_MCR_SELECTOR);
@@ -198,10 +197,10 @@ static u32 uncore_rw_with_mcr_steering_fw(struct intel_uncore *uncore,
        return val;
 }
 
-static u32 uncore_rw_with_mcr_steering(struct intel_uncore *uncore,
-                                      i915_reg_t reg, u8 rw_flag,
-                                      int slice, int subslice,
-                                      u32 value)
+static u32 rw_with_mcr_steering(struct intel_uncore *uncore,
+                               i915_reg_t reg, u8 rw_flag,
+                               int group, int instance,
+                               u32 value)
 {
        enum forcewake_domains fw_domains;
        u32 val;
@@ -215,8 +214,7 @@ static u32 uncore_rw_with_mcr_steering(struct intel_uncore *uncore,
        spin_lock_irq(&uncore->lock);
        intel_uncore_forcewake_get__locked(uncore, fw_domains);
 
-       val = uncore_rw_with_mcr_steering_fw(uncore, reg, rw_flag,
-                                            slice, subslice, value);
+       val = rw_with_mcr_steering_fw(uncore, reg, rw_flag, group, instance, value);
 
        intel_uncore_forcewake_put__locked(uncore, fw_domains);
        spin_unlock_irq(&uncore->lock);
@@ -224,31 +222,73 @@ static u32 uncore_rw_with_mcr_steering(struct intel_uncore *uncore,
        return val;
 }
 
-u32 intel_uncore_read_with_mcr_steering_fw(struct intel_uncore *uncore,
-                                          i915_reg_t reg, int slice, int subslice)
+/**
+ * intel_gt_mcr_read - read a specific instance of an MCR register
+ * @gt: GT structure
+ * @reg: the MCR register to read
+ * @group: the MCR group
+ * @instance: the MCR instance
+ *
+ * Returns the value read from an MCR register after steering toward a specific
+ * group/instance.
+ */
+u32 intel_gt_mcr_read(struct intel_gt *gt,
+                     i915_reg_t reg,
+                     int group, int instance)
 {
-       return uncore_rw_with_mcr_steering_fw(uncore, reg, FW_REG_READ,
-                                             slice, subslice, 0);
+       return rw_with_mcr_steering(gt->uncore, reg, FW_REG_READ, group, instance, 0);
 }
 
-u32 intel_uncore_read_with_mcr_steering(struct intel_uncore *uncore,
-                                       i915_reg_t reg, int slice, int subslice)
+/**
+ * intel_gt_mcr_unicast_write - write a specific instance of an MCR register
+ * @gt: GT structure
+ * @reg: the MCR register to write
+ * @value: value to write
+ * @group: the MCR group
+ * @instance: the MCR instance
+ *
+ * Write an MCR register in unicast mode after steering toward a specific
+ * group/instance.
+ */
+void intel_gt_mcr_unicast_write(struct intel_gt *gt, i915_reg_t reg, u32 value,
+                               int group, int instance)
 {
-       return uncore_rw_with_mcr_steering(uncore, reg, FW_REG_READ,
-                                          slice, subslice, 0);
+       rw_with_mcr_steering(gt->uncore, reg, FW_REG_WRITE, group, instance, value);
 }
 
-void intel_uncore_write_with_mcr_steering(struct intel_uncore *uncore,
-                                         i915_reg_t reg, u32 value,
-                                         int slice, int subslice)
+/**
+ * intel_gt_mcr_multicast_write - write a value to all instances of an MCR register
+ * @gt: GT structure
+ * @reg: the MCR register to write
+ * @value: value to write
+ *
+ * Write an MCR register in multicast mode to update all instances.
+ */
+void intel_gt_mcr_multicast_write(struct intel_gt *gt,
+                               i915_reg_t reg, u32 value)
 {
-       uncore_rw_with_mcr_steering(uncore, reg, FW_REG_WRITE,
-                                   slice, subslice, value);
+       intel_uncore_write(gt->uncore, reg, value);
 }
 
 /**
- * intel_gt_reg_needs_read_steering - determine whether a register read
- *     requires explicit steering
+ * intel_gt_mcr_multicast_write_fw - write a value to all instances of an MCR register
+ * @gt: GT structure
+ * @reg: the MCR register to write
+ * @value: value to write
+ *
+ * Write an MCR register in multicast mode to update all instances.  This
+ * function assumes the caller is already holding any necessary forcewake
+ * domains; use intel_gt_mcr_multicast_write() in cases where forcewake should
+ * be obtained automatically.
+ */
+void intel_gt_mcr_multicast_write_fw(struct intel_gt *gt, i915_reg_t reg, u32 value)
+{
+       intel_uncore_write_fw(gt->uncore, reg, value);
+}
+
+/*
+ * reg_needs_read_steering - determine whether a register read requires
+ *     explicit steering
  * @gt: GT structure
  * @reg: the register to check steering requirements for
  * @type: type of multicast steering to check
@@ -260,14 +300,14 @@ void intel_uncore_write_with_mcr_steering(struct intel_uncore *uncore,
  * steering type, or if the default (subslice-based) steering IDs are suitable
  * for @type steering too.
  */
-static bool intel_gt_reg_needs_read_steering(struct intel_gt *gt,
-                                            i915_reg_t reg,
-                                            enum intel_steering_type type)
+static bool reg_needs_read_steering(struct intel_gt *gt,
+                                   i915_reg_t reg,
+                                   enum intel_steering_type type)
 {
        const u32 offset = i915_mmio_reg_offset(reg);
        const struct intel_mmio_range *entry;
 
-       if (likely(!intel_gt_needs_read_steering(gt, type)))
+       if (likely(!gt->steering_table[type]))
                return false;
 
        for (entry = gt->steering_table[type]; entry->end; entry++) {
@@ -278,29 +318,29 @@ static bool intel_gt_reg_needs_read_steering(struct intel_gt *gt,
        return false;
 }
 
-/**
- * intel_gt_get_valid_steering - determines valid IDs for a class of MCR steering
+/*
+ * get_nonterminated_steering - determines valid IDs for a class of MCR steering
  * @gt: GT structure
  * @type: multicast register type
- * @sliceid: Slice ID returned
- * @subsliceid: Subslice ID returned
+ * @group: Group ID returned
+ * @instance: Instance ID returned
  *
- * Determines sliceid and subsliceid values that will steer reads
- * of a specific multicast register class to a valid value.
+ * Determines group and instance values that will steer reads of the specified
+ * MCR class to a non-terminated instance.
  */
-static void intel_gt_get_valid_steering(struct intel_gt *gt,
-                                       enum intel_steering_type type,
-                                       u8 *sliceid, u8 *subsliceid)
+static void get_nonterminated_steering(struct intel_gt *gt,
+                                      enum intel_steering_type type,
+                                      u8 *group, u8 *instance)
 {
        switch (type) {
        case L3BANK:
-               *sliceid = 0;           /* unused */
-               *subsliceid = __ffs(gt->info.l3bank_mask);
+               *group = 0;             /* unused */
+               *instance = __ffs(gt->info.l3bank_mask);
                break;
        case MSLICE:
                GEM_WARN_ON(!HAS_MSLICE_STEERING(gt->i915));
-               *sliceid = __ffs(gt->info.mslice_mask);
-               *subsliceid = 0;        /* unused */
+               *group = __ffs(gt->info.mslice_mask);
+               *instance = 0;  /* unused */
                break;
        case LNCF:
                /*
@@ -308,96 +348,105 @@ static void intel_gt_get_valid_steering(struct intel_gt *gt,
                 * can safely just steer to LNCF 0 in all cases.
                 */
                GEM_WARN_ON(!HAS_MSLICE_STEERING(gt->i915));
-               *sliceid = __ffs(gt->info.mslice_mask) << 1;
-               *subsliceid = 0;        /* unused */
+               *group = __ffs(gt->info.mslice_mask) << 1;
+               *instance = 0;  /* unused */
                break;
        case INSTANCE0:
                /*
                 * There are a lot of MCR types for which instance (0, 0)
                 * will always provide a non-terminated value.
                 */
-               *sliceid = 0;
-               *subsliceid = 0;
+               *group = 0;
+               *instance = 0;
                break;
        default:
                MISSING_CASE(type);
-               *sliceid = 0;
-               *subsliceid = 0;
+               *group = 0;
+               *instance = 0;
        }
 }
 
 /**
- * intel_gt_get_valid_steering_for_reg - get a valid steering for a register
+ * intel_gt_mcr_get_nonterminated_steering - find group/instance values that
+ *    will steer a register to a non-terminated instance
  * @gt: GT structure
  * @reg: register for which the steering is required
- * @sliceid: return variable for slice steering
- * @subsliceid: return variable for subslice steering
+ * @group: return variable for group steering
+ * @instance: return variable for instance steering
  *
- * This function returns a slice/subslice pair that is guaranteed to work for
+ * This function returns a group/instance pair that is guaranteed to work for
  * read steering of the given register. Note that a value will be returned even
  * if the register is not replicated and therefore does not actually require
  * steering.
  */
-void intel_gt_get_valid_steering_for_reg(struct intel_gt *gt, i915_reg_t reg,
-                                        u8 *sliceid, u8 *subsliceid)
+void intel_gt_mcr_get_nonterminated_steering(struct intel_gt *gt,
+                                            i915_reg_t reg,
+                                            u8 *group, u8 *instance)
 {
        int type;
 
        for (type = 0; type < NUM_STEERING_TYPES; type++) {
-               if (intel_gt_reg_needs_read_steering(gt, reg, type)) {
-                       intel_gt_get_valid_steering(gt, type, sliceid,
-                                                   subsliceid);
+               if (reg_needs_read_steering(gt, reg, type)) {
+                       get_nonterminated_steering(gt, type, group, instance);
                        return;
                }
        }
 
-       *sliceid = gt->default_steering.groupid;
-       *subsliceid = gt->default_steering.instanceid;
+       *group = gt->default_steering.groupid;
+       *instance = gt->default_steering.instanceid;
 }
 
 /**
- * intel_gt_read_register_fw - reads a GT register with support for multicast
+ * intel_gt_mcr_read_any_fw - reads one instance of an MCR register
  * @gt: GT structure
  * @reg: register to read
  *
- * This function will read a GT register.  If the register is a multicast
- * register, the read will be steered to a valid instance (i.e., one that
- * isn't fused off or powered down by power gating).
+ * Reads a GT MCR register.  The read will be steered to a non-terminated
+ * instance (i.e., one that isn't fused off or powered down by power gating).
+ * This function assumes the caller is already holding any necessary forcewake
+ * domains; use intel_gt_mcr_read_any() in cases where forcewake should be
+ * obtained automatically.
  *
- * Returns the value from a valid instance of @reg.
+ * Returns the value from a non-terminated instance of @reg.
  */
-u32 intel_gt_read_register_fw(struct intel_gt *gt, i915_reg_t reg)
+u32 intel_gt_mcr_read_any_fw(struct intel_gt *gt, i915_reg_t reg)
 {
        int type;
-       u8 sliceid, subsliceid;
+       u8 group, instance;
 
        for (type = 0; type < NUM_STEERING_TYPES; type++) {
-               if (intel_gt_reg_needs_read_steering(gt, reg, type)) {
-                       intel_gt_get_valid_steering(gt, type, &sliceid,
-                                                   &subsliceid);
-                       return intel_uncore_read_with_mcr_steering_fw(gt->uncore,
-                                                                     reg,
-                                                                     sliceid,
-                                                                     subsliceid);
+               if (reg_needs_read_steering(gt, reg, type)) {
+                       get_nonterminated_steering(gt, type, &group, &instance);
+                       return rw_with_mcr_steering_fw(gt->uncore, reg,
+                                                      FW_REG_READ,
+                                                      group, instance, 0);
                }
        }
 
        return intel_uncore_read_fw(gt->uncore, reg);
 }
 
-u32 intel_gt_read_register(struct intel_gt *gt, i915_reg_t reg)
+/**
+ * intel_gt_mcr_read_any - reads one instance of an MCR register
+ * @gt: GT structure
+ * @reg: register to read
+ *
+ * Reads a GT MCR register.  The read will be steered to a non-terminated
+ * instance (i.e., one that isn't fused off or powered down by power gating).
+ *
+ * Returns the value from a non-terminated instance of @reg.
+ */
+u32 intel_gt_mcr_read_any(struct intel_gt *gt, i915_reg_t reg)
 {
        int type;
-       u8 sliceid, subsliceid;
+       u8 group, instance;
 
        for (type = 0; type < NUM_STEERING_TYPES; type++) {
-               if (intel_gt_reg_needs_read_steering(gt, reg, type)) {
-                       intel_gt_get_valid_steering(gt, type, &sliceid,
-                                                   &subsliceid);
-                       return intel_uncore_read_with_mcr_steering(gt->uncore,
-                                                                  reg,
-                                                                  sliceid,
-                                                                  subsliceid);
+               if (reg_needs_read_steering(gt, reg, type)) {
+                       get_nonterminated_steering(gt, type, &group, &instance);
+                       return rw_with_mcr_steering(gt->uncore, reg,
+                                                   FW_REG_READ,
+                                                   group, instance, 0);
                }
        }
 
@@ -410,7 +459,7 @@ static void report_steering_type(struct drm_printer *p,
                                 bool dump_table)
 {
        const struct intel_mmio_range *entry;
-       u8 slice, subslice;
+       u8 group, instance;
 
        BUILD_BUG_ON(ARRAY_SIZE(intel_steering_types) != NUM_STEERING_TYPES);
 
@@ -420,9 +469,9 @@ static void report_steering_type(struct drm_printer *p,
                return;
        }
 
-       intel_gt_get_valid_steering(gt, type, &slice, &subslice);
-       drm_printf(p, "%s steering: sliceid=0x%x, subsliceid=0x%x\n",
-                  intel_steering_types[type], slice, subslice);
+       get_nonterminated_steering(gt, type, &group, &instance);
+       drm_printf(p, "%s steering: group=0x%x, instance=0x%x\n",
+                  intel_steering_types[type], group, instance);
 
        if (!dump_table)
                return;
@@ -431,10 +480,10 @@ static void report_steering_type(struct drm_printer *p,
                drm_printf(p, "\t0x%06x - 0x%06x\n", entry->start, entry->end);
 }
 
-void intel_gt_report_steering(struct drm_printer *p, struct intel_gt *gt,
-                             bool dump_table)
+void intel_gt_mcr_report_steering(struct drm_printer *p, struct intel_gt *gt,
+                                 bool dump_table)
 {
-       drm_printf(p, "Default steering: sliceid=0x%x, subsliceid=0x%x\n",
+       drm_printf(p, "Default steering: group=0x%x, instance=0x%x\n",
                   gt->default_steering.groupid,
                   gt->default_steering.instanceid);
 
index b570c15..506b0cb 100644 (file)
 
 void intel_gt_mcr_init(struct intel_gt *gt);
 
-u32 intel_uncore_read_with_mcr_steering_fw(struct intel_uncore *uncore,
-                                          i915_reg_t reg,
-                                          int slice, int subslice);
-u32 intel_uncore_read_with_mcr_steering(struct intel_uncore *uncore,
-                                       i915_reg_t reg, int slice, int subslice);
-void intel_uncore_write_with_mcr_steering(struct intel_uncore *uncore,
-                                         i915_reg_t reg, u32 value,
-                                         int slice, int subslice);
-
-u32 intel_gt_read_register_fw(struct intel_gt *gt, i915_reg_t reg);
-u32 intel_gt_read_register(struct intel_gt *gt, i915_reg_t reg);
-
-static inline bool intel_gt_needs_read_steering(struct intel_gt *gt,
-                                               enum intel_steering_type type)
-{
-       return gt->steering_table[type];
-}
-
-void intel_gt_get_valid_steering_for_reg(struct intel_gt *gt, i915_reg_t reg,
-                                        u8 *sliceid, u8 *subsliceid);
-
-void intel_gt_report_steering(struct drm_printer *p, struct intel_gt *gt,
-                             bool dump_table);
+u32 intel_gt_mcr_read(struct intel_gt *gt,
+                     i915_reg_t reg,
+                     int group, int instance);
+u32 intel_gt_mcr_read_any_fw(struct intel_gt *gt, i915_reg_t reg);
+u32 intel_gt_mcr_read_any(struct intel_gt *gt, i915_reg_t reg);
+
+void intel_gt_mcr_unicast_write(struct intel_gt *gt,
+                               i915_reg_t reg, u32 value,
+                               int group, int instance);
+void intel_gt_mcr_multicast_write(struct intel_gt *gt,
+                                 i915_reg_t reg, u32 value);
+void intel_gt_mcr_multicast_write_fw(struct intel_gt *gt,
+                                    i915_reg_t reg, u32 value);
+
+void intel_gt_mcr_get_nonterminated_steering(struct intel_gt *gt,
+                                            i915_reg_t reg,
+                                            u8 *group, u8 *instance);
+
+void intel_gt_mcr_report_steering(struct drm_printer *p, struct intel_gt *gt,
+                                 bool dump_table);
 
 #endif /* __INTEL_GT_MCR__ */
index 1f4e723..2ff4480 100644 (file)
@@ -105,11 +105,11 @@ static struct intel_memory_region *setup_lmem(struct intel_gt *gt)
                resource_size_t lmem_range;
                u64 tile_stolen, flat_ccs_base;
 
-               lmem_range = intel_gt_read_register(&i915->gt0, XEHPSDV_TILE0_ADDR_RANGE) & 0xFFFF;
+               lmem_range = intel_gt_mcr_read_any(&i915->gt0, XEHPSDV_TILE0_ADDR_RANGE) & 0xFFFF;
                lmem_size = lmem_range >> XEHPSDV_TILE_LMEM_RANGE_SHIFT;
                lmem_size *= SZ_1G;
 
-               flat_ccs_base = intel_gt_read_register(gt, XEHPSDV_FLAT_CCS_BASE_ADDR);
+               flat_ccs_base = intel_gt_mcr_read_any(gt, XEHPSDV_FLAT_CCS_BASE_ADDR);
                flat_ccs_base = (flat_ccs_base >> XEHPSDV_CCS_BASE_SHIFT) * SZ_64K;
 
                /* FIXME: Remove this when we have small-bar enabled */
index e4913ae..3213c59 100644 (file)
@@ -1083,7 +1083,7 @@ static void __add_mcr_wa(struct intel_gt *gt, struct i915_wa_list *wal,
        gt->default_steering.instanceid = subslice;
 
        if (drm_debug_enabled(DRM_UT_DRIVER))
-               intel_gt_report_steering(&p, gt, false);
+               intel_gt_mcr_report_steering(&p, gt, false);
 }
 
 static void
@@ -1624,13 +1624,13 @@ wa_list_apply(struct intel_gt *gt, const struct i915_wa_list *wal)
                u32 val, old = 0;
 
                /* open-coded rmw due to steering */
-               old = wa->clr ? intel_gt_read_register_fw(gt, wa->reg) : 0;
+               old = wa->clr ? intel_gt_mcr_read_any_fw(gt, wa->reg) : 0;
                val = (old & ~wa->clr) | wa->set;
                if (val != old || !wa->clr)
                        intel_uncore_write_fw(uncore, wa->reg, val);
 
                if (IS_ENABLED(CONFIG_DRM_I915_DEBUG_GEM))
-                       wa_verify(wa, intel_gt_read_register_fw(gt, wa->reg),
+                       wa_verify(wa, intel_gt_mcr_read_any_fw(gt, wa->reg),
                                  wal->name, "application");
        }
 
@@ -1661,7 +1661,7 @@ static bool wa_list_verify(struct intel_gt *gt,
 
        for (i = 0, wa = wal->list; i < wal->count; i++, wa++)
                ok &= wa_verify(wa,
-                               intel_gt_read_register_fw(gt, wa->reg),
+                               intel_gt_mcr_read_any_fw(gt, wa->reg),
                                wal->name, from);
 
        intel_uncore_forcewake_put__locked(uncore, fw);
index dea138d..ba7541f 100644 (file)
@@ -314,7 +314,7 @@ static long __must_check guc_mmio_reg_add(struct intel_gt *gt,
         * tracking, it is easier to just program the default steering for all
         * regs that don't need a non-default one.
         */
-       intel_gt_get_valid_steering_for_reg(gt, reg, &group, &inst);
+       intel_gt_mcr_get_nonterminated_steering(gt, reg, &group, &inst);
        entry.flags |= GUC_REGSET_STEERING(group, inst);
 
        slot = __mmio_reg_add(regset, &entry);