Staging: rtl8188eu: Fix warning of prefer ether_addr_copy().
authorElena Oat <oat.elena@gmail.com>
Thu, 6 Mar 2014 22:10:34 +0000 (00:10 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 7 Mar 2014 22:44:04 +0000 (14:44 -0800)
This patch fixes the warning produced by checkpatch "Prefer
ether_addr_copy() over memcpy() if the Ethernet addresses are
__aligned(2)". The address that is copied is a field of
nat25_network_db_entry structure. Definition for this struct is found in
rtw_br_ext.h file. Pahole shows that the addresses are 2-byte aligned.

Signed-off-by: Elena Oat <oat.elena@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/core/rtw_br_ext.c

index 96c8a93..c5a703a 100644 (file)
@@ -409,7 +409,7 @@ static void __nat25_db_network_insert(struct adapter *priv,
        db = priv->nethash[hash];
        while (db != NULL) {
                if (!memcmp(db->networkAddr, networkAddr, MAX_NETWORK_ADDR_LEN)) {
-                       memcpy(db->macAddr, macAddr, ETH_ALEN);
+                       ether_addr_copy(db->macAddr, macAddr);
                        db->ageing_timer = jiffies;
                        spin_unlock_bh(&priv->br_ext_lock);
                        return;
@@ -422,7 +422,7 @@ static void __nat25_db_network_insert(struct adapter *priv,
                return;
        }
        memcpy(db->networkAddr, networkAddr, MAX_NETWORK_ADDR_LEN);
-       memcpy(db->macAddr, macAddr, ETH_ALEN);
+       ether_addr_copy(db->macAddr, macAddr);
        atomic_set(&db->use_count, 1);
        db->ageing_timer = jiffies;