gpu: nvgpu: remove duplicate \n from log messages

nvgpu_log/info/warn/err() internally add a \n to the end of the message.
Hence, callers should not include a \n at the end of the message. Doing
so results in duplicate \n being printed, which ends up creating empty
log messages. Remove the duplicate \n from all err/warn messages.

Bug 1928311

Change-Id: I99362c5327f36146f28ba63d4e68181589735c39
Signed-off-by: Stephen Warren <swarren@nvidia.com>
Reviewed-on: http://git-master/r/1487232
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

2 files changed, 4 insertions, 4 deletions

diff --git a/drivers/gpu/nvgpu/gp10b/gp10b_sysfs.c b/drivers/gpu/nvgpu/gp10b/gp10b_sysfs.c
index 1bd95b40..5a242bb5 100644
--- a/drivers/gpu/nvgpu/gp10b/gp10b_sysfs.c
+++ b/drivers/gpu/nvgpu/gp10b/gp10b_sysfs.c
@@ -35,7 +35,7 @@ static ssize_t ecc_enable_store(struct device *dev,
                 err = g->ops.pmu.send_lrf_tex_ltc_dram_overide_en_dis_cmd
                         (g, ecc_mask);
                 if (err)
-                        nvgpu_err(g, "ECC override did not happen\n");
+                        nvgpu_err(g, "ECC override did not happen");
         } else
                 return -EINVAL;
         return count;
@@ -90,7 +90,7 @@ void gp10b_create_sysfs(struct device *dev)
         error |= device_create_file(dev, &dev_attr_ecc_enable);
         error |= device_create_file(dev, &dev_attr_czf_bypass);
         if (error)
-                nvgpu_err(g, "Failed to create sysfs attributes!\n");
+                nvgpu_err(g, "Failed to create sysfs attributes!");
 }
 
 void gp10b_remove_sysfs(struct device *dev)
diff --git a/drivers/gpu/nvgpu/gp10b/gr_gp10b.c b/drivers/gpu/nvgpu/gp10b/gr_gp10b.c
index 2c85a667..98a8be2f 100644
--- a/drivers/gpu/nvgpu/gp10b/gr_gp10b.c
+++ b/drivers/gpu/nvgpu/gp10b/gr_gp10b.c
@@ -1631,7 +1631,7 @@ static int gr_gp10b_disable_channel_or_tsg(struct gk20a *g, struct channel_gk20a
         ret = gk20a_disable_channel_tsg(g, fault_ch);
         if (ret) {
                 nvgpu_err(g,
-                                "CILP: failed to disable channel/TSG!\n");
+                                "CILP: failed to disable channel/TSG!");
                 return ret;
         }
 
@@ -1833,7 +1833,7 @@ static int gr_gp10b_pre_process_sm_exception(struct gk20a *g,
                         gk20a_dbg(gpu_dbg_fn | gpu_dbg_gpu_dbg, "CILP: Setting CILP preempt pending\n");
                         ret = gr_gp10b_set_cilp_preempt_pending(g, fault_ch);
                         if (ret) {
-                                nvgpu_err(g, "CILP: error while setting CILP preempt pending!\n");
+                                nvgpu_err(g, "CILP: error while setting CILP preempt pending!");
                                 return ret;
                         }