All of lore.kernel.org
 help / color / mirror / Atom feed
From: Timur Tabi <timur@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS reference board
Date: Thu, 27 May 2010 15:11:22 -0500	[thread overview]
Message-ID: <4BFED1EA.5060104@freescale.com> (raw)
In-Reply-To: <20100527195358.2A6C1EAC238@gemini.denx.de>

Wolfgang Denk wrote:

>> Ah, that's a serious limitation.
> 
> No, it is not. Testing is done per bank.

Ok, I get it now.  I'm supposed to call get_ram_size() on each DIMM I
locate.  That's going to be complicated because we don't have any TLBs set
up when we do that.

And what if we have a 4GB DIMM?  Because virtual addresses are still
32-bits, we would need to change get_ram_size() to modify a temporary TLB
while it's reading/writing memory.  So we'd need to have an e300 version of
get_ram_size(), an e500 version, an e600 version, and so on.

So I have a question: are you still going to require me to use
get_ram_size() in my P1022DS board patch?

-- 
Timur Tabi
Linux kernel developer at Freescale

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

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-20 22:01 [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS reference board Timur Tabi
2010-05-20 22:33 ` Wolfgang Denk
2010-05-20 23:13   ` Kumar Gala
2010-05-21  6:50     ` Wolfgang Denk
2010-05-20 23:23   ` Timur Tabi
2010-05-21  7:07     ` Wolfgang Denk
2010-05-21 13:45       ` Timur Tabi
2010-05-21 14:22         ` Wolfgang Denk
2010-05-21 14:33           ` Timur Tabi
2010-05-21 16:07           ` Timur Tabi
2010-05-26 18:12       ` Timur Tabi
2010-05-26 18:17         ` Scott Wood
2010-05-26 18:19           ` Timur Tabi
2010-05-26 19:04             ` Scott Wood
2010-05-26 19:34               ` Timur Tabi
2010-05-26 19:46                 ` Scott Wood
2010-05-26 21:59                   ` Timur Tabi
2010-05-25 18:30   ` Timur Tabi
2010-05-26 20:10     ` Wolfgang Denk
2010-05-26 20:18       ` Timur Tabi
2010-05-26 20:24       ` Timur Tabi
2010-05-27  7:02         ` Wolfgang Denk
2010-05-27 14:31           ` Timur Tabi
2010-05-27 18:11             ` Wolfgang Denk
2010-05-27 18:25               ` Timur Tabi
2010-05-27 19:03                 ` Scott Wood
2010-05-27 19:07                   ` Timur Tabi
2010-05-27 19:10                     ` Scott Wood
2010-05-27 19:54                       ` Wolfgang Denk
2010-05-27 19:53                     ` Wolfgang Denk
2010-05-27 20:11                       ` Timur Tabi [this message]
2010-05-27 21:10                         ` Wolfgang Denk
2010-05-27 19:53                   ` Wolfgang Denk
2010-05-27 20:03                     ` Timur Tabi
2010-05-27 20:59                       ` Wolfgang Denk
2010-05-27 20:05                     ` Scott Wood
2010-05-27 21:03                       ` Wolfgang Denk
2010-05-27 19:45                 ` Wolfgang Denk
2010-05-27 19:54                   ` Timur Tabi
2010-05-27 20:00                     ` Wolfgang Denk
2010-05-27 20:10                       ` Scott Wood
2010-05-21  0:26 ` [U-Boot] [PATCH] powerpc: add support for the FreescaleP1022DS " Liu Dave-R63238
2010-05-21 15:25   ` Timur Tabi
2010-05-21  9:46 ` [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS " 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=4BFED1EA.5060104@freescale.com \
    --to=timur@freescale.com \
    --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.