Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/74738 )
Change subject: mb/starlabs/starbook/adl: Correct port for Hot Plug ......................................................................
mb/starlabs/starbook/adl: Correct port for Hot Plug
Commit 5103b87a4d7b ("mb/starlabs/starbook/adl: Add an option to enable Hot Plug") introduced an option to enable Hot Plug for the SSD. The port was set to 4 (RP5) which is the wireless card. Change this to 8 (RP9) which is the SSD.
Signed-off-by: Sean Rhodes sean@starlabs.systems Change-Id: I884f4997d73e31bd422477952466f168afad66a1 Reviewed-on: https://review.coreboot.org/c/coreboot/+/74738 Reviewed-by: Paul Menzel paulepanter@mailbox.org Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Lean Sheng Tan sheng.tan@9elements.com --- M src/mainboard/starlabs/starbook/variants/adl/ramstage.c 1 file changed, 20 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Lean Sheng Tan: Looks good to me, approved
diff --git a/src/mainboard/starlabs/starbook/variants/adl/ramstage.c b/src/mainboard/starlabs/starbook/variants/adl/ramstage.c index 7accb1d..a2c3926 100644 --- a/src/mainboard/starlabs/starbook/variants/adl/ramstage.c +++ b/src/mainboard/starlabs/starbook/variants/adl/ramstage.c @@ -11,5 +11,5 @@ * third-party drives are detected. */ if (get_uint_option("pci_hot_plug", 0) == 1) - supd->PcieRpHotPlug[4] = 1; + supd->PcieRpHotPlug[8] = 1; }