diff options
| author | Adrian Bunk <bunk@stusta.de> | 2006-12-12 17:24:39 +0100 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2007-02-05 16:58:47 -0500 |
| commit | 079ca7da1e6d05c7cb82e9c4f2e1d98839332664 (patch) | |
| tree | 849faa0b78df712cbfa62d2454843cb0add04c3d | |
| parent | bd36b0ac5d06378c95b5149b6df5f413a6c985a5 (diff) | |
| download | op-kernel-dev-079ca7da1e6d05c7cb82e9c4f2e1d98839332664.zip op-kernel-dev-079ca7da1e6d05c7cb82e9c4f2e1d98839332664.tar.gz | |
bonding.h: "extern inline" -> "static inline"
"extern inline" generates a warning with -Wmissing-prototypes and I'm
currently working on getting the kernel cleaned up for adding this to
the CFLAGS since it will help us to avoid a nasty class of runtime
errors.
If there are places that really need a forced inline, __always_inline
would be the correct solution.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
| -rw-r--r-- | drivers/net/bonding/bonding.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/net/bonding/bonding.h b/drivers/net/bonding/bonding.h index 0978c9a..731cfb6 100644 --- a/drivers/net/bonding/bonding.h +++ b/drivers/net/bonding/bonding.h @@ -237,12 +237,13 @@ static inline struct bonding *bond_get_bond_by_slave(struct slave *slave) #define BOND_ARP_VALIDATE_ALL (BOND_ARP_VALIDATE_ACTIVE | \ BOND_ARP_VALIDATE_BACKUP) -extern inline int slave_do_arp_validate(struct bonding *bond, struct slave *slave) +static inline int slave_do_arp_validate(struct bonding *bond, + struct slave *slave) { return bond->params.arp_validate & (1 << slave->state); } -extern inline unsigned long slave_last_rx(struct bonding *bond, +static inline unsigned long slave_last_rx(struct bonding *bond, struct slave *slave) { if (slave_do_arp_validate(bond, slave)) |
