All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@pobox.com>
To: Netdev <netdev@oss.sgi.com>
Cc: David Woodhouse <dwmw2@infradead.org>,
	Kumar Gala <kumar.gala@freescale.com>, jamal <hadi@cyberus.ca>
Subject: Phy layer notes (was Re: [RFR] gianfar ethernet driver)
Date: Fri, 09 Jul 2004 12:47:08 -0400	[thread overview]
Message-ID: <40EECC0C.9040304@pobox.com> (raw)
In-Reply-To: <1089375760.28614.1365.camel@hades.cambridge.redhat.com>

David Woodhouse wrote:
> Hmmm. 
> 
> eth0: Running with NAPI disabled
> eth0: 64/64 RX/TX BD ring size
> eth1: Gianfar Ethernet Controller Version 1.0, 05:e6:9e:c0:05:e6
> eth1: Running with NAPI disabled
> eth1: 64/64 RX/TX BD ring size      
> eth0: PHY id 2060e1 is not supported!
> eth0: No PHY found
> IP-Config: Failed to open eth0
> IP-Config: Device `eth0' not found
> 
> The PHY is a BCM5421S. Does it really have to give up completely? Isn't
> there a subset of common MII support it could use?
> 
> Is it expected that every NIC driver will include its own version of
> support for the PHYs which have actually been seen paired with that NIC,
> or is there some more generic support planned?


David W and I discussed some of this on IRC.

1) most gige phys can be treated with generic GMII code.  unfortunately 
experience shows that phy init can often involve phy-specific magic 
sequences.

2) we do need a generic phy layer, and I'm looking for volunteers who 
want to prototype a nice, small, compact one.  BenH has a nice template 
in sungem_phy.c.

3) drivers/net/mii.c and include/linux/mii.h need GMII code and 
constants.  Feel free to add.

4) generic TBI code would be nice, too.  (TBI is a standard gige fibre 
interface)

5) sometimes a MAC+phy pairing is unique enough that you need 
MAC-specific support for a particular phy.  For example, one might wind 
up with two phy drivers, one a generic Broadcom GMII phy driver (for use 
by any NIC driver), and one a tg3-specific Broadcom GMII phy driver.

      parent reply	other threads:[~2004-07-09 16:47 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <8F52CF1D-C916-11D8-BB6A-000393DBC2E8@freescale.com>
2004-07-05 17:28 ` [RFR] gianfar ethernet driver Jeff Garzik
2004-07-06  2:38   ` jamal
     [not found]   ` <20040708231131.GA20305@infradead.org>
2004-07-08 23:25     ` Jeff Garzik
2004-07-08 23:35       ` Christoph Hellwig
     [not found]   ` <1089375760.28614.1365.camel@hades.cambridge.redhat.com>
2004-07-09 16:47     ` Jeff Garzik [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=40EECC0C.9040304@pobox.com \
    --to=jgarzik@pobox.com \
    --cc=dwmw2@infradead.org \
    --cc=hadi@cyberus.ca \
    --cc=kumar.gala@freescale.com \
    --cc=netdev@oss.sgi.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 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.