From bcfce1af622e0827677d70e23978c8cb30f70976 Mon Sep 17 00:00:00 2001 From: Debarshi Dutta Date: Fri, 30 Nov 2018 15:01:03 +0530 Subject: [PATCH] gpu: nvgpu: fixed misra-c 16.6 violation The switch statement "switch (interleave_level)" has no conforming switch clauses as none of the clauses end with unconditional break statement. The above switch statement is now fixed in accordance to misra-c standards. Jira NVGPU-1555 Change-Id: Id2ea98826b5fff51f42eed83a597d8e0e273ebde Signed-off-by: Debarshi Dutta Reviewed-on: https://git-master.nvidia.com/r/1962545 Reviewed-by: mobile promotions Tested-by: mobile promotions --- drivers/gpu/nvgpu/gk20a/fifo_gk20a.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/nvgpu/gk20a/fifo_gk20a.c b/drivers/gpu/nvgpu/gk20a/fifo_gk20a.c index 8943f907b..17c57545c 100644 --- a/drivers/gpu/nvgpu/gk20a/fifo_gk20a.c +++ b/drivers/gpu/nvgpu/gk20a/fifo_gk20a.c @@ -4499,19 +4499,27 @@ u32 gk20a_fifo_pbdma_acquire_val(u64 timeout) const char *gk20a_fifo_interleave_level_name(u32 interleave_level) { + const char *ret_string = NULL; + switch (interleave_level) { case NVGPU_FIFO_RUNLIST_INTERLEAVE_LEVEL_LOW: - return "LOW"; + ret_string = "LOW"; + break; case NVGPU_FIFO_RUNLIST_INTERLEAVE_LEVEL_MEDIUM: - return "MEDIUM"; + ret_string = "MEDIUM"; + break; case NVGPU_FIFO_RUNLIST_INTERLEAVE_LEVEL_HIGH: - return "HIGH"; + ret_string = "HIGH"; + break; default: - return "?"; + ret_string = "?"; + break; } + + return ret_string; } u32 gk20a_fifo_get_sema_wait_cmd_size(void)