Author: blogic
Date: 2014-12-01 17:14:32 +0100 (Mon, 01 Dec 2014)
New Revision: 43466

Modified:
   trunk/target/linux/generic/files/drivers/net/phy/ar8216.c
Log:
ar8216: factor out info whether switch should be configured at probe stage to 
ar8xxx_chip

Factor out info whether switch should be configured at probe stage
to ar8xxx_chip. Remove related chip_is_... checks.

Signed-off-by: Heiner Kallweit <[email protected]>

Modified: trunk/target/linux/generic/files/drivers/net/phy/ar8216.c
===================================================================
--- trunk/target/linux/generic/files/drivers/net/phy/ar8216.c   2014-12-01 
16:14:26 UTC (rev 43465)
+++ trunk/target/linux/generic/files/drivers/net/phy/ar8216.c   2014-12-01 
16:14:32 UTC (rev 43466)
@@ -70,6 +70,7 @@
 
 struct ar8xxx_chip {
        unsigned long caps;
+       bool config_at_probe;
 
        int (*hw_init)(struct ar8xxx_priv *priv);
        void (*cleanup)(struct ar8xxx_priv *priv);
@@ -1821,6 +1822,7 @@
 
 static const struct ar8xxx_chip ar8327_chip = {
        .caps = AR8XXX_CAP_GIGE | AR8XXX_CAP_MIB_COUNTERS,
+       .config_at_probe = true,
        .hw_init = ar8327_hw_init,
        .cleanup = ar8327_cleanup,
        .init_globals = ar8327_init_globals,
@@ -2797,7 +2799,7 @@
        if (WARN_ON(!priv))
                return -ENODEV;
 
-       if (chip_is_ar8327(priv) || chip_is_ar8337(priv))
+       if (priv->chip->config_at_probe)
                return ar8xxx_phy_check_aneg(phydev);
 
        priv->phy = phydev;
@@ -2973,7 +2975,7 @@
                        phydev->advertising = ADVERTISED_100baseT_Full;
                }
 
-               if (chip_is_ar8327(priv) || chip_is_ar8337(priv)) {
+               if (priv->chip->config_at_probe) {
                        priv->phy = phydev;
 
                        ret = ar8xxx_start(priv);
_______________________________________________
openwrt-commits mailing list
[email protected]
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-commits

Reply via email to