From mboxrd@z Thu Jan  1 00:00:00 1970
From: Timur Tabi <timur@freescale.com>
Date: Thu, 27 May 2010 15:11:22 -0500
Subject: [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS
 reference board
In-Reply-To: <20100527195358.2A6C1EAC238@gemini.denx.de>
References: <1274392909-16422-1-git-send-email-timur@freescale.com>
	<20100520223324.50594CCF026@gemini.denx.de>
	<4BFC1736.5030902@freescale.com>
	<20100526201014.97886EAC238@gemini.denx.de>
	<4BFD837D.2040508@freescale.com>
	<20100527070235.E8812EAC238@gemini.denx.de>
	<4BFE823A.1080409@freescale.com>
	<20100527181118.3C9F7EAC238@gemini.denx.de>
	<4BFEB922.9040106@freescale.com>
	<20100527190340.GA5915@schlenkerla.am.freescale.net>
	<4BFEC2FD.5050103@freescale.com>
	<20100527195358.2A6C1EAC238@gemini.denx.de>
Message-ID: <4BFED1EA.5060104@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.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