From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Thu, 13 Jun 2013 10:58:07 -0500
Subject: [U-Boot] [PATCH v2] powerpc/85xx: Add P1023RDB board support
In-Reply-To: <51B96E96.1020105@freescale.com> (from b25806@freescale.com on
	Thu Jun 13 02:02:46 2013)
References: <1370973632.18413.28@snotra> <51B96E96.1020105@freescale.com>
Message-ID: <1371139087.2028.3@snotra>
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

On 06/13/2013 02:02:46 AM, Chunhe Lan wrote:
> On 06/12/2013 02:00 AM, Scott Wood wrote:
>> On 06/09/2013 05:24:05 AM, Chunhe Lan wrote:
>>> +    /* W**G* - Flash, localbus */
>>> +    /* This will be changed to *I*G* after relocation to RAM. */
>>> +    SET_TLB_ENTRY(1, CONFIG_SYS_FLASH_BASE,  
>>> CONFIG_SYS_FLASH_BASE_PHYS,
>>> +              MAS3_SX|MAS3_SW|MAS3_SR, MAS2_W|MAS2_G,
>>> +              0, 2, BOOKE_PAGESZ_256M, 1),
>> 
>> Do not set MAS3_SX on I/O regions.
> 
>           Here is need to use MAS3_SX, or u-boot does NOT work.

Sorry, I misread the comment as saying that this was the  
after-relocation TLB entry, not that it was modified after relocation.   
When changing to *I*G* it should also have MAS3_SX removed.

>>> +/*
>>> + * Memory map
>>> + *
>>> + * 0x0000_0000    0x1fff_ffff    DDR            500M Cacheable
>> 
>> 500M?  Are you sure?
> 
>      Yes, system has the 500MB size of ddr.

Not 512 MiB?

-Scott