2005-02-11 20:45:20

by Michael Chan

[permalink] [raw]
Subject: [PATCH] tg3: capacitive coupling detection fix

This patch fixes the problem reported in:

http://marc.theaimsgroup.com/?l=linux-kernel&m=110798711911645&w=2


The 5700 link problem was caused by reading uninitialized values in sram and
causing capacitive coupling mode to be enabled by mistake. This patch fixes
the problem by properly validating the sram contents.


Signed-off-by: Michael Chan <[email protected]>


Attachments:
tg3_cap_cpling.patch (1.28 kB)
tg3_cap_cpling.patch

2005-02-11 21:16:15

by David Miller

[permalink] [raw]
Subject: Re: [PATCH] tg3: capacitive coupling detection fix

On Fri, 11 Feb 2005 12:44:10 -0800
"Michael Chan" <[email protected]> wrote:

> This patch fixes the problem reported in:
>
> http://marc.theaimsgroup.com/?l=linux-kernel&m=110798711911645&w=2
>
> The 5700 link problem was caused by reading uninitialized values in sram and
> causing capacitive coupling mode to be enabled by mistake. This patch fixes
> the problem by properly validating the sram contents.
>
> Signed-off-by: Michael Chan <[email protected]>

Applied, thanks a lot Michael.

2005-02-11 21:20:07

by john stultz

[permalink] [raw]
Subject: Re: [PATCH] tg3: capacitive coupling detection fix

On Fri, 2005-02-11 at 12:44 -0800, Michael Chan wrote:
> This patch fixes the problem reported in:
>
> http://marc.theaimsgroup.com/?l=linux-kernel&m=110798711911645&w=2
>
>
> The 5700 link problem was caused by reading uninitialized values in sram and
> causing capacitive coupling mode to be enabled by mistake. This patch fixes
> the problem by properly validating the sram contents.

Verified as working on the x440 I was seeing the trouble with.

Thanks!
-john