drm/i915/uc: kill <g,h>uc_to_i915
authorDaniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Sat, 13 Jul 2019 10:00:16 +0000 (11:00 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Sat, 13 Jul 2019 19:11:54 +0000 (20:11 +0100)
Get rid of them to avoid more users being added while the guc code
transitions to use gt more than i915.

Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Acked-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20190713100016.8026-11-chris@chris-wilson.co.uk
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
drivers/gpu/drm/i915/gt/uc/intel_guc.c
drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c
drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c
drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
drivers/gpu/drm/i915/gt/uc/intel_huc.c
drivers/gpu/drm/i915/gt/uc/intel_huc_fw.c
drivers/gpu/drm/i915/gt/uc/intel_uc.c
drivers/gpu/drm/i915/i915_drv.h

index 6b56f39072b14c341d6df280b5ab135403947089..83f2c197375ffeb637ba502190a48d712ce580fe 100644 (file)
@@ -77,7 +77,7 @@ void intel_guc_init_send_regs(struct intel_guc *guc)
 
 void intel_guc_init_early(struct intel_guc *guc)
 {
-       struct drm_i915_private *i915 = guc_to_i915(guc);
+       struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
 
        intel_guc_fw_init_early(guc);
        intel_guc_ct_init_early(&guc->ct);
@@ -204,7 +204,7 @@ static u32 guc_ctl_feature_flags(struct intel_guc *guc)
 {
        u32 flags = 0;
 
-       if (!USES_GUC_SUBMISSION(guc_to_i915(guc)))
+       if (!intel_uc_is_using_guc_submission(&guc_to_gt(guc)->uc))
                flags |= GUC_CTL_DISABLE_SCHEDULER;
 
        return flags;
@@ -214,7 +214,7 @@ static u32 guc_ctl_ctxinfo_flags(struct intel_guc *guc)
 {
        u32 flags = 0;
 
-       if (USES_GUC_SUBMISSION(guc_to_i915(guc))) {
+       if (intel_uc_is_using_guc_submission(&guc_to_gt(guc)->uc)) {
                u32 ctxnum, base;
 
                base = intel_guc_ggtt_offset(guc, guc->stage_desc_pool);
@@ -414,7 +414,7 @@ int intel_guc_to_host_process_recv_msg(struct intel_guc *guc,
 
 int intel_guc_sample_forcewake(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct drm_i915_private *dev_priv = guc_to_gt(guc)->i915;
        u32 action[2];
 
        action[0] = INTEL_GUC_ACTION_SAMPLE_FORCEWAKE;
index 69859d1e047fed78897da8753c0b76c06650afd7..a0da80241f22bc7ad9a0f8d7019c2b46d8f8a01d 100644 (file)
@@ -22,6 +22,7 @@
  *
  */
 
+#include "gt/intel_gt.h"
 #include "intel_guc_ads.h"
 #include "intel_uc.h"
 #include "i915_drv.h"
@@ -85,7 +86,7 @@ struct __guc_ads_blob {
 
 static void __guc_ads_init(struct intel_guc *guc)
 {
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct drm_i915_private *dev_priv = guc_to_gt(guc)->i915;
        struct __guc_ads_blob *blob = guc->ads_blob;
        const u32 skipped_size = LRC_PPHWSP_SZ * PAGE_SIZE + LR_HW_CONTEXT_SIZE;
        u32 base;
index 98305e3fd42c7ef31914791c0566beda0edebbb6..3dfa40fdbe997b5fa05085cf1c84d0d709e01235 100644 (file)
@@ -76,7 +76,7 @@ MODULE_FIRMWARE(ICL_GUC_FIRMWARE_PATH);
 static void guc_fw_select(struct intel_uc_fw *guc_fw)
 {
        struct intel_guc *guc = container_of(guc_fw, struct intel_guc, fw);
-       struct drm_i915_private *i915 = guc_to_i915(guc);
+       struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
 
        GEM_BUG_ON(guc_fw->type != INTEL_UC_FW_TYPE_GUC);
 
index 9be5d3a6fb5ffa3e51f20d3f5b5ddef9e45a2368..77fda1e85d3becc895b1f0facc94eb5cacfec6da 100644 (file)
@@ -24,6 +24,7 @@
 
 #include <linux/debugfs.h>
 
+#include "gt/intel_gt.h"
 #include "intel_guc_log.h"
 #include "i915_drv.h"
 
@@ -209,7 +210,7 @@ static bool guc_check_log_buf_overflow(struct intel_guc_log *log,
                        log->stats[type].sampled_overflow += 16;
                }
 
-               dev_notice_ratelimited(guc_to_i915(log_to_guc(log))->drm.dev,
+               dev_notice_ratelimited(guc_to_gt(log_to_guc(log))->i915->drm.dev,
                                       "GuC log buffer overflow\n");
        }
 
@@ -383,7 +384,7 @@ void intel_guc_log_init_early(struct intel_guc_log *log)
 static int guc_log_relay_create(struct intel_guc_log *log)
 {
        struct intel_guc *guc = log_to_guc(log);
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct drm_i915_private *dev_priv = guc_to_gt(guc)->i915;
        struct rchan *guc_log_relay_chan;
        size_t n_subbufs, subbuf_size;
        int ret;
@@ -429,7 +430,7 @@ static void guc_log_relay_destroy(struct intel_guc_log *log)
 static void guc_log_capture_logs(struct intel_guc_log *log)
 {
        struct intel_guc *guc = log_to_guc(log);
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct drm_i915_private *dev_priv = guc_to_gt(guc)->i915;
        intel_wakeref_t wakeref;
 
        guc_read_update_log_buffer(log);
@@ -498,7 +499,7 @@ void intel_guc_log_destroy(struct intel_guc_log *log)
 int intel_guc_log_set_level(struct intel_guc_log *log, u32 level)
 {
        struct intel_guc *guc = log_to_guc(log);
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct drm_i915_private *dev_priv = guc_to_gt(guc)->i915;
        intel_wakeref_t wakeref;
        int ret = 0;
 
@@ -593,7 +594,7 @@ out_unlock:
 void intel_guc_log_relay_flush(struct intel_guc_log *log)
 {
        struct intel_guc *guc = log_to_guc(log);
-       struct drm_i915_private *i915 = guc_to_i915(guc);
+       struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
        intel_wakeref_t wakeref;
 
        /*
@@ -612,7 +613,7 @@ void intel_guc_log_relay_flush(struct intel_guc_log *log)
 void intel_guc_log_relay_close(struct intel_guc_log *log)
 {
        struct intel_guc *guc = log_to_guc(log);
-       struct drm_i915_private *i915 = guc_to_i915(guc);
+       struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
 
        guc_log_disable_flush_events(log);
        intel_synchronize_irq(i915);
index 99f0fa2e8ff570fef555ff5668c83d4d96426bf0..bc14439173d76f2a0c0a388cebbab86c67ffef11 100644 (file)
@@ -30,7 +30,7 @@
 
 void intel_huc_init_early(struct intel_huc *huc)
 {
-       struct drm_i915_private *i915 = huc_to_i915(huc);
+       struct drm_i915_private *i915 = huc_to_gt(huc)->i915;
 
        intel_huc_fw_init_early(huc);
 
index becfd34a26c11cd70dcc4817b1aef534f3768b1c..543854c42d9dc8c81ac06d8bd74e97e4f1a225a9 100644 (file)
@@ -70,7 +70,7 @@ MODULE_FIRMWARE(I915_ICL_HUC_UCODE);
 static void huc_fw_select(struct intel_uc_fw *huc_fw)
 {
        struct intel_huc *huc = container_of(huc_fw, struct intel_huc, fw);
-       struct drm_i915_private *dev_priv = huc_to_i915(huc);
+       struct drm_i915_private *dev_priv = huc_to_gt(huc)->i915;
 
        GEM_BUG_ON(huc_fw->type != INTEL_UC_FW_TYPE_HUC);
 
index 41a770865477b25a0c0d4b9a8f05531dd14f9607..5ebb0a534718fd8bea4a427ff7216e67b635550d 100644 (file)
@@ -252,7 +252,7 @@ static void guc_get_mmio_msg(struct intel_guc *guc)
 
 static void guc_handle_mmio_msg(struct intel_guc *guc)
 {
-       struct drm_i915_private *i915 = guc_to_i915(guc);
+       struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
 
        /* we need communication to be enabled to reply to GuC */
        GEM_BUG_ON(guc->handler == intel_guc_to_host_event_handler_nop);
@@ -284,7 +284,7 @@ static void guc_disable_interrupts(struct intel_guc *guc)
 
 static int guc_enable_communication(struct intel_guc *guc)
 {
-       struct drm_i915_private *i915 = guc_to_i915(guc);
+       struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
        int ret;
 
        ret = intel_guc_ct_enable(&guc->ct);
index c4e4d136c937f070931ad8e816e4d283fff00d57..d14e073155123d08d405ac3c43b7eb603c505170 100644 (file)
@@ -1902,16 +1902,6 @@ static inline struct drm_i915_private *wopcm_to_i915(struct intel_wopcm *wopcm)
        return container_of(wopcm, struct drm_i915_private, wopcm);
 }
 
-static inline struct drm_i915_private *guc_to_i915(struct intel_guc *guc)
-{
-       return container_of(guc, struct drm_i915_private, gt.uc.guc);
-}
-
-static inline struct drm_i915_private *huc_to_i915(struct intel_huc *huc)
-{
-       return container_of(huc, struct drm_i915_private, gt.uc.huc);
-}
-
 /* Simple iterator over all initialised engines */
 #define for_each_engine(engine__, dev_priv__, id__) \
        for ((id__) = 0; \