From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Tue, 11 Feb 2014 10:51:18 +0100 Subject: [U-Boot] [PATCH] NET: fec_mxc: fix MDIO clock prescaler calculation In-Reply-To: <1391594051-4947-1-git-send-email-list-09_u-boot@tqsc.de> References: <1391594051-4947-1-git-send-email-list-09_u-boot@tqsc.de> Message-ID: <52F9F296.1090108@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Marcus, On 05/02/2014 10:54, Markus Niebel wrote: > From: Markus Niebel > > The prescaler value for MDC is calculated wrong for > MX6S. Implement the same logic as linux here which handles > ENET_MAC different then original FEC. > > Tested on a custom board with i.MX6S and 100MBit PHY > > Signed-off-by: Markus Niebel > --- > drivers/net/fec_mxc.c | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c > index 3b2b995..4cefda4 100644 > --- a/drivers/net/fec_mxc.c > +++ b/drivers/net/fec_mxc.c > @@ -128,8 +128,12 @@ static void fec_mii_setspeed(struct ethernet_regs *eth) > * Set MII_SPEED = (1/(mii_speed * 2)) * System Clock > * and do not drop the Preamble. > */ > - writel((((imx_get_fecclk() / 1000000) + 2) / 5) << 1, > - ð->mii_speed); > + register u32 speed = DIV_ROUND_UP(imx_get_fecclk(), 5000000); > +#ifdef FEC_QUIRK_ENET_MAC > + speed--; > +#endif > + speed <<= 1; > + writel(speed, ð->mii_speed); > debug("%s: mii_speed %08x\n", __func__, readl(ð->mii_speed)); > } Thanks for this - I go for merging.. Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================