2021-06-09 13:09:11

by Pu Lehui

[permalink] [raw]
Subject: [PATCH -next] clk/qcom: Remove unused variables

Fix gcc '-Wunused-const-variable' warnings:

drivers/clk/qcom/gcc-mdm9607.c:122:37: warning: 'gcc_xo_gpll0_gpll1'
defined but not used [-Wunused-const-variable=]
drivers/clk/qcom/gcc-mdm9607.c:116:32: warning: 'gcc_xo_gpll0_gpll1_map'
defined but not used [-Wunused-const-variable=]
drivers/clk/qcom/gcc-mdm9607.c:42:37: warning: 'gcc_xo_sleep' defined
but not used [-Wunused-const-variable=]
drivers/clk/qcom/gcc-mdm9607.c:37:32: warning: 'gcc_xo_sleep_map'
defined but not used [-Wunused-const-variable=]

Let's remove them.

Signed-off-by: Pu Lehui <[email protected]>
---
drivers/clk/qcom/gcc-mdm9607.c | 23 -----------------------
1 file changed, 23 deletions(-)

diff --git a/drivers/clk/qcom/gcc-mdm9607.c b/drivers/clk/qcom/gcc-mdm9607.c
index 5918dc51ffcd..4c9078e99bb3 100644
--- a/drivers/clk/qcom/gcc-mdm9607.c
+++ b/drivers/clk/qcom/gcc-mdm9607.c
@@ -34,16 +34,6 @@ enum {
P_SLEEP_CLK,
};

-static const struct parent_map gcc_xo_sleep_map[] = {
- { P_XO, 0 },
- { P_SLEEP_CLK, 6 }
-};
-
-static const struct clk_parent_data gcc_xo_sleep[] = {
- { .fw_name = "xo" },
- { .fw_name = "sleep_clk" },
-};
-
static struct clk_alpha_pll gpll0_early = {
.offset = 0x21000,
.regs = clk_alpha_pll_regs[CLK_ALPHA_PLL_TYPE_DEFAULT],
@@ -113,19 +103,6 @@ static struct clk_regmap gpll1_vote = {
},
};

-static const struct parent_map gcc_xo_gpll0_gpll1_map[] = {
- { P_XO, 0 },
- { P_GPLL0, 1 },
- { P_GPLL1, 2 },
-};
-
-static const struct clk_parent_data gcc_xo_gpll0_gpll1[] = {
- { .fw_name = "xo" },
- { .hw = &gpll0.clkr.hw },
- { .hw = &gpll1_vote.hw },
-};
-
-
static const struct parent_map gcc_xo_gpll0_gpll1_sleep_map[] = {
{ P_XO, 0 },
{ P_GPLL0, 1 },
--
2.17.1


2021-06-28 00:06:22

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH -next] clk/qcom: Remove unused variables

Quoting Pu Lehui (2021-06-08 23:18:48)
> Fix gcc '-Wunused-const-variable' warnings:
>
> drivers/clk/qcom/gcc-mdm9607.c:122:37: warning: 'gcc_xo_gpll0_gpll1'
> defined but not used [-Wunused-const-variable=]
> drivers/clk/qcom/gcc-mdm9607.c:116:32: warning: 'gcc_xo_gpll0_gpll1_map'
> defined but not used [-Wunused-const-variable=]
> drivers/clk/qcom/gcc-mdm9607.c:42:37: warning: 'gcc_xo_sleep' defined
> but not used [-Wunused-const-variable=]
> drivers/clk/qcom/gcc-mdm9607.c:37:32: warning: 'gcc_xo_sleep_map'
> defined but not used [-Wunused-const-variable=]
>
> Let's remove them.
>
> Signed-off-by: Pu Lehui <[email protected]>
> ---

Applied to clk-next