Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1490857 > unrolled thread

[PATCH] net: hip04: mark tx_done() static

Started byBaoyou Xie <baoyou.xie@linaro.org>
First post2016-09-25 11:20 +0200
Last post2016-09-26 08:30 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] net: hip04: mark tx_done() static Baoyou Xie <baoyou.xie@linaro.org> - 2016-09-25 11:20 +0200
    Re: [PATCH] net: hip04: mark tx_done() static David Miller <davem@davemloft.net> - 2016-09-26 08:30 +0200

#1490857 — [PATCH] net: hip04: mark tx_done() static

FromBaoyou Xie <baoyou.xie@linaro.org>
Date2016-09-25 11:20 +0200
Subject[PATCH] net: hip04: mark tx_done() static
Message-ID<sldLj-4J9-1@gated-at.bofh.it>
We get 1 warning when building kernel with W=1:
drivers/net/ethernet/hisilicon/hip04_eth.c:603:22: warning: no previous prototype for 'tx_done' [-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/hisilicon/hip04_eth.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/hisilicon/hip04_eth.c b/drivers/net/ethernet/hisilicon/hip04_eth.c
index 415ffa1..3977889 100644
--- a/drivers/net/ethernet/hisilicon/hip04_eth.c
+++ b/drivers/net/ethernet/hisilicon/hip04_eth.c
@@ -600,7 +600,7 @@ static irqreturn_t hip04_mac_interrupt(int irq, void *dev_id)
 	return IRQ_HANDLED;
 }
 
-enum hrtimer_restart tx_done(struct hrtimer *hrtimer)
+static enum hrtimer_restart tx_done(struct hrtimer *hrtimer)
 {
 	struct hip04_priv *priv;
 
-- 
2.7.4

[toc] | [next] | [standalone]


#1491080

FromDavid Miller <davem@davemloft.net>
Date2016-09-26 08:30 +0200
Message-ID<slxAm-h3-1@gated-at.bofh.it>
In reply to#1490857
From: Baoyou Xie <baoyou.xie@linaro.org>
Date: Sun, 25 Sep 2016 17:19:04 +0800

> We get 1 warning when building kernel with W=1:
> drivers/net/ethernet/hisilicon/hip04_eth.c:603:22: warning: no previous prototype for 'tx_done' [-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.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web