Lean Sheng Tan has submitted this change. ( https://review.coreboot.org/c/coreboot/+/78329?usp=email )
Change subject: device/device.h: Fix outdated comment ......................................................................
device/device.h: Fix outdated comment
LAPIC devices in devicetree is not possible any longer since commit 3eba665 "util/sconfig: Remove lapic devices from devicetree parsers".
TEST=intel/archercity CRB
Signed-off-by: Arthur Heymans arthur@aheymans.xyz Change-Id: I02192c9a11c35d9625837a8a9f3ba798ff0ae611 Reviewed-on: https://review.coreboot.org/c/coreboot/+/78329 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Shuo Liu shuo.liu@intel.com Reviewed-by: Lean Sheng Tan sheng.tan@9elements.com --- M src/include/device/device.h 1 file changed, 1 insertion(+), 8 deletions(-)
Approvals: build bot (Jenkins): Verified Lean Sheng Tan: Looks good to me, approved Shuo Liu: Looks good to me, but someone else must approve
diff --git a/src/include/device/device.h b/src/include/device/device.h index 50307e6..752e1c0 100644 --- a/src/include/device/device.h +++ b/src/include/device/device.h @@ -229,14 +229,7 @@ void mp_init_cpus(DEVTREE_CONST struct bus *cpu_bus); static inline void mp_cpu_bus_init(struct device *dev) { - /* - * When no LAPIC device is specified in the devicetree inside the CPU cluster device, - * neither a LAPIC device nor the link/bus between the CPU cluster and the LAPIC device - * will be present in the static device tree and the link_list struct element of the - * CPU cluster device will be NULL. In this case add one link, so that the - * alloc_find_dev calls in init_bsp and allocate_cpu_devices will be able to add a - * LAPIC device for the BSP and the APs on this link/bus. - */ + /* Make sure the cpu cluster has a downstream bus for LAPICs to be allocated. */ if (!dev->link_list) add_more_links(dev, 1);