drm/i915/guc: Make scratch register base and count flexible
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Wed, 10 May 2017 12:59:27 +0000 (12:59 +0000)
committerJoonas Lahtinen <joonas.lahtinen@linux.intel.com>
Thu, 11 May 2017 09:49:26 +0000 (12:49 +0300)
We are using some scratch registers in MMIO based send function.
Make their base and count flexible in preparation of upcoming
GuC firmware/hardware changes. While around, change cmd len
parameter verification from WARN_ON to GEM_BUG_ON as we don't
need this all the time.

v2: call out WARN/GEM_BUG change in the commit msg (Daniele)
v3: don't overqualify the ints (Chris)
v4: rebase and use proper enum

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Suggested-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
drivers/gpu/drm/i915/intel_uc.c
drivers/gpu/drm/i915/intel_uc.h

index 72f49e679707cad9c1b836f1d027b5fcaa080860..07c5658c4b35f1b95a1b420777127d043475f1f0 100644 (file)
@@ -260,9 +260,36 @@ void intel_uc_fini_fw(struct drm_i915_private *dev_priv)
        __intel_uc_fw_fini(&dev_priv->huc.fw);
 }
 
+static inline i915_reg_t guc_send_reg(struct intel_guc *guc, u32 i)
+{
+       GEM_BUG_ON(!guc->send_regs.base);
+       GEM_BUG_ON(!guc->send_regs.count);
+       GEM_BUG_ON(i >= guc->send_regs.count);
+
+       return _MMIO(guc->send_regs.base + 4 * i);
+}
+
+static void guc_init_send_regs(struct intel_guc *guc)
+{
+       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       enum forcewake_domains fw_domains = 0;
+       unsigned int i;
+
+       guc->send_regs.base = i915_mmio_reg_offset(SOFT_SCRATCH(0));
+       guc->send_regs.count = SOFT_SCRATCH_COUNT - 1;
+
+       for (i = 0; i < guc->send_regs.count; i++) {
+               fw_domains |= intel_uncore_forcewake_for_reg(dev_priv,
+                                       guc_send_reg(guc, i),
+                                       FW_REG_READ | FW_REG_WRITE);
+       }
+       guc->send_regs.fw_domains = fw_domains;
+}
+
 static int guc_enable_communication(struct intel_guc *guc)
 {
        /* XXX: placeholder for alternate setup */
+       guc_init_send_regs(guc);
        guc->send = intel_guc_send_mmio;
        return 0;
 }
@@ -407,19 +434,19 @@ int intel_guc_send_mmio(struct intel_guc *guc, const u32 *action, u32 len)
        int i;
        int ret;
 
-       if (WARN_ON(len < 1 || len > 15))
-               return -EINVAL;
+       GEM_BUG_ON(!len);
+       GEM_BUG_ON(len > guc->send_regs.count);
 
        mutex_lock(&guc->send_mutex);
-       intel_uncore_forcewake_get(dev_priv, FORCEWAKE_BLITTER);
+       intel_uncore_forcewake_get(dev_priv, guc->send_regs.fw_domains);
 
        dev_priv->guc.action_count += 1;
        dev_priv->guc.action_cmd = action[0];
 
        for (i = 0; i < len; i++)
-               I915_WRITE(SOFT_SCRATCH(i), action[i]);
+               I915_WRITE(guc_send_reg(guc, i), action[i]);
 
-       POSTING_READ(SOFT_SCRATCH(i - 1));
+       POSTING_READ(guc_send_reg(guc, i - 1));
 
        intel_guc_notify(guc);
 
@@ -428,7 +455,7 @@ int intel_guc_send_mmio(struct intel_guc *guc, const u32 *action, u32 len)
         * Fast commands should still complete in 10us.
         */
        ret = __intel_wait_for_register_fw(dev_priv,
-                                          SOFT_SCRATCH(0),
+                                          guc_send_reg(guc, 0),
                                           INTEL_GUC_RECV_MASK,
                                           INTEL_GUC_RECV_MASK,
                                           10, 10, &status);
@@ -450,7 +477,7 @@ int intel_guc_send_mmio(struct intel_guc *guc, const u32 *action, u32 len)
        }
        dev_priv->guc.action_status = status;
 
-       intel_uncore_forcewake_put(dev_priv, FORCEWAKE_BLITTER);
+       intel_uncore_forcewake_put(dev_priv, guc->send_regs.fw_domains);
        mutex_unlock(&guc->send_mutex);
 
        return ret;
index 2af5633b8a0917533eb7ba969ba32e5b6346fc94..7618b710017561c92af04a48fc679495116de5e6 100644 (file)
@@ -205,6 +205,13 @@ struct intel_guc {
        uint64_t submissions[I915_NUM_ENGINES];
        uint32_t last_seqno[I915_NUM_ENGINES];
 
+       /* GuC's FW specific registers used in MMIO send */
+       struct {
+               u32 base;
+               unsigned int count;
+               enum forcewake_domains fw_domains;
+       } send_regs;
+
        /* To serialize the intel_guc_send actions */
        struct mutex send_mutex;