Closed StefanLellwitz closed 2 years ago
please contact the rockchip bsp u-boot maintainer about this issue, likely to be chenjh@rock-chips.com.
this repo is just a mirror, i don't know much about this u-boot...as far as i know, the spl size limit here should be the chip's sram size, and bootroom would load ddr init code(spl) into it before ddr inited.
so guessing they need to double check the sram size or truncate the ddr bin.
there seems a few meaningless padding at the end: 00027f0 0000 0000 0030 0000 0004 0000 ffff ffff 0002800 ffff ffff beaf dead
so you can try to truncate that bin to the expected size
hmm, tested on rk3399, the ddr binary's padding bytes are important.
checking px30's TRM, the "Chapter 6 Embedded SRAM" says "SYSTEM_SRAM Provide 16KB access space", so guessing the 0x2800 should be 0x4000 - (bootrom's reserved size(mostly stack, and likely to be 4K))
Hi Jeffy, thanks for reviewing this.
checking px30's TRM, the "Chapter 6 Embedded SRAM" says "SYSTEM_SRAM Provide 16KB access space", so guessing the 0x2800 should be 0x4000
This was the whole reason for my Pull Request https://github.com/JeffyCN/mirrors/pull/4. I found this information of 16K SYSTEM_SRAM in PX30 datasheet. But I wasn't sure about that. So the absolute minimum which works is 12K (0x3000).
But should be rather 0x4000 based on this information I guess...
Anyway, this seems to be wrong.
I will contact U-Boot maintainer as you suggested.
0x4000 - (bootrom's reserved size(mostly stack, and likely to be 4K))
Sorry, didn't get it. Would be 0x3000 then.
0x4000 - (bootrom's reserved size(mostly stack, and likely to be 4K))
Sorry, didn't get it. Would be 0x3000 then.
right, the bootrom is close-source, so the reserved size(6k) in the original commit might just be a guess value.
anyway, the bootrom guys did confirm it's 4k reserved in px30/3326. the fix patch is going through the rockchip internal review flow now.
Not possible to build U-Boot for PX30.
This problem exists since 413d2f5.
Before
px30_ddr_333MHz_v1.15.bin
was exactly 10.240 Bytes (= 0x2800) and everything was fine. But since update topx30_ddr_333MHz_v1.16.bin
it is 10.248 Bytes and therefore does not fit.