From ee60394abc9a52182892ab54554d3e9c841080bf Mon Sep 17 00:00:00 2001 From: Sunny He Date: Mon, 31 Jul 2017 14:43:11 -0700 Subject: 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 Reviewed-on: https://git-master.nvidia.com/r/1537746 Reviewed-by: Automatic_Commit_Validation_User Reviewed-by: svccoveritychecker GVS: Gerrit_Virtual_Submit Reviewed-by: Terje Bergstrom --- drivers/gpu/nvgpu/gm20b/fb_gm20b.c | 36 +++++++++--------------------------- drivers/gpu/nvgpu/gm20b/fb_gm20b.h | 13 +++++++++++-- drivers/gpu/nvgpu/gm20b/hal_gm20b.c | 21 ++++++++++++++++++++- 3 files changed, 40 insertions(+), 30 deletions(-) (limited to 'drivers/gpu/nvgpu/gm20b') diff --git a/drivers/gpu/nvgpu/gm20b/fb_gm20b.c b/drivers/gpu/nvgpu/gm20b/fb_gm20b.c index 47b4313b..31947ad0 100644 --- a/drivers/gpu/nvgpu/gm20b/fb_gm20b.c +++ b/drivers/gpu/nvgpu/gm20b/fb_gm20b.c @@ -25,7 +25,7 @@ #define VPR_INFO_FETCH_WAIT (5) -static void fb_gm20b_init_fs_state(struct gk20a *g) +void fb_gm20b_init_fs_state(struct gk20a *g) { gk20a_dbg_info("initialize gm20b fb"); @@ -434,7 +434,7 @@ void gm20b_init_kind_attr(void) } } -static void gm20b_fb_set_mmu_page_size(struct gk20a *g) +void gm20b_fb_set_mmu_page_size(struct gk20a *g) { /* set large page size in fb */ u32 fb_mmu_ctrl = gk20a_readl(g, fb_mmu_ctrl_r()); @@ -442,7 +442,7 @@ static void gm20b_fb_set_mmu_page_size(struct gk20a *g) gk20a_writel(g, fb_mmu_ctrl_r(), fb_mmu_ctrl); } -static bool gm20b_fb_set_use_full_comp_tag_line(struct gk20a *g) +bool gm20b_fb_set_use_full_comp_tag_line(struct gk20a *g) { /* set large page size in fb */ u32 fb_mmu_ctrl = gk20a_readl(g, fb_mmu_ctrl_r()); @@ -452,17 +452,17 @@ static bool gm20b_fb_set_use_full_comp_tag_line(struct gk20a *g) return true; } -static unsigned int gm20b_fb_compression_page_size(struct gk20a *g) +unsigned int gm20b_fb_compression_page_size(struct gk20a *g) { return SZ_128K; } -static unsigned int gm20b_fb_compressible_page_size(struct gk20a *g) +unsigned int gm20b_fb_compressible_page_size(struct gk20a *g) { return SZ_64K; } -static void gm20b_fb_dump_vpr_wpr_info(struct gk20a *g) +void gm20b_fb_dump_vpr_wpr_info(struct gk20a *g) { u32 val; @@ -511,7 +511,7 @@ static int gm20b_fb_vpr_info_fetch_wait(struct gk20a *g, return -ETIMEDOUT; } -static int gm20b_fb_vpr_info_fetch(struct gk20a *g) +int gm20b_fb_vpr_info_fetch(struct gk20a *g) { if (gm20b_fb_vpr_info_fetch_wait(g, VPR_INFO_FETCH_WAIT)) { return -ETIME; @@ -523,14 +523,14 @@ static int gm20b_fb_vpr_info_fetch(struct gk20a *g) return gm20b_fb_vpr_info_fetch_wait(g, VPR_INFO_FETCH_WAIT); } -static bool gm20b_fb_debug_mode_enabled(struct gk20a *g) +bool gm20b_fb_debug_mode_enabled(struct gk20a *g) { u32 debug_ctrl = gk20a_readl(g, gr_gpcs_pri_mmu_debug_ctrl_r()); return gr_gpcs_pri_mmu_debug_ctrl_debug_v(debug_ctrl) == gr_gpcs_pri_mmu_debug_ctrl_debug_enabled_v(); } -static void gm20b_fb_set_debug_mode(struct gk20a *g, bool enable) +void gm20b_fb_set_debug_mode(struct gk20a *g, bool enable) { u32 reg_val, fb_debug_ctrl, gpc_debug_ctrl; @@ -554,21 +554,3 @@ static void gm20b_fb_set_debug_mode(struct gk20a *g, bool enable) gr_gpcs_pri_mmu_debug_ctrl_debug_m(), gpc_debug_ctrl); gk20a_writel(g, gr_gpcs_pri_mmu_debug_ctrl_r(), reg_val); } - -void gm20b_init_fb(struct gpu_ops *gops) -{ - gops->fb.reset = fb_gk20a_reset; - gops->fb.init_hw = gk20a_fb_init_hw; - gops->fb.init_fs_state = fb_gm20b_init_fs_state; - gops->fb.set_mmu_page_size = gm20b_fb_set_mmu_page_size; - gops->fb.set_use_full_comp_tag_line = gm20b_fb_set_use_full_comp_tag_line; - gops->fb.compression_page_size = gm20b_fb_compression_page_size; - gops->fb.compressible_page_size = gm20b_fb_compressible_page_size; - gops->fb.vpr_info_fetch = gm20b_fb_vpr_info_fetch; - gops->fb.dump_vpr_wpr_info = gm20b_fb_dump_vpr_wpr_info; - gops->fb.is_debug_mode_enabled = gm20b_fb_debug_mode_enabled; - gops->fb.set_debug_mode = gm20b_fb_set_debug_mode; - gops->fb.tlb_invalidate = gk20a_fb_tlb_invalidate; - gm20b_init_uncompressed_kind_map(); - gm20b_init_kind_attr(); -} diff --git a/drivers/gpu/nvgpu/gm20b/fb_gm20b.h b/drivers/gpu/nvgpu/gm20b/fb_gm20b.h index 22b848d6..42e86407 100644 --- a/drivers/gpu/nvgpu/gm20b/fb_gm20b.h +++ b/drivers/gpu/nvgpu/gm20b/fb_gm20b.h @@ -1,7 +1,7 @@ /* * GM20B 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, @@ -17,7 +17,16 @@ #define _NVHOST_GM20B_FB struct gk20a; -void gm20b_init_fb(struct gpu_ops *gops); +void fb_gm20b_init_fs_state(struct gk20a *g); +void gm20b_fb_set_mmu_page_size(struct gk20a *g); +bool gm20b_fb_set_use_full_comp_tag_line(struct gk20a *g); +unsigned int gm20b_fb_compression_page_size(struct gk20a *g); +unsigned int gm20b_fb_compressible_page_size(struct gk20a *g); +void gm20b_fb_dump_vpr_wpr_info(struct gk20a *g); +int gm20b_fb_vpr_info_fetch(struct gk20a *g); +bool gm20b_fb_debug_mode_enabled(struct gk20a *g); +void gm20b_fb_set_debug_mode(struct gk20a *g, bool enable); + void gm20b_init_uncompressed_kind_map(void); void gm20b_init_kind_attr(void); #endif diff --git a/drivers/gpu/nvgpu/gm20b/hal_gm20b.c b/drivers/gpu/nvgpu/gm20b/hal_gm20b.c index 7861e438..a540de64 100644 --- a/drivers/gpu/nvgpu/gm20b/hal_gm20b.c +++ b/drivers/gpu/nvgpu/gm20b/hal_gm20b.c @@ -16,6 +16,7 @@ #include "gk20a/gk20a.h" #include "gk20a/ce2_gk20a.h" #include "gk20a/dbg_gpu_gk20a.h" +#include "gk20a/fb_gk20a.h" #include "gk20a/fifo_gk20a.h" #include "gk20a/therm_gk20a.h" #include "gk20a/css_gr_gk20a.h" @@ -161,6 +162,21 @@ static const struct gpu_ops gm20b_ops = { .isr_stall = gk20a_ce2_isr, .isr_nonstall = gk20a_ce2_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 = gm20b_fb_compression_page_size, + .compressible_page_size = gm20b_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 = gm20b_slcg_bus_load_gating_prod, @@ -378,6 +394,7 @@ int gm20b_init_hal(struct gk20a *g) gops->ltc = gm20b_ops.ltc; gops->ce2 = gm20b_ops.ce2; + gops->fb = gm20b_ops.fb; gops->clock_gating = gm20b_ops.clock_gating; gops->fifo = gm20b_ops.fifo; gops->gr_ctx = gm20b_ops.gr_ctx; @@ -445,10 +462,12 @@ int gm20b_init_hal(struct gk20a *g) #endif g->bootstrap_owner = LSF_BOOTSTRAP_OWNER_DEFAULT; gm20b_init_gr(g); - gm20b_init_fb(gops); gm20b_init_mm(gops); gm20b_init_pmu_ops(g); + gm20b_init_uncompressed_kind_map(); + gm20b_init_kind_attr(); + g->name = "gm20b"; c->twod_class = FERMI_TWOD_A; -- cgit v1.2.2