[PATCH] Change in inline keyword usage.

Subsystems: networking drivers, the rest

STALE5627d

2 messages, 2 authors, 2011-03-28 · open the first message on its own page

[PATCH] Change in inline keyword usage.

From: G.Balaji <hidden>
Date: 2011-03-27 08:49:04

Signed-off-by: G.Balaji <redacted>
---
 drivers/net/bnx2.c |    2 +-
 drivers/net/tg3.c  |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/bnx2.c b/drivers/net/bnx2.c
index d1865cc..8e6d618 100644
--- a/drivers/net/bnx2.c
+++ b/drivers/net/bnx2.c
@@ -8317,7 +8317,7 @@ static const struct net_device_ops bnx2_netdev_ops = {
 #endif
 };
 
-static void inline vlan_features_add(struct net_device *dev, u32 flags)
+static inline void vlan_features_add(struct net_device *dev, u32 flags)
 {
 	dev->vlan_features |= flags;
 }
diff --git a/drivers/net/tg3.c b/drivers/net/tg3.c
index ebec888..bddcf25 100644
--- a/drivers/net/tg3.c
+++ b/drivers/net/tg3.c
@@ -13118,7 +13118,7 @@ done:
 
 static struct pci_dev * __devinit tg3_find_peer(struct tg3 *);
 
-static void inline vlan_features_add(struct net_device *dev, unsigned long flags)
+static inline void vlan_features_add(struct net_device *dev, unsigned long flags)
 {
 	dev->vlan_features |= flags;
 }
-- 
1.7.4

Re: [PATCH] Change in inline keyword usage.

From: David Miller <davem@davemloft.net>
Date: 2011-03-28 01:42:12

From: "G.Balaji" <redacted>
Date: Sun, 27 Mar 2011 14:18:30 +0530
Signed-off-by: G.Balaji <redacted>
The tg3.c case was already taken care of by another patch
submission, please resend this patch with just the other
portion and please write a better commit message.

"Change in inline keyword usage" doesn't tell us anything.

What are you changing?  Why are you changing it?  What subsystem
are you making these changes in?

F.e. a better submission would be:

--------------------
Subject: [PATCH] drivers net: Fix declaration ordering in inline functions.

It should be "static inline foo" instead of "static foo inline."

Signed-off-by: ...
--------------------
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help