summaryrefslogtreecommitdiffstats
path: root/drivers/gpu/nvgpu/gk20a/ce2_gk20a.c
diff options
context:
space:
mode:
authorScott Long <scottl@nvidia.com>2018-08-08 21:30:09 -0400
committermobile promotions <svcmobile_promotions@nvidia.com>2018-08-22 20:31:42 -0400
commit07f6739285140d7c5335ddcb8996450966bfc175 (patch)
treec7f1b3a73d519e44e4fe0d29516e6516740edb26 /drivers/gpu/nvgpu/gk20a/ce2_gk20a.c
parent1c13da1d29c344cb60953eabeca56b601446c64a (diff)
gpu: nvgpu: switch gk20a nonstall ops to #defines
Fix MISRA rule 10.1 violations involving gk20a_nonstall_ops enums by replacing them with with corresponding #defines. Because these values can be used in expressions that require unsigned values (e.g. bitwise OR) we cannot use enums. The g->ce2.isr_nonstall() function was previously returning an int that was a combination of gk20a_nonstall_ops enum bits which led to the violations. JIRA NVGPU-650 Change-Id: I6210aacec8829b3c8d339c5fe3db2f3069c67406 Signed-off-by: Scott Long <scottl@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/1796242 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.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/gpu/nvgpu/gk20a/ce2_gk20a.c b/drivers/gpu/nvgpu/gk20a/ce2_gk20a.c
index 5aae0613..0248b0ca 100644
--- a/drivers/gpu/nvgpu/gk20a/ce2_gk20a.c
+++ b/drivers/gpu/nvgpu/gk20a/ce2_gk20a.c
@@ -88,9 +88,9 @@ void gk20a_ce2_isr(struct gk20a *g, u32 inst_id, u32 pri_base)
88 return; 88 return;
89} 89}
90 90
91int gk20a_ce2_nonstall_isr(struct gk20a *g, u32 inst_id, u32 pri_base) 91u32 gk20a_ce2_nonstall_isr(struct gk20a *g, u32 inst_id, u32 pri_base)
92{ 92{
93 int ops = 0; 93 u32 ops = 0;
94 u32 ce2_intr = gk20a_readl(g, ce2_intr_status_r()); 94 u32 ce2_intr = gk20a_readl(g, ce2_intr_status_r());
95 95
96 nvgpu_log(g, gpu_dbg_intr, "ce2 nonstall isr %08x\n", ce2_intr); 96 nvgpu_log(g, gpu_dbg_intr, "ce2 nonstall isr %08x\n", ce2_intr);
@@ -98,8 +98,8 @@ int gk20a_ce2_nonstall_isr(struct gk20a *g, u32 inst_id, u32 pri_base)
98 if (ce2_intr & ce2_intr_status_nonblockpipe_pending_f()) { 98 if (ce2_intr & ce2_intr_status_nonblockpipe_pending_f()) {
99 gk20a_writel(g, ce2_intr_status_r(), 99 gk20a_writel(g, ce2_intr_status_r(),
100 ce2_nonblockpipe_isr(g, ce2_intr)); 100 ce2_nonblockpipe_isr(g, ce2_intr));
101 ops |= (gk20a_nonstall_ops_wakeup_semaphore | 101 ops |= (GK20A_NONSTALL_OPS_WAKEUP_SEMAPHORE |
102 gk20a_nonstall_ops_post_events); 102 GK20A_NONSTALL_OPS_POST_EVENTS);
103 } 103 }
104 return ops; 104 return ops;
105} 105}