[coreboot-gerrit] Change in coreboot[master]: soc/mediatek: Save dram info to cbmem

[Julius Werner (Code Review)]

Attention is currently required from: Hung-Te Lin, Xi Chen, Paul Menzel, Xixi Chen, Yu-Ping Wu.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61334 )

Change subject: soc/mediatek: Save dram info to cbmem
......................................................................

Patch Set 25:

(1 comment)

File src/soc/mediatek/common/memory.c:

https://review.coreboot.org/c/coreboot/+/61334/comment/fca5dca8_584b195e 
PS22, Line 123: 	p = (void *)((uintptr_t)mc + sizeof(*mc));
> Now I see what the problem is. […]

I still can't reproduce locally if I make x a global tbh, although that would make sense since then the compiler can know for (pretty) sure that it's invalid. The difference here is that we're using a pointer, not just a local stack allocation, so then the compiler can't tell how it was allocated and if there's sufficient space reserved behind it.

Anyway, as long as the problem is gone now that's good enough for me.