diff mbox series

clk: qcom: camcc-sm6350: Make camcc_sm6350_hws static

Message ID 20230107120434.1902666-1-konrad.dybcio@linaro.org
State Accepted
Commit bfc7486991c2b1b514cee90854af0f90c56b6bf2
Headers show
Series clk: qcom: camcc-sm6350: Make camcc_sm6350_hws static | expand

Commit Message

Konrad Dybcio Jan. 7, 2023, 12:04 p.m. UTC
There's no reason for it not to be static, and some compilers don't
like not it being that way. Make it so.

Reported-by: kernel test robot <lkp@intel.com>
Fixes: 80f5451d9a7c ("clk: qcom: Add camera clock controller driver for SM6350")
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 drivers/clk/qcom/camcc-sm6350.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stephen Boyd Jan. 11, 2023, 7:22 p.m. UTC | #1
Quoting Konrad Dybcio (2023-01-07 04:04:34)
> There's no reason for it not to be static, and some compilers don't
> like not it being that way. Make it so.
> 
> Reported-by: kernel test robot <lkp@intel.com>
> Fixes: 80f5451d9a7c ("clk: qcom: Add camera clock controller driver for SM6350")
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>
Bjorn Andersson Jan. 11, 2023, 7:49 p.m. UTC | #2
On Sat, 7 Jan 2023 13:04:34 +0100, Konrad Dybcio wrote:
> There's no reason for it not to be static, and some compilers don't
> like not it being that way. Make it so.
> 
> 

Applied, thanks!

[1/1] clk: qcom: camcc-sm6350: Make camcc_sm6350_hws static
      commit: bfc7486991c2b1b514cee90854af0f90c56b6bf2

Best regards,
diff mbox series

Patch

diff --git a/drivers/clk/qcom/camcc-sm6350.c b/drivers/clk/qcom/camcc-sm6350.c
index ea8600530046..acba9f99d960 100644
--- a/drivers/clk/qcom/camcc-sm6350.c
+++ b/drivers/clk/qcom/camcc-sm6350.c
@@ -1742,7 +1742,7 @@  static struct gdsc titan_top_gdsc = {
 	.pwrsts = PWRSTS_OFF_ON,
 };
 
-struct clk_hw *camcc_sm6350_hws[] = {
+static struct clk_hw *camcc_sm6350_hws[] = {
 	[CAMCC_PLL2_OUT_EARLY] = &camcc_pll2_out_early.hw,
 };