r8169: rtl8169_set_speed_xmii cleanup.
authorFrancois Romieu <romieu@fr.zoreil.com>
Fri, 11 Mar 2011 19:30:24 +0000 (20:30 +0100)
committerFrancois romieu <romieu@fr.zoreil.com>
Mon, 9 May 2011 19:03:01 +0000 (21:03 +0200)
Shorten chipset version test.

No functional change.

Careful readers will notice that the 'supports_gmii' flag is deduced
from the device PCI id. Though less specific than the chipset related
RTL_GIGA_MAC_VER_XY, it is good enough to detect a GMII deprieved 810x.
Some features push for a device specific configuration (improved jumbo
frame support for instance). 'supports_gmii' will follow this path
if / when the device PCI id test stops working.

Signed-off-by: Francois Romieu <romieu@fr.zoreil.com>
Cc: Realtek linux nic maintainers <nic_swsd@realtek.com>
drivers/net/r8169.c

index 976bb31b2092bdddd0c768aca0e2a624bd721d07..182c794744344ac3d46b8deb639dc117a207bb75 100644 (file)
@@ -1258,16 +1258,7 @@ static int rtl8169_set_speed_xmii(struct net_device *dev,
                giga_ctrl &= ~(ADVERTISE_1000FULL | ADVERTISE_1000HALF);
 
                /* The 8100e/8101e/8102e do Fast Ethernet only. */
-               if (tp->mac_version != RTL_GIGA_MAC_VER_07 &&
-                   tp->mac_version != RTL_GIGA_MAC_VER_08 &&
-                   tp->mac_version != RTL_GIGA_MAC_VER_09 &&
-                   tp->mac_version != RTL_GIGA_MAC_VER_10 &&
-                   tp->mac_version != RTL_GIGA_MAC_VER_13 &&
-                   tp->mac_version != RTL_GIGA_MAC_VER_14 &&
-                   tp->mac_version != RTL_GIGA_MAC_VER_15 &&
-                   tp->mac_version != RTL_GIGA_MAC_VER_16 &&
-                   tp->mac_version != RTL_GIGA_MAC_VER_29 &&
-                   tp->mac_version != RTL_GIGA_MAC_VER_30) {
+               if (tp->mii.supports_gmii) {
                        if (adv & ADVERTISED_1000baseT_Half)
                                giga_ctrl |= ADVERTISE_1000HALF;
                        if (adv & ADVERTISED_1000baseT_Full)