From cd48c7ece3d38acaf67d25e35b1a66a47728aec8 Mon Sep 17 00:00:00 2001 From: Sean Rhodes Date: Wed, 19 Apr 2023 14:48:51 +0100 Subject: [PATCH] mb/starlabs/starbook: Let coreboot configure ASPM FSP is fractionally faster at configuring ASPM (1,118,688 vs 1,122,205) but coreboot's configuration results in lower power consumption of approximately 0.5W when idling - the reason why is unknown. Signed-off-by: Sean Rhodes Change-Id: Ib15eaede956f0aa55118d093fdff0fd9487df250 Reviewed-on: https://review.coreboot.org/c/coreboot/+/74520 Reviewed-by: Paul Menzel Reviewed-by: Lean Sheng Tan Tested-by: build bot (Jenkins) --- src/mainboard/starlabs/starbook/Kconfig | 21 ------------------- .../starbook/variants/adl/devicetree.cb | 4 ---- 2 files changed, 25 deletions(-) diff --git a/src/mainboard/starlabs/starbook/Kconfig b/src/mainboard/starlabs/starbook/Kconfig index 5ad7e2747f..b49a86d4b3 100644 --- a/src/mainboard/starlabs/starbook/Kconfig +++ b/src/mainboard/starlabs/starbook/Kconfig @@ -147,30 +147,9 @@ config EDK2_BOOTSPLASH_FILE string default "3rdparty/blobs/mainboard/starlabs/Logo.bmp" -config PCIEXP_ASPM - bool - default n - help - FSP is already taking care of ASPM, which is configured through the devicetree in coreboot - on Alderlake Platforms. Disable it to save some boot time. - config PCIEXP_DEFAULT_MAX_RESIZABLE_BAR_BITS default 32 -config PCIEXP_L1_SUB_STATE - bool - default n - help - Enabling PCIe L1 sub states is already done in FSP. - Disable it to save some boot time. - -config PCIEXP_CLK_PM - bool - default n - help - Enabling PCIe clock power management is already done in FSP. - Disable it to save some boot time - config SOC_INTEL_CSE_SEND_EOP_EARLY default n if BOARD_STARLABS_STARBOOK_ADL diff --git a/src/mainboard/starlabs/starbook/variants/adl/devicetree.cb b/src/mainboard/starlabs/starbook/variants/adl/devicetree.cb index 5c54f4da35..298ec199b2 100644 --- a/src/mainboard/starlabs/starbook/variants/adl/devicetree.cb +++ b/src/mainboard/starlabs/starbook/variants/adl/devicetree.cb @@ -100,8 +100,6 @@ chip soc/intel/alderlake .clk_src = 2, .clk_req = 2, .flags = PCIE_RP_LTR | PCIE_RP_AER, - .PcieRpL1Substates = L1_SS_L1_2, - .pcie_rp_aspm = ASPM_L0S_L1, }" smbios_slot_desc "SlotTypePciExpressGen3X1" "SlotLengthShort" @@ -119,8 +117,6 @@ chip soc/intel/alderlake .clk_src = 1, .clk_req = 1, .flags = PCIE_RP_LTR | PCIE_RP_AER, - .PcieRpL1Substates = L1_SS_L1_2, - .pcie_rp_aspm = ASPM_L0S_L1, }" smbios_slot_desc "SlotTypeM2Socket3"