Yes, the 1000BASE-T capabilities are clearly the wrong way round.

Patch:

--- net-tools-1.60.orig/mii-tool.c
+++ net-tools-1.60/mii-tool.c
@@ -355,9 +355,9 @@
            printf("remote fault, ");
        printf((bmsr & MII_BMSR_LINK_VALID) ? "link ok" : "no link");
        printf("\n  capabilities:%s", media_list(bmsr >> 6, bmcr2, 0));
-       printf("\n  advertising: %s", media_list(advert, lpa2 >> 2, 0));
+       printf("\n  advertising: %s", media_list(advert, bmcr2, 0));
        if (lkpar & MII_AN_ABILITY_MASK)
-           printf("\n  link partner:%s", media_list(lkpar, bmcr2, 0));
+           printf("\n  link partner:%s", media_list(lkpar, lpa2 >> 2, 0));
        printf("\n");
     }
     fflush(stdout);
--- END ---

Ben.

-- 
Ben Hutchings
Who are all these weirdos? - David Bowie, about L-Space IRC channel #afp

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to