From 2b582c5141752ff272c5d059b56433155bc3985a Mon Sep 17 00:00:00 2001 From: Sunny He Date: Wed, 28 Jun 2017 17:41:55 -0700 Subject: 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 Reviewed-on: https://git-master.nvidia.com/r/1514102 Reviewed-by: Automatic_Commit_Validation_User GVS: Gerrit_Virtual_Submit Reviewed-by: Alex Waterman Reviewed-by: Vijayakumar Subbu --- drivers/gpu/nvgpu/gp10b/hal_gp10b.c | 6 +++++- drivers/gpu/nvgpu/gp10b/priv_ring_gp10b.c | 7 +------ drivers/gpu/nvgpu/gp10b/priv_ring_gp10b.h | 4 ++-- 3 files changed, 8 insertions(+), 9 deletions(-) (limited to 'drivers/gpu/nvgpu/gp10b') 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 #include -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__*/ -- cgit v1.2.2