Re: [U-Boot] [PATCH v2 0/3] Add sdram capacity auto detect for rk3288

2016-09-21 Thread Kever Yang
Hi Sandy, On 09/21/2016 08:41 PM, Sandy Patterson wrote: On Tue, Sep 20, 2016 at 10:56 PM, Kever Yang mailto:kever.y...@rock-chips.com>> wrote: parameters changes from dts to auto-detect including those I removed from dts and ddrconfig, stride, they should be the same as without

Re: [U-Boot] [PATCH v2 0/3] Add sdram capacity auto detect for rk3288

2016-09-21 Thread Sandy Patterson
On Tue, Sep 20, 2016 at 10:56 PM, Kever Yang wrote: > > parameters changes from dts to auto-detect including those I removed from > dts and ddrconfig, stride, they should be the same as without my patch, > which means my patch suppose to not change any parameter for DDR other than > how we get tho

Re: [U-Boot] [PATCH v2 0/3] Add sdram capacity auto detect for rk3288

2016-09-20 Thread Kever Yang
Hi Sandy, On 09/20/2016 11:04 PM, Sandy Patterson wrote: I can confirm that this boots on rock2 with current master branch. 9b1b6d42256a4c2e59c803afdbf90d39371e61ba Thanks for your test. It doesn't boot for me using rock2 on v2016.09 tag. I get: U-Boot SPL 2016.09-2-g690a8a3 (Sep 20 20

Re: [U-Boot] [PATCH v2 0/3] Add sdram capacity auto detect for rk3288

2016-09-20 Thread Sandy Patterson
I can confirm that this boots on rock2 with current master branch. 9b1b6d42256a4c2e59c803afdbf90d39371e61ba It doesn't boot for me using rock2 on v2016.09 tag. I get: U-Boot SPL 2016.09-2-g690a8a3 (Sep 20 2016 - 10:47:44) Trying to boot from MMC1 It continues to detect 2GB of ram. on master.

[U-Boot] [PATCH v2 0/3] Add sdram capacity auto detect for rk3288

2016-09-19 Thread Kever Yang
The rk3288 spl size is very close to 32KB while the rk3288 bootrom has the limitation of maximum size of SPL is 32KB. After apply this patch, the SPL size will exceed 32KB if we do not enable macro CONFIG_ROCKCHIP_SPL_BACK_TO_BROM. I think this patch is usful and should be go upstream other than