From 0d35c3fcc3e7cc52f7400a65b8a23b6da677e740 Mon Sep 17 00:00:00 2001 From: Sean Rhodes Date: Tue, 4 Nov 2025 20:37:38 +0000 Subject: [PATCH] mb/starlabs/starbook: Fix inclusion of CPU RP ASPM option The ASPM option for CPU root ports was guarded against the SOC supporting CPU root ports. This meant that the option was visible for boards that didn't utilise the CPU root ports. Adjust this to guard against BOARD_STARLABS_STARBOOK_RPL, which is the only board to actually use the CPU root ports. Change-Id: Id632a8279e8c1cb07536b4198c3752d57eee657a Signed-off-by: Sean Rhodes Reviewed-on: https://review.coreboot.org/c/coreboot/+/89908 Reviewed-by: Matt DeVillier Tested-by: build bot (Jenkins) --- src/mainboard/starlabs/starbook/cfr.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/mainboard/starlabs/starbook/cfr.c b/src/mainboard/starlabs/starbook/cfr.c index e1f3f6dd56..cddc9af16f 100644 --- a/src/mainboard/starlabs/starbook/cfr.c +++ b/src/mainboard/starlabs/starbook/cfr.c @@ -90,9 +90,10 @@ static struct sm_obj_form pci = { #endif #if CONFIG(SOC_INTEL_COMMON_BLOCK_ASPM) &pciexp_clk_pm, - &pciexp_aspm, - #if CONFIG(HAS_INTEL_CPU_ROOT_PORTS) + #if CONFIG(BOARD_STARLABS_STARBOOK_RPL) &pciexp_aspm_cpu, + #else + &pciexp_aspm, #endif &pciexp_l1ss, #endif