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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.