From d29afd2c9e990799b470bb95a97935cf5b5020db Mon Sep 17 00:00:00 2001 From: Terje Bergstrom Date: Wed, 9 Nov 2016 15:53:16 -0800 Subject: gpu: nvgpu: Fix signed comparison bugs Fix small problems related to signed versus unsigned comparisons throughout the driver. Bump up the warning level to prevent such problems from occuring in future. Change-Id: I8ff5efb419f664e8a2aedadd6515ae4d18502ae0 Signed-off-by: Terje Bergstrom Reviewed-on: http://git-master/r/1252068 Reviewed-by: mobile promotions Tested-by: mobile promotions --- drivers/gpu/nvgpu/gk20a/fifo_gk20a.h | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) (limited to 'drivers/gpu/nvgpu/gk20a/fifo_gk20a.h') diff --git a/drivers/gpu/nvgpu/gk20a/fifo_gk20a.h b/drivers/gpu/nvgpu/gk20a/fifo_gk20a.h index 64bdeabb..c32142e3 100644 --- a/drivers/gpu/nvgpu/gk20a/fifo_gk20a.h +++ b/drivers/gpu/nvgpu/gk20a/fifo_gk20a.h @@ -26,7 +26,9 @@ #define MAX_RUNLIST_BUFFERS 2 -#define FIFO_INVAL_ENGINE_ID ~0 +#define FIFO_INVAL_ENGINE_ID ((u32)~0) +#define FIFO_INVAL_CHANNEL_ID ((u32)~0) +#define FIFO_INVAL_TSG_ID ((u32)~0) /* generally corresponds to the "pbdma" engine */ @@ -96,11 +98,11 @@ struct fifo_engine_info_gk20a { struct fifo_gk20a { struct gk20a *g; - int num_channels; - int runlist_entry_size; - int num_runlist_entries; + unsigned int num_channels; + unsigned int runlist_entry_size; + unsigned int num_runlist_entries; - int num_pbdma; + unsigned int num_pbdma; u32 *pbdma_map; struct fifo_engine_info_gk20a *engine_info; @@ -114,7 +116,7 @@ struct fifo_gk20a { struct mem_desc userd; u32 userd_entry_size; - int used_channels; + unsigned int used_channels; struct channel_gk20a *channel; /* zero-kref'd channels here */ struct list_head free_chs; -- cgit v1.2.2