summaryrefslogtreecommitdiffstats
path: root/src/mainboard/intel/adlrvp
diff options
context:
space:
mode:
authorMAULIK V VAGHELA <maulik.v.vaghela@intel.com>2021-08-06 18:49:56 +0530
committerNick Vaccaro <nvaccaro@google.com>2021-08-10 21:17:14 +0000
commit563a6cc6f2690f2594df51d8d16b3e9f4ef5ca8d (patch)
treef4f98746dbc83164b0c373ad73141efaaa4f00c3 /src/mainboard/intel/adlrvp
parentbde3c56d2cd0fc117d8ab9a51e8b67ba0b73d090 (diff)
downloadcoreboot-563a6cc6f2690f2594df51d8d16b3e9f4ef5ca8d.tar.gz
coreboot-563a6cc6f2690f2594df51d8d16b3e9f4ef5ca8d.tar.bz2
coreboot-563a6cc6f2690f2594df51d8d16b3e9f4ef5ca8d.zip
mb/*/brya/adlrvp: Remove hardcoding of BSP APIC ID
coreboot always assumes that BSP APIC ID will be 0 and core enumeration logic will look for lapic id from the mainboard. As per Intel 64 and IA-32 Architectures Software Developer’s Manual Volume 3: 8.4.1 BSP and AP Processors, this assumption might not hold true and we may have any other core as BSP. To handle this, we need to remove hardcoding of APIC ID 0 from mainboard. BUG=None BRANCH=None TEST=Check if there is no functional impact on the board. Change-Id: Ibc60494b0032a3139c1e6c79251fb2da750c8de8 Signed-off-by: MAULIK V VAGHELA <maulik.v.vaghela@intel.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/56852 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-by: Nick Vaccaro <nvaccaro@google.com>
Diffstat (limited to 'src/mainboard/intel/adlrvp')
-rw-r--r--src/mainboard/intel/adlrvp/devicetree.cb4
-rw-r--r--src/mainboard/intel/adlrvp/devicetree_m.cb4
2 files changed, 2 insertions, 6 deletions
diff --git a/src/mainboard/intel/adlrvp/devicetree.cb b/src/mainboard/intel/adlrvp/devicetree.cb
index 65dc9ca2e5a1..f4852b920fdb 100644
--- a/src/mainboard/intel/adlrvp/devicetree.cb
+++ b/src/mainboard/intel/adlrvp/devicetree.cb
@@ -1,8 +1,6 @@
chip soc/intel/alderlake
- device cpu_cluster 0 on
- device lapic 0 on end
- end
+ device cpu_cluster 0 on end
# GPE configuration
# Note that GPE events called out in ASL code rely on this
diff --git a/src/mainboard/intel/adlrvp/devicetree_m.cb b/src/mainboard/intel/adlrvp/devicetree_m.cb
index ac80caff6b8e..3669c8624375 100644
--- a/src/mainboard/intel/adlrvp/devicetree_m.cb
+++ b/src/mainboard/intel/adlrvp/devicetree_m.cb
@@ -6,9 +6,7 @@ fw_config
end
chip soc/intel/alderlake
- device cpu_cluster 0 on
- device lapic 0 on end
- end
+ device cpu_cluster 0 on end
# GPE configuration
# Note that GPE events called out in ASL code rely on this