soc: qcom: ramp_controller: Make things static
authorBjorn Andersson <quic_bjorande@quicinc.com>
Tue, 10 Jan 2023 04:20:04 +0000 (20:20 -0800)
committerBjorn Andersson <andersson@kernel.org>
Wed, 11 Jan 2023 04:00:27 +0000 (22:00 -0600)
The five msm8976_cfg_* objects ought to be static, as reported by LKP
and sparse, fix this.

drivers/soc/qcom/ramp_controller.c:235:27: sparse: sparse: symbol 'msm8976_cfg_dfs_sid' was not declared. Should it be static?
drivers/soc/qcom/ramp_controller.c:246:27: sparse: sparse: symbol 'msm8976_cfg_link_sid' was not declared. Should it be static?
drivers/soc/qcom/ramp_controller.c:250:27: sparse: sparse: symbol 'msm8976_cfg_lmh_sid' was not declared. Should it be static?
drivers/soc/qcom/ramp_controller.c:256:27: sparse: sparse: symbol 'msm8976_cfg_ramp_en' was not declared. Should it be static?
drivers/soc/qcom/ramp_controller.c:262:27: sparse: sparse: symbol 'msm8976_cfg_ramp_dis' was not declared. Should it be static?

Fixes: a723c95fa137 ("soc: qcom: Add Qualcomm Ramp Controller driver")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Bjorn Andersson <quic_bjorande@quicinc.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20230110042004.2378444-1-quic_bjorande@quicinc.com
drivers/soc/qcom/ramp_controller.c

index 93652b0..dc74d2a 100644 (file)
@@ -232,7 +232,7 @@ static const struct regmap_config qrc_regmap_config = {
        .fast_io = true,
 };
 
-const struct reg_sequence msm8976_cfg_dfs_sid[] = {
+static const struct reg_sequence msm8976_cfg_dfs_sid[] = {
        { 0x10, 0xfefebff7 },
        { 0x14, 0xfdff7fef },
        { 0x18, 0xfbffdefb },
@@ -243,23 +243,23 @@ const struct reg_sequence msm8976_cfg_dfs_sid[] = {
        { 0x2c, 0x8102 }
 };
 
-const struct reg_sequence msm8976_cfg_link_sid[] = {
+static const struct reg_sequence msm8976_cfg_link_sid[] = {
        { 0x40, 0xfc987 }
 };
 
-const struct reg_sequence msm8976_cfg_lmh_sid[] = {
+static const struct reg_sequence msm8976_cfg_lmh_sid[] = {
        { 0x30, 0x77706db },
        { 0x34, 0x5550249 },
        { 0x38, 0x111 }
 };
 
-const struct reg_sequence msm8976_cfg_ramp_en[] = {
+static const struct reg_sequence msm8976_cfg_ramp_en[] = {
        { 0x50, 0x800 }, /* pre_en */
        { 0x50, 0xc00 }, /* en */
        { 0x50, 0x400 }  /* post_en */
 };
 
-const struct reg_sequence msm8976_cfg_ramp_dis[] = {
+static const struct reg_sequence msm8976_cfg_ramp_dis[] = {
        { 0x50, 0x0 }
 };