mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: Renaud Barbier <renaud.barbier@ge.com>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 0/1 v7] net: introduce phylib
Date: Tue, 2 Oct 2012 18:04:21 +0200	[thread overview]
Message-ID: <20121002160421.GA26553@game.jcrosoft.org> (raw)
In-Reply-To: <506AFDEB.4010805@ge.com>

On 15:44 Tue 02 Oct     , Renaud Barbier wrote:
> On 24/09/12 10:31, Jean-Christophe PLAGNIOL-VILLARD wrote:
> 
> > 
> > Adapt phylib from linux
> > 
> > This will allow to have
> >  - phy drivers
> >  - to only connect the phy at then opening of the device
> >  - if the phy is not ready or down fail on open
> > 
> > Same behaviour as in linux and will allow to share code and simplify porting.
> > 
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> > 
> > ----------------------------------------------------------------
> > Jean-Christophe PLAGNIOL-VILLARD (1):
> >       net: introduce phylib
> > 
> >  arch/arm/boards/freescale-mx6-arm2/board.c      |   43 ++---
> >  arch/arm/boards/freescale-mx6-sabrelite/board.c |   40 ++--
> >  arch/arm/configs/phycard_a_l1_defconfig         |    2 +-
> >  arch/arm/configs/tx51stk5_defconfig             |    2 +-
> >  arch/ppc/boards/pcm030/pcm030.c                 |    4 +-
> >  arch/ppc/mach-mpc5xxx/include/mach/fec.h        |   14 --
> >  drivers/net/Kconfig                             |   30 +--
> >  drivers/net/Makefile                            |    2 +-
> >  drivers/net/altera_tse.c                        |   48 +++--
> >  drivers/net/altera_tse.h                        |    3 +-
> >  drivers/net/at91_ether.c                        |   49 ++---
> >  drivers/net/designware.c                        |   62 ++++---
> >  drivers/net/dm9k.c                              |   50 +++--
> >  drivers/net/ep93xx.c                            |   35 ++--
> >  drivers/net/ep93xx.h                            |    2 +-
> >  drivers/net/fec_imx.c                           |   89 +++++----
> >  drivers/net/fec_imx.h                           |    6 +-
> >  drivers/net/fec_mpc5200.c                       |   58 +++---
> >  drivers/net/fec_mpc5200.h                       |    4 +-
> >  drivers/net/gianfar.c                           |   61 +++---
> 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

Best Regards,
J.

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

  reply	other threads:[~2012-10-02 16:06 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 [this message]
2012-10-02 16:58     ` Sascha Hauer
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=20121002160421.GA26553@game.jcrosoft.org \
    --to=plagnioj@jcrosoft.com \
    --cc=barebox@lists.infradead.org \
    --cc=renaud.barbier@ge.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