From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/1 v3] ppc4xx: Add support for GPCS, SGMII and M88E1112 PHY
Date: Thu, 4 Sep 2008 17:34:19 +0200 [thread overview]
Message-ID: <200809041734.19573.sr@denx.de> (raw)
In-Reply-To: <48BF8DB3.70104@gmail.com>
On Thursday 04 September 2008, Ben Warren wrote:
> I like the idea very much, but am not sure about the implementation.
> This problem has been around for a while (just search the archives for
> people wondering how to deal with a switch chip connected via rvMII or
> whatever). The trickiest part of this is how to get the information to
> the driver. I've always thought that the best way would be for board
> code to initialize each controller through proper C code (i.e. not
> CONFIG macros). But there's definitely something to be said for doing
> it all through macros, since that's how Kconfig works. Please have a
> look at the code that Andy Fleming recently submitted for the TSEC
> driver (it's in the main branch now). He passes a tsec_info_struct into
> each call of tsec_initialize(), allowing all type of custom information
> to go in. In my mind, that could be generalized to something that more
> than just TSEC, but let's take baby steps.
Yes, this looks like a good approach. Not sure if we should go all the way or
accept Victors approach for now. Moving to this parameter based
initialization is a different matter that should really be done soon.
> Incidentally, the term "Fixed PHY" has already been coined for what
> you're calling "PHY-less". I suggest we standardize.
Yes. Is there already a define available?
> Anyway, I have to go to bed. Eyes are starting to close and brain's
> sloowwwiing doowwn.
Heh :)
Best regards,
Stefan
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
next prev parent reply other threads:[~2008-09-04 15:34 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-03 6:45 [U-Boot] [PATCH 1/1 v3] ppc4xx: Add support for GPCS, SGMII and M88E1112 PHY Victor Gallardo
2008-09-03 7:22 ` Stefan Roese
2008-09-03 17:17 ` Victor Gallardo
2008-09-03 20:07 ` Victor Gallardo
2008-09-04 7:26 ` Ben Warren
2008-09-04 15:34 ` Stefan Roese [this message]
2008-09-04 17:19 ` Victor Gallardo
2008-09-04 17:22 ` Ben Warren
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=200809041734.19573.sr@denx.de \
--to=sr@denx.de \
--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.