Revert "gpu: nvgpu: Use gv11b_css_hw_set_handled_snapshots for GV11B"

This reverts commit 2f2e51bbae39009d0305f6aaf01596571a8f5d5c.

Bug 2068936

Change-Id: I539cdc12a3bd0d9d7fe0ce7dbe9cb7a274eeaa57
Signed-off-by: Srikar Srimath Tirumala <srikars@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1664647
Reviewed-by: Alex Waterman <alexw@nvidia.com>
Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com>
GVS: Gerrit_Virtual_Submit

3 files changed, 10 insertions, 13 deletions

diff --git a/drivers/gpu/nvgpu/gv11b/css_gr_gv11b.c b/drivers/gpu/nvgpu/gv11b/css_gr_gv11b.c
index d8ba332b..617ea61d 100644
--- a/drivers/gpu/nvgpu/gv11b/css_gr_gv11b.c
+++ b/drivers/gpu/nvgpu/gv11b/css_gr_gv11b.c
@@ -57,6 +57,15 @@ static inline u32 css_hw_get_pending_snapshots(struct gk20a *g)
                         sizeof(struct gk20a_cs_snapshot_fifo_entry);
 }
 
+/* informs hw how many snapshots have been processed (frees up fifo space) */
+static inline void gv11b_css_hw_set_handled_snapshots(struct gk20a *g, u32 done)
+{
+        if (done > 0) {
+                gk20a_writel(g, perf_pmasys_mem_bump_r(),
+                     done * sizeof(struct gk20a_cs_snapshot_fifo_entry));
+        }
+}
+
 /* disable streaming to memory */
 static void gv11b_css_hw_reset_streaming(struct gk20a *g)
 {
@@ -77,17 +86,6 @@ static void gv11b_css_hw_reset_streaming(struct gk20a *g)
         gv11b_css_hw_set_handled_snapshots(g, css_hw_get_pending_snapshots(g));
 }
 
-/* informs hw how many snapshots have been processed (frees up fifo space) */
-void gv11b_css_hw_set_handled_snapshots(struct gk20a *g, u32 done)
-{
-        if (done == 0u) {
-                return;
-        }
-
-        gk20a_writel(g, perf_pmasys_mem_bump_r(),
-                done * sizeof(struct gk20a_cs_snapshot_fifo_entry));
-}
-
 int gv11b_css_hw_enable_snapshot(struct channel_gk20a *ch,
                                 struct gk20a_cs_snapshot_client *cs_client)
 {
diff --git a/drivers/gpu/nvgpu/gv11b/css_gr_gv11b.h b/drivers/gpu/nvgpu/gv11b/css_gr_gv11b.h
index 186f5fcc..6b11a62e 100644
--- a/drivers/gpu/nvgpu/gv11b/css_gr_gv11b.h
+++ b/drivers/gpu/nvgpu/gv11b/css_gr_gv11b.h
@@ -30,6 +30,5 @@ int gv11b_css_hw_enable_snapshot(struct channel_gk20a *ch,
 void gv11b_css_hw_disable_snapshot(struct gr_gk20a *gr);
 int gv11b_css_hw_check_data_available(struct channel_gk20a *ch, u32 *pending,
                         bool *hw_overflow);
-void gv11b_css_hw_set_handled_snapshots(struct gk20a *g, u32 done);
 
 #endif /* CSS_GR_GV11B_H */
diff --git a/drivers/gpu/nvgpu/gv11b/hal_gv11b.c b/drivers/gpu/nvgpu/gv11b/hal_gv11b.c
index 8d78d524..5282af05 100644
--- a/drivers/gpu/nvgpu/gv11b/hal_gv11b.c
+++ b/drivers/gpu/nvgpu/gv11b/hal_gv11b.c
@@ -688,7 +688,7 @@ static const struct gpu_ops gv11b_ops = {
                 .enable_snapshot = gv11b_css_hw_enable_snapshot,
                 .disable_snapshot = gv11b_css_hw_disable_snapshot,
                 .check_data_available = gv11b_css_hw_check_data_available,
-                .set_handled_snapshots = gv11b_css_hw_set_handled_snapshots,
+                .set_handled_snapshots = css_hw_set_handled_snapshots,
                 .allocate_perfmon_ids = css_gr_allocate_perfmon_ids,
                 .release_perfmon_ids = css_gr_release_perfmon_ids,
         },