s390/qeth: use LINK_MODE_* to report the link characteristics
authorJulian Wiedmann <jwi@linux.vnet.ibm.com>
Tue, 11 Apr 2017 14:11:18 +0000 (16:11 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 11 Apr 2017 18:51:35 +0000 (14:51 -0400)
LINK_MODE_* replaces the u32-limited SUPPORTED_* / ENABLED_*
definitions.

Signed-off-by: Julian Wiedmann <jwi@linux.vnet.ibm.com>
Reviewed-by: Thomas Richter <tmricht@linux.vnet.ibm.com>
Reviewed-by: Ursula Braun <ubraun@linux.vnet.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/s390/net/qeth_core_main.c

index 4304bc2a6620c09b60622e90cbe16d79628f1ea3..88f4c6cff9a41a721d510758f2c78126e4272e1b 100644 (file)
@@ -6031,52 +6031,75 @@ EXPORT_SYMBOL_GPL(qeth_core_get_drvinfo);
 static void qeth_set_cmd_adv_sup(struct ethtool_link_ksettings *cmd,
                                int maxspeed, int porttype)
 {
-       u32 sup, adv;
+       ethtool_link_ksettings_zero_link_mode(cmd, supported);
+       ethtool_link_ksettings_zero_link_mode(cmd, advertising);
+       ethtool_link_ksettings_zero_link_mode(cmd, lp_advertising);
 
-       sup = SUPPORTED_Autoneg;
-       adv = ADVERTISED_Autoneg;
+       ethtool_link_ksettings_add_link_mode(cmd, supported, Autoneg);
+       ethtool_link_ksettings_add_link_mode(cmd, advertising, Autoneg);
 
        switch (porttype) {
        case PORT_TP:
-               sup |= SUPPORTED_TP;
-               adv |= ADVERTISED_TP;
+               ethtool_link_ksettings_add_link_mode(cmd, supported, TP);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising, TP);
                break;
        case PORT_FIBRE:
-               sup |= SUPPORTED_FIBRE;
-               adv |= ADVERTISED_FIBRE;
+               ethtool_link_ksettings_add_link_mode(cmd, supported, FIBRE);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising, FIBRE);
                break;
        default:
-               sup |= SUPPORTED_TP;
-               adv |= ADVERTISED_TP;
+               ethtool_link_ksettings_add_link_mode(cmd, supported, TP);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising, TP);
                WARN_ON_ONCE(1);
        }
 
        /* fallthrough from high to low, to select all legal speeds: */
        switch (maxspeed) {
        case SPEED_10000:
-               sup |= SUPPORTED_10000baseT_Full;
-               adv |= ADVERTISED_10000baseT_Full;
+               ethtool_link_ksettings_add_link_mode(cmd, supported,
+                                                    10000baseT_Full);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising,
+                                                    10000baseT_Full);
        case SPEED_1000:
-               sup |= SUPPORTED_1000baseT_Half | SUPPORTED_1000baseT_Full;
-               adv |= ADVERTISED_1000baseT_Half | ADVERTISED_1000baseT_Full;
+               ethtool_link_ksettings_add_link_mode(cmd, supported,
+                                                    1000baseT_Full);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising,
+                                                    1000baseT_Full);
+               ethtool_link_ksettings_add_link_mode(cmd, supported,
+                                                    1000baseT_Half);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising,
+                                                    1000baseT_Half);
        case SPEED_100:
-               sup |= SUPPORTED_100baseT_Half | SUPPORTED_100baseT_Full;
-               adv |= ADVERTISED_100baseT_Half | ADVERTISED_100baseT_Full;
+               ethtool_link_ksettings_add_link_mode(cmd, supported,
+                                                    100baseT_Full);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising,
+                                                    100baseT_Full);
+               ethtool_link_ksettings_add_link_mode(cmd, supported,
+                                                    100baseT_Half);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising,
+                                                    100baseT_Half);
        case SPEED_10:
-               sup |= SUPPORTED_10baseT_Half | SUPPORTED_10baseT_Full;
-               adv |= ADVERTISED_10baseT_Half | ADVERTISED_10baseT_Full;
+               ethtool_link_ksettings_add_link_mode(cmd, supported,
+                                                    10baseT_Full);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising,
+                                                    10baseT_Full);
+               ethtool_link_ksettings_add_link_mode(cmd, supported,
+                                                    10baseT_Half);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising,
+                                                    10baseT_Half);
                /* end fallthrough */
                break;
        default:
-               sup |= SUPPORTED_10baseT_Half | SUPPORTED_10baseT_Full;
-               adv |= ADVERTISED_10baseT_Half | ADVERTISED_10baseT_Full;
+               ethtool_link_ksettings_add_link_mode(cmd, supported,
+                                                    10baseT_Full);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising,
+                                                    10baseT_Full);
+               ethtool_link_ksettings_add_link_mode(cmd, supported,
+                                                    10baseT_Half);
+               ethtool_link_ksettings_add_link_mode(cmd, advertising,
+                                                    10baseT_Half);
                WARN_ON_ONCE(1);
        }
-
-       ethtool_convert_legacy_u32_to_link_mode(cmd->link_modes.supported,
-                                               sup);
-       ethtool_convert_legacy_u32_to_link_mode(cmd->link_modes.advertising,
-                                               adv);
 }
 
 int qeth_core_ethtool_get_link_ksettings(struct net_device *netdev,