diff options
| author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2021-06-06 12:03:27 +0300 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2021-06-09 05:19:21 +0000 |
| commit | 4cf65e9cc37f6f1e0467d57a680fcbdeb1a8614d (patch) | |
| tree | bc809b2861de67ec9564ff5ad5e2e1db2ae85294 /src/cpu | |
| parent | 863b753918381fea66615185c3367cda0cb42d0a (diff) | |
cpu/x86/lapic: Drop IOAPIC test
For the purpose of LAPIC IPI messaging it is not required to
evaluate if IOAPIC is enabled. The necessary enable_lapic()
will still be called as part of setup_lapic() within cpu init.
Change-Id: I8b6a34e39f755452f0af63ae0ced7279747c28fc
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/55251
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/cpu')
| -rw-r--r-- | src/cpu/x86/lapic/lapic_cpu_init.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/src/cpu/x86/lapic/lapic_cpu_init.c b/src/cpu/x86/lapic/lapic_cpu_init.c index ba88a3662d..589b8fa0dc 100644 --- a/src/cpu/x86/lapic/lapic_cpu_init.c +++ b/src/cpu/x86/lapic/lapic_cpu_init.c @@ -506,18 +506,13 @@ void initialize_cpus(struct bus *cpu_bus) /* Find the info struct for this CPU */ info = cpu_info(); - if (need_lapic_init()) { - /* Ensure the local APIC is enabled */ + /* Ensure the local APIC is enabled */ + if (is_smp_boot()) enable_lapic(); - /* Get the device path of the boot CPU */ - cpu_path.type = DEVICE_PATH_APIC; - cpu_path.apic.apic_id = lapicid(); - } else { - /* Get the device path of the boot CPU */ - cpu_path.type = DEVICE_PATH_CPU; - cpu_path.cpu.id = 0; - } + /* Get the device path of the boot CPU */ + cpu_path.type = DEVICE_PATH_APIC; + cpu_path.apic.apic_id = lapicid(); /* Find the device structure for the boot CPU */ info->cpu = alloc_find_dev(cpu_bus, &cpu_path); |