eth: Make is_link_local() consistent with other address tests

Function name should include '_ether_addr'.
Return type should be bool.
Parameter name should be 'addr' not 'dest' (also matching kernel-doc).

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Acked-by: John Fastabend <john.r.fastabend@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

2 files changed, 2 insertions, 2 deletions

diff --git a/net/bridge/br_input.c b/net/bridge/br_input.c
index d047978bf025..4b34207419b1 100644
--- a/net/bridge/br_input.c
+++ b/net/bridge/br_input.c
@@ -147,7 +147,7 @@ rx_handler_result_t br_handle_frame(struct sk_buff **pskb)
 
         p = br_port_get_rcu(skb->dev);
 
-        if (unlikely(is_link_local(dest))) {
+        if (unlikely(is_link_local_ether_addr(dest))) {
                 /*
                  * See IEEE 802.1D Table 7-10 Reserved addresses
                  *
diff --git a/net/bridge/br_sysfs_br.c b/net/bridge/br_sysfs_br.c
index 82385fd2f187..cffb76e2161c 100644
--- a/net/bridge/br_sysfs_br.c
+++ b/net/bridge/br_sysfs_br.c
@@ -309,7 +309,7 @@ static ssize_t store_group_addr(struct device *d,
                    &new_addr[3], &new_addr[4], &new_addr[5]) != 6)
                 return -EINVAL;
 
-        if (!is_link_local(new_addr))
+        if (!is_link_local_ether_addr(new_addr))
                 return -EINVAL;
 
         if (new_addr[5] == 1 ||         /* 802.3x Pause address */