Skip to content

Commit

Permalink
mb/starlabs/lite/glk: Disconnect unused GPIOs
Browse files Browse the repository at this point in the history 
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>
  • Loading branch information
@Sean-StarLabs @martinlroth
Sean-StarLabs authored and martinlroth committed May 31, 2022
1 parent ec56d6f commit 642c6b1
Showing 1 changed file with 61 additions and 78 deletions.
139 changes: 61 additions & 78 deletions src/mainboard/starlabs/lite/variants/glk/gpio.c
View file
Expand Up @@ -57,8 +57,7 @@ const struct pad_config gpio_table[] = {
/* 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 */
Expand All @@ -81,10 +80,10 @@ const struct pad_config gpio_table[] = {
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 */
Expand All @@ -97,7 +96,7 @@ const struct pad_config gpio_table[] = {
_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),
Expand All @@ -116,15 +115,12 @@ const struct pad_config gpio_table[] = {
/* 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 */
Expand All @@ -137,38 +133,33 @@ const struct pad_config gpio_table[] = {
/* 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),
Expand All @@ -195,7 +186,7 @@ const struct pad_config gpio_table[] = {
_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 */
Expand All @@ -211,25 +202,21 @@ const struct pad_config gpio_table[] = {
/* 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 */
Expand All @@ -241,13 +228,13 @@ const struct pad_config gpio_table[] = {
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 */
Expand All @@ -274,47 +261,44 @@ const struct pad_config gpio_table[] = {
_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)),
/* GPIO_107: PMU_RSTBTN# */
_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 */
Expand Down Expand Up @@ -422,13 +406,13 @@ const struct pad_config gpio_table[] = {
/* 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 */
Expand Down Expand Up @@ -456,10 +440,10 @@ const struct pad_config gpio_table[] = {
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 */
Expand All @@ -481,8 +465,7 @@ const struct pad_config gpio_table[] = {
/* 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 */
Expand All @@ -500,14 +483,14 @@ const struct pad_config gpio_table[] = {
_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 */
Expand Down

0 comments on commit 642c6b1

Please sign in to comment.