drm/i915/guc: Don't pass CTB while writing
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Fri, 17 Jan 2020 08:20:36 +0000 (08:20 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Fri, 17 Jan 2020 14:02:05 +0000 (14:02 +0000)
Since we only have one SEND buffer we don't need to explicitly pass
it to the write function.

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20200117082039.65644-3-michal.wajdeczko@intel.com
drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c

index 46e4faf..a373ca1 100644 (file)
@@ -311,12 +311,13 @@ static u32 ct_get_next_fence(struct intel_guc_ct *ct)
  *                   ^-----------------len-------------------^
  */
 
-static int ctb_write(struct intel_guc_ct_buffer *ctb,
-                    const u32 *action,
-                    u32 len /* in dwords */,
-                    u32 fence,
-                    bool want_response)
+static int ct_write(struct intel_guc_ct *ct,
+                   const u32 *action,
+                   u32 len /* in dwords */,
+                   u32 fence,
+                   bool want_response)
 {
+       struct intel_guc_ct_buffer *ctb = &ct->ctbs[CTB_SEND];
        struct guc_ct_buffer_desc *desc = ctb->desc;
        u32 head = desc->head / 4;      /* in dwords */
        u32 tail = desc->tail / 4;      /* in dwords */
@@ -492,7 +493,7 @@ static int ct_send(struct intel_guc_ct *ct,
        list_add_tail(&request.link, &ct->requests.pending);
        spin_unlock_irqrestore(&ct->requests.lock, flags);
 
-       err = ctb_write(ctb, action, len, fence, !!response_buf);
+       err = ct_write(ct, action, len, fence, !!response_buf);
        if (unlikely(err))
                goto unlink;