[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Add get_system_memory_map()

[Marc Jones (Code Review)]

Marc Jones has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45847 )

Change subject: soc/intel/xeon_sp: Add get_system_memory_map()
......................................................................

Patch Set 5: -Code-Review

(1 comment)

https://review.coreboot.org/c/coreboot/+/45847/5/src/soc/intel/xeon_sp/skx/s... 
File src/soc/intel/xeon_sp/skx/soc_util.c:

https://review.coreboot.org/c/coreboot/+/45847/5/src/soc/intel/xeon_sp/skx/s... 
PS5, Line 383: fsp_find_extension_hob_by_guid
> This seems to return a `struct SystemMemoryMapHob *` for SKX.

Yes, The HOB data access is different between the SKX and CPX FSP. Since this is split out already, we can do the correct access here. We will just need to be careful in the de-duplication.