diff options
author | Sai Nikhil <snikhil@nvidia.com> | 2018-08-22 01:12:37 -0400 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2018-08-22 20:30:19 -0400 |
commit | d28a401e6d872f7ea6abb0c5cfc8f63e0235fe21 (patch) | |
tree | 66b5c4ad42135dbd9f5535fa3c86f8ecdd1a067c /drivers/gpu/nvgpu/common/pmu/pmu.c | |
parent | 650171566bff59e9eb372f213fdce4dfbb6da5bd (diff) |
gpu: nvgpu: common: fix MISRA 10.4 violations
MISRA Rule 10.4 only allows the usage of arithmetic operations on
operands of the same essential type category.
Adding "U" at the end of the integer literals to have same type of
operands when an arithmetic operation is performed.
This fix violations where an arithmetic operation is performed on
signed and unsigned int types.
Jira NVGPU-992
Change-Id: Iab512139a025e035ec82a9dd74245bcf1f3869fb
Signed-off-by: Sai Nikhil <snikhil@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1789425
Reviewed-by: svc-misra-checker <svc-misra-checker@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Adeel Raza <araza@nvidia.com>
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/pmu/pmu.c')
-rw-r--r-- | drivers/gpu/nvgpu/common/pmu/pmu.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/nvgpu/common/pmu/pmu.c b/drivers/gpu/nvgpu/common/pmu/pmu.c index d72629b5..86e56d9e 100644 --- a/drivers/gpu/nvgpu/common/pmu/pmu.c +++ b/drivers/gpu/nvgpu/common/pmu/pmu.c | |||
@@ -512,7 +512,7 @@ int nvgpu_pmu_destroy(struct gk20a *g) | |||
512 | { | 512 | { |
513 | struct nvgpu_pmu *pmu = &g->pmu; | 513 | struct nvgpu_pmu *pmu = &g->pmu; |
514 | struct pmu_pg_stats_data pg_stat_data = { 0 }; | 514 | struct pmu_pg_stats_data pg_stat_data = { 0 }; |
515 | int i; | 515 | u32 i; |
516 | 516 | ||
517 | nvgpu_log_fn(g, " "); | 517 | nvgpu_log_fn(g, " "); |
518 | 518 | ||
@@ -539,7 +539,7 @@ int nvgpu_pmu_destroy(struct gk20a *g) | |||
539 | pmu->isr_enabled = false; | 539 | pmu->isr_enabled = false; |
540 | nvgpu_mutex_release(&pmu->isr_mutex); | 540 | nvgpu_mutex_release(&pmu->isr_mutex); |
541 | 541 | ||
542 | for (i = 0; i < PMU_QUEUE_COUNT; i++) { | 542 | for (i = 0U; i < PMU_QUEUE_COUNT; i++) { |
543 | nvgpu_flcn_queue_free(pmu->flcn, &pmu->queue[i]); | 543 | nvgpu_flcn_queue_free(pmu->flcn, &pmu->queue[i]); |
544 | } | 544 | } |
545 | 545 | ||
@@ -559,7 +559,7 @@ void nvgpu_pmu_surface_describe(struct gk20a *g, struct nvgpu_mem *mem, | |||
559 | { | 559 | { |
560 | fb->address.lo = u64_lo32(mem->gpu_va); | 560 | fb->address.lo = u64_lo32(mem->gpu_va); |
561 | fb->address.hi = u64_hi32(mem->gpu_va); | 561 | fb->address.hi = u64_hi32(mem->gpu_va); |
562 | fb->params = ((u32)mem->size & 0xFFFFFF); | 562 | fb->params = ((u32)mem->size & 0xFFFFFFU); |
563 | fb->params |= (GK20A_PMU_DMAIDX_VIRT << 24); | 563 | fb->params |= (GK20A_PMU_DMAIDX_VIRT << 24); |
564 | } | 564 | } |
565 | 565 | ||