diff mbox series

[-next] clk: qcom: msm8996: Make symbol 'cpu_msm8996_clks' static

Message ID 20200714142155.35085-1-weiyongjun1@huawei.com (mailing list archive)
State Accepted
Commit 8607fa169178a4a38f232612e036def96b5ecd1d
Headers show
Series [-next] clk: qcom: msm8996: Make symbol 'cpu_msm8996_clks' static | expand

Commit Message

Wei Yongjun July 14, 2020, 2:21 p.m. UTC
The sparse tool complains as follows:

drivers/clk/qcom/clk-cpu-8996.c:341:19: warning:
 symbol 'cpu_msm8996_clks' was not declared. Should it be static?

This variable is not used outside of clk-cpu-8996.c, so this commit
marks it static.

Fixes: 03e342dc45c9 ("clk: qcom: Add CPU clock driver for msm8996")
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
---
 drivers/clk/qcom/clk-cpu-8996.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stephen Boyd July 21, 2020, 12:40 a.m. UTC | #1
Quoting Wei Yongjun (2020-07-14 07:21:55)
> The sparse tool complains as follows:
> 
> drivers/clk/qcom/clk-cpu-8996.c:341:19: warning:
>  symbol 'cpu_msm8996_clks' was not declared. Should it be static?
> 
> This variable is not used outside of clk-cpu-8996.c, so this commit
> marks it static.
> 
> Fixes: 03e342dc45c9 ("clk: qcom: Add CPU clock driver for msm8996")
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
> ---

Applied to clk-next
diff mbox series

Patch

diff --git a/drivers/clk/qcom/clk-cpu-8996.c b/drivers/clk/qcom/clk-cpu-8996.c
index 77a2d2806e58..4a4fde8dd12d 100644
--- a/drivers/clk/qcom/clk-cpu-8996.c
+++ b/drivers/clk/qcom/clk-cpu-8996.c
@@ -338,7 +338,7 @@  static const struct regmap_config cpu_msm8996_regmap_config = {
 	.val_format_endian	= REGMAP_ENDIAN_LITTLE,
 };
 
-struct clk_regmap *cpu_msm8996_clks[] = {
+static struct clk_regmap *cpu_msm8996_clks[] = {
 	&perfcl_pll.clkr,
 	&pwrcl_pll.clkr,
 	&perfcl_alt_pll.clkr,