gpu: nvgpu: Reorg priv_ring HAL initialization

Reorganize HAL initialization to remove inheritance and construct
the gpu_ops struct at compile time. This patch only covers the
priv_ring sub-module of the gpu_ops struct.

Perform HAL function assignments in hal_gxxxx.c through the
population of a chip-specific copy of gpu_ops.

Jira NVGPU-74

Change-Id: I9ebf27619f771262e5dc398b1200d6c19d6aef16
Signed-off-by: Sunny He <suhe@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1514102
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Alex Waterman <alexw@nvidia.com>
Reviewed-by: Vijayakumar Subbu <vsubbu@nvidia.com>

3 files changed, 8 insertions, 9 deletions

diff --git a/drivers/gpu/nvgpu/gp10b/hal_gp10b.c b/drivers/gpu/nvgpu/gp10b/hal_gp10b.c
index fab1c238..4479be42 100644
--- a/drivers/gpu/nvgpu/gp10b/hal_gp10b.c
+++ b/drivers/gpu/nvgpu/gp10b/hal_gp10b.c
@@ -266,6 +266,9 @@ static const struct gpu_ops gp10b_ops = {
         .falcon = {
                 .falcon_hal_sw_init = gk20a_falcon_hal_sw_init,
         },
+        .priv_ring = {
+                .isr = gp10b_priv_ring_isr,
+        },
         .chip_init_gpu_characteristics = gp10b_init_gpu_characteristics,
         .get_litter_value = gp10b_get_litter_value,
 };
@@ -288,6 +291,8 @@ int gp10b_init_hal(struct gk20a *g)
 #endif
         gops->falcon = gp10b_ops.falcon;
 
+        gops->priv_ring = gp10b_ops.priv_ring;
+
         /* Lone Functions */
         gops->chip_init_gpu_characteristics =
                 gp10b_ops.chip_init_gpu_characteristics;
@@ -332,7 +337,6 @@ int gp10b_init_hal(struct gk20a *g)
 #endif
 
         g->bootstrap_owner = LSF_BOOTSTRAP_OWNER_DEFAULT;
-        gp10b_init_priv_ring(gops);
         gp10b_init_gr(gops);
         gp10b_init_fecs_trace_ops(gops);
         gp10b_init_fb(gops);
diff --git a/drivers/gpu/nvgpu/gp10b/priv_ring_gp10b.c b/drivers/gpu/nvgpu/gp10b/priv_ring_gp10b.c
index 8aaa7bff..0ca29809 100644
--- a/drivers/gpu/nvgpu/gp10b/priv_ring_gp10b.c
+++ b/drivers/gpu/nvgpu/gp10b/priv_ring_gp10b.c
@@ -27,7 +27,7 @@
 #include <nvgpu/hw/gp10b/hw_pri_ringstation_sys_gp10b.h>
 #include <nvgpu/hw/gp10b/hw_pri_ringstation_gpc_gp10b.h>
 
-static void gp10b_priv_ring_isr(struct gk20a *g)
+void gp10b_priv_ring_isr(struct gk20a *g)
 {
         u32 status0, status1;
         u32 cmd;
@@ -76,8 +76,3 @@ static void gp10b_priv_ring_isr(struct gk20a *g)
         if (retry <= 0)
                 nvgpu_warn(g, "priv ringmaster cmd ack too many retries");
 }
-
-void gp10b_init_priv_ring(struct gpu_ops *gops)
-{
-        gops->priv_ring.isr = gp10b_priv_ring_isr;
-}
diff --git a/drivers/gpu/nvgpu/gp10b/priv_ring_gp10b.h b/drivers/gpu/nvgpu/gp10b/priv_ring_gp10b.h
index acd0857f..e12a2f27 100644
--- a/drivers/gpu/nvgpu/gp10b/priv_ring_gp10b.h
+++ b/drivers/gpu/nvgpu/gp10b/priv_ring_gp10b.h
@@ -15,8 +15,8 @@
 #ifndef __PRIV_RING_GP10B_H__
 #define __PRIV_RING_GP10B_H__
 
-struct gpu_ops;
+struct gk20a;
 
-void gp10b_init_priv_ring(struct gpu_ops *gops);
+void gp10b_priv_ring_isr(struct gk20a *g);
 
 #endif /*__PRIV_RING_GP10B_H__*/