[coreboot-gerrit] [S] Change in coreboot[master]: cpu/x86/mp_init.c: Set topology on BSP

[Arthur Heymans (Code Review)]

Attention is currently required from: Arthur Heymans, Lean Sheng Tan, Yu-Ping Wu.

Hello build bot (Jenkins), Tarun Tuli, Subrata Banik, Kapil Porwal, Lean Sheng Tan, Yu-Ping Wu,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/74374

to look at the new patch set (#2).

Change subject: cpu/x86/mp_init.c: Set topology on BSP
......................................................................

cpu/x86/mp_init.c: Set topology on BSP

The BSP might have non-zero lapicid so set the topology accordingly,
without assuming it is 0. This fixes a cpu exception on at least Intel
Meteorlake. This was caused by FSP CPU PPI being giving incorrect
information about the BSP topology.

This problem was introduced by 8b8400a "drivers/fsp2_0/mp_service_ppi:
Use struct device to fill in buffer" which sets the PPI struct based on
struct device.

TESTED on google/rex

Change-Id: I3fae5efa86d8efc474c129b48bdfa1d1e2306acf
Signed-off-by: Arthur Heymans 
---
M src/cpu/x86/mp_init.c
1 file changed, 22 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/74/74374/2