diff options
author | Terje Bergstrom <tbergstrom@nvidia.com> | 2016-11-09 18:53:16 -0500 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2016-11-17 00:35:36 -0500 |
commit | d29afd2c9e990799b470bb95a97935cf5b5020db (patch) | |
tree | 48f14dffe90956e9b42b02a87b95fd7a1dfeef3e /drivers/gpu/nvgpu/gk20a/ce2_gk20a.c | |
parent | 5494e846c78dd0da74635905ead3abe45502375f (diff) |
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 <tbergstrom@nvidia.com>
Reviewed-on: http://git-master/r/1252068
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Diffstat (limited to 'drivers/gpu/nvgpu/gk20a/ce2_gk20a.c')
-rw-r--r-- | drivers/gpu/nvgpu/gk20a/ce2_gk20a.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/nvgpu/gk20a/ce2_gk20a.c b/drivers/gpu/nvgpu/gk20a/ce2_gk20a.c index bfd183fb..235bc027 100644 --- a/drivers/gpu/nvgpu/gk20a/ce2_gk20a.c +++ b/drivers/gpu/nvgpu/gk20a/ce2_gk20a.c | |||
@@ -211,10 +211,10 @@ static void gk20a_ce_delete_gpu_context(struct gk20a_gpu_ctx *ce_ctx) | |||
211 | kfree(ce_ctx); | 211 | kfree(ce_ctx); |
212 | } | 212 | } |
213 | 213 | ||
214 | static inline int gk20a_ce_get_method_size(int request_operation) | 214 | static inline unsigned int gk20a_ce_get_method_size(int request_operation) |
215 | { | 215 | { |
216 | /* failure size */ | 216 | /* failure size */ |
217 | int methodsize = ~0; | 217 | unsigned int methodsize = UINT_MAX; |
218 | 218 | ||
219 | if (request_operation & NVGPU_CE_PHYS_MODE_TRANSFER) | 219 | if (request_operation & NVGPU_CE_PHYS_MODE_TRANSFER) |
220 | methodsize = 10 * 2 * sizeof(u32); | 220 | methodsize = 10 * 2 * sizeof(u32); |
@@ -518,7 +518,7 @@ u32 gk20a_ce_create_context_with_cb(struct device *dev, | |||
518 | ce_ctx->gpu_ctx_state = NVGPU_CE_GPU_CTX_ALLOCATED; | 518 | ce_ctx->gpu_ctx_state = NVGPU_CE_GPU_CTX_ALLOCATED; |
519 | 519 | ||
520 | end: | 520 | end: |
521 | if (ctx_id == ~0) { | 521 | if (ctx_id == (u32)~0) { |
522 | mutex_lock(&ce_app->app_mutex); | 522 | mutex_lock(&ce_app->app_mutex); |
523 | gk20a_ce_delete_gpu_context(ce_ctx); | 523 | gk20a_ce_delete_gpu_context(ce_ctx); |
524 | mutex_unlock(&ce_app->app_mutex); | 524 | mutex_unlock(&ce_app->app_mutex); |