Hi guys,
I see that "Simplify RAM size detection" patch is reverted now. It was breaking boot on DRA7XX EVM as well. But I think I found the actual bug in that patch. This line (from patch):
- for (cnt = (maxsize / sizeof(long)) >> 1; cnt > 0; cnt >>= 1) { + for (cnt = (maxsize / sizeof(long)) >> 1; cnt >= 0; cnt >>= 1) {
is changing loop condition to "cnt >= 0". And once cnt is 0, loop continue iterating, turning into forever loop. To fix this one can add "break" line here:
for (cnt = (maxsize / sizeof(long)) >> 1; cnt >= 0; cnt >>= 1) { addr = base + cnt; /* pointer arith! */ sync(); save[i] = *addr; sync(); if (cnt) { i++; *addr = ~cnt; } else { *addr = 0; + break; }
So if someone is going to rework it and push again -- this note can be helpful.