summaryrefslogtreecommitdiff
path: root/src/mainboard/google/skyrim
diff options
context:
space:
mode:
authorMatt DeVillier <matt.devillier@amd.corp-partner.google.com>2023-03-03 14:17:00 -0600
committerMartin L Roth <gaumless@gmail.com>2023-03-09 17:55:20 +0000
commit6337180ba90c5a2a8b986bf7837cb05ed68b2647 (patch)
tree5064522d5c11ab7317dc1ee646f7280005e62cde /src/mainboard/google/skyrim
parentdcd7ec25cdfbe097ac4d269facd00582e52d980c (diff)
mb/google/skyrim: Enable L1 ASPM substates for PCIe devices
Enable both L1.1 and L1.2 substates for the WiFi, SD card reader, and SSD (both NVMe and eMMC). If a given device does not support a particular substate, then it will not be enabled during PCIe enumeration by coreboot. BUG=b:270690572 TEST=build/boot multiple skyrim/whiterun/frostflow SKUs with different storage configs, verify WiFi/SD card/SSD all functional and have L1 substates enabled insofar as they are supported by the device. BRANCH=skyrim Change-Id: Ib84df8b9d97282ae696414e52c4a65cfb0a81194 Signed-off-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/73438 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin L Roth <gaumless@gmail.com> Reviewed-by: Jon Murphy <jpmurphy@google.com>
Diffstat (limited to 'src/mainboard/google/skyrim')
-rw-r--r--src/mainboard/google/skyrim/port_descriptors.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/src/mainboard/google/skyrim/port_descriptors.c b/src/mainboard/google/skyrim/port_descriptors.c
index 067b656065..3e754916e7 100644
--- a/src/mainboard/google/skyrim/port_descriptors.c
+++ b/src/mainboard/google/skyrim/port_descriptors.c
@@ -20,6 +20,8 @@ static const fsp_dxio_descriptor skyrim_mdn_dxio_descriptors[] = {
.turn_off_unused_lanes = true,
.link_aspm = ASPM_L1,
.link_hotplug = 3,
+ .link_aspm_L1_1 = true,
+ .link_aspm_L1_2 = true,
.clk_req = CLK_REQ2,
},
{ /* SD */
@@ -33,6 +35,8 @@ static const fsp_dxio_descriptor skyrim_mdn_dxio_descriptors[] = {
.turn_off_unused_lanes = true,
.link_aspm = ASPM_L1,
.link_hotplug = 3,
+ .link_aspm_L1_1 = true,
+ .link_aspm_L1_2 = true,
.gpio_group_id = GPIO_27,
.clk_req = CLK_REQ1,
},
@@ -47,6 +51,8 @@ static const fsp_dxio_descriptor skyrim_mdn_dxio_descriptors[] = {
.turn_off_unused_lanes = true,
.link_aspm = ASPM_L1,
.link_hotplug = 3,
+ .link_aspm_L1_1 = true,
+ .link_aspm_L1_2 = true,
.gpio_group_id = GPIO_6,
.clk_req = CLK_REQ0,
},