[coreboot-gerrit] [M] Change in coreboot[main]: soc/intel/xeon_sp: Add soc_add_dram_resources

[Shuo Liu (Code Review)]

Attention is currently required from: Arthur Heymans, Christian Walter, Johnny Lin, Lean Sheng Tan, Patrick Rudolph, Tim Chu.

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

Change subject: soc/intel/xeon_sp: Add soc_add_dram_resources
......................................................................

Patch Set 2:

(5 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/81110/comment/680bbfd5_8d72df1f :
PS1, Line 9: SoC
> This seems to move code around for no apparent reason. […]

Right, will have.

https://review.coreboot.org/c/coreboot/+/81110/comment/8e2eaa57_1e8bac29 :
PS1, Line 10: acorss
> across

Acknowledged

Patchset:

PS2: 
Good catches, thanks!

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

https://review.coreboot.org/c/coreboot/+/81110/comment/931861dc_02ae77a9 :
PS1, Line 201: uint64_t
> static uint64_t

Acknowledged

https://review.coreboot.org/c/coreboot/+/81110/comment/f42b9af4_97b9ac43 :
PS1, Line 304: Ahe
> The

Acknowledged