drm/amd/display: change several dcn30 variables storage-class-specifier to static
authorTom Rix <trix@redhat.com>
Sun, 5 Mar 2023 12:52:26 +0000 (07:52 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 7 Mar 2023 19:22:42 +0000 (14:22 -0500)
smatch reports these similar problems in dcn30
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn30/dcn30_dwb.c:223:25:
  warning: symbol 'dcn30_dwbc_funcs' was not declared. Should it be static?
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn30/dcn30_mmhubbub.c:214:28:
  warning: symbol 'dcn30_mmhubbub_funcs' was not declared. Should it be static?
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn30/dcn30_mpc.c:1402:24:
  warning: symbol 'dcn30_mpc_funcs' was not declared. Should it be static?

All of these are only used in their definition file, so they should be static

Signed-off-by: Tom Rix <trix@redhat.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dcn30/dcn30_dwb.c
drivers/gpu/drm/amd/display/dc/dcn30/dcn30_mmhubbub.c
drivers/gpu/drm/amd/display/dc/dcn30/dcn30_mpc.c

index f14f696166920a0fa437e2c27cf5caf4e8f959d2..0d98918bf0fc4138a2aebedf4690753970e89535 100644 (file)
@@ -220,7 +220,7 @@ void dwb3_set_denorm(struct dwbc *dwbc, struct dc_dwb_params *params)
 }
 
 
-const struct dwbc_funcs dcn30_dwbc_funcs = {
+static const struct dwbc_funcs dcn30_dwbc_funcs = {
        .get_caps               = dwb3_get_caps,
        .enable                 = dwb3_enable,
        .disable                = dwb3_disable,
index 7a93eff183d981a7bf0d2fdecaba489195287eb1..6f2a0d5d963bd2fc7feff3055620568742574707 100644 (file)
@@ -211,7 +211,7 @@ static void mmhubbub3_config_mcif_arb(struct mcif_wb *mcif_wb,
        REG_UPDATE(MCIF_WB_ARBITRATION_CONTROL, MCIF_WB_CLIENT_ARBITRATION_SLICE,  params->arbitration_slice);
 }
 
-const struct mcif_wb_funcs dcn30_mmhubbub_funcs = {
+static const struct mcif_wb_funcs dcn30_mmhubbub_funcs = {
        .warmup_mcif            = mmhubbub3_warmup_mcif,
        .enable_mcif            = mmhubbub2_enable_mcif,
        .disable_mcif           = mmhubbub2_disable_mcif,
index ad1c1b703874e1fe38151d94ceda6d4e7c890a72..6cf40c1332bc3751c365648dac3893a3294a1302 100644 (file)
@@ -1399,7 +1399,7 @@ static void mpc3_set_mpc_mem_lp_mode(struct mpc *mpc)
        }
 }
 
-const struct mpc_funcs dcn30_mpc_funcs = {
+static const struct mpc_funcs dcn30_mpc_funcs = {
        .read_mpcc_state = mpc1_read_mpcc_state,
        .insert_plane = mpc1_insert_plane,
        .remove_mpcc = mpc1_remove_mpcc,