diff options
| author | Riana Tauro <riana.tauro@intel.com> | 2025-12-18 16:21:53 +0530 |
|---|---|---|
| committer | Rodrigo Vivi <rodrigo.vivi@intel.com> | 2025-12-18 17:47:37 -0500 |
| commit | fb084e4183e1d79ead103265cafc4b6bc0ecba03 (patch) | |
| tree | 9921d1be44f5ef5ad3b263bc5717399611b339e2 | |
| parent | 9dde74fd9e65ddd952413d98a8a756e12d087627 (diff) | |
drm/xe/xe_survivability: Remove unused index
Remove unused index variable and fix for loop.
Fixes: f4e9fc967afd ("drm/xe/xe_survivability: Redesign survivability mode")
Reported-by: Nathan Chancellor <nathan@kernel.org>
Closes: https://lore.kernel.org/intel-xe/20251210075757.GA1206705@ax162/
Signed-off-by: Riana Tauro <riana.tauro@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://patch.msgid.link/20251218105151.586575-5-riana.tauro@intel.com
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
| -rw-r--r-- | drivers/gpu/drm/xe/xe_survivability_mode.c | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/drivers/gpu/drm/xe/xe_survivability_mode.c b/drivers/gpu/drm/xe/xe_survivability_mode.c index 7520d7b7f9b8..2869866537ad 100644 --- a/drivers/gpu/drm/xe/xe_survivability_mode.c +++ b/drivers/gpu/drm/xe/xe_survivability_mode.c @@ -128,11 +128,6 @@ xe_survivability_attribute *dev_attr_to_survivability_attr(struct device_attribu return container_of(attr, struct xe_survivability_attribute, attr); } -static u32 aux_history_offset(u32 reg_value) -{ - return REG_FIELD_GET(AUXINFO_HISTORY_OFFSET, reg_value); -} - static void set_survivability_info(struct xe_mmio *mmio, u32 *info, int id) { info[id] = xe_mmio_read32(mmio, PCODE_SCRATCH(id)); @@ -144,7 +139,6 @@ static void populate_survivability_info(struct xe_device *xe) u32 *info = survivability->info; struct xe_mmio *mmio; u32 id = 0, reg_value; - int index; mmio = xe_root_tile_mmio(xe); set_survivability_info(mmio, info, CAPABILITY_INFO); @@ -162,13 +156,12 @@ static void populate_survivability_info(struct xe_device *xe) set_survivability_info(mmio, info, POSTCODE_TRACE_OVERFLOW); } + /* Traverse the linked list of aux info registers */ if (reg_value & AUXINFO_SUPPORT) { - id = REG_FIELD_GET(AUXINFO_REG_OFFSET, reg_value); - - for (index = 0; id >= AUX_INFO0 && id < MAX_SCRATCH_REG; index++) { + for (id = REG_FIELD_GET(AUXINFO_REG_OFFSET, reg_value); + id >= AUX_INFO0 && id < MAX_SCRATCH_REG; + id = REG_FIELD_GET(AUXINFO_HISTORY_OFFSET, info[id])) set_survivability_info(mmio, info, id); - id = aux_history_offset(info[id]); - } } } |