All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v2] rk3399: Reserve space for ARM Trust Firmware
@ 2016-07-25  3:45 Kever Yang
  2016-07-28  3:11 ` Simon Glass
  2016-11-03  1:32 ` Andreas Färber
  0 siblings, 2 replies; 5+ messages in thread
From: Kever Yang @ 2016-07-25  3:45 UTC (permalink / raw)
  To: u-boot

RK3399 needs reserve 0x200000 at the beginning of DRAM, for ATF bl31.

Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
---

Changes in v2:
- correct some typo on commit message and comment

 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 39a26fa..84a00fc 100644
--- a/board/rockchip/evb_rk3399/evb-rk3399.c
+++ b/board/rockchip/evb_rk3399/evb-rk3399.c
@@ -35,6 +35,7 @@ int dram_init(void)
 
 void dram_init_banksize(void)
 {
-	gd->bd->bi_dram[0].start = 0;
+	/* Reserve 0x200000 for ATF bl31 */
+	gd->bd->bi_dram[0].start = 0x200000;
 	gd->bd->bi_dram[0].size = 0x80000000;
 }
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH v2] rk3399: Reserve space for ARM Trust Firmware
  2016-07-25  3:45 [U-Boot] [PATCH v2] rk3399: Reserve space for ARM Trust Firmware Kever Yang
@ 2016-07-28  3:11 ` Simon Glass
  2016-07-28  3:33   ` Simon Glass
  2016-11-03  1:32 ` Andreas Färber
  1 sibling, 1 reply; 5+ messages in thread
From: Simon Glass @ 2016-07-28  3:11 UTC (permalink / raw)
  To: u-boot

On 24 July 2016 at 21:45, Kever Yang <kever.yang@rock-chips.com> wrote:
> RK3399 needs reserve 0x200000 at the beginning of DRAM, for ATF bl31.
>
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> ---
>
> Changes in v2:
> - correct some typo on commit message and comment
>
>  board/rockchip/evb_rk3399/evb-rk3399.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Acked-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH v2] rk3399: Reserve space for ARM Trust Firmware
  2016-07-28  3:11 ` Simon Glass
@ 2016-07-28  3:33   ` Simon Glass
  0 siblings, 0 replies; 5+ messages in thread
From: Simon Glass @ 2016-07-28  3:33 UTC (permalink / raw)
  To: u-boot

On 27 July 2016 at 21:11, Simon Glass <sjg@chromium.org> wrote:
> On 24 July 2016 at 21:45, Kever Yang <kever.yang@rock-chips.com> wrote:
>> RK3399 needs reserve 0x200000 at the beginning of DRAM, for ATF bl31.
>>
>> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
>> ---
>>
>> Changes in v2:
>> - correct some typo on commit message and comment
>>
>>  board/rockchip/evb_rk3399/evb-rk3399.c | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot-rockchip, thanks!

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH v2] rk3399: Reserve space for ARM Trust Firmware
  2016-07-25  3:45 [U-Boot] [PATCH v2] rk3399: Reserve space for ARM Trust Firmware Kever Yang
  2016-07-28  3:11 ` Simon Glass
@ 2016-11-03  1:32 ` Andreas Färber
  2016-11-04  0:43   ` Kever Yang
  1 sibling, 1 reply; 5+ messages in thread
From: Andreas Färber @ 2016-11-03  1:32 UTC (permalink / raw)
  To: u-boot

Hello,

Am 25.07.2016 um 05:45 schrieb Kever Yang:
> RK3399 needs reserve 0x200000 at the beginning of DRAM, for ATF bl31.
> 
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> ---
> 
> Changes in v2:
> - correct some typo on commit message and comment
> 
>  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 39a26fa..84a00fc 100644
> --- a/board/rockchip/evb_rk3399/evb-rk3399.c
> +++ b/board/rockchip/evb_rk3399/evb-rk3399.c
> @@ -35,6 +35,7 @@ int dram_init(void)
>  
>  void dram_init_banksize(void)
>  {
> -	gd->bd->bi_dram[0].start = 0;
> +	/* Reserve 0x200000 for ATF bl31 */
> +	gd->bd->bi_dram[0].start = 0x200000;
>  	gd->bd->bi_dram[0].size = 0x80000000;

Doesn't the size need to be deduced by the same amount then?

Regards,
Andreas

>  }
> 


-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany
GF: Felix Imend?rffer, Jane Smithard, Graham Norton
HRB 21284 (AG N?rnberg)

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH v2] rk3399: Reserve space for ARM Trust Firmware
  2016-11-03  1:32 ` Andreas Färber
@ 2016-11-04  0:43   ` Kever Yang
  0 siblings, 0 replies; 5+ messages in thread
From: Kever Yang @ 2016-11-04  0:43 UTC (permalink / raw)
  To: u-boot

Hi Andreas,

On 11/03/2016 09:32 AM, Andreas F?rber wrote:
> Hello,
>
> Am 25.07.2016 um 05:45 schrieb Kever Yang:
>> RK3399 needs reserve 0x200000 at the beginning of DRAM, for ATF bl31.
>>
>> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
>> ---
>>
>> Changes in v2:
>> - correct some typo on commit message and comment
>>
>>   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 39a26fa..84a00fc 100644
>> --- a/board/rockchip/evb_rk3399/evb-rk3399.c
>> +++ b/board/rockchip/evb_rk3399/evb-rk3399.c
>> @@ -35,6 +35,7 @@ int dram_init(void)
>>   
>>   void dram_init_banksize(void)
>>   {
>> -	gd->bd->bi_dram[0].start = 0;
>> +	/* Reserve 0x200000 for ATF bl31 */
>> +	gd->bd->bi_dram[0].start = 0x200000;
>>   	gd->bd->bi_dram[0].size = 0x80000000;
> Doesn't the size need to be deduced by the same amount then?

Yes, you are right, will send a patch to fix this.

Thanks,
- Kever
>
> Regards,
> Andreas
>
>>   }
>>
>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-11-04  0:43 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-25  3:45 [U-Boot] [PATCH v2] rk3399: Reserve space for ARM Trust Firmware Kever Yang
2016-07-28  3:11 ` Simon Glass
2016-07-28  3:33   ` Simon Glass
2016-11-03  1:32 ` Andreas Färber
2016-11-04  0:43   ` Kever Yang

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.