[coreboot-gerrit] [S] Change in coreboot[main]: soc/intel/xeon_sp: Use MemoryMapDataHob to add high RAM resources

[Shuo Liu (Code Review)]

Attention is currently required from: Alicja Michalska, Angel Pons, Arthur Heymans, Christian Walter, Johnny Lin, Jonathan Zhang, Patrick Rudolph, Tim Chu.

Shuo Liu has posted comments on this change by Shuo Liu. ( https://review.coreboot.org/c/coreboot/+/84304?usp=email )

Change subject: soc/intel/xeon_sp: Use MemoryMapDataHob to add high RAM resources
......................................................................

Patch Set 2:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84304/comment/166392ba_f294563c?usp... :
PS1, Line 9: window
> Use plural: window*s*

Done

https://review.coreboot.org/c/coreboot/+/84304/comment/8817d2ef_5fc16f1e?usp... :
PS1, Line 10: no
> typo: on

Done

https://review.coreboot.org/c/coreboot/+/84304/comment/72715974_3e21d388?usp... :
PS1, Line 13: mechanism for GNR and previous generation SoCs.
> Has this change been tested? If so, on which platforms and how? I would recommend testing on previou […]

Sure, updated the tests.

File src/soc/intel/xeon_sp/uncore.c:

https://review.coreboot.org/c/coreboot/+/84304/comment/ca2cda10_1b02ca6c?usp... :
PS1, Line 288:         const struct SystemMemoryMapHob *mm = get_system_memory_map();
> > `code indent should use tabs where possible` […]

Done