Re: SUNGEM config

[Jens Schmalzing <jens.schmalzing@physik.uni-muenchen.de>]

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

Hi,

> I know there are some laptop specific tools that monitor the link
> and do the shutdown of the interface when there is no link but I
> don't have any precise pointer at hand.

On Debian systems, this is done by a small daemon named ifd in the
laptop-net package.

Unfortunately, it uses the SIOCGMIIPHY and SIOCGMIIREG ioctls for link
detection, which on my Pismo reproducibly lead to machine check
oopses.  I've managed to track down the problem, a patch is included
below.  Basically, instead of really reading the registers it fakes
and returns a register setting from the cached status of the device.
Right now, it is fairly ugly, since it only works for that relevant
one bit.

Regards, Jens.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Patch to SUNGEM driver for GMAC --]
[-- Type: text/x-patch, Size: 662 bytes --]

--- linux-benh/drivers/net/sungem.c	Wed Feb  6 20:01:22 2002
+++ linux/drivers/net/sungem.c	Thu Feb 14 01:37:41 2002
@@ -2553,11 +2540,16 @@
 		break;

 	case SIOCGMIIPHY:		/* Get address of MII PHY in use. */
+	case SIOCDEVPRIVATE:		/* for binary compat, remove in 2.5 */
 		data->phy_id = gp->mii_phy_addr;
 		/* Fallthrough... */

 	case SIOCGMIIREG:		/* Read MII PHY register. */
-		data->val_out = __phy_read(gp, data->reg_num & 0x1f, data->phy_id & 0x1f);
+	case SIOCDEVPRIVATE+1:		/* for binary compat, remove in 2.5 */
+		if(data->reg_num==1 && gp->lstate==link_up)
+			data->val_out = PCS_MIISTAT_LS;
+		else
+			data->val_out = 0;
 		rc = 0;
 		break;

[-- Attachment #3: Type: text/plain, Size: 196 bytes --]


P.S.: I forget whether to include or to attach patches.  My apologies
if I have done it incorrectly.

--
J'qbpbe, le m'en fquz pe j'qbpbe!
Le veux aimeb et mqubib panz je pézqbpbe je djuz tqtaj!