Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69302 )
Change subject: soc/intel/common: Don't add more links ......................................................................
soc/intel/common: Don't add more links
This is now properly handled in the common mp_init code.
TESTED on prodrive/hermes.
Change-Id: I8f4588094fa0c128963e5ee0ccd36402b588586b Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/soc/intel/common/block/cpu/mp_init.c 1 file changed, 14 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/02/69302/1
diff --git a/src/soc/intel/common/block/cpu/mp_init.c b/src/soc/intel/common/block/cpu/mp_init.c index d8ca0b3..f7a4593 100644 --- a/src/soc/intel/common/block/cpu/mp_init.c +++ b/src/soc/intel/common/block/cpu/mp_init.c @@ -142,10 +142,6 @@ struct device *dev = dev_find_path(NULL, DEVICE_PATH_CPU_CLUSTER); assert(dev != NULL);
- /* In case link to APIC device is not found, create the one */ - if (!dev->link_list) - add_more_links(dev, 1); - soc_init_cpus(dev->link_list); }