Thread (3 messages) 3 messages, 3 authors, 2023-02-13

Re: [PATCH] clk: imx: set imx_clk_gpr_mux_ops storage-class-specifier to static

From: Peng Fan <hidden>
Date: 2023-02-13 12:47:41
Also in: linux-clk, lkml


On 2/5/2023 11:01 AM, Tom Rix wrote:
smatch reports
drivers/clk/imx/clk-gpr-mux.c:73:22: warning: symbol 'imx_clk_gpr_mux_ops' was not declared. Should it be static?

imx_clk_gpr_mux_ops is only used in clk-gpr-mux.c, so it should be static.

Signed-off-by: Tom Rix <trix@redhat.com>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
quoted hunk ↗ jump to hunk
---
  drivers/clk/imx/clk-gpr-mux.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/clk/imx/clk-gpr-mux.c b/drivers/clk/imx/clk-gpr-mux.c
index 47a3e3cdcc82..c8d6090f15d6 100644
--- a/drivers/clk/imx/clk-gpr-mux.c
+++ b/drivers/clk/imx/clk-gpr-mux.c
@@ -70,7 +70,7 @@ static int imx_clk_gpr_mux_determine_rate(struct clk_hw *hw,
  	return clk_mux_determine_rate_flags(hw, req, 0);
  }
  
-const struct clk_ops imx_clk_gpr_mux_ops = {
+static const struct clk_ops imx_clk_gpr_mux_ops = {
  	.get_parent = imx_clk_gpr_mux_get_parent,
  	.set_parent = imx_clk_gpr_mux_set_parent,
  	.determine_rate = imx_clk_gpr_mux_determine_rate,
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help