I noticed the latest BSP from Freescale has this patch:<br><br>From: Chen Hongjun <<a href="mailto:Hong-jun.chen@freecale.com">Hong-jun.chen@freecale.com</a>><br>Date: Thu, 16 Apr 2009 20:22:52 +0800<br>Subject: [PATCH] Fixed FEC bug for bluestone board.<br>
<br>Signed-off-by: Chen Hongjun <<a href="mailto:Hong-jun.chen@freecale.com">Hong-jun.chen@freecale.com</a>><br>---<br> drivers/net/fs_enet/mii-fec.c | 2 +-<br> 1 files changed, 1 insertions(+), 1 deletions(-)<br>
<br>diff --git a/drivers/net/fs_enet/mii-fec.c b/drivers/net/fs_enet/mii-fec.c<br>index 13a7d66..53d01a8 100644<br>--- a/drivers/net/fs_enet/mii-fec.c<br>+++ b/drivers/net/fs_enet/mii-fec.c<br>@@ -208,7 +208,7 @@ static int __devinit fs_enet_mdio_probe(struct of_device *ofdev,<br>
if (!fec->fecp)<br> goto out_fec;<br> <br>- fec->mii_speed = ((ppc_proc_freq + 4999999) / 5000000) << 1;<br>+ fec->mii_speed = ((((ppc_proc_freq + 4999999) / 2500000) / 2) & 0x3F) << 1;<br>
<br> setbits32(&fec->fecp->fec_r_cntrl, FEC_RCNTRL_MII_MODE);<br> setbits32(&fec->fecp->fec_ecntrl, FEC_ECNTRL_PINMUX |<br>-- <br>1.5.4<br><br><br><div class="gmail_quote">On Sat, Jun 6, 2009 at 4:16 PM, Wolfgang Denk <span dir="ltr"><<a href="mailto:wd@denx.de">wd@denx.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Dear John,<br>
<div class="im"><br>
In message <<a href="mailto:4b73d43f0905071909v6e6e8b2el9eb6d4a1b9038f45@mail.gmail.com">4b73d43f0905071909v6e6e8b2el9eb6d4a1b9038f45@mail.gmail.com</a>> you wrote:<br>
><br>
> I think the fec's parent clock is the ipb clock not the ppc core clock.<br>
> Could that be the problem?<br>
<br>
</div>I don't think so.<br>
<br>
When debugging, I printed the actual clock frequencies, and they<br>
looked as expected. And "arch/powerpc/platforms/512x/clock.c" has<br>
this:<br>
<br>
385 static struct clk fec_clk = {<br>
386 .name = "fec_clk",<br>
387 .flags = CLK_HAS_CTRL,<br>
388 .reg = 0,<br>
389 .bit = 13,<br>
390 .calc = unity_clk_calc,<br>
391 .parent = &ips_clk,<br>
392 };<br>
<br>
which looks OK to me.<br>
<div class="im"><br>
Best regards,<br>
<br>
Wolfgang Denk<br>
<br>
--<br>
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel<br>
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany<br>
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: <a href="mailto:wd@denx.de">wd@denx.de</a><br>
</div>The human mind ordinarily operates at only ten percent of its<br>
capacity. The rest is overhead for the operating system.<br>
</blockquote></div><br>