[PATCH] eth_v10.c phy fix

["Pieter Steyn" <pieterste@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 899 bytes --]

Hi guys,

This patch allows you to set phy address parameters through ethtool, like:

/bin/ethtool -s eth0 phyad 8

--- drivers/net/cris/eth_v10.c  2008-06-12 14:12:33.000000000 +0200
+++ drivers/net/cris/eth_v10.c  2008-06-12 14:19:35.000000000 +0200
@@ -1430,7 +1430,11 @@
                e100_set_duplex(dev, ecmd->duplex == DUPLEX_HALF ? half : full);
                e100_set_speed(dev, ecmd->speed == SPEED_10 ? 10: 100);
        }
-
+        //Honor ethtool phy address parameter, which was simply
discarded before this.
+        if (ecmd->phy_address <= 31)
+                mdio_phy_addr = ecmd->phy_address;
+        else if (ecmd->phy_address == 32)
+                return -ENODEV;
        return 0;
 }

I've searched the MAINTAINERS file and haven't seen anything regarding
either the source file name or cris.  Please consider applying my
patch to main kernel...

Thanks,
Pieter Steyn

[-- Attachment #2: eth_v10.patch --]
[-- Type: application/octet-stream, Size: 558 bytes --]

--- drivers/net/cris/eth_v10.c	2008-06-12 14:12:33.000000000 +0200
+++ drivers/net/cris/eth_v10.c	2008-06-12 14:19:35.000000000 +0200
@@ -1430,7 +1430,11 @@
 		e100_set_duplex(dev, ecmd->duplex == DUPLEX_HALF ? half : full);
 		e100_set_speed(dev, ecmd->speed == SPEED_10 ? 10: 100);
 	}
-
+        //Honor ethtool phy address parameter, which was simply unused before this.
+        if (ecmd->phy_address <= 31)
+                mdio_phy_addr = ecmd->phy_address;
+        else if (ecmd->phy_address == 32)
+                return -ENODEV;
 	return 0;
 }