[U-Boot] [PATCH] rk3399: reseve space for trust

[Simon Glass]

Hi again,

On 22 July 2016 at 20:57, Simon Glass <sjg at chromium.org> wrote:
> Hi Kever,
>
> On 22 July 2016 at 03:34, Kever Yang <kever.yang at rock-chips.com> wrote:
>
> For the commit message, you have "rk3399: reseve space for trust".
>
> I think it should be "rk3399: Reseve space for ARM Trusted Firmware"
> or perhaps ATF

Ooops I mean 'Reserve' !

>
>
>> rk3399 need reserve 0x200000 at the begining of DRAM, for ATF usage.
>
> needs
> beginning
> s/usage/use/
>
>>
>> Signed-off-by: Kever Yang <kever.yang at rock-chips.com>
>> ---
>>
>>  board/rockchip/evb_rk3399/evb-rk3399.c | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/board/rockchip/evb_rk3399/evb-rk3399.c b/board/rockchip/evb_rk3399/evb-rk3399.c
>> index dffacd0..37d3851 100644
>> --- a/board/rockchip/evb_rk3399/evb-rk3399.c
>> +++ b/board/rockchip/evb_rk3399/evb-rk3399.c
>> @@ -21,6 +21,7 @@ int dram_init(void)
>>
>>  void dram_init_banksize(void)
>>  {
>> -       gd->bd->bi_dram[0].start = 0;
>> +       /* Reserve 0x200000 for trust word */
>
> What does 'trust word' mean? Do you mean space for ATF, or a single
> word of memory?
>
>> +       gd->bd->bi_dram[0].start = 0x200000;
>>         gd->bd->bi_dram[0].size = 0x80000000;
>>  }
>> --
>> 1.9.1
>>
>>
>
> Regards,
> Simon


Regards,
Simon