All of lore.kernel.org
 help / color / mirror / Atom feed
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

      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.