gpu: nvgpu: Fix nvgpu_sim_esc_readl MISRA 17.7 violations

MISRA Rule-17.7 requires the return value of all functions to be used.
Fix is either to use the return value or change the function to return
void. This patch contains fix for calls to nvgpu_sim_esc_readl by
changing its return to void and printing an error message instead.

JIRA NVGPU-677

Change-Id: I949bea253dafc316c7dfbf870eb1225b6fc5b9c3
Signed-off-by: Nicolas Benech <nbenech@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1812887
Reviewed-by: Alex Waterman <alexw@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

1 files changed, 7 insertions, 3 deletions

diff --git a/drivers/gpu/nvgpu/common/sim.c b/drivers/gpu/nvgpu/common/sim.c
index 8261f270..618e8ffb 100644
--- a/drivers/gpu/nvgpu/common/sim.c
+++ b/drivers/gpu/nvgpu/common/sim.c
@@ -28,6 +28,7 @@
 #include <nvgpu/hw_sim.h>
 #include <nvgpu/sim.h>
 #include <nvgpu/utils.h>
+#include <nvgpu/bug.h>
 
 #include "gk20a/gk20a.h"
 
@@ -215,7 +216,7 @@ static int issue_rpc_and_wait(struct gk20a *g)
         return 0;
 }
 
-static int nvgpu_sim_esc_readl(struct gk20a *g,
+static void nvgpu_sim_esc_readl(struct gk20a *g,
                 char *path, u32 index, u32 *data)
 {
         int err;
@@ -232,9 +233,12 @@ static int nvgpu_sim_esc_readl(struct gk20a *g,
 
         err = issue_rpc_and_wait(g);
 
-        if (!err)
+        if (err == 0) {
                 memcpy(data, sim_msg_param(g, data_offset), sizeof(u32));
-        return err;
+        } else {
+                *data = 0xffffffff;
+                WARN(1, "issue_rpc_and_wait failed err=%d", err);
+        }
 }
 
 static void nvgpu_sim_init_late(struct gk20a *g)