diff options
author | Srirangan <smadhavan@nvidia.com> | 2018-08-01 03:32:14 -0400 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2018-08-02 16:56:31 -0400 |
commit | 63e6e8ee3ee9ce01b1f7d4c0014bb589df105d71 (patch) | |
tree | f4db057c93973a9ad4afa15d1c6e5e1f2440c241 /drivers/gpu/nvgpu/common/mm/vm.c | |
parent | e6c135ecb70e8b966fc036587301c979ae70e869 (diff) |
gpu: nvgpu: common: Fix MISRA 15.6 violations
This fixes errors due to single statement loop bodies
without braces, which is part of Rule 15.6 of MISRA.
This patch covers in gpu/nvgpu/common/
JIRA NVGPU-989
Change-Id: Ic6a98a1cd04e4524dabf650e2f6e73c6b5a1db9d
Signed-off-by: Srirangan <smadhavan@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1786207
Reviewed-by: svc-misra-checker <svc-misra-checker@nvidia.com>
Reviewed-by: Adeel Raza <araza@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Alex Waterman <alexw@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Diffstat (limited to 'drivers/gpu/nvgpu/common/mm/vm.c')
-rw-r--r-- | drivers/gpu/nvgpu/common/mm/vm.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/drivers/gpu/nvgpu/common/mm/vm.c b/drivers/gpu/nvgpu/common/mm/vm.c index 3d9dd174..3b682e28 100644 --- a/drivers/gpu/nvgpu/common/mm/vm.c +++ b/drivers/gpu/nvgpu/common/mm/vm.c | |||
@@ -75,8 +75,9 @@ int nvgpu_vm_pde_coverage_bit_count(struct vm_gk20a *vm) | |||
75 | * heirarchy: the last level is PTEs so we really want the level | 75 | * heirarchy: the last level is PTEs so we really want the level |
76 | * before that which is the last level of PDEs. | 76 | * before that which is the last level of PDEs. |
77 | */ | 77 | */ |
78 | while (vm->mmu_levels[final_pde_level + 2].update_entry) | 78 | while (vm->mmu_levels[final_pde_level + 2].update_entry) { |
79 | final_pde_level++; | 79 | final_pde_level++; |
80 | } | ||
80 | 81 | ||
81 | return vm->mmu_levels[final_pde_level].lo_bit[0]; | 82 | return vm->mmu_levels[final_pde_level].lo_bit[0]; |
82 | } | 83 | } |
@@ -93,9 +94,10 @@ static void __nvgpu_vm_free_entries(struct vm_gk20a *vm, | |||
93 | } | 94 | } |
94 | 95 | ||
95 | if (pd->entries) { | 96 | if (pd->entries) { |
96 | for (i = 0; i < pd->num_entries; i++) | 97 | for (i = 0; i < pd->num_entries; i++) { |
97 | __nvgpu_vm_free_entries(vm, &pd->entries[i], | 98 | __nvgpu_vm_free_entries(vm, &pd->entries[i], |
98 | level + 1); | 99 | level + 1); |
100 | } | ||
99 | nvgpu_vfree(vm->mm->g, pd->entries); | 101 | nvgpu_vfree(vm->mm->g, pd->entries); |
100 | pd->entries = NULL; | 102 | pd->entries = NULL; |
101 | } | 103 | } |
@@ -112,8 +114,9 @@ static void nvgpu_vm_free_entries(struct vm_gk20a *vm, | |||
112 | if (!pdb->entries) | 114 | if (!pdb->entries) |
113 | return; | 115 | return; |
114 | 116 | ||
115 | for (i = 0; i < pdb->num_entries; i++) | 117 | for (i = 0; i < pdb->num_entries; i++) { |
116 | __nvgpu_vm_free_entries(vm, &pdb->entries[i], 1); | 118 | __nvgpu_vm_free_entries(vm, &pdb->entries[i], 1); |
119 | } | ||
117 | 120 | ||
118 | nvgpu_vfree(g, pdb->entries); | 121 | nvgpu_vfree(g, pdb->entries); |
119 | pdb->entries = NULL; | 122 | pdb->entries = NULL; |
@@ -750,8 +753,9 @@ void nvgpu_vm_put_buffers(struct vm_gk20a *vm, | |||
750 | nvgpu_vm_mapping_batch_start(&batch); | 753 | nvgpu_vm_mapping_batch_start(&batch); |
751 | vm->kref_put_batch = &batch; | 754 | vm->kref_put_batch = &batch; |
752 | 755 | ||
753 | for (i = 0; i < num_buffers; ++i) | 756 | for (i = 0; i < num_buffers; ++i) { |
754 | nvgpu_ref_put(&mapped_buffers[i]->ref, __nvgpu_vm_unmap_ref); | 757 | nvgpu_ref_put(&mapped_buffers[i]->ref, __nvgpu_vm_unmap_ref); |
758 | } | ||
755 | 759 | ||
756 | vm->kref_put_batch = NULL; | 760 | vm->kref_put_batch = NULL; |
757 | nvgpu_vm_mapping_batch_finish_locked(vm, &batch); | 761 | nvgpu_vm_mapping_batch_finish_locked(vm, &batch); |