public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] 85xx/p2020ds: Use is_serdes_configured() to determine of PCIe enabled
Date: Thu, 27 May 2010 15:20:20 +0200	[thread overview]
Message-ID: <20100527132020.84853EAC238@gemini.denx.de> (raw)
In-Reply-To: <7D2BA6A1-2EEF-4BF5-8B76-07AD55537BEE@kernel.crashing.org>

Dear Kumar Gala,

In message <7D2BA6A1-2EEF-4BF5-8B76-07AD55537BEE@kernel.crashing.org> you wrote:
> 
> > i. e. the highest number is at the lowest address??
>
> Correct, that is matching FSL HW docs numbering/naming.
>
> in the .dts the alias:
> * "pci0" is @ 0x8000	- FSL HW calls it PCIE3
> * "pci1" is @ 0x9000	- FSL HW calls it PCIE2
> * "pci2" is @ 0xa000	- FSL HW calls it PCIE1

IMHO there were several decisions that I would not exactly call
intelligent - the first to assign HW names in descending order, and
the second to chose a different order for the bus names.

Now we sit here and have to suffer from this :-(

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
There are two ways of constructing a software design. One way  is  to
make  it  so  simple that there are obviously no deficiencies and the
other is to make it so complicated that there are  no  obvious  defi-
ciencies.                             - Charles Anthony Richard Hoare

  reply	other threads:[~2010-05-27 13:20 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-21  9:17 [U-Boot] [PATCH] 85xx: Add is_serdes_configured() support for P2020 SERDES Kumar Gala
2010-05-21  9:17 ` [U-Boot] [PATCH] 85xx/p2020ds: Use is_serdes_configured() to determine of PCIe enabled Kumar Gala
2010-05-26 20:46   ` Timur Tabi
2010-05-26 21:37   ` Timur Tabi
2010-05-27  7:08   ` Wolfgang Denk
2010-05-27  8:42     ` Kumar Gala
2010-05-27 11:20       ` Wolfgang Denk
2010-05-27 12:45         ` Kumar Gala
2010-05-27 13:20           ` Wolfgang Denk [this message]
2010-05-27 15:47             ` Kumar Gala
2010-05-27 19:38           ` Scott Wood
2010-05-27 21:27             ` Kumar Gala
2010-05-27 21:28               ` Timur Tabi
2010-05-27 21:38   ` Timur Tabi
2010-05-28 12:53     ` Kumar Gala
2010-05-28 14:33       ` Timur Tabi
2010-05-21 11:43 ` [U-Boot] [PATCH] 85xx: Add is_serdes_configured() support for P2020SERDES Li Yang
     [not found] ` <6D3E1D972AA8904199D0D412F4A64CBD024CF99A@zch01exm26.fsl.freescale.net>
2010-05-21 12:51   ` Kumar Gala

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=20100527132020.84853EAC238@gemini.denx.de \
    --to=wd@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox