ar8216: adjust ATU flushing in case of link changes
authorFelix Fietkau <nbd@openwrt.org>
Wed, 15 Jul 2015 08:17:36 +0000 (08:17 +0000)
committerFelix Fietkau <nbd@openwrt.org>
Wed, 15 Jul 2015 08:17:36 +0000 (08:17 +0000)
If a link goes down, don't flush the complete ARL table.
Only flush the entries for the respective port.
Don't touch ARL table if a link goes up.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
SVN-Revision: 46381

target/linux/generic/files/drivers/net/phy/ar8216.c

index f4c7b8d1e9eaf79ce0d231f1ce228adbe2f86e2a..9065915f4bbb9dd4b529425a065c86e5ccbe769c 100644 (file)
@@ -1882,13 +1882,13 @@ ar8xxx_check_link_states(struct ar8xxx_priv *priv)
 
                priv->link_up[i] = link_new;
                changed = true;
+               /* flush ARL entries for this port if it went down*/
+               if (!link_new)
+                       priv->chip->atu_flush_port(priv, i);
                dev_info(&priv->phy->dev, "Port %d is %s\n",
                         i, link_new ? "up" : "down");
        }
 
-       if (changed)
-               priv->chip->atu_flush(priv);
-
        mutex_unlock(&priv->reg_mutex);
 
        return changed;
@@ -1900,9 +1900,7 @@ ar8xxx_phy_read_status(struct phy_device *phydev)
        struct ar8xxx_priv *priv = phydev->priv;
        struct switch_port_link link;
 
-       /* check for link changes and flush ATU
-        * if a change was detected
-        */
+       /* check for switch port link changes */
        if (phydev->state == PHY_CHANGELINK)
                ar8xxx_check_link_states(priv);