drm/xe/lrc: Remove leftover TODO/FIXME

There isn't anything to set for CTX_TIMESTAMP handling in the empty
LRC: that is set on every LRC init since it should always start from 0
rather than the value saved in the image after first submission.

The FIXME about perma-pinning also doesn't make much sense as we will
always going to pin the lrc and the GGTT mapping has nothing to do with
VM bind.

Nuke these leftover comments.

Reviewed-by: Matthew Brost <matthew.brost@intel.com>
Link: https://lore.kernel.org/r/20250710-lrc-refactors-v2-5-a5e2ca03f6bd@intel.com
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>

diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
index 682c5b69fee8c6a3968a6560ec6a3ea2f6626da9..8bb2ca3af7d12bb6a89448b89065f8d979bdfa7b 100644 (file)
--- a/drivers/gpu/drm/xe/xe_lrc.c
+++ b/drivers/gpu/drm/xe/xe_lrc.c
@@ -589,8 +589,6 @@ static void set_context_control(u32 *regs, struct xe_hw_engine *hwe)
        if (xe_gt_has_indirect_ring_state(hwe->gt))
                regs[CTX_CONTEXT_CONTROL] |=
                        _MASKED_BIT_ENABLE(CTX_CTRL_INDIRECT_RING_STATE_ENABLE);
-
-       /* TODO: Timestamp */
 }
 
 static void set_memory_based_intr(u32 *regs, struct xe_hw_engine *hwe)
@@ -1181,10 +1179,6 @@ static int xe_lrc_init(struct xe_lrc *lrc, struct xe_hw_engine *hwe,
        if (vm && vm->xef) /* userspace */
                bo_flags |= XE_BO_FLAG_PINNED_LATE_RESTORE;
 
-       /*
-        * FIXME: Perma-pinning LRC as we don't yet support moving GGTT address
-        * via VM bind calls.
-        */
        lrc->bo = xe_bo_create_pin_map(xe, tile, NULL, bo_size,
                                       ttm_bo_type_kernel,
                                       bo_flags);