GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/intel/xeon_sp: Print HOB for all sockets 

Use the FSP define to iterate over all sockets as the runtime value of
numofIIO is the detected number of sockets, not the highest working
socket.

This fixes printing the HOB on multi-socket platforms where a CPU has
been removed or has been disabled (4S system running as 3S).

Change-Id: Ieed67cd48d26c7634636c0aae6a56f3b6fbdf640
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/76492
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Patrick Rudolph 2023-07-14 17:00:17 +02:00 committed by Lean Sheng Tan
parent 9e4968a623
commit 971ea286dd
3 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/soc/intel/xeon_sp/cpx/hob_display.c
View File

@ -102,7 +102,7 @@ static void soc_display_iio_universal_data_hob(const IIO_UDS *hob)
printk(BIOS_DEBUG, "\tPci64BitResourceAllocation: %d\n", printk(BIOS_DEBUG, "\tPci64BitResourceAllocation: %d\n",
hob->PlatformData.Pci64BitResourceAllocation); hob->PlatformData.Pci64BitResourceAllocation);
for (uint8_t s = 0; s < hob->PlatformData.numofIIO; ++s) { for (uint8_t s = 0; s < MAX_SOCKET; ++s) {
printk(BIOS_DEBUG, "\t============ Socket %d Info ================\n", s); printk(BIOS_DEBUG, "\t============ Socket %d Info ================\n", s);
printk(BIOS_DEBUG, "\tValid: 0x%x\n", printk(BIOS_DEBUG, "\tValid: 0x%x\n",
hob->PlatformData.IIO_resource[s].Valid); hob->PlatformData.IIO_resource[s].Valid);

2
src/soc/intel/xeon_sp/skx/hob_display.c
View File

@ -151,7 +151,7 @@ void soc_display_iio_universal_data_hob(void)
printk(BIOS_DEBUG, "\tMmiohGranularity: hi: 0x%x, lo:0x%x\n", printk(BIOS_DEBUG, "\tMmiohGranularity: hi: 0x%x, lo:0x%x\n",
hob->PlatformData.MmiohGranularity.hi, hob->PlatformData.MmiohGranularity.lo); hob->PlatformData.MmiohGranularity.hi, hob->PlatformData.MmiohGranularity.lo);
for (int s = 0; s < hob->PlatformData.numofIIO; ++s) { for (int s = 0; s < MAX_SOCKET; ++s) {
printk(BIOS_DEBUG, "\t============ Socket %d Info ================\n", s); printk(BIOS_DEBUG, "\t============ Socket %d Info ================\n", s);
printk(BIOS_DEBUG, "\tSocketID: 0x%x\n", printk(BIOS_DEBUG, "\tSocketID: 0x%x\n",
hob->PlatformData.IIO_resource[s].SocketID); hob->PlatformData.IIO_resource[s].SocketID);

2
src/soc/intel/xeon_sp/spr/hob_display.c
View File

@ -130,7 +130,7 @@ void soc_display_iio_universal_data_hob(const IIO_UDS *hob)
printk(BIOS_DEBUG, "\tMmiohGranularity: hi: 0x%x, lo:0x%x\n", printk(BIOS_DEBUG, "\tMmiohGranularity: hi: 0x%x, lo:0x%x\n",
hob->PlatformData.MmiohGranularity.hi, hob->PlatformData.MmiohGranularity.lo); hob->PlatformData.MmiohGranularity.hi, hob->PlatformData.MmiohGranularity.lo);
for (uint8_t s = 0; s < hob->PlatformData.numofIIO; ++s) { for (uint8_t s = 0; s < MAX_SOCKET; ++s) {
printk(BIOS_DEBUG, "\t============ Socket %d Info ================\n", s); printk(BIOS_DEBUG, "\t============ Socket %d Info ================\n", s);
printk(BIOS_DEBUG, "\tValid: 0x%x\n", hob->PlatformData.IIO_resource[s].Valid); printk(BIOS_DEBUG, "\tValid: 0x%x\n", hob->PlatformData.IIO_resource[s].Valid);
printk(BIOS_DEBUG, "\tSocketID: 0x%x\n", printk(BIOS_DEBUG, "\tSocketID: 0x%x\n",