[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-ia64
Subject: Re: ethtool -d MCAs rx2600
From: Jack Steiner <steiner () sgi ! com>
Date: 2004-01-29 20:41:08
Message-ID: 20040129204108.GC6997 () sgi ! com
[Download RAW message or body]
On Thu, Jan 29, 2004 at 11:18:37AM -0800, Grant Grundler wrote:
> On Mon, Jan 26, 2004 at 12:39:42PM -0500, Jeff Garzik wrote:
> > I get lockups occasionally on x86 too, but have had higher priority
> > things to look at. Since regdump is mainly an engineer's tool, we felt
> > it was a "use at your own risk" feature.
> >
> > But if we can fix it, all the better.
>
> tg3_get_regs() is reading registers that don't exist.
> Neither HPUX nor tru64 drivers attempt to touch NVRAM on BCM5700/1 chips.
> And tg3 in most other places doesn't either.
> It just needs to check TG3_FLAG_NVRAM before reading NVRAM regs.
>
> Jack, you also using the bcm5701 chip?
Yes. Tigon3 [rev 0105 PHY(5701)] (PCI:66MHz:64-bit
We'll apply the patch.
Thanks....
>
> Jeff, please apply. Following patch is against 2.6.2-rc2.
>
> thanks,
> grant
>
> ===== drivers/net/tg3.c 1.81 vs edited =====
> --- 1.81/drivers/net/tg3.c Wed Dec 31 23:40:32 2003
> +++ edited/drivers/net/tg3.c Thu Jan 29 10:19:46 2004
> @@ -5904,7 +5904,9 @@
> GET_REG32_LOOP(MSGINT_MODE, 0x0c);
> GET_REG32_1(DMAC_MODE);
> GET_REG32_LOOP(GRC_MODE, 0x4c);
> - GET_REG32_LOOP(NVRAM_CMD, 0x24);
> + if (tp->tg3_flags & TG3_FLAG_NVRAM) {
> + GET_REG32_LOOP(NVRAM_CMD, 0x24);
> + }
>
> #undef __GET_REG32
> #undef GET_REG32_LOOP
> -
> To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Thanks
Jack Steiner (steiner@sgi.com) 651-683-5302
Principal Engineer SGI - Silicon Graphics, Inc.
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic