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>

1 files changed, 61 insertions, 78 deletions

diff --git a/src/mainboard/starlabs/lite/variants/glk/gpio.c b/src/mainboard/starlabs/lite/variants/glk/gpio.c
index 9d83554cdd..928a2116ab 100644
--- a/src/mainboard/starlabs/lite/variants/glk/gpio.c
+++ b/src/mainboard/starlabs/lite/variants/glk/gpio.c
@@ -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		*/
@@ -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		*/
@@ -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),
@@ -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		*/
@@ -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),
@@ -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		*/
@@ -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		*/
@@ -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		*/
@@ -274,9 +261,9 @@ 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)),
@@ -284,37 +271,34 @@ const struct pad_config gpio_table[] = {
 	_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 @@ 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		*/
@@ -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		*/
@@ -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		*/
@@ -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		*/