mb/starlabs/lite/glk: Disconnect unused GPIOs
Disconnect GPIOs that are unused or not connected.
Also, update comments that are vague or have errors.
Signed-off-by: Sean Rhodes <sean@starlabs.systems>
Change-Id: Ic83797b8a8e05eed99db0356f360a329f6fbf347
Reviewed-on: https://review.coreboot.org/c/coreboot/+/64452
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin L Roth <gaumless@tutanota.com>
diff --git a/src/mainboard/starlabs/lite/variants/glk/gpio.c b/src/mainboard/starlabs/lite/variants/glk/gpio.c
index 9d83554..928a211 100644
--- a/src/mainboard/starlabs/lite/variants/glk/gpio.c
+++ b/src/mainboard/starlabs/lite/variants/glk/gpio.c
@@ -57,8 +57,7 @@
/* GPIO_15: Not Connected */
PAD_NC(GPIO_15, DN_20K),
/* GPIO_16: SATA_DEVSLP */
- _PAD_CFG_STRUCT(GPIO_16, PAD_FUNC(GPIO) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- PAD_BUF(TX_RX_DISABLE), PAD_IOSSTATE(IGNORE)),
+ PAD_NC(GPIO_16, NONE),
/* GPIO_17: Not Connected */
PAD_NC(GPIO_17, DN_20K),
/* GPIO_18: Not Connected */
@@ -81,10 +80,10 @@
PAD_NC(GPIO_25, DN_20K),
/* GPIO_26: TPNL_INT# */
PAD_NC(GPIO_26, DN_20K),
- /* GPIO_27: GPIO_27 */
- PAD_NC(GPIO_27, DN_20K),
- /* GPIO_28: GPIO_28 */
- PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_28, 1, DEEP, UP_20K, TxLASTRxE, ENPD),
+ /* GPIO_27: eMMC Strap */
+ PAD_NC(GPIO_27, NONE),
+ /* GPIO_28: SPI Strap */
+ PAD_NC(GPIO_28, NONE),
/* GPIO_29: Not Connected */
PAD_NC(GPIO_29, DN_20K),
/* GPIO_30: Not Connected */
@@ -97,7 +96,7 @@
_PAD_CFG_STRUCT(GPIO_33, PAD_FUNC(GPIO) | PAD_RESET(DEEP) | PAD_IRQ_ROUTE(IOAPIC) |
PAD_RX_POL(INVERT) | PAD_BUF(TX_DISABLE) | (1 << 1), PAD_PULL(UP_20K) |
PAD_IOSSTATE(IGNORE)),
- /* GPIO_33: WIFI_ON_SOC */
+ /* GPIO_34: WIFI_ON_SOC */
PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_34, 1, DEEP, UP_20K, IGNORE, ENPU),
/* GPIO_35: Not Connected */
PAD_NC(GPIO_35, DN_20K),
@@ -116,15 +115,12 @@
/* GPIO_41: Not Connected */
PAD_NC(GPIO_41, DN_20K),
/* GPIO_42: SECURITY_FLASH */
- _PAD_CFG_STRUCT(GPIO_42, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- PAD_BUF(TX_RX_DISABLE), PAD_PULL(DN_20K)),
+ PAD_NC(GPIO_42, DN_20K),
/* GPIO_43: GPIO_43 */
- _PAD_CFG_STRUCT(GPIO_43, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- PAD_BUF(TX_RX_DISABLE), PAD_PULL(DN_20K)),
- /* GPIO_44: GPIO_44 */
- _PAD_CFG_STRUCT(GPIO_44, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- (1 << 1), PAD_PULL(UP_20K) | PAD_IOSTERM(ENPU)),
- /* GPIO_45: GPIO_45 */
+ PAD_NC(GPIO_43, DN_20K),
+ /* GPIO_44: USB OC0 */
+ PAD_NC(GPIO_44, DN_20K),
+ /* GPIO_45: USB_OC1 */
_PAD_CFG_STRUCT(GPIO_45, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
(1 << 1), PAD_PULL(UP_20K) | PAD_IOSTERM(ENPU)),
/* GPIO_46: Not Connected */
@@ -137,38 +133,33 @@
/* GPIO_49: PMIC_I2C_SCL */
_PAD_CFG_STRUCT(GPIO_49, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
(1 << 1), PAD_PULL(UP_1K) | PAD_IOSSTATE(IGNORE) | PAD_IOSTERM(ENPU)),
- /* GPIO_50: Not Connected */
+ /* GPIO_50: I2C_0 SDA */
PAD_NC(GPIO_50, DN_20K),
- /* GPIO_51: Not Connected */
+ /* GPIO_51: I2C_0 SDL */
PAD_NC(GPIO_51, DN_20K),
- /* GPIO_52: Not Connected */
+ /* GPIO_52: I2C_1 SDA */
PAD_NC(GPIO_52, DN_20K),
- /* GPIO_53: Not Connected */
+ /* GPIO_53: I2C_1 SDL */
PAD_NC(GPIO_53, DN_20K),
- /* GPIO_54: Not Connected */
+ /* GPIO_54: I2C_2 SDA */
PAD_NC(GPIO_54, DN_20K),
- /* GPIO_55: Not Connected */
+ /* GPIO_55: I2C_2 SDL */
PAD_NC(GPIO_55, DN_20K),
- /* GPIO_56: I2C_SDA_GS */
- _PAD_CFG_STRUCT(GPIO_56, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- (1 << 1), PAD_PULL(UP_1K) | PAD_IOSSTATE(HIZCRx1) | PAD_IOSTERM(ENPU)),
- /* GPIO_57: I2C_SCL_GS */
- _PAD_CFG_STRUCT(GPIO_57, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- (1 << 1), PAD_PULL(UP_1K) | PAD_IOSSTATE(HIZCRx1) | PAD_IOSTERM(ENPU)),
+ /* GPIO_56: I2C_3 SDA */
+ PAD_NC(GPIO_56, DN_20K),
+ /* GPIO_57: I2C_3 SDL */
+ PAD_NC(GPIO_57, DN_20K),
/* GPIO_58: I2C4_SDA */
- _PAD_CFG_STRUCT(GPIO_58, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- (1 << 1), PAD_PULL(UP_1K) | PAD_IOSSTATE(HIZCRx1) | PAD_IOSTERM(ENPU)),
+ PAD_NC(GPIO_58, DN_20K),
/* GPIO_59: I2C4_SCL */
- _PAD_CFG_STRUCT(GPIO_59, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- (1 << 1), PAD_PULL(UP_1K) | PAD_IOSSTATE(HIZCRx1) | PAD_IOSTERM(ENPU)),
- /* GPIO_60: GPIO_60 */
+ PAD_NC(GPIO_59, DN_20K),
+ /* GPIO_60: UART0 RXD */
PAD_NC(GPIO_60, DN_20K),
- /* GPIO_61: GPIO_61 */
- _PAD_CFG_STRUCT(GPIO_61, PAD_FUNC(GPIO) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- PAD_BUF(TX_RX_DISABLE), PAD_PULL(UP_20K)),
- /* GPIO_62: Not Connected */
+ /* GPIO_61: UART0 TXD */
+ PAD_NC(GPIO_61, DN_20K),
+ /* GPIO_62: UART0 RTS */
PAD_NC(GPIO_62, DN_20K),
- /* GPIO_63: Not Connected */
+ /* GPIO_63: UART0 CTS */
PAD_NC(GPIO_63, DN_20K),
/* GPIO_66: GPIO_66 */
PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_66, 1, DEEP, UP_20K, TxLASTRxE, ENPU),
@@ -195,7 +186,7 @@
_PAD_CFG_STRUCT(GPIO_75, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
PAD_BUF(TX_RX_DISABLE) | (1 << 1), PAD_PULL(UP_20K)),
/* GPIO_211: EMMC_RST#_CPU */
- PAD_NC(GPIO_211, DN_20K),
+ PAD_NC(GPIO_211, UP_20K),
/* GPIO_212: BRD_ID_0 */
PAD_NC(GPIO_212, DN_20K),
/* GPIO_213: BRD_ID_1 */
@@ -211,25 +202,21 @@
/* GPIO_78: Not Connected */
PAD_NC(GPIO_78, DN_20K),
/* GPIO_79: GPIO_79 */
- _PAD_CFG_STRUCT(GPIO_79, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF),
- PAD_PULL(DN_20K) | PAD_IOSSTATE(IGNORE) | PAD_IOSTERM(ENPD)),
+ PAD_NC(GPIO_79, DN_20K),
/* GPIO_80: FP_FS0 */
PAD_NC(GPIO_80, DN_20K),
/* GPIO_81: GPIO_81 */
- _PAD_CFG_STRUCT(GPIO_81, PAD_FUNC(NF3) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- (1 << 1), PAD_PULL(DN_20K) | PAD_IOSSTATE(IGNORE)),
+ PAD_NC(GPIO_81, DN_20K),
/* GPIO_82: FP_MISO */
PAD_NC(GPIO_82, DN_20K),
- /* GPIO_83: GPIO_83 */
- _PAD_CFG_STRUCT(GPIO_83, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF),
- PAD_PULL(DN_20K) | PAD_IOSSTATE(IGNORE) | PAD_IOSTERM(ENPD)),
- /* GPIO_84: GPIO_84 */
- PAD_CFG_GPI_TRIG_OWN(GPIO_84, DN_20K, DEEP, OFF, ACPI),
+ /* GPIO_83: LPC 1.8V/3.3V Select */
+ PAD_NC(GPIO_83, DN_20K),
+ /* GPIO_84: Allow SPI Boot */
+ PAD_NC(GPIO_84, UP_20K),
/* GPIO_85: Not Connected */
PAD_NC(GPIO_85, DN_20K),
/* GPIO_86: GPIO_86 */
- _PAD_CFG_STRUCT(GPIO_86, PAD_FUNC(GPIO) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- PAD_BUF(TX_RX_DISABLE), PAD_PULL(DN_20K)),
+ PAD_NC(GPIO_86, DN_20K),
/* GPIO_87: Not Connected */
PAD_NC(GPIO_87, DN_20K),
/* GPIO_88: Not Connected */
@@ -241,13 +228,13 @@
PAD_BUF(TX_DISABLE) | (1 << 1), PAD_PULL(NATIVE)),
/* GPIO_91: Not Connected */
PAD_NC(GPIO_91, DN_20K),
- /* GPIO_91: FLASH_SPI_MOSI */
+ /* GPIO_92: FLASH_SPI_MOSI */
_PAD_CFG_STRUCT(GPIO_92, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
PAD_BUF(TX_DISABLE), PAD_PULL(NATIVE)),
- /* GPIO_92: FLASH_SPI_MISO */
+ /* GPIO_93: FLASH_SPI_MISO */
_PAD_CFG_STRUCT(GPIO_93, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
PAD_BUF(TX_DISABLE) | (1 << 1), PAD_PULL(NATIVE)),
- /* GPIO_93: FLASH_SPI_IO2 */
+ /* GPIO_94: FLASH_SPI_IO2 */
_PAD_CFG_STRUCT(GPIO_94, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
PAD_BUF(TX_DISABLE) | (1 << 1), PAD_PULL(NATIVE)),
/* GPIO_95: FLASH_SPI_IO3 */
@@ -274,9 +261,9 @@
_PAD_CFG_STRUCT(GPIO_103, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF),
PAD_IOSSTATE(IGNORE)),
/* GPIO_104: Not Connected */
- PAD_NC(GPIO_104, DN_20K),
+ PAD_NC(GPIO_104, UP_20K),
/* GPIO_105: GPIO_105 */
- PAD_NC(GPIO_105, DN_20K),
+ PAD_NC(GPIO_105, UP_20K),
/* GPIO_106: PMU_BATLOW# */
_PAD_CFG_STRUCT(GPIO_106, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
(1 << 1), PAD_PULL(UP_20K) | PAD_IOSSTATE(IGNORE) | PAD_IOSTERM(ENPU)),
@@ -284,37 +271,34 @@
_PAD_CFG_STRUCT(GPIO_107, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
(1 << 1), PAD_PULL(UP_20K) | PAD_IOSSTATE(IGNORE) | PAD_IOSTERM(ENPU)),
/* GPIO_108: SUS_CLK */
- _PAD_CFG_STRUCT(GPIO_108, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF),
- PAD_IOSSTATE(IGNORE)),
+ PAD_NC(GPIO_108, NONE),
/* GPIO_109: PMU_SUS_STAT# */
_PAD_CFG_STRUCT(GPIO_109, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF),
PAD_IOSSTATE(IGNORE)),
- /* GPIO_110: Not Connected */
+ /* GPIO_110: I2C_5 SDA */
PAD_NC(GPIO_110, DN_20K),
- /* GPIO_111: Not Connected */
+ /* GPIO_111: I2C_5 SDL */
PAD_NC(GPIO_111, DN_20K),
- /* GPIO_112: Not Connected */
+ /* GPIO_112: I2C_6 SDA */
PAD_NC(GPIO_112, DN_20K),
- /* GPIO_113: Not Connected */
+ /* GPIO_113: I2C_6 SCL */
PAD_NC(GPIO_113, DN_20K),
- /* GPIO_114: I2C_SDA_CPU */
+ /* GPIO_114: I2C_7 SDA */
_PAD_CFG_STRUCT(GPIO_114, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
(1 << 1), PAD_PULL(UP_1K) | PAD_IOSSTATE(HIZCRx1) | PAD_IOSTERM(ENPU)),
- /* GPIO_115: I2C_SDL_CPU */
+ /* GPIO_115: I2C_7 SCL */
_PAD_CFG_STRUCT(GPIO_115, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
(1 << 1), PAD_PULL(UP_1K) | PAD_IOSSTATE(HIZCRx1) | PAD_IOSTERM(ENPU)),
- /* GPIO_166: WLAN_PCIE_WAKE# */
- _PAD_CFG_STRUCT(GPIO_116, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- (1 << 1), PAD_PULL(UP_20K) | PAD_IOSSTATE(IGNORE) | PAD_IOSTERM(ENPU)),
+ /* GPIO_116: WLAN_PCIE_WAKE# */
+ PAD_NC(GPIO_116, DN_20K),
/* GPIO_117: Not Connected */
PAD_NC(GPIO_117, DN_20K),
/* GPIO_118: Not Connected */
PAD_NC(GPIO_118, DN_20K),
/* GPIO_119: Not Connected */
PAD_NC(GPIO_119, DN_20K),
- /* GPIO_121: WLAN_CLK_REQ# */
- _PAD_CFG_STRUCT(GPIO_120, PAD_FUNC(NF1) | PAD_RESET(DEEP) | PAD_TRIG(OFF) |
- (1 << 1), PAD_PULL(UP_20K) | PAD_IOSSTATE(HIZCRx1) | PAD_IOSTERM(ENPU)),
+ /* GPIO_120: WLAN_CLK_REQ# */
+ PAD_NC(GPIO_120, DN_20K),
/* GPIO_121: Not Connected */
PAD_NC(GPIO_121, DN_20K),
/* GPIO_122: Not Connected */
@@ -422,13 +406,13 @@
/* GPIO_160: Not Connected */
PAD_NC(GPIO_160, DN_20K),
/* GPIO_161: GPIO_161 */
- PAD_CFG_GPI_TRIG_OWN(GPIO_161, DN_20K, DEEP, OFF, ACPI),
+ PAD_NC(GPIO_161, DN_20K),
/* GPIO_162: Not Connected */
PAD_NC(GPIO_162, DN_20K),
/* GPIO_163: GPIO_163 */
- PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_163, 1, DEEP, UP_20K, IGNORE, ENPU),
+ PAD_NC(GPIO_163, DN_20K),
/* GPIO_164: GPIO_164 */
- PAD_CFG_GPI_TRIG_OWN(GPIO_164, DN_20K, DEEP, OFF, ACPI),
+ PAD_NC(GPIO_164, DN_20K),
/* GPIO_165: GPIO_165 */
PAD_NC(GPIO_165, DN_20K),
/* GPIO_166: HDA_BCLK_CPU_R */
@@ -456,10 +440,10 @@
PAD_NC(GPIO_172, DN_20K),
/* GPIO_173: Not Connected */
PAD_NC(GPIO_173, DN_20K),
- /* GPIO_174: GPIO_174 */
+ /* GPIO_174: VDD2 1.20V / 1.24V Select */
PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_174, 1, DEEP, DN_20K, TxLASTRxE, ENPU),
/* GPIO_175: GPIO_175 */
- PAD_CFG_GPI_TRIG_OWN(GPIO_175, DN_20K, DEEP, OFF, ACPI),
+ PAD_NC(GPIO_175, DN_20K),
/* ----- GPIO Group SCC ----- */
/* GPIO_176: Not Connected */
@@ -481,8 +465,7 @@
/* GPIO_184: Not Connected */
PAD_NC(GPIO_184, DN_20K),
/* GPIO_185: WLAN_RST_N_R */
- _PAD_CFG_STRUCT(GPIO_185, PAD_FUNC(GPIO) | PAD_RESET(DEEP) |
- PAD_TRIG(OFF) | PAD_BUF(TX_RX_DISABLE), PAD_IOSSTATE(IGNORE)),
+ PAD_NC(GPIO_185, DN_20K),
/* GPIO_186: Not Connected */
PAD_NC(GPIO_186, DN_20K),
/* GPIO_188: Not Connected */
@@ -500,14 +483,14 @@
_PAD_CFG_STRUCT(GPIO_192, PAD_FUNC(NF1) | PAD_RESET(DEEP) |
PAD_TRIG(OFF) | (1 << 1), PAD_PULL(UP_20K) | PAD_IOSSTATE(IGNORE) |
PAD_IOSTERM(ENPU)),
- /* GPIO_192: CNVI_RGI_DT */
+ /* GPIO_193: CNVI_RGI_DT */
_PAD_CFG_STRUCT(GPIO_193, PAD_FUNC(NF1) | PAD_RESET(DEEP) |
PAD_TRIG(OFF), PAD_IOSSTATE(IGNORE)),
- /* GPIO_192: GPIO_194 */
+ /* GPIO_194: GPIO_194 */
_PAD_CFG_STRUCT(GPIO_194, PAD_FUNC(NF1) | PAD_RESET(DEEP) |
PAD_TRIG(OFF), PAD_PULL(UP_20K) | PAD_IOSSTATE(IGNORE) |
PAD_IOSTERM(ENPU)),
- /* GPIO_192: CNVI_RF_RESET_N */
+ /* GPIO_195: CNVI_RF_RESET_N */
_PAD_CFG_STRUCT(GPIO_195, PAD_FUNC(NF1) | PAD_RESET(DEEP) |
PAD_TRIG(OFF), PAD_IOSSTATE(IGNORE)),
/* GPIO_196: GPIO_196 */