It is better to use batadv_compate_eth instead of memcpy for
concise style.

Cc: Marek Lindner <mareklind...@neomailbox.ch>
Cc: Simon Wunderlich <s...@simonwunderlich.de>
Cc: Antonio Quartulli <anto...@meshcoding.com>
Cc: "David S. Miller" <da...@davemloft.net>
Cc: b.a.t.m....@lists.open-mesh.org
Cc: net...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Tan Xiaojun <tanxiao...@huawei.com>
Signed-off-by: Ding Tianhong <dingtianh...@huawei.com>
Acked-by: Antonio Quartulli <anto...@meshcoding.com>
---
 net/batman-adv/originator.c        | 2 +-
 net/batman-adv/translation-table.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/batman-adv/originator.c b/net/batman-adv/originator.c
index 8ab1434..803ab4b 100644
--- a/net/batman-adv/originator.c
+++ b/net/batman-adv/originator.c
@@ -41,7 +41,7 @@ int batadv_compare_orig(const struct hlist_node *node, const 
void *data2)
        const void *data1 = container_of(node, struct batadv_orig_node,
                                         hash_entry);
 
-       return (memcmp(data1, data2, ETH_ALEN) == 0 ? 1 : 0);
+       return batadv_compare_eth(data1, data2);
 }
 
 /**
diff --git a/net/batman-adv/translation-table.c 
b/net/batman-adv/translation-table.c
index 4add57d..06506e6 100644
--- a/net/batman-adv/translation-table.c
+++ b/net/batman-adv/translation-table.c
@@ -51,7 +51,7 @@ static int batadv_compare_tt(const struct hlist_node *node, 
const void *data2)
        const void *data1 = container_of(node, struct batadv_tt_common_entry,
                                         hash_entry);
 
-       return (memcmp(data1, data2, ETH_ALEN) == 0 ? 1 : 0);
+       return batadv_compare_eth(data1, data2);
 }
 
 /**
-- 
1.8.0


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to