mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 0/1 v7] net: introduce phylib
Date: Tue, 2 Oct 2012 18:58:47 +0200	[thread overview]
Message-ID: <20121002165847.GA1322@pengutronix.de> (raw)
In-Reply-To: <20121002160421.GA26553@game.jcrosoft.org>

On Tue, Oct 02, 2012 at 06:04:21PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > The mdiobus_scan performed when registering the mdio bus at probe time
> > has an impact on boot time. I have a 85xx board whose PHY needs 450ms to
> > be ready for access.
> > 
> > I will send a patch for the gianfar driver to register the mdio bus only
> > when the device is open.
> That why I did at the first open before
> 
> and I guess your case is not the only one
> 
> I do not like the idea to register the bus later
> 
> we need to have defer probe

Yes, that's probably the best. Normally, when a phy address is given, we
do not have to scan the bus at all, we only have to do it when -1 is
given as phy address.

Another idea would be to pass a bitmask for valid phy addresses rather
than a number. Maybe some board has a phy at a known address and another
one at some to be detected address. Then for one phy the mask would be
0x00010000 and for the other 0xfffeffff. Given that this would be more
effort to change all the platform data we can do this should the need
arise.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

  reply	other threads:[~2012-10-02 16:58 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-24  9:31 Jean-Christophe PLAGNIOL-VILLARD
2012-09-24  9:36 ` [PATCH 1/1] " Jean-Christophe PLAGNIOL-VILLARD
2012-09-24 19:20 ` [PATCH 0/1 v7] " Sascha Hauer
2012-09-24 20:14   ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-25 17:26 ` Jan Lübbe
2012-09-25 18:19   ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-26 11:02 ` Sascha Hauer
2012-09-26 11:20   ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-26 11:31     ` Sascha Hauer
2012-09-26 11:36       ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-26 11:37   ` [PATCH 1/2] calao: at91 qil/tny/usb auto detect the phy addr Jean-Christophe PLAGNIOL-VILLARD
2012-09-26 11:37     ` [PATCH 2/2] net/eth: fix missing check on eth active Jean-Christophe PLAGNIOL-VILLARD
2012-09-26 11:53     ` [PATCH 3/3] net/eth: fix link handling Jean-Christophe PLAGNIOL-VILLARD
2012-09-26 12:43       ` Sascha Hauer
2012-09-26 12:50         ` Jan Lübbe
2012-09-26 13:03           ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-26 13:08             ` Sascha Hauer
2012-09-26 13:43               ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-26 13:05         ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-26 13:59 ` [PATCH 1/1] " Jean-Christophe PLAGNIOL-VILLARD
2012-09-26 15:17   ` Sascha Hauer
2012-09-26 15:51     ` Jean-Christophe PLAGNIOL-VILLARD
2012-10-02 14:44 ` [PATCH 0/1 v7] net: introduce phylib Renaud Barbier
2012-10-02 16:04   ` Jean-Christophe PLAGNIOL-VILLARD
2012-10-02 16:58     ` Sascha Hauer [this message]
2012-10-02 18:32       ` Jean-Christophe PLAGNIOL-VILLARD

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=20121002165847.GA1322@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=plagnioj@jcrosoft.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