From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 1/1] net: introduce phylib
Date: Sat, 8 Sep 2012 19:28:44 +0200 [thread overview]
Message-ID: <20120908172844.GF20330@game.jcrosoft.org> (raw)
In-Reply-To: <20120908150243.GF18243@pengutronix.de>
On 17:02 Sat 08 Sep , Sascha Hauer wrote:
> On Fri, Sep 07, 2012 at 12:26:06PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > Adapt phylib from linux
> >
> > switch all the driver to it
> >
> > This will allow to have phy drivers and to only connect the phy at then
> > opening of the device. And if the phy is not ready fail on open.
> >
> > This is needed by the boot sequence support to do not endup in a loop or spend
> > time on bootp when no ethernet cable is connected.
>
> We do not 'need' this for the above reasons, we could also fix the
> current code. The above may be an advantage of your code, but the major
> selling point is code sharing with Linux (at least that's what you told
> me)
yes the major point is to touch the phy only when the ethernet is opening
yes the commit is true but it's not the major selling point
Best Regards,
J.
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2012-09-08 17:28 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-07 10:26 [PATCH 1/1] net: introduce phylib Jean-Christophe PLAGNIOL-VILLARD
2012-09-08 15:02 ` Sascha Hauer
2012-09-08 17:28 ` Jean-Christophe PLAGNIOL-VILLARD [this message]
-- strict thread matches above, loose matches on Subject: below --
2012-09-16 12:42 [PATCH 0/3 v4] " Jean-Christophe PLAGNIOL-VILLARD
2012-09-16 13:45 ` [PATCH 1/1] " Jean-Christophe PLAGNIOL-VILLARD
2012-09-16 18:07 ` Sascha Hauer
2012-09-17 5:23 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-16 17:54 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-16 21:01 ` Sascha Hauer
2012-09-17 5:20 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-17 5:43 [PATCH 0/1 v5] " Jean-Christophe PLAGNIOL-VILLARD
2012-09-17 5:59 ` [PATCH 1/1] " Jean-Christophe PLAGNIOL-VILLARD
2012-09-17 14:13 ` Sascha Hauer
2012-09-17 14:26 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-22 10:07 [PATCH 0/1 v6] " Jean-Christophe PLAGNIOL-VILLARD
2012-09-22 10:12 ` [PATCH 1/1] " Jean-Christophe PLAGNIOL-VILLARD
2012-09-23 14:50 ` Sascha Hauer
2012-09-23 17:21 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-24 9:31 [PATCH 0/1 v7] " Jean-Christophe PLAGNIOL-VILLARD
2012-09-24 9:36 ` [PATCH 1/1] " 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=20120908172844.GF20330@game.jcrosoft.org \
--to=plagnioj@jcrosoft.com \
--cc=barebox@lists.infradead.org \
--cc=s.hauer@pengutronix.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.