Skip to content

Commit 424e2cc

Browse files
elongbugmwiniars
authored andcommitted
drm/xe: Remove never used code in xe_vm_create()
Clang is not happy with set but unused variable (this is visible with `make LLVM=1` build: drivers/gpu/drm/xe/xe_vm.c:1462:11: error: variable 'number_tiles' set but not used [-Werror,-Wunused-but-set-variable] The use of this variable was removed in the commit mentioned below as "Fixes:" but only its declaration and update remain. It seems like the variable is not used along with the assignment that does not have side effects as far as I can see. Remove those altogether. Fixes: cb99e12 ("drm/xe: Decouple bind queue last fence from TLB invalidations") Signed-off-by: Gwan-gyeong Mun <gwan-gyeong.mun@intel.com> Reviewed-by: Michał Winiarski <michal.winiarski@intel.com> Link: https://patch.msgid.link/20251105011311.3177875-1-gwan-gyeong.mun@intel.com Signed-off-by: Michał Winiarski <michal.winiarski@intel.com>
1 parent 816e127 commit 424e2cc

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

drivers/gpu/drm/xe/xe_vm.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1459,7 +1459,7 @@ struct xe_vm *xe_vm_create(struct xe_device *xe, u32 flags, struct xe_file *xef)
14591459
struct xe_validation_ctx ctx;
14601460
struct drm_exec exec;
14611461
struct xe_vm *vm;
1462-
int err, number_tiles = 0;
1462+
int err;
14631463
struct xe_tile *tile;
14641464
u8 id;
14651465

@@ -1620,7 +1620,6 @@ struct xe_vm *xe_vm_create(struct xe_device *xe, u32 flags, struct xe_file *xef)
16201620
goto err_close;
16211621
}
16221622
vm->q[id] = q;
1623-
number_tiles++;
16241623
}
16251624
}
16261625

0 commit comments

Comments
 (0)