From ed60c25d3840c9d198e7b4b5f852382b02ed64bd Mon Sep 17 00:00:00 2001 From: Vijayakumar Date: Mon, 1 May 2017 12:26:14 +0530 Subject: gpu: nvgpu: fix error for static code analysis use memset to fill structures with zero instead of assigning zero. mark functions local to the file as static fixing errors in clk, perf and therm modules. Bug 200299572 Change-Id: I0470298803c35b6faed2edc2a0c1dbf0e47e842e Signed-off-by: Vijayakumar Reviewed-on: http://git-master/r/1472940 Reviewed-by: Automatic_Commit_Validation_User Reviewed-by: svccoveritychecker Reviewed-by: Sachin Nikam GVS: Gerrit_Virtual_Submit Reviewed-by: Terje Bergstrom --- drivers/gpu/nvgpu/clk/clk.c | 30 +++++++++++++++++++++--------- 1 file changed, 21 insertions(+), 9 deletions(-) (limited to 'drivers/gpu/nvgpu/clk/clk.c') diff --git a/drivers/gpu/nvgpu/clk/clk.c b/drivers/gpu/nvgpu/clk/clk.c index 8b36394d..c1b8d5e1 100644 --- a/drivers/gpu/nvgpu/clk/clk.c +++ b/drivers/gpu/nvgpu/clk/clk.c @@ -49,15 +49,19 @@ int clk_pmu_freq_controller_load(struct gk20a *g, bool bload) { struct pmu_cmd cmd; struct pmu_msg msg; - struct pmu_payload payload = { {0} }; + struct pmu_payload payload; u32 status; u32 seqdesc; - struct nv_pmu_clk_rpc rpccall = {0}; - struct clkrpc_pmucmdhandler_params handler = {0}; + struct nv_pmu_clk_rpc rpccall; + struct clkrpc_pmucmdhandler_params handler; struct nv_pmu_clk_load *clkload; struct clk_freq_controllers *pclk_freq_controllers; struct ctrl_boardobjgrp_mask_e32 *load_mask; + memset(&payload, 0, sizeof(struct pmu_payload)); + memset(&rpccall, 0, sizeof(struct nv_pmu_clk_rpc)); + memset(&handler, 0, sizeof(struct clkrpc_pmucmdhandler_params)); + pclk_freq_controllers = &g->clk_pmu.clk_freq_controllers; rpccall.function = NV_PMU_CLK_RPC_ID_LOAD; clkload = &rpccall.params.clk_load; @@ -120,13 +124,17 @@ u32 clk_pmu_vin_load(struct gk20a *g) { struct pmu_cmd cmd; struct pmu_msg msg; - struct pmu_payload payload = { {0} }; + struct pmu_payload payload; u32 status; u32 seqdesc; - struct nv_pmu_clk_rpc rpccall = {0}; - struct clkrpc_pmucmdhandler_params handler = {0}; + struct nv_pmu_clk_rpc rpccall; + struct clkrpc_pmucmdhandler_params handler; struct nv_pmu_clk_load *clkload; + memset(&payload, 0, sizeof(struct pmu_payload)); + memset(&rpccall, 0, sizeof(struct nv_pmu_clk_rpc)); + memset(&handler, 0, sizeof(struct clkrpc_pmucmdhandler_params)); + rpccall.function = NV_PMU_CLK_RPC_ID_LOAD; clkload = &rpccall.params.clk_load; clkload->feature = NV_NV_PMU_CLK_LOAD_FEATURE_VIN; @@ -179,13 +187,17 @@ static u32 clk_pmu_vf_inject(struct gk20a *g, struct set_fll_clk *setfllclk) { struct pmu_cmd cmd; struct pmu_msg msg; - struct pmu_payload payload = { {0} }; + struct pmu_payload payload; u32 status; u32 seqdesc; - struct nv_pmu_clk_rpc rpccall = {0}; - struct clkrpc_pmucmdhandler_params handler = {0}; + struct nv_pmu_clk_rpc rpccall; + struct clkrpc_pmucmdhandler_params handler; struct nv_pmu_clk_vf_change_inject *vfchange; + memset(&payload, 0, sizeof(struct pmu_payload)); + memset(&rpccall, 0, sizeof(struct nv_pmu_clk_rpc)); + memset(&handler, 0, sizeof(struct clkrpc_pmucmdhandler_params)); + if ((setfllclk->gpc2clkmhz == 0) || (setfllclk->xbar2clkmhz == 0) || (setfllclk->sys2clkmhz == 0) || (setfllclk->voltuv == 0)) return -EINVAL; -- cgit v1.2.2