Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/64452 )
Change subject: mb/starlabs/lite/glk: Disconnect unused GPIOs ......................................................................
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 --- M src/mainboard/starlabs/lite/variants/glk/gpio.c 1 file changed, 61 insertions(+), 78 deletions(-)
Approvals: build bot (Jenkins): Verified Martin L Roth: Looks good to me, approved
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 */
-
Martin L Roth (Code Review)