From: "Thomas Dörfler" <Thomas.Doerfler@embedded-brains.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Bugfix: mpc512x_fec MII accesses fail after ethernet transfers
Date: Sun, 19 Jul 2009 19:51:27 +0200 [thread overview]
Message-ID: <4A635D1F.9030402@embedded-brains.de> (raw)
In-Reply-To: <20090719172209.83C95832E416@gemini.denx.de>
Hello,
on our custom MPC5121 board I used the following sequence to see the bug:
- After U-Boot stated, I could inspect the status of the PHY with "mii
info". This works fine even if repeated.
- then I trigger some network activity, for example with "ping"
- if I try to get the PHY status with "mii info" AFTER that network
activity, the command fails (it does not even detect, that a PHY is
connected).
The reason I could track down is, that:
- the FEC driver performs a FEC reset before the initial packet is
transmitted
- this reset also clears the MII speed register, which defines the
communication speed on MDC/MDIO
- the PHY access works again, when the MII speed register is set to its
proper value.
I have tested this behaviour on a old (M34K) mask, but I am sure it
applies also to the current M36P.
wkr,
Thomas Doerfler.
Wolfgang Denk wrote:
> Dear =?ISO-8859-15?Q?Thomas_D=F6rfler?=,
>
> In message <4A633D5A.3060803@embedded-brains.de> you wrote:
>> The mpc512x FEC network driver resets the FEC before transferring
>> packets. With this reset, the FEC clears the previous setting of MII
>> transfer speed. This patch ensures, that it is set again to a reasonable
>> value after a FEC reset.
>
> Hmmm... what is the exact effect of the bug you are trying to fix
> here? Do you have any example how to trigger erroneous behaviour?
>
> Best regards,
>
> Wolfgang Denk
>
--
--------------------------------------------
Embedded Brains GmbH
Thomas Doerfler Obere Lagerstrasse 30
D-82178 Puchheim Germany
email: Thomas.Doerfler at embedded-brains.de
Phone: +49-89-18908079-2
Fax: +49-89-18908079-9
next prev parent reply other threads:[~2009-07-19 17:51 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-19 15:35 [U-Boot] [PATCH] Bugfix: mpc512x_fec MII accesses fail after ethernet transfers Thomas Dörfler
2009-07-19 17:22 ` Wolfgang Denk
2009-07-19 17:51 ` Thomas Dörfler [this message]
2009-07-19 18:50 ` Wolfgang Denk
2009-07-19 19:26 ` Wolfgang Denk
2009-07-19 19:59 ` Thomas Dörfler
2009-07-23 19:29 ` Wolfgang Denk
2009-07-21 6:53 ` Ben Warren
2009-07-21 7:44 ` Wolfgang Denk
2009-07-21 7:46 ` Ben Warren
2009-07-21 7:48 ` Ben Warren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A635D1F.9030402@embedded-brains.de \
--to=thomas.doerfler@embedded-brains.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox