gpu: nvgpu: Reorg fb HAL initialization

Reorganize HAL initialization to remove inheritance and construct
the gpu_ops struct at compile time. This patch only covers the
fb 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: I593d00ffccc40a3721e792ef7ca259583852eb28
Signed-off-by: Sunny He <suhe@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1537746
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: svccoveritychecker <svccoveritychecker@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>

3 files changed, 32 insertions, 18 deletions

diff --git a/drivers/gpu/nvgpu/gp10b/fb_gp10b.c b/drivers/gpu/nvgpu/gp10b/fb_gp10b.c
index 8ba9ff8e..775dc5c9 100644
--- a/drivers/gpu/nvgpu/gp10b/fb_gp10b.c
+++ b/drivers/gpu/nvgpu/gp10b/fb_gp10b.c
@@ -20,7 +20,7 @@
 
 #include <nvgpu/hw/gp10b/hw_gmmu_gp10b.h>
 
-static noinline_for_stack void gp10b_init_uncompressed_kind_map(void)
+noinline_for_stack void gp10b_init_uncompressed_kind_map(void)
 {
         int i;
 
@@ -438,7 +438,7 @@ static noinline_for_stack bool gp10b_kind_zbc(u8 k)
                         k <= gmmu_pte_kind_c128_ms8_ms16_2cr_v());
 }
 
-static void gp10b_init_kind_attr(void)
+void gp10b_init_kind_attr(void)
 {
         u16 k;
 
@@ -456,22 +456,12 @@ static void gp10b_init_kind_attr(void)
         }
 }
 
-static unsigned int gp10b_fb_compression_page_size(struct gk20a *g)
+unsigned int gp10b_fb_compression_page_size(struct gk20a *g)
 {
         return SZ_64K;
 }
 
-static unsigned int gp10b_fb_compressible_page_size(struct gk20a *g)
+unsigned int gp10b_fb_compressible_page_size(struct gk20a *g)
 {
         return SZ_4K;
 }
-
-void gp10b_init_fb(struct gpu_ops *gops)
-{
-        gm20b_init_fb(gops);
-        gops->fb.compression_page_size = gp10b_fb_compression_page_size;
-        gops->fb.compressible_page_size = gp10b_fb_compressible_page_size;
-
-        gp10b_init_uncompressed_kind_map();
-        gp10b_init_kind_attr();
-}
diff --git a/drivers/gpu/nvgpu/gp10b/fb_gp10b.h b/drivers/gpu/nvgpu/gp10b/fb_gp10b.h
index 76efd331..626cf54d 100644
--- a/drivers/gpu/nvgpu/gp10b/fb_gp10b.h
+++ b/drivers/gpu/nvgpu/gp10b/fb_gp10b.h
@@ -1,7 +1,7 @@
 /*
  * GP10B FB
  *
- * Copyright (c) 2014, NVIDIA CORPORATION.  All rights reserved.
+ * Copyright (c) 2014-2017, NVIDIA CORPORATION.  All rights reserved.
  *
  * This program is free software; you can redistribute it and/or modify it
  * under the terms and conditions of the GNU General Public License,
@@ -15,7 +15,11 @@
 
 #ifndef _NVGPU_GP10B_FB
 #define _NVGPU_GP10B_FB
-struct gpu_ops;
+struct gk20a;
+
+noinline_for_stack void gp10b_init_uncompressed_kind_map(void);
+void gp10b_init_kind_attr(void);
+unsigned int gp10b_fb_compression_page_size(struct gk20a *g);
+unsigned int gp10b_fb_compressible_page_size(struct gk20a *g);
 
-void gp10b_init_fb(struct gpu_ops *gops);
 #endif
diff --git a/drivers/gpu/nvgpu/gp10b/hal_gp10b.c b/drivers/gpu/nvgpu/gp10b/hal_gp10b.c
index 197c4fad..b0871155 100644
--- a/drivers/gpu/nvgpu/gp10b/hal_gp10b.c
+++ b/drivers/gpu/nvgpu/gp10b/hal_gp10b.c
@@ -24,6 +24,7 @@
 #include "gk20a/flcn_gk20a.h"
 #include "gk20a/regops_gk20a.h"
 #include "gk20a/mc_gk20a.h"
+#include "gk20a/fb_gk20a.h"
 
 #include "gp10b/gr_gp10b.h"
 #include "gp10b/fecs_trace_gp10b.h"
@@ -47,6 +48,7 @@
 #include "gm20b/pmu_gm20b.h"
 #include "gm20b/clk_gm20b.h"
 #include "gm20b/fifo_gm20b.h"
+#include "gm20b/fb_gm20b.h"
 
 #include "gp10b.h"
 #include "hal_gp10b.h"
@@ -170,6 +172,21 @@ static const struct gpu_ops gp10b_ops = {
                 .isr_stall = gp10b_ce_isr,
                 .isr_nonstall = gp10b_ce_nonstall_isr,
         },
+        .fb = {
+                .reset = fb_gk20a_reset,
+                .init_hw = gk20a_fb_init_hw,
+                .init_fs_state = fb_gm20b_init_fs_state,
+                .set_mmu_page_size = gm20b_fb_set_mmu_page_size,
+                .set_use_full_comp_tag_line =
+                        gm20b_fb_set_use_full_comp_tag_line,
+                .compression_page_size = gp10b_fb_compression_page_size,
+                .compressible_page_size = gp10b_fb_compressible_page_size,
+                .vpr_info_fetch = gm20b_fb_vpr_info_fetch,
+                .dump_vpr_wpr_info = gm20b_fb_dump_vpr_wpr_info,
+                .is_debug_mode_enabled = gm20b_fb_debug_mode_enabled,
+                .set_debug_mode = gm20b_fb_set_debug_mode,
+                .tlb_invalidate = gk20a_fb_tlb_invalidate,
+        },
         .clock_gating = {
                 .slcg_bus_load_gating_prod =
                         gp10b_slcg_bus_load_gating_prod,
@@ -405,6 +422,7 @@ int gp10b_init_hal(struct gk20a *g)
 
         gops->ltc = gp10b_ops.ltc;
         gops->ce2 = gp10b_ops.ce2;
+        gops->fb = gp10b_ops.fb;
         gops->clock_gating = gp10b_ops.clock_gating;
         gops->fifo = gp10b_ops.fifo;
         gops->gr_ctx = gp10b_ops.gr_ctx;
@@ -471,10 +489,12 @@ int gp10b_init_hal(struct gk20a *g)
 
         g->bootstrap_owner = LSF_BOOTSTRAP_OWNER_DEFAULT;
         gp10b_init_gr(g);
-        gp10b_init_fb(gops);
         gp10b_init_mm(gops);
         gp10b_init_pmu_ops(g);
 
+        gp10b_init_uncompressed_kind_map();
+        gp10b_init_kind_attr();
+
         g->name = "gp10b";
 
         c->twod_class = FERMI_TWOD_A;