gpu: nvgpu: Make LTC disabling common code

Refactor the sync_debugfs LTC HAL op so that the logic to enable
or disable LTC goes to common code nvgpu_ltc_sync_enabled() and
the LTC HAL set_enabled only performs the hardware register access.

Create a new common function nvgpu_init_ltc_support() to initialize
the LTC software variable, and move hardware initialization of LTC to
be called from it.

JIRA NVGPU-62

Change-Id: Ib1cf4f5b83ca3dac08407464ed56a732e0a33923
Signed-off-by: Terje Bergstrom <tbergstrom@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1528262
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 files changed, 11 insertions, 23 deletions

diff --git a/drivers/gpu/nvgpu/gp10b/hal_gp10b.c b/drivers/gpu/nvgpu/gp10b/hal_gp10b.c
index e2479530..197c4fad 100644
--- a/drivers/gpu/nvgpu/gp10b/hal_gp10b.c
+++ b/drivers/gpu/nvgpu/gp10b/hal_gp10b.c
@@ -164,9 +164,7 @@ static const struct gpu_ops gp10b_ops = {
                 .isr = gp10b_ltc_isr,
                 .cbc_fix_config = gm20b_ltc_cbc_fix_config,
                 .flush = gm20b_flush_ltc,
-#ifdef CONFIG_DEBUG_FS
-                .sync_debugfs = gp10b_ltc_sync_debugfs,
-#endif
+                .set_enabled = gp10b_ltc_set_enabled,
         },
         .ce2 = {
                 .isr_stall = gp10b_ce_isr,
diff --git a/drivers/gpu/nvgpu/gp10b/ltc_gp10b.c b/drivers/gpu/nvgpu/gp10b/ltc_gp10b.c
index baa275c7..43619f80 100644
--- a/drivers/gpu/nvgpu/gp10b/ltc_gp10b.c
+++ b/drivers/gpu/nvgpu/gp10b/ltc_gp10b.c
@@ -205,25 +205,17 @@ void gp10b_ltc_init_fs_state(struct gk20a *g)
                         ltc_intr);
 }
 
-#ifdef CONFIG_DEBUG_FS
-void gp10b_ltc_sync_debugfs(struct gk20a *g)
+void gp10b_ltc_set_enabled(struct gk20a *g, bool enabled)
 {
         u32 reg_f = ltc_ltcs_ltss_tstg_set_mgmt_2_l2_bypass_mode_enabled_f();
+        u32 reg = gk20a_readl(g, ltc_ltcs_ltss_tstg_set_mgmt_2_r());
 
-        nvgpu_spinlock_acquire(&g->debugfs_lock);
-        if (g->mm.ltc_enabled != g->mm.ltc_enabled_debug) {
-                u32 reg = gk20a_readl(g, ltc_ltcs_ltss_tstg_set_mgmt_2_r());
-
-                if (g->mm.ltc_enabled_debug)
-                        /* bypass disabled (normal caching ops)*/
-                        reg &= ~reg_f;
-                else
-                        /* bypass enabled (no caching) */
-                        reg |= reg_f;
+        if (enabled)
+                /* bypass disabled (normal caching ops)*/
+                reg &= ~reg_f;
+        else
+                /* bypass enabled (no caching) */
+                reg |= reg_f;
 
-                gk20a_writel(g, ltc_ltcs_ltss_tstg_set_mgmt_2_r(), reg);
-                g->mm.ltc_enabled = g->mm.ltc_enabled_debug;
-        }
-        nvgpu_spinlock_release(&g->debugfs_lock);
+        gk20a_writel(g, ltc_ltcs_ltss_tstg_set_mgmt_2_r(), reg);
 }
-#endif
diff --git a/drivers/gpu/nvgpu/gp10b/ltc_gp10b.h b/drivers/gpu/nvgpu/gp10b/ltc_gp10b.h
index b5f2cda6..385754ba 100644
--- a/drivers/gpu/nvgpu/gp10b/ltc_gp10b.h
+++ b/drivers/gpu/nvgpu/gp10b/ltc_gp10b.h
@@ -20,7 +20,5 @@ void gp10b_ltc_isr(struct gk20a *g);
 int gp10b_determine_L2_size_bytes(struct gk20a *g);
 int gp10b_ltc_init_comptags(struct gk20a *g, struct gr_gk20a *gr);
 void gp10b_ltc_init_fs_state(struct gk20a *g);
-#ifdef CONFIG_DEBUG_FS
-void gp10b_ltc_sync_debugfs(struct gk20a *g);
-#endif
+void gp10b_ltc_set_enabled(struct gk20a *g, bool enabled);
 #endif