Masanori Ogino has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45927 )
Change subject: nb/intel/*/memmap.c: Use `postcar_frame_add_cbmem_top` ......................................................................
Patch Set 2:
(2 comments)
This is nice de-duplication work, but I have some minor comments.
https://review.coreboot.org/c/coreboot/+/45927/2/src/northbridge/intel/sandy... File src/northbridge/intel/sandybridge/memmap.c:
https://review.coreboot.org/c/coreboot/+/45927/2/src/northbridge/intel/sandy... PS2, Line 41: uintptr_t top_of_ram = (uintptr_t)cbmem_top(); This variable can be eliminated now.
https://review.coreboot.org/c/coreboot/+/45927/2/src/northbridge/intel/sandy... PS2, Line 49: postcar_frame_add_cbmem_top(pcf, 8 * MiB, 1); Is the change of alignment from 8 MiB to 1 byte here intentional?