From: Ben Warren <biggerbadderben@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] MII / RMII
Date: Tue, 08 Jan 2008 11:27:45 -0500 [thread overview]
Message-ID: <4783A481.1040004@gmail.com> (raw)
In-Reply-To: <200801080543.18095.sr@denx.de>
Stefan Roese wrote:
> On Tuesday 08 January 2008, Andy Fleming wrote:
>
>>> I think that if you use an Ethernet and need to differentiate
>>> between MII and RMII they should be mutually exclusive.
>>> You either define CONFIG_RMII OR CONFIG_MII but not both.
>>> If you have a PHY on the chip, then you do not define any of the two.
>>> MII uses more pins and maybe different pins, so a port for a CPU
>>> should differentiate.
>>>
>> I'm a little late to this, but I feel I should make sure people are aware
>> that CONFIG_MII and CONFIG_RMII are not, in this case, two different
>> options for the same concept. CONFIG_MII is the option you enable to allow
>> MII Management operations to be done. It should *probably* be called
>> CONFIG_MDIO.
>>
>
> Yes, this would be good. Anyone interested in fixing up a patch for this?
>
I think it's a noble idea, but probably a can of worms in practice. The
use of the term 'MII' is rampant, for example we have 'mii' commands,
and 'bbmii' commands. These both really concern MDIO, but I doubt we
want to change them.
The real culprit is RMII. As Andy has pointed out, RMII is a data plane
alternative to MII, but shares the MDIO control plane. RMII is a
hardware configuration, and as such is 'set and forget' and highly
implementation-specific. Having a global CONFIG_RMII makes no sense. If
anything, CONFIG_RMII should be changed to stuff like CONFIG_TSEC_RMII_2
etc. (a quickly made-up example with no grounding in reality), where the
hardware binding is obvious.
regards,
Ben
prev parent reply other threads:[~2008-01-08 16:27 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-29 18:51 [U-Boot-Users] MII / RMII Guennadi Liakhovetski
2007-11-29 20:31 ` Ben Warren
2007-11-29 22:25 ` Haavard Skinnemoen
2007-11-29 22:25 ` Ulf Samuelsson
2007-11-29 22:55 ` Ben Warren
2007-11-29 23:45 ` Ulf Samuelsson
2008-01-08 1:35 ` Andy Fleming
2008-01-08 4:43 ` Stefan Roese
2008-01-08 16:27 ` Ben Warren [this message]
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=4783A481.1040004@gmail.com \
--to=biggerbadderben@gmail.com \
--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.