drm/i915/uc: prefer intel_gt over i915 in GuC/HuC paths

With our HW interface logic moving from i915 to gt and with GuC and HuC
being part of the gt HW, it makes sense to use the intel_gt structure
instead of i915 as our reference object in GuC/HuC paths.

Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20190713100016.8026-9-chris@chris-wilson.co.uk
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

diff --git a/drivers/gpu/drm/i915/gt/intel_gt.h b/drivers/gpu/drm/i915/gt/intel_gt.h
index 26c94521ad1b394731a7ca93af6d9b0d08a77824..640bb0531f5b52a523ab2a91f21addb49301972c 100644 (file)
--- a/drivers/gpu/drm/i915/gt/intel_gt.h
+++ b/drivers/gpu/drm/i915/gt/intel_gt.h
@@ -17,6 +17,16 @@ static inline struct intel_gt *uc_to_gt(struct intel_uc *uc)
        return container_of(uc, struct intel_gt, uc);
 }
 
+static inline struct intel_gt *guc_to_gt(struct intel_guc *guc)
+{
+       return container_of(guc, struct intel_gt, uc.guc);
+}
+
+static inline struct intel_gt *huc_to_gt(struct intel_huc *huc)
+{
+       return container_of(huc, struct intel_gt, uc.huc);
+}
+
 void intel_gt_init_early(struct intel_gt *gt, struct drm_i915_private *i915);
 void intel_gt_init_hw(struct drm_i915_private *i915);
 

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc.c b/drivers/gpu/drm/i915/gt/uc/intel_guc.c
index 4173b35bf104b7239266323cfbef5453ffb89d84..6b56f39072b14c341d6df280b5ab135403947089 100644 (file)
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc.c
@@ -22,6 +22,7 @@
  *
  */
 
+#include "gt/intel_gt.h"
 #include "intel_guc.h"
 #include "intel_guc_ads.h"
 #include "intel_guc_submission.h"
@@ -29,16 +30,16 @@
 
 static void gen8_guc_raise_irq(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_gt *gt = guc_to_gt(guc);
 
-       I915_WRITE(GUC_SEND_INTERRUPT, GUC_SEND_TRIGGER);
+       intel_uncore_write(gt->uncore, GUC_SEND_INTERRUPT, GUC_SEND_TRIGGER);
 }
 
 static void gen11_guc_raise_irq(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_gt *gt = guc_to_gt(guc);
 
-       I915_WRITE(GEN11_GUC_HOST_INTERRUPT, 0);
+       intel_uncore_write(gt->uncore, GEN11_GUC_HOST_INTERRUPT, 0);
 }
 
 static inline i915_reg_t guc_send_reg(struct intel_guc *guc, u32 i)
@@ -52,11 +53,11 @@ static inline i915_reg_t guc_send_reg(struct intel_guc *guc, u32 i)
 
 void intel_guc_init_send_regs(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_gt *gt = guc_to_gt(guc);
        enum forcewake_domains fw_domains = 0;
        unsigned int i;
 
-       if (INTEL_GEN(dev_priv) >= 11) {
+       if (INTEL_GEN(gt->i915) >= 11) {
                guc->send_regs.base =
                                i915_mmio_reg_offset(GEN11_SOFT_SCRATCH(0));
                guc->send_regs.count = GEN11_SOFT_SCRATCH_COUNT;
@@ -67,7 +68,7 @@ void intel_guc_init_send_regs(struct intel_guc *guc)
        }
 
        for (i = 0; i < guc->send_regs.count; i++) {
-               fw_domains |= intel_uncore_forcewake_for_reg(&dev_priv->uncore,
+               fw_domains |= intel_uncore_forcewake_for_reg(gt->uncore,
                                        guc_send_reg(guc, i),
                                        FW_REG_READ | FW_REG_WRITE);
        }
@@ -127,7 +128,7 @@ static void guc_shared_data_destroy(struct intel_guc *guc)
 
 int intel_guc_init(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_gt *gt = guc_to_gt(guc);
        int ret;
 
        ret = intel_uc_fw_init(&guc->fw);
@@ -153,7 +154,7 @@ int intel_guc_init(struct intel_guc *guc)
                goto err_ads;
 
        /* We need to notify the guc whenever we change the GGTT */
-       i915_ggtt_enable_guc(dev_priv);
+       i915_ggtt_enable_guc(gt->ggtt);
 
        return 0;
 
@@ -172,9 +173,9 @@ err_fetch:
 
 void intel_guc_fini(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_gt *gt = guc_to_gt(guc);
 
-       i915_ggtt_disable_guc(dev_priv);
+       i915_ggtt_disable_guc(gt->ggtt);
 
        intel_guc_ct_fini(&guc->ct);
 
@@ -282,7 +283,7 @@ static u32 guc_ctl_ads_flags(struct intel_guc *guc)
  */
 void intel_guc_init_params(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_uncore *uncore = guc_to_gt(guc)->uncore;
        u32 params[GUC_CTL_MAX_DWORDS];
        int i;
 
@@ -302,14 +303,14 @@ void intel_guc_init_params(struct intel_guc *guc)
         * they are power context saved so it's ok to release forcewake
         * when we are done here and take it again at xfer time.
         */
-       intel_uncore_forcewake_get(&dev_priv->uncore, FORCEWAKE_BLITTER);
+       intel_uncore_forcewake_get(uncore, FORCEWAKE_BLITTER);
 
-       I915_WRITE(SOFT_SCRATCH(0), 0);
+       intel_uncore_write(uncore, SOFT_SCRATCH(0), 0);
 
        for (i = 0; i < GUC_CTL_MAX_DWORDS; i++)
-               I915_WRITE(SOFT_SCRATCH(1 + i), params[i]);
+               intel_uncore_write(uncore, SOFT_SCRATCH(1 + i), params[i]);
 
-       intel_uncore_forcewake_put(&dev_priv->uncore, FORCEWAKE_BLITTER);
+       intel_uncore_forcewake_put(uncore, FORCEWAKE_BLITTER);
 }
 
 int intel_guc_send_nop(struct intel_guc *guc, const u32 *action, u32 len,
@@ -330,8 +331,7 @@ void intel_guc_to_host_event_handler_nop(struct intel_guc *guc)
 int intel_guc_send_mmio(struct intel_guc *guc, const u32 *action, u32 len,
                        u32 *response_buf, u32 response_buf_size)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-       struct intel_uncore *uncore = &dev_priv->uncore;
+       struct intel_uncore *uncore = guc_to_gt(guc)->uncore;
        u32 status;
        int i;
        int ret;
@@ -380,7 +380,8 @@ int intel_guc_send_mmio(struct intel_guc *guc, const u32 *action, u32 len,
                int count = min(response_buf_size, guc->send_regs.count - 1);
 
                for (i = 0; i < count; i++)
-                       response_buf[i] = I915_READ(guc_send_reg(guc, i + 1));
+                       response_buf[i] = intel_uncore_read(uncore,
+                                                           guc_send_reg(guc, i + 1));
        }
 
        /* Use data from the GuC response as our return value */
@@ -454,7 +455,7 @@ int intel_guc_auth_huc(struct intel_guc *guc, u32 rsa_offset)
  */
 int intel_guc_suspend(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_uncore *uncore = guc_to_gt(guc)->uncore;
        int ret;
        u32 status;
        u32 action[] = {
@@ -472,13 +473,14 @@ int intel_guc_suspend(struct intel_guc *guc)
         * in progress so we need to take care of that ourselves as well.
         */
 
-       I915_WRITE(SOFT_SCRATCH(14), INTEL_GUC_SLEEP_STATE_INVALID_MASK);
+       intel_uncore_write(uncore, SOFT_SCRATCH(14),
+                          INTEL_GUC_SLEEP_STATE_INVALID_MASK);
 
        ret = intel_guc_send(guc, action, ARRAY_SIZE(action));
        if (ret)
                return ret;
 
-       ret = __intel_wait_for_register(&dev_priv->uncore, SOFT_SCRATCH(14),
+       ret = __intel_wait_for_register(uncore, SOFT_SCRATCH(14),
                                        INTEL_GUC_SLEEP_STATE_INVALID_MASK,
                                        0, 0, 10, &status);
        if (ret)
@@ -574,17 +576,17 @@ int intel_guc_resume(struct intel_guc *guc)
  */
 struct i915_vma *intel_guc_allocate_vma(struct intel_guc *guc, u32 size)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_gt *gt = guc_to_gt(guc);
        struct drm_i915_gem_object *obj;
        struct i915_vma *vma;
        u64 flags;
        int ret;
 
-       obj = i915_gem_object_create_shmem(dev_priv, size);
+       obj = i915_gem_object_create_shmem(gt->i915, size);
        if (IS_ERR(obj))
                return ERR_CAST(obj);
 
-       vma = i915_vma_instance(obj, &dev_priv->ggtt.vm, NULL);
+       vma = i915_vma_instance(obj, &gt->ggtt->vm, NULL);
        if (IS_ERR(vma))
                goto err;
 

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c
index ee95d4960c5cc620a7d38f13aa44167e183cedc8..98305e3fd42c7ef31914791c0566beda0edebbb6 100644 (file)
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c
@@ -27,6 +27,7 @@
  *    Alex Dai <yu.dai@intel.com>
  */
 
+#include "gt/intel_gt.h"
 #include "intel_guc_fw.h"
 #include "i915_drv.h"
 
@@ -129,35 +130,37 @@ void intel_guc_fw_init_early(struct intel_guc *guc)
 
 static void guc_prepare_xfer(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_gt *gt = guc_to_gt(guc);
+       struct intel_uncore *uncore = gt->uncore;
+       u32 shim_flags = GUC_DISABLE_SRAM_INIT_TO_ZEROES |
+                        GUC_ENABLE_READ_CACHE_LOGIC |
+                        GUC_ENABLE_MIA_CACHING |
+                        GUC_ENABLE_READ_CACHE_FOR_SRAM_DATA |
+                        GUC_ENABLE_READ_CACHE_FOR_WOPCM_DATA |
+                        GUC_ENABLE_MIA_CLOCK_GATING;
 
        /* Must program this register before loading the ucode with DMA */
-       I915_WRITE(GUC_SHIM_CONTROL, GUC_DISABLE_SRAM_INIT_TO_ZEROES |
-                                    GUC_ENABLE_READ_CACHE_LOGIC |
-                                    GUC_ENABLE_MIA_CACHING |
-                                    GUC_ENABLE_READ_CACHE_FOR_SRAM_DATA |
-                                    GUC_ENABLE_READ_CACHE_FOR_WOPCM_DATA |
-                                    GUC_ENABLE_MIA_CLOCK_GATING);
-
-       if (IS_GEN9_LP(dev_priv))
-               I915_WRITE(GEN9LP_GT_PM_CONFIG, GT_DOORBELL_ENABLE);
+       intel_uncore_write(uncore, GUC_SHIM_CONTROL, shim_flags);
+
+       if (IS_GEN9_LP(gt->i915))
+               intel_uncore_write(uncore, GEN9LP_GT_PM_CONFIG, GT_DOORBELL_ENABLE);
        else
-               I915_WRITE(GEN9_GT_PM_CONFIG, GT_DOORBELL_ENABLE);
+               intel_uncore_write(uncore, GEN9_GT_PM_CONFIG, GT_DOORBELL_ENABLE);
 
-       if (IS_GEN(dev_priv, 9)) {
+       if (IS_GEN(gt->i915, 9)) {
                /* DOP Clock Gating Enable for GuC clocks */
-               I915_WRITE(GEN7_MISCCPCTL, (GEN8_DOP_CLOCK_GATE_GUC_ENABLE |
-                                           I915_READ(GEN7_MISCCPCTL)));
+               intel_uncore_rmw(uncore, GEN7_MISCCPCTL,
+                                0, GEN8_DOP_CLOCK_GATE_GUC_ENABLE);
 
                /* allows for 5us (in 10ns units) before GT can go to RC6 */
-               I915_WRITE(GUC_ARAT_C6DIS, 0x1FF);
+               intel_uncore_write(uncore, GUC_ARAT_C6DIS, 0x1FF);
        }
 }
 
 /* Copy RSA signature from the fw image to HW for verification */
 static void guc_xfer_rsa(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_uncore *uncore = guc_to_gt(guc)->uncore;
        struct intel_uc_fw *fw = &guc->fw;
        struct sg_table *pages = fw->obj->mm.pages;
        u32 rsa[UOS_RSA_SCRATCH_COUNT];
@@ -167,15 +170,13 @@ static void guc_xfer_rsa(struct intel_guc *guc)
                           rsa, sizeof(rsa), fw->rsa_offset);
 
        for (i = 0; i < UOS_RSA_SCRATCH_COUNT; i++)
-               I915_WRITE(UOS_RSA_SCRATCH(i), rsa[i]);
+               intel_uncore_write(uncore, UOS_RSA_SCRATCH(i), rsa[i]);
 }
 
-static bool guc_xfer_completed(struct intel_guc *guc, u32 *status)
+static bool guc_xfer_completed(struct intel_uncore *uncore, u32 *status)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-
        /* Did we complete the xfer? */
-       *status = I915_READ(DMA_CTRL);
+       *status = intel_uncore_read(uncore, DMA_CTRL);
        return !(*status & START_DMA);
 }
 
@@ -188,10 +189,9 @@ static bool guc_xfer_completed(struct intel_guc *guc, u32 *status)
  * This is used for polling the GuC status in a wait_for()
  * loop below.
  */
-static inline bool guc_ready(struct intel_guc *guc, u32 *status)
+static inline bool guc_ready(struct intel_uncore *uncore, u32 *status)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-       u32 val = I915_READ(GUC_STATUS);
+       u32 val = intel_uncore_read(uncore, GUC_STATUS);
        u32 uk_val = val & GS_UKERNEL_MASK;
 
        *status = val;
@@ -199,9 +199,8 @@ static inline bool guc_ready(struct intel_guc *guc, u32 *status)
                ((val & GS_MIA_CORE_STATE) && (uk_val == GS_UKERNEL_LAPIC_DONE));
 }
 
-static int guc_wait_ucode(struct intel_guc *guc)
+static int guc_wait_ucode(struct intel_uncore *uncore)
 {
-       struct drm_i915_private *i915 = guc_to_i915(guc);
        u32 status;
        int ret;
 
@@ -213,7 +212,7 @@ static int guc_wait_ucode(struct intel_guc *guc)
         * (Higher levels of the driver may decide to reset the GuC and
         * attempt the ucode load again if this happens.)
         */
-       ret = wait_for(guc_ready(guc, &status), 100);
+       ret = wait_for(guc_ready(uncore, &status), 100);
        DRM_DEBUG_DRIVER("GuC status %#x\n", status);
 
        if ((status & GS_BOOTROM_MASK) == GS_BOOTROM_RSA_FAILED) {
@@ -223,11 +222,11 @@ static int guc_wait_ucode(struct intel_guc *guc)
 
        if ((status & GS_UKERNEL_MASK) == GS_UKERNEL_EXCEPTION) {
                DRM_ERROR("GuC firmware exception. EIP: %#x\n",
-                         intel_uncore_read(&i915->uncore, SOFT_SCRATCH(13)));
+                         intel_uncore_read(uncore, SOFT_SCRATCH(13)));
                ret = -ENXIO;
        }
 
-       if (ret == 0 && !guc_xfer_completed(guc, &status)) {
+       if (ret == 0 && !guc_xfer_completed(uncore, &status)) {
                DRM_ERROR("GuC is ready, but the xfer %08x is incomplete\n",
                          status);
                ret = -ENXIO;
@@ -245,7 +244,7 @@ static int guc_wait_ucode(struct intel_guc *guc)
  */
 static int guc_xfer_ucode(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_uncore *uncore = guc_to_gt(guc)->uncore;
        struct intel_uc_fw *guc_fw = &guc->fw;
        unsigned long offset;
 
@@ -253,24 +252,26 @@ static int guc_xfer_ucode(struct intel_guc *guc)
         * The header plus uCode will be copied to WOPCM via DMA, excluding any
         * other components
         */
-       I915_WRITE(DMA_COPY_SIZE, guc_fw->header_size + guc_fw->ucode_size);
+       intel_uncore_write(uncore, DMA_COPY_SIZE,
+                          guc_fw->header_size + guc_fw->ucode_size);
 
        /* Set the source address for the new blob */
        offset = intel_uc_fw_ggtt_offset(guc_fw) + guc_fw->header_offset;
-       I915_WRITE(DMA_ADDR_0_LOW, lower_32_bits(offset));
-       I915_WRITE(DMA_ADDR_0_HIGH, upper_32_bits(offset) & 0xFFFF);
+       intel_uncore_write(uncore, DMA_ADDR_0_LOW, lower_32_bits(offset));
+       intel_uncore_write(uncore, DMA_ADDR_0_HIGH, upper_32_bits(offset) & 0xFFFF);
 
        /*
         * Set the DMA destination. Current uCode expects the code to be
         * loaded at 8k; locations below this are used for the stack.
         */
-       I915_WRITE(DMA_ADDR_1_LOW, 0x2000);
-       I915_WRITE(DMA_ADDR_1_HIGH, DMA_ADDRESS_SPACE_WOPCM);
+       intel_uncore_write(uncore, DMA_ADDR_1_LOW, 0x2000);
+       intel_uncore_write(uncore, DMA_ADDR_1_HIGH, DMA_ADDRESS_SPACE_WOPCM);
 
        /* Finally start the DMA */
-       I915_WRITE(DMA_CTRL, _MASKED_BIT_ENABLE(UOS_MOVE | START_DMA));
+       intel_uncore_write(uncore, DMA_CTRL,
+                          _MASKED_BIT_ENABLE(UOS_MOVE | START_DMA));
 
-       return guc_wait_ucode(guc);
+       return guc_wait_ucode(uncore);
 }
 /*
  * Load the GuC firmware blob into the MinuteIA.
@@ -278,12 +279,12 @@ static int guc_xfer_ucode(struct intel_guc *guc)
 static int guc_fw_xfer(struct intel_uc_fw *guc_fw)
 {
        struct intel_guc *guc = container_of(guc_fw, struct intel_guc, fw);
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_uncore *uncore = guc_to_gt(guc)->uncore;
        int ret;
 
        GEM_BUG_ON(guc_fw->type != INTEL_UC_FW_TYPE_GUC);
 
-       intel_uncore_forcewake_get(&dev_priv->uncore, FORCEWAKE_ALL);
+       intel_uncore_forcewake_get(uncore, FORCEWAKE_ALL);
 
        guc_prepare_xfer(guc);
 
@@ -296,7 +297,7 @@ static int guc_fw_xfer(struct intel_uc_fw *guc_fw)
 
        ret = guc_xfer_ucode(guc);
 
-       intel_uncore_forcewake_put(&dev_priv->uncore, FORCEWAKE_ALL);
+       intel_uncore_forcewake_put(uncore, FORCEWAKE_ALL);
 
        return ret;
 }

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
index fa22d377ae48d6b58fd101692fa70fc76c39fab2..a0f2a01365bce4c3acc3b36ed5e5a4db07f76245 100644 (file)
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
@@ -28,6 +28,7 @@
 
 #include "gt/intel_context.h"
 #include "gt/intel_engine_pm.h"
+#include "gt/intel_gt.h"
 #include "gt/intel_lrc_reg.h"
 #include "intel_guc_submission.h"
 
@@ -201,10 +202,10 @@ static struct guc_doorbell_info *__get_doorbell(struct intel_guc_client *client)
 
 static bool __doorbell_valid(struct intel_guc *guc, u16 db_id)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_uncore *uncore = guc_to_gt(guc)->uncore;
 
        GEM_BUG_ON(db_id >= GUC_NUM_DOORBELLS);
-       return I915_READ(GEN8_DRBREGL(db_id)) & GEN8_DRB_VALID;
+       return intel_uncore_read(uncore, GEN8_DRBREGL(db_id)) & GEN8_DRB_VALID;
 }
 
 static void __init_doorbell(struct intel_guc_client *client)
@@ -1001,9 +1002,10 @@ void intel_guc_submission_fini(struct intel_guc *guc)
                guc_stage_desc_pool_destroy(guc);
 }
 
-static void guc_interrupts_capture(struct drm_i915_private *dev_priv)
+static void guc_interrupts_capture(struct intel_gt *gt)
 {
-       struct intel_rps *rps = &dev_priv->gt_pm.rps;
+       struct intel_rps *rps = &gt->i915->gt_pm.rps;
+       struct intel_uncore *uncore = gt->uncore;
        struct intel_engine_cs *engine;
        enum intel_engine_id id;
        int irqs;
@@ -1012,16 +1014,16 @@ static void guc_interrupts_capture(struct drm_i915_private *dev_priv)
         * to GuC
         */
        irqs = _MASKED_BIT_ENABLE(GFX_INTERRUPT_STEERING);
-       for_each_engine(engine, dev_priv, id)
+       for_each_engine(engine, gt->i915, id)
                ENGINE_WRITE(engine, RING_MODE_GEN7, irqs);
 
        /* route USER_INTERRUPT to Host, all others are sent to GuC. */
        irqs = GT_RENDER_USER_INTERRUPT << GEN8_RCS_IRQ_SHIFT |
               GT_RENDER_USER_INTERRUPT << GEN8_BCS_IRQ_SHIFT;
        /* These three registers have the same bit definitions */
-       I915_WRITE(GUC_BCS_RCS_IER, ~irqs);
-       I915_WRITE(GUC_VCS2_VCS1_IER, ~irqs);
-       I915_WRITE(GUC_WD_VECS_IER, ~irqs);
+       intel_uncore_write(uncore, GUC_BCS_RCS_IER, ~irqs);
+       intel_uncore_write(uncore, GUC_VCS2_VCS1_IER, ~irqs);
+       intel_uncore_write(uncore, GUC_WD_VECS_IER, ~irqs);
 
        /*
         * The REDIRECT_TO_GUC bit of the PMINTRMSK register directs all
@@ -1046,9 +1048,10 @@ static void guc_interrupts_capture(struct drm_i915_private *dev_priv)
        rps->pm_intrmsk_mbz &= ~GEN8_PMINTR_DISABLE_REDIRECT_TO_GUC;
 }
 
-static void guc_interrupts_release(struct drm_i915_private *dev_priv)
+static void guc_interrupts_release(struct intel_gt *gt)
 {
-       struct intel_rps *rps = &dev_priv->gt_pm.rps;
+       struct intel_rps *rps = &gt->i915->gt_pm.rps;
+       struct intel_uncore *uncore = gt->uncore;
        struct intel_engine_cs *engine;
        enum intel_engine_id id;
        int irqs;
@@ -1059,13 +1062,13 @@ static void guc_interrupts_release(struct drm_i915_private *dev_priv)
         */
        irqs = _MASKED_FIELD(GFX_FORWARD_VBLANK_MASK, GFX_FORWARD_VBLANK_NEVER);
        irqs |= _MASKED_BIT_DISABLE(GFX_INTERRUPT_STEERING);
-       for_each_engine(engine, dev_priv, id)
+       for_each_engine(engine, gt->i915, id)
                ENGINE_WRITE(engine, RING_MODE_GEN7, irqs);
 
        /* route all GT interrupts to the host */
-       I915_WRITE(GUC_BCS_RCS_IER, 0);
-       I915_WRITE(GUC_VCS2_VCS1_IER, 0);
-       I915_WRITE(GUC_WD_VECS_IER, 0);
+       intel_uncore_write(uncore, GUC_BCS_RCS_IER, 0);
+       intel_uncore_write(uncore, GUC_VCS2_VCS1_IER, 0);
+       intel_uncore_write(uncore, GUC_WD_VECS_IER, 0);
 
        rps->pm_intrmsk_mbz |= GEN8_PMINTR_DISABLE_REDIRECT_TO_GUC;
        rps->pm_intrmsk_mbz &= ~ARAT_EXPIRED_INTRMSK;
@@ -1115,7 +1118,7 @@ static void guc_set_default_submission(struct intel_engine_cs *engine)
 
 int intel_guc_submission_enable(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_gt *gt = guc_to_gt(guc);
        struct intel_engine_cs *engine;
        enum intel_engine_id id;
        int err;
@@ -1140,9 +1143,9 @@ int intel_guc_submission_enable(struct intel_guc *guc)
                return err;
 
        /* Take over from manual control of ELSP (execlists) */
-       guc_interrupts_capture(dev_priv);
+       guc_interrupts_capture(gt);
 
-       for_each_engine(engine, dev_priv, id) {
+       for_each_engine(engine, gt->i915, id) {
                engine->set_default_submission = guc_set_default_submission;
                engine->set_default_submission(engine);
        }
@@ -1152,11 +1155,11 @@ int intel_guc_submission_enable(struct intel_guc *guc)
 
 void intel_guc_submission_disable(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_gt *gt = guc_to_gt(guc);
 
-       GEM_BUG_ON(dev_priv->gt.awake); /* GT should be parked first */
+       GEM_BUG_ON(gt->awake); /* GT should be parked first */
 
-       guc_interrupts_release(dev_priv);
+       guc_interrupts_release(gt);
        guc_clients_disable(guc);
 }
 

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_huc.c b/drivers/gpu/drm/i915/gt/uc/intel_huc.c
index 581c9c3d4fc0926d380f582a44da673cb4f85d95..99f0fa2e8ff570fef555ff5668c83d4d96426bf0 100644 (file)
--- a/drivers/gpu/drm/i915/gt/uc/intel_huc.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_huc.c
@@ -24,6 +24,7 @@
 
 #include <linux/types.h>
 
+#include "gt/intel_gt.h"
 #include "intel_huc.h"
 #include "i915_drv.h"
 
@@ -46,8 +47,8 @@ void intel_huc_init_early(struct intel_huc *huc)
 
 static int intel_huc_rsa_data_create(struct intel_huc *huc)
 {
-       struct drm_i915_private *i915 = huc_to_i915(huc);
-       struct intel_guc *guc = &i915->gt.uc.guc;
+       struct intel_gt *gt = huc_to_gt(huc);
+       struct intel_guc *guc = &gt->uc.guc;
        struct i915_vma *vma;
        void *vaddr;
 
@@ -112,8 +113,8 @@ void intel_huc_fini(struct intel_huc *huc)
  */
 int intel_huc_auth(struct intel_huc *huc)
 {
-       struct drm_i915_private *i915 = huc_to_i915(huc);
-       struct intel_guc *guc = &i915->gt.uc.guc;
+       struct intel_gt *gt = huc_to_gt(huc);
+       struct intel_guc *guc = &gt->uc.guc;
        int ret;
 
        if (huc->fw.load_status != INTEL_UC_FIRMWARE_SUCCESS)
@@ -127,7 +128,7 @@ int intel_huc_auth(struct intel_huc *huc)
        }
 
        /* Check authentication status, it should be done by now */
-       ret = __intel_wait_for_register(&i915->uncore,
+       ret = __intel_wait_for_register(gt->uncore,
                                        huc->status.reg,
                                        huc->status.mask,
                                        huc->status.value,
@@ -159,16 +160,15 @@ fail:
  */
 int intel_huc_check_status(struct intel_huc *huc)
 {
-       struct drm_i915_private *dev_priv = huc_to_i915(huc);
+       struct intel_gt *gt = huc_to_gt(huc);
        intel_wakeref_t wakeref;
        bool status = false;
 
-       if (!HAS_HUC(dev_priv))
+       if (!intel_uc_is_using_huc(&gt->uc))
                return -ENODEV;
 
-       with_intel_runtime_pm(&dev_priv->runtime_pm, wakeref)
-               status = (I915_READ(huc->status.reg) & huc->status.mask) ==
-                         huc->status.value;
+       with_intel_runtime_pm(&gt->i915->runtime_pm, wakeref)
+               status = intel_uncore_read(gt->uncore, huc->status.reg);
 
-       return status;
+       return (status & huc->status.mask) == huc->status.value;
 }

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_huc_fw.c b/drivers/gpu/drm/i915/gt/uc/intel_huc_fw.c
index 06e726ba98632643fa89a2e81ace29950961ec7f..becfd34a26c11cd70dcc4817b1aef534f3768b1c 100644 (file)
--- a/drivers/gpu/drm/i915/gt/uc/intel_huc_fw.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_huc_fw.c
@@ -4,6 +4,7 @@
  * Copyright © 2014-2018 Intel Corporation
  */
 
+#include "gt/intel_gt.h"
 #include "intel_huc_fw.h"
 #include "i915_drv.h"
 
@@ -139,8 +140,7 @@ static void huc_xfer_rsa(struct intel_huc *huc)
 static int huc_xfer_ucode(struct intel_huc *huc)
 {
        struct intel_uc_fw *huc_fw = &huc->fw;
-       struct drm_i915_private *dev_priv = huc_to_i915(huc);
-       struct intel_uncore *uncore = &dev_priv->uncore;
+       struct intel_uncore *uncore = huc_to_gt(huc)->uncore;
        unsigned long offset = 0;
        u32 size;
        int ret;

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc.c b/drivers/gpu/drm/i915/gt/uc/intel_uc.c
index 0c43d547bc9475e228c073fd2ef64c2e2b80d3b9..41a770865477b25a0c0d4b9a8f05531dd14f9607 100644 (file)
--- a/drivers/gpu/drm/i915/gt/uc/intel_uc.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_uc.c
@@ -228,7 +228,7 @@ static void guc_free_load_err_log(struct intel_guc *guc)
  */
 static void guc_clear_mmio_msg(struct intel_guc *guc)
 {
-       intel_uncore_write(&guc_to_i915(guc)->uncore, SOFT_SCRATCH(15), 0);
+       intel_uncore_write(guc_to_gt(guc)->uncore, SOFT_SCRATCH(15), 0);
 }
 
 static void guc_get_mmio_msg(struct intel_guc *guc)
@@ -237,7 +237,7 @@ static void guc_get_mmio_msg(struct intel_guc *guc)
 
        spin_lock_irq(&guc->irq_lock);
 
-       val = intel_uncore_read(&guc_to_i915(guc)->uncore, SOFT_SCRATCH(15));
+       val = intel_uncore_read(guc_to_gt(guc)->uncore, SOFT_SCRATCH(15));
        guc->mmio_msg |= val & guc->msg_enabled_mask;
 
        /*

diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 6da564e27a64bce333b394a02457c7909f4f8051..1e3ccdf7fe1b311b3250208ab8fb6a0b570077d0 100644 (file)
--- a/drivers/gpu/drm/i915/i915_gem_gtt.c
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
@@ -3333,10 +3333,8 @@ int i915_ggtt_enable_hw(struct drm_i915_private *dev_priv)
        return 0;
 }
 
-void i915_ggtt_enable_guc(struct drm_i915_private *i915)
+void i915_ggtt_enable_guc(struct i915_ggtt *ggtt)
 {
-       struct i915_ggtt *ggtt = &i915->ggtt;
-
        GEM_BUG_ON(ggtt->invalidate != gen6_ggtt_invalidate);
 
        ggtt->invalidate = guc_ggtt_invalidate;
@@ -3344,10 +3342,8 @@ void i915_ggtt_enable_guc(struct drm_i915_private *i915)
        ggtt->invalidate(ggtt);
 }
 
-void i915_ggtt_disable_guc(struct drm_i915_private *i915)
+void i915_ggtt_disable_guc(struct i915_ggtt *ggtt)
 {
-       struct i915_ggtt *ggtt = &i915->ggtt;
-
        /* XXX Temporary pardon for error unload */
        if (ggtt->invalidate == gen6_ggtt_invalidate)
                return;

diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.h b/drivers/gpu/drm/i915/i915_gem_gtt.h
index a450349b3a5026b6394a6dd8ffecc3794fe2b039..de156634a8894b32c1c0ea1618a026345ba17bcc 100644 (file)
--- a/drivers/gpu/drm/i915/i915_gem_gtt.h
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.h
@@ -631,8 +631,8 @@ i915_vm_to_ppgtt(struct i915_address_space *vm)
 int i915_ggtt_probe_hw(struct drm_i915_private *dev_priv);
 int i915_ggtt_init_hw(struct drm_i915_private *dev_priv);
 int i915_ggtt_enable_hw(struct drm_i915_private *dev_priv);
-void i915_ggtt_enable_guc(struct drm_i915_private *i915);
-void i915_ggtt_disable_guc(struct drm_i915_private *i915);
+void i915_ggtt_enable_guc(struct i915_ggtt *ggtt);
+void i915_ggtt_disable_guc(struct i915_ggtt *ggtt);
 int i915_init_ggtt(struct drm_i915_private *dev_priv);
 void i915_ggtt_driver_release(struct drm_i915_private *dev_priv);