Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/73512 )
Change subject: mb/google/skyrim: drop link_hotplug from port descriptors ......................................................................
mb/google/skyrim: drop link_hotplug from port descriptors
These ports are not hot pluggable, so drop the parameter, which will result it in being set to zero / not enabled.
BUG=none TEST=build boot skyrim, verify all PCIe devices functional.
Change-Id: Iaa55cc765e8f073b31f25771633789ac13e2fffa Signed-off-by: Matt DeVillier matt.devillier@amd.corp-partner.google.com --- M src/mainboard/google/skyrim/port_descriptors.c 1 file changed, 16 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/73512/1
diff --git a/src/mainboard/google/skyrim/port_descriptors.c b/src/mainboard/google/skyrim/port_descriptors.c index 11748d8..c16041b 100644 --- a/src/mainboard/google/skyrim/port_descriptors.c +++ b/src/mainboard/google/skyrim/port_descriptors.c @@ -16,7 +16,6 @@ .link_speed_capability = GEN3, .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, @@ -31,7 +30,6 @@ .link_speed_capability = GEN3, .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, @@ -47,7 +45,6 @@ .link_speed_capability = GEN3, .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,