Sean Rhodes has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86178?usp=email )
Change subject: mb/starlabs/starbook/adl: Disconnect unused GPIOs ......................................................................
mb/starlabs/starbook/adl: Disconnect unused GPIOs
These GPIO's are not used, so disconnect them or set the outputs low accordingly.
Change-Id: I9472e003b730646fea9860d9da960d7f766bdda9 Signed-off-by: Sean Rhodes sean@starlabs.systems Reviewed-on: https://review.coreboot.org/c/coreboot/+/86178 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Matt DeVillier matt.devillier@gmail.com --- M src/mainboard/starlabs/starbook/variants/adl/gpio.c 1 file changed, 18 insertions(+), 19 deletions(-)
Approvals: build bot (Jenkins): Verified Matt DeVillier: Looks good to me, approved
diff --git a/src/mainboard/starlabs/starbook/variants/adl/gpio.c b/src/mainboard/starlabs/starbook/variants/adl/gpio.c index 00a4a08..f3f26f2 100644 --- a/src/mainboard/starlabs/starbook/variants/adl/gpio.c +++ b/src/mainboard/starlabs/starbook/variants/adl/gpio.c @@ -27,11 +27,11 @@ /* GPD3: Power Button */ PAD_CFG_NF(GPD3, UP_20K, DEEP, NF1), /* GPD4: Sleep S3 */ - PAD_CFG_NF(GPD4, NONE, DEEP, NF1), + PAD_NC(GPD4, NONE), /* GPD5: Sleep S4 */ - PAD_CFG_NF(GPD5, NONE, DEEP, NF1), + PAD_NC(GPD5, NONE), /* GPD6: Sleep A */ - PAD_CFG_NF(GPD6, NONE, DEEP, NF1), + PAD_NC(GPD6, NONE), /* GPD7: Power Adapter Disable */ PAD_CFG_GPO(GPD7, 0, PWROK), /* GPD8: Suspend Clock */ @@ -39,7 +39,7 @@ /* GPD9: Wireless LAN Sleep */ PAD_CFG_NF(GPD9, NONE, DEEP, NF1), /* GPD10: Sleep S5 */ - PAD_CFG_NF(GPD10, NONE, DEEP, NF1), + PAD_NC(GPD10, NONE), /* GPD11: LAN PHY Enable */ PAD_NC(GPD11, NONE),
@@ -85,7 +85,6 @@ /* A23: Not Connected */ PAD_NC(GPP_A23, NONE),
- /* B0: Core Vendor ID 0 */ PAD_CFG_NF(GPP_B0, NONE, DEEP, NF1), /* B1: Core Vendor ID 1 */ @@ -111,13 +110,13 @@ /* B11: I2C PMC PD Interrupt Test Point 28 */ PAD_NC(GPP_B11, NONE), /* B12: PM SLP S0 */ - PAD_CFG_NF(GPP_B12, NONE, DEEP, NF1), + PAD_NC(GPP_B12, NONE), /* B13: PLT RST */ PAD_CFG_NF(GPP_B13, NONE, DEEP, NF1), /* B14: Top Swap Override Weak Internal PD 20K High: Enabled Low: Disabled */ - PAD_CFG_GPO(GPP_B14, 0, PLTRST), + PAD_NC(GPP_B14, NONE), /* B15: Not Connected */ PAD_NC(GPP_B15, NONE), /* B16: Not Connected */ @@ -242,7 +241,7 @@ /* E1: Not used Accelerometer Interrupt */ PAD_NC(GPP_E1, NONE), /* E2: Not Connected */ - PAD_CFG_GPO(GPP_E2, 1, PLTRST), + PAD_NC(GPP_E2, NONE), /* E3: WiFi RF Kill */ PAD_CFG_GPO_GPIO_DRIVER(GPP_E3, 1, DEEP, NONE), /* E4: Test Point 14 */ @@ -260,7 +259,7 @@ /* E9: USB OverCurrent 0 */ PAD_CFG_NF(GPP_E9, NONE, DEEP, NF1), /* E10: PWD Amplifier Input */ - PAD_CFG_GPO(GPP_E10, 0, PLTRST), + PAD_NC(GPP_E10, NONE), /* E11: TPM IRQ */ PAD_CFG_NF(GPP_E11, NONE, DEEP, NF1), /* E12: Touchpad Interrupt */ @@ -274,7 +273,7 @@ /* E16: Not Connected */ PAD_NC(GPP_E16, NONE), /* E17: Not Connected */ - PAD_CFG_GPO(GPP_E17, 1, PLTRST), + PAD_NC(GPP_E17, NONE), /* E18: Not Connected */ PAD_NC(GPP_E18, NONE), /* E19: Thunderbolt LSX RXD */ @@ -305,15 +304,15 @@ /* F7: TBT LSX VCCIO Weak Internal PD 20K Low: 1.8V High: 3.3V */ - PAD_NC(GPP_F7, NONE), + PAD_CFG_GPO(GPP_F7, 0, DEEP), /* F8: Not Connected */ PAD_NC(GPP_F8, NONE), /* F9: EC Sleep S0 */ - PAD_CFG_GPO(GPP_F9, 1, PLTRST), + PAD_NC(GPP_F9, NONE), /* F10: Weak Internal PD 20K */ - PAD_CFG_GPO(GPP_F10, 1, PLTRST), + PAD_CFG_GPO(GPP_F10, 0, PLTRST), /* F11: TPM ID */ - PAD_CFG_GPI_TRIG_OWN(GPP_F11, NONE, DEEP, OFF, ACPI), + PAD_NC(GPP_F11, NONE), /* F12: Not Connected */ PAD_NC(GPP_F12, NONE), /* F13: Not Connected */ @@ -323,7 +322,7 @@ /* F15: Not used Accelerometer Interrupt 2*/ PAD_NC(GPP_F15, NONE), /* F16: Not Connected */ - PAD_CFG_GPO(GPP_F16, 1, RSMRST), + PAD_NC(GPP_F16, NONE), /* F17: Not used Touch Panel Reset */ PAD_NC(GPP_F17, NONE), /* F18: Not used Touch Panel Interrupt */ @@ -333,7 +332,7 @@ /* F20: CPU M.2 SSD Reset */ PAD_NC(GPP_F20, NONE), /* F21: GPPC_F21 */ - PAD_CFG_NF(GPP_F21, NONE, DEEP, NF1), + PAD_NC(GPP_F21, NONE), /* F22: Not Connected */ PAD_NC(GPP_F22, NONE), /* F23: Not Connected */ @@ -342,7 +341,7 @@ /* H0: PCH M.2 SSD Reset */ PAD_CFG_GPO(GPP_H0, 1, PLTRST), /* H1: BFX Strap 2 Bit 3 Weak Internal PD 20K */ - PAD_CFG_GPO(GPP_H1, 1, PLTRST), + PAD_CFG_GPO(GPP_H1, 0, DEEP), /* H2: Wireless LAN Reset */ PAD_CFG_GPO(GPP_H2, 1, PLTRST), /* H3: Not Connected */ @@ -389,9 +388,9 @@ /* S1: Not Connected */ PAD_NC(GPP_S1, NONE), /* S2: DMIC Clock */ - PAD_CFG_NF(GPP_S2, NONE, DEEP, NF2), + PAD_NC(GPP_S2, NONE), /* S3: DMIC Data */ - PAD_CFG_NF(GPP_S3, NONE, DEEP, NF2), + PAD_NC(GPP_S3, NONE), /* S4: Not Connected */ PAD_NC(GPP_S4, NONE), /* S5: Not Connected */