diff mbox

cxgb4: mark cxgb_setup_tc() static

Message ID 1475220865-18321-1-git-send-email-baoyou.xie@linaro.org
State Accepted
Commit 8efebd6e5e93283a72d7a014d6dd8130e6601352
Headers show

Commit Message

Baoyou Xie Sept. 30, 2016, 7:34 a.m. UTC
We get 1 warning when building kernel with W=1:
drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c:2715:5: warning: no previous prototype for 'cxgb_setup_tc' [-Wmissing-prototypes]

In fact, this function is only used in the file in which it is
declared and don't need a declaration, but can be made static.
so this patch marks this function with 'static'.

Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>

---
 drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
2.7.4

Comments

David Miller Oct. 3, 2016, 5:20 a.m. UTC | #1
From: Baoyou Xie <baoyou.xie@linaro.org>

Date: Fri, 30 Sep 2016 15:34:25 +0800

> We get 1 warning when building kernel with W=1:

> drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c:2715:5: warning: no previous prototype for 'cxgb_setup_tc' [-Wmissing-prototypes]

> 

> In fact, this function is only used in the file in which it is

> declared and don't need a declaration, but can be made static.

> so this patch marks this function with 'static'.

> 

> Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>


Applied.
diff mbox

Patch

diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
index c8a5c43..08e69d3 100644
--- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
+++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
@@ -2712,8 +2712,8 @@  static int cxgb_set_tx_maxrate(struct net_device *dev, int index, u32 rate)
 	return err;
 }
 
-int cxgb_setup_tc(struct net_device *dev, u32 handle, __be16 proto,
-		  struct tc_to_netdev *tc)
+static int cxgb_setup_tc(struct net_device *dev, u32 handle, __be16 proto,
+			 struct tc_to_netdev *tc)
 {
 	struct port_info *pi = netdev2pinfo(dev);
 	struct adapter *adap = netdev2adap(dev);