From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:6f8:1178:4:290:27ff:fe1d:cc33]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1TgDMN-0008SK-40 for barebox@lists.infradead.org; Wed, 05 Dec 2012 11:41:51 +0000 Date: Wed, 5 Dec 2012 12:41:49 +0100 From: Sascha Hauer Message-ID: <20121205114149.GF10369@pengutronix.de> References: <1354706925-13432-1-git-send-email-w.sang@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1354706925-13432-1-git-send-email-w.sang@pengutronix.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: barebox-bounces@lists.infradead.org Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH] mxs: get_fecclk is hclk, not a constant To: Wolfram Sang Cc: barebox@lists.infradead.org, jbe@pengutronix.de On Wed, Dec 05, 2012 at 12:28:45PM +0100, Wolfram Sang wrote: > The rate is not constant as the comment said, but is hclk. The result > was that MII clock was often calculated wrong. Some more prosa would be good here: The fec has multiple clock inputs: - 50MHz clock for generating the (R)MII clock - bus clock The MDIO clock is derived from the bus clock, not the 50MHz clock, so pass this into the driver so that it can correctly configure the MDIO clock divider. Sascha > > Reported-by: Michael Grzeschik > Signed-off-by: Wolfram Sang > --- > arch/arm/mach-mxs/speed-imx28.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/arch/arm/mach-mxs/speed-imx28.c b/arch/arm/mach-mxs/speed-imx28.c > index df55f64..2cab42d 100644 > --- a/arch/arm/mach-mxs/speed-imx28.c > +++ b/arch/arm/mach-mxs/speed-imx28.c > @@ -121,8 +121,7 @@ unsigned imx_get_xtalclk(void) > > unsigned imx_get_fecclk(void) > { > - /* this PLL always runs at 50 MHz */ > - return 50000000; > + return imx_get_hclk(); > } > > > -- > 1.7.10.4 > > > _______________________________________________ > barebox mailing list > barebox@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/barebox > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox