Received: with ECARTIS (v1.0.0; list netdev); Fri, 11 Feb 2005 13:19:50 -0800 (PST) Received: from e32.co.us.ibm.com (e32.co.us.ibm.com [32.97.110.130]) by oss.sgi.com (8.13.0/8.13.0) with ESMTP id j1BLJkTV029887 for ; Fri, 11 Feb 2005 13:19:46 -0800 Received: from westrelay02.boulder.ibm.com (westrelay02.boulder.ibm.com [9.17.195.11]) by e32.co.us.ibm.com (8.12.10/8.12.9) with ESMTP id j1BLJeuA021794 for ; Fri, 11 Feb 2005 16:19:40 -0500 Received: from d03av02.boulder.ibm.com (d03av02.boulder.ibm.com [9.17.195.168]) by westrelay02.boulder.ibm.com (8.12.10/NCO/VER6.6) with ESMTP id j1BLJeHb414608 for ; Fri, 11 Feb 2005 14:19:40 -0700 Received: from d03av02.boulder.ibm.com (loopback [127.0.0.1]) by d03av02.boulder.ibm.com (8.12.11/8.12.11) with ESMTP id j1BLJdns006890 for ; Fri, 11 Feb 2005 14:19:40 -0700 Received: from cog.beaverton.ibm.com (cog.beaverton.ibm.com [9.47.21.16]) by d03av02.boulder.ibm.com (8.12.11/8.12.11) with ESMTP id j1BLJcd8006832; Fri, 11 Feb 2005 14:19:39 -0700 Subject: Re: [PATCH] tg3: capacitive coupling detection fix From: john stultz To: Michael Chan Cc: "David S. Miller" , netdev@oss.sgi.com, lkml , Chris McDermott , keith maanthey In-Reply-To: References: Content-Type: text/plain Date: Fri, 11 Feb 2005 13:19:37 -0800 Message-Id: <1108156778.11683.212.camel@cog.beaverton.ibm.com> Mime-Version: 1.0 X-Mailer: Evolution 2.0.2 (2.0.2-3) Content-Transfer-Encoding: 7bit X-Virus-Scanned: ClamAV version 0.81, clamav-milter version 0.81b on oss.sgi.com X-Virus-Status: Clean X-archive-position: 1534 X-ecartis-version: Ecartis v1.0.0 Sender: netdev-bounce@oss.sgi.com Errors-to: netdev-bounce@oss.sgi.com X-original-sender: johnstul@us.ibm.com Precedence: bulk X-list: netdev Content-Length: 470 Lines: 15 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