[PATCH] Change in inline keyword usage.

[PATCH] Change in inline keyword usage.

[G.Balaji]

Signed-off-by: G.Balaji <balajig81@gmail.com>
---
 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.

[David Miller]

From: "G.Balaji" <balajig81@gmail.com>
Date: Sun, 27 Mar 2011 14:18:30 +0530

> 
> Signed-off-by: G.Balaji <balajig81@gmail.com>

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: ...
--------------------