From: Gerlando Falauto <gerlando.falauto@keymile.com>
To: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Cc: Grant Likely <grant.likely@secretlab.ca>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
"Brunck, Holger" <Holger.Brunck@keymile.com>,
Florian Fainelli <f.fainelli@gmail.com>
Subject: Disabling autoneg and enforcing speed/duplex
Date: Thu, 06 Feb 2014 13:00:43 +0100 [thread overview]
Message-ID: <52F3796B.8050809@keymile.com> (raw)
Hi,
I'm using the Kirkwood Ethernet controller (mv643xx_eth.c) with a
Marvell 88E3018 PHY which needs to be set in forced 100Base-TX mode.
Thanks to Sebastian's addition of DT support to the ethernet driver, I
can easily set speed and duplex within the ethernet's port node,
therefore leaving the phy unmanaged -- this works fine (I guess this
mode is set on the phy by the bootloader or by strap settings).
However, this PHY has an erratum whose workaround requires some
registers be written -- I believe the natural solution would be to start
managing the PHY (i.e. set "phy-handle") and implement the proper
workaround within drivers/net/phy/marvell.c. Making the PHY managed does
however enable autoneg and therefore break everything.
Which brings me to my question: shouldn't there be a way to specify some
forced settings within the PHY's node for such cases?
Only thing I found vaguely resembling what I'm looking for is Florian's
patch introducing "max-speed" in the PHY -- not quite the same thing
though. Which, if I understand it correctly, implements it as a property
of the PHY, whereas ePAPR specifies it as a property of the ethernet
device (which makes sense, since you might want to connect a 10/100 MII
to a 10/100/1000 PHY and therefore have the MII restrict the
capabilities of the PHY).
Or perhaps I'm missing some important bits here?
Thanks!
Gerlando
next reply other threads:[~2014-02-06 12:00 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-06 12:00 Gerlando Falauto [this message]
[not found] ` <52F3796B.8050809-SkAbAL50j+5BDgjK7y7TUQ@public.gmane.org>
2014-02-06 14:50 ` Disabling autoneg and enforcing speed/duplex Rob Herring
2014-02-06 15:00 ` Gerlando Falauto
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=52F3796B.8050809@keymile.com \
--to=gerlando.falauto@keymile.com \
--cc=Holger.Brunck@keymile.com \
--cc=devicetree@vger.kernel.org \
--cc=f.fainelli@gmail.com \
--cc=grant.likely@secretlab.ca \
--cc=netdev@vger.kernel.org \
--cc=sebastian.hesselbarth@gmail.com \
/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).