Re: [U-Boot] [PATCH v2] rockchip: reserve memory for rk3399 ATF data

2017-04-20 Thread Kever Yang
Hi Andreas, Thanks for fixing my pool English, will update in next version. Thanks, - Kever On 04/19/2017 09:00 PM, Andreas Färber wrote: Hi Kever, Some grammar nits below: Am 19.04.2017 um 08:42 schrieb Kever Yang: There are 3 region used by rk3399 ATF: "regions" - bl31 code, locate

Re: [U-Boot] [PATCH v2] rockchip: reserve memory for rk3399 ATF data

2017-04-19 Thread Andreas Färber
Hi Kever, Some grammar nits below: Am 19.04.2017 um 08:42 schrieb Kever Yang: > There are 3 region used by rk3399 ATF: "regions" > - bl31 code, locate at 0x1; > - cortex-m0 code and data, locate at 0xff8c; > - bl31 data, locate at 0xff8c1000 ~ 0xff8c4000; 3x "located" > > SPL_TEXT_BA

Re: [U-Boot] [PATCH v2] rockchip: reserve memory for rk3399 ATF data

2017-04-19 Thread Dr. Philipp Tomsich
Reviewed-by: Philipp Tomsich mailto:philipp.toms...@theobroma-systems.com>> > On 19 Apr 2017, at 08:42, Kever Yang wrote: > > There are 3 region used by rk3399 ATF: > - bl31 code, locate at 0x1; > - cortex-m0 code and data, locate at 0xff8c; > - bl31 data, locate at 0xff8c1000 ~ 0xff8c40

[U-Boot] [PATCH v2] rockchip: reserve memory for rk3399 ATF data

2017-04-18 Thread Kever Yang
There are 3 region used by rk3399 ATF: - bl31 code, locate at 0x1; - cortex-m0 code and data, locate at 0xff8c; - bl31 data, locate at 0xff8c1000 ~ 0xff8c4000; SPL_TEXT_BASE starts from 0xff8c2000, we need to reserve memory for ATF data, or else there will have memory corrupt after SPL loa