diff options
author | Deepak Nibade <dnibade@nvidia.com> | 2015-11-19 03:56:15 -0500 |
---|---|---|
committer | Sachin Nikam <snikam@nvidia.com> | 2015-11-25 03:45:58 -0500 |
commit | 10f6da09eb6fffea2d50c8fe4634e0d5f8226d7d (patch) | |
tree | 4141867911601aed374dba50b031cb5aa5d9e9d3 /drivers/gpu/nvgpu/gk20a/ltc_gk20a.c | |
parent | 836027806dab4a59c0faadca4f530371ef5869d0 (diff) |
gpu: nvgpu: fix Coverity issues
- operands not affecting result (id = 12845)
- logically dead code (id = 12890)
- dereference after null check (id = 12968)
- unsigned compared to 0 (id = 13176)
- resource leak (id = 13338, 18673)
- unused pointer value (id = 13916)
Bug 1703084
Change-Id: I2f401dd93126af27748c53fa1b3a59cb154af36b
Signed-off-by: Deepak Nibade <dnibade@nvidia.com>
Reviewed-on: http://git-master/r/835143
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>
Reviewed-by: Seshendra Gadagottu <sgadagottu@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Sachin Nikam <snikam@nvidia.com>
Diffstat (limited to 'drivers/gpu/nvgpu/gk20a/ltc_gk20a.c')
-rw-r--r-- | drivers/gpu/nvgpu/gk20a/ltc_gk20a.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/nvgpu/gk20a/ltc_gk20a.c b/drivers/gpu/nvgpu/gk20a/ltc_gk20a.c index 7cb386f0..c7f9a55d 100644 --- a/drivers/gpu/nvgpu/gk20a/ltc_gk20a.c +++ b/drivers/gpu/nvgpu/gk20a/ltc_gk20a.c | |||
@@ -105,7 +105,7 @@ static int gk20a_ltc_cbc_ctrl(struct gk20a *g, enum gk20a_cbc_op op, | |||
105 | int err = 0; | 105 | int err = 0; |
106 | struct gr_gk20a *gr = &g->gr; | 106 | struct gr_gk20a *gr = &g->gr; |
107 | u32 fbp, slice, ctrl1, val, hw_op = 0; | 107 | u32 fbp, slice, ctrl1, val, hw_op = 0; |
108 | u32 retry = 200; | 108 | int retry = 200; |
109 | u32 slices_per_fbp = | 109 | u32 slices_per_fbp = |
110 | ltc_ltcs_ltss_cbc_param_slices_per_fbp_v( | 110 | ltc_ltcs_ltss_cbc_param_slices_per_fbp_v( |
111 | gk20a_readl(g, ltc_ltcs_ltss_cbc_param_r())); | 111 | gk20a_readl(g, ltc_ltcs_ltss_cbc_param_r())); |