From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.dvmed.net (srv5.dvmed.net [207.36.208.214]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 1AAFCDDF01 for ; Sun, 13 Apr 2008 06:28:46 +1000 (EST) Message-ID: <48011B70.1020102@garzik.org> Date: Sat, 12 Apr 2008 16:28:32 -0400 From: Jeff Garzik MIME-Version: 1.0 To: Josh Boyer Subject: Re: [PATCH 1/2] ibm_newemac: PowerPC 440GX EMAC PHY clock workaround References: <20080327144044.GA8831@ru.mvista.com> <47EDA6F1.9080206@garzik.org> <20080411092417.57825115@zod.rchland.ibm.com> In-Reply-To: <20080411092417.57825115@zod.rchland.ibm.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Cc: linuxppc-dev@ozlabs.org, netdev@vger.kernel.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Josh Boyer wrote: > On Fri, 28 Mar 2008 22:18:25 -0400 > Jeff Garzik wrote: > >> Valentine Barshak wrote: >>> The PowerPC 440GX Taishan board fails to reset EMAC3 (reset timeout error) >>> if there's no link. Because of that it fails to find PHY chip. The older ibm_emac >>> driver had a workaround for that: the EMAC_CLK_INTERNAL/EMAC_CLK_EXTERNAL macros, >>> which toggle the Ethernet Clock Select bit in the SDR0_MFR register. This patch >>> does the same for "ibm,emac-440gx" compatible chips. The workaround forces >>> clock on -all- EMACs, so we select clock under global emac_phy_map_lock. >>> >>> Signed-off-by: Valentine Barshak >>> --- >>> drivers/net/ibm_newemac/core.c | 16 +++++++++++++++- >>> drivers/net/ibm_newemac/core.h | 8 ++++++-- >>> 2 files changed, 21 insertions(+), 3 deletions(-) >> is this for 2.6.25-rc? > > Jeff, can I get an ack from you on this patch, and patch 2 in this > set? They depend on a patch in my tree and I'd like to include them in > my next push to Paul for 2.6.26. ACK I had queried the status of these patches, and didn't receive any reply initially from my query...