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-00002-g690a8a3 (Sep 20 2016 - 10:47:44) Trying to boot from MMC1
It continues to detect 2GB of ram. on master.
I get nervous with these memory changes. We found that some memory parameters persisted through to linux and had the ability to cause instability. I don't run have linux setup right now on my rock2, so I can't test that.
I also wanted to test that the parameters detected via the new routine matched those removed from the dts file. I tried enabling DEBUG but that somehow caused no console output anymore (on master). I'm not sure what's going on. I also tried doing some printfs but they had the same effect. It looks like somehow the console init has moved and maybe isn't setup yet.
Sandy