From: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
To: Brian Lilly <brian@crystalfontz.com>
Cc: "David S. Miller" <davem@davemloft.net>,
Fabio Estevam <fabio.estevam@freescale.com>,
Jim Baxter <jim_baxter@mentor.com>,
Frank Li <Frank.Li@freescale.com>,
Fugang Duan <B38611@freescale.com>,
netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
kernel@pengutronix.de
Subject: Re: i.MX28 based system losing eth0 on boot
Date: Tue, 6 May 2014 20:11:51 +0200 [thread overview]
Message-ID: <20140506181151.GU28564@pengutronix.de> (raw)
In-Reply-To: <CAOk88BLR=nwNGdT++39vLdOt=TwZmemF-trSQPY8AdMN9tKtKQ@mail.gmail.com>
Hello Brian,
On Tue, May 06, 2014 at 09:44:34AM -0700, Brian Lilly wrote:
> With commit a264b981f2c76e281ef27e7232774bf6c54ec865 we're having eth0
> come up, then brought right back down with an MDIO rx timeout moments
> after. Adding back in the removed code keeps the interface alive and
> it's working afterward without trouble. I've tested the re-inserted
> code in 3.12, 3.14 without issue on our boards.
So you can reliably trigger that problem? You're just doing
ifconfig eth0 1.2.3.4 up
(or equivalent) and the interface goes down without further
interference with the above mentioned commit? The exact error you're
seeing is
MDIO read timeout
(with some prefix saying something about fec and eth0 I think)?
This error is also present with a264b981f2 reverted, just doesn't affect
eth0 being functional? Does the timeout always happen, or only on
specific addresses?
This is not a proper fix, but does it help to increment FEC_MII_TIMEOUT?
> Is there something else that can be done to prevent the MDIO timeouts?
> We are using basically the same schematic for networking as the
> imx28evk.
Hard to say, but assuming it works just fine on the imx28evk for you,
too, there seems to be some hardware difference that makes your machine
fail. (That doesn't mean it's not fixable in software.)
I don't know if a mdio read error is intended to make the device go
down, maybe one the the netdev guys can answer that.
Assuming that it's not intended, instrument the code, find out how that
timeout makes your device go down and find the wrong branch. I'd start
with adding stackdumps when the mdio timeout happens and when
fec_enet_start_xmit is called with fep->link == 0.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
next prev parent reply other threads:[~2014-05-06 18:11 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-06 16:44 i.MX28 based system losing eth0 on boot Brian Lilly
2014-05-06 18:11 ` Uwe Kleine-König [this message]
2014-05-06 18:39 ` Florian Fainelli
2014-05-06 19:12 ` Brian Lilly
2014-05-06 19:24 ` Florian Fainelli
2014-05-06 21:40 ` Brian Lilly
2014-05-06 22:06 ` Florian Fainelli
2014-05-06 22:27 ` Brian Lilly
2014-05-07 3:07 ` Florian Fainelli
2014-05-07 19:16 ` Brian Lilly
2014-05-07 19:34 ` Florian Fainelli
2014-05-07 19:51 ` Brian Lilly
2014-05-08 1:47 ` fugang.duan
2014-05-07 3:17 ` Fabio Estevam
2014-05-07 19:00 ` Brian Lilly
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=20140506181151.GU28564@pengutronix.de \
--to=u.kleine-koenig@pengutronix.de \
--cc=B38611@freescale.com \
--cc=Frank.Li@freescale.com \
--cc=brian@crystalfontz.com \
--cc=davem@davemloft.net \
--cc=fabio.estevam@freescale.com \
--cc=jim_baxter@mentor.com \
--cc=kernel@pengutronix.de \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
/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;
as well as URLs for NNTP newsgroup(s).