diff options
| author | Nikolay Aleksandrov <nikolay@redhat.com> | 2014-07-15 16:08:57 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-07-15 22:55:57 -0700 |
| commit | ff11d8b27dcd85e7b96ac2570116d5061130bd5e (patch) | |
| tree | c768872890f225fa73efa8ee20b26719bdef69ea /drivers/net/bonding/bond_options.c | |
| parent | f54424412b6b2f64cae4d7c39d981ca14ce0052c (diff) | |
| download | lwn-ff11d8b27dcd85e7b96ac2570116d5061130bd5e.tar.gz lwn-ff11d8b27dcd85e7b96ac2570116d5061130bd5e.zip | |
bonding: fix bond_option_mode_set warning
During the conversion to "static" functions this one got left out, only its
prototype was converted, thus resulting in:
drivers/net/bonding//bond_options.c:674:5: warning: symbol
'bond_option_mode_set' was not declared. Should it be static?
Fix it by making it static and also break the line in two as it was too
long.
CC: Stephen Hemminger <stephen@networkplumber.org>
CC: Jay Vosburgh <j.vosburgh@gmail.com>
CC: Veaceslav Falico <vfalico@gmail.com>
CC: Andy Gospodarek <andy@greyhouse.net>
CC: David S. Miller <davem@davemloft.net>
Signed-off-by: Nikolay Aleksandrov <nikolay@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding/bond_options.c')
| -rw-r--r-- | drivers/net/bonding/bond_options.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/bonding/bond_options.c b/drivers/net/bonding/bond_options.c index f908e65e86c1..cf720ce1b69e 100644 --- a/drivers/net/bonding/bond_options.c +++ b/drivers/net/bonding/bond_options.c @@ -671,7 +671,8 @@ const struct bond_option *bond_opt_get(unsigned int option) return &bond_opts[option]; } -int bond_option_mode_set(struct bonding *bond, const struct bond_opt_value *newval) +static int bond_option_mode_set(struct bonding *bond, + const struct bond_opt_value *newval) { if (!bond_mode_uses_arp(newval->value) && bond->params.arp_interval) { pr_info("%s: %s mode is incompatible with arp monitoring, start mii monitoring\n", |