mb/starlabs/starbook/*: Tidy GPIO comments for the touchpad

This is a non-functional change, and only makes the GPIOs easier to read.

Change-Id: I9563f1d3f464b3be35d18d6cd6fbbcee314fce28
Signed-off-by: Sean Rhodes <sean@starlabs.systems>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/87102
Reviewed-by: Matt DeVillier <matt.devillier@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Sean Rhodes 2025-04-02 11:34:46 +01:00
commit e7de832d12
7 changed files with 35 additions and 42 deletions

View file

@ -34,6 +34,11 @@ const struct pad_config gpio_table[] = {
// PAD_CFG_NF_IOSTANDBY_IGNORE(GPP_A9, UP_20K, DEEP, NF1), /* eSPI Clk */
// PAD_CFG_NF_IOSTANDBY_IGNORE(GPP_A10, NONE, DEEP, NF1), /* eSPI Reset */
/* Touchpad */
PAD_CFG_NF(GPP_H4, NONE, DEEP, NF1), /* Data */
PAD_CFG_NF(GPP_H5, NONE, DEEP, NF1), /* Clock */
PAD_CFG_GPI_APIC_LOW(GPP_E12, NONE, PLTRST), /* Interrupt */
/* GPD2: LAN Wake */
PAD_NC(GPD2, NONE),
/* GPD6: Sleep A */
@ -259,8 +264,6 @@ const struct pad_config gpio_table[] = {
PAD_NC(GPP_E10, NONE),
/* E11: TPM IRQ */
PAD_CFG_GPI_APIC_LOW(GPP_E11, NONE, PLTRST),
/* E12: Touchpad Interrupt */
PAD_CFG_GPI_APIC_LOW(GPP_E12, NONE, PLTRST),
/* E13: Not connected */
PAD_NC(GPP_E13, NONE),
/* E14: EDP HPD */
@ -343,10 +346,6 @@ const struct pad_config gpio_table[] = {
PAD_CFG_GPO(GPP_H2, 1, PLTRST),
/* H3: Not Connected */
PAD_NC(GPP_H3, NONE),
/* H4: I2C 0 SDA Touchpad */
PAD_CFG_NF(GPP_H4, NONE, DEEP, NF1),
/* H5: I2C 0 SDL Touchpad */
PAD_CFG_NF(GPP_H5, NONE, DEEP, NF1),
/* H6: Not Connected */
PAD_NC(GPP_H6, NONE),
/* H7: Not Connected */

View file

@ -34,6 +34,11 @@ const struct pad_config gpio_table[] = {
PAD_CFG_NF(GPD5, NONE, DEEP, NF1), /* Sleep S4 */
PAD_CFG_NF(GPD8, NONE, DEEP, NF1), /* Bluetooth Suspend */
/* Touchpad */
PAD_CFG_NF(GPP_C16, NONE, DEEP, NF1), /* Data */
PAD_CFG_NF(GPP_C17, NONE, DEEP, NF1), /* Clock */
PAD_CFG_GPI_APIC_LOW(GPP_B3, NONE, PLTRST), /* Interrupt */
/* GPD2: LAN_WAKE# */
PAD_NC(GPD2, NONE),
/* GPD6: SIO_SLP_A# */
@ -102,8 +107,6 @@ const struct pad_config gpio_table[] = {
PAD_NC(GPP_B1, NONE),
/* B2: Not Connected */
PAD_NC(GPP_B2, NONE),
/* B3: CLICK_PAD_INT_R_N */
PAD_CFG_GPI_APIC_LOW(GPP_B3, NONE, PLTRST),
/* B4: BT_RF_KILL_N */
PAD_CFG_GPO(GPP_B4, 1, DEEP),
/* B5: WLAN_CLKREQ# */
@ -177,10 +180,6 @@ const struct pad_config gpio_table[] = {
PAD_NC(GPP_C14, NONE),
/* C15: TBT_RST_N */
PAD_NC(GPP_C15, NONE),
/* C16: I2C0_SDA */
PAD_CFG_NF(GPP_C16, NONE, DEEP, NF1),
/* C17: I2C0_SCL */
PAD_CFG_NF(GPP_C17, NONE, DEEP, NF1),
/* C18: TOUCH_I2C_SDA */
PAD_NC(GPP_C18, NONE),
/* C19: TOUCH_I2C_SDL */

View file

@ -25,6 +25,11 @@ const struct pad_config gpio_table[] = {
PAD_CFG_NF(GPD5, NONE, PWROK, NF1), /* Sleep S4 */
PAD_CFG_NF(GPD8, NONE, PWROK, NF1), /* Bluetooth Suspend */
/* Touchpad */
PAD_CFG_NF(GPP_C16, NONE, DEEP, NF1), /* Data */
PAD_CFG_NF(GPP_C17, NONE, DEEP, NF1), /* Clock */
PAD_CFG_GPI_APIC_LOW(GPP_C23, NONE, PLTRST), /* Interrupt */
/* GPD2: Not Connected */
PAD_NC(GPD2, NONE),
/* GPD6: Not Connected */
@ -165,18 +170,12 @@ const struct pad_config gpio_table[] = {
PAD_NC(GPP_C14, UP_20K),
/* GPP_C15: Not Connected */
PAD_NC(GPP_C15, UP_20K),
/* GPP_C16: DAT_I2C_TP */
PAD_CFG_NF(GPP_C16, NONE, DEEP, NF1),
/* GPP_C17: CLK_I2C_TP */
PAD_CFG_NF(GPP_C17, NONE, DEEP, NF1),
/* GPP_C18: TOUCH_I2C_SDA */
PAD_CFG_NF(GPP_C18, NONE, DEEP, NF1),
/* GPP_C19: TOUCH_I2C_CLK */
PAD_CFG_NF(GPP_C19, NONE, DEEP, NF1),
/* GPP_C22: Not Connected */
PAD_NC(GPP_C22, NONE),
/* GPP_C23: TOUCHPAD_INT */
PAD_CFG_GPI_APIC_LOW(GPP_C23, NONE, PLTRST),
/* GPP_D0: Not Connected */
PAD_NC(GPP_D0, DN_20K),
/* GPP_D1: Not Connected */

View file

@ -39,6 +39,11 @@ const struct pad_config gpio_table[] = {
// PAD_CFG_NF_IOSTANDBY_IGNORE(GPP_A05, UP_20K, DEEP, NF1), /* eSPI Clk */
// PAD_CFG_NF_IOSTANDBY_IGNORE(GPP_A06, NONE, DEEP, NF1), /* eSPI Reset */
/* Touchpad */
PAD_CFG_NF(GPP_H19, NONE, DEEP, NF1), /* Data */
PAD_CFG_NF(GPP_H20, NONE, DEEP, NF1), /* Clock */
PAD_CFG_GPI_APIC(GPP_B00, NONE, DEEP, LEVEL, INVERT), /* Interrupt */
/* V02: PCIE_WAKE_LAN */
PAD_NC(GPP_V02, NONE),
/* V06: GPD_6_SLP_A_N */
@ -272,10 +277,6 @@ const struct pad_config gpio_table[] = {
PAD_CFG_NF(GPP_H17, NONE, DEEP, NF1),
/* H18: */
PAD_NC(GPP_H18, NONE),
/* H19: TCHPAD_I2C0_SDA_N */
PAD_CFG_NF(GPP_H19, NONE, DEEP, NF1),
/* H20: TCHPAD_I2C0_SCL_N */
PAD_CFG_NF(GPP_H20, NONE, DEEP, NF1),
/* H21: */
PAD_NC(GPP_H21, NONE),
/* H22: */
@ -373,8 +374,6 @@ const struct pad_config gpio_table[] = {
* Start: GPP_B00
* End: GPP_B23
*/
/* B00: TCHPAD_INT_N */
PAD_CFG_GPI_APIC(GPP_B00, NONE, DEEP, LEVEL, INVERT),
/* B01: */
PAD_NC(GPP_B01, NONE),
/* B02: */

View file

@ -34,6 +34,11 @@ const struct pad_config gpio_table[] = {
// PAD_CFG_NF_IOSTANDBY_IGNORE(GPP_A9, UP_20K, DEEP, NF1), /* eSPI Clk */
// PAD_CFG_NF_IOSTANDBY_IGNORE(GPP_A10, NONE, DEEP, NF1), /* eSPI Reset */
/* Touchpad */
PAD_CFG_NF(GPP_H4, NONE, DEEP, NF1), /* Data */
PAD_CFG_NF(GPP_H5, NONE, DEEP, NF1), /* Clock */
PAD_CFG_GPI_APIC_LOW(GPP_E12, NONE, PLTRST), /* Interrupt */
/* GPD2: LAN Wake */
PAD_NC(GPD2, NONE),
/* GPD6: Sleep A */
@ -260,8 +265,6 @@ const struct pad_config gpio_table[] = {
PAD_NC(GPP_E10, NONE),
/* E11: TPM IRQ */
PAD_CFG_GPI_APIC_LOW(GPP_E11, NONE, PLTRST),
/* E12: Touchpad Interrupt */
PAD_CFG_GPI_APIC_LOW(GPP_E12, NONE, PLTRST),
/* E13: Not connected */
PAD_NC(GPP_E13, NONE),
/* E14: EDP HPD */
@ -344,10 +347,6 @@ const struct pad_config gpio_table[] = {
PAD_CFG_GPO(GPP_H2, 1, PLTRST),
/* H3: Not Connected */
PAD_NC(GPP_H3, NONE),
/* H4: I2C 0 SDA Touchpad */
PAD_CFG_NF(GPP_H4, NONE, DEEP, NF1),
/* H5: I2C 0 SDL Touchpad */
PAD_CFG_NF(GPP_H5, NONE, DEEP, NF1),
/* H6: Not Connected */
PAD_NC(GPP_H6, NONE),
/* H7: Not Connected */

View file

@ -25,6 +25,11 @@ const struct pad_config gpio_table[] = {
PAD_CFG_NF(GPD5, NONE, DEEP, NF1), /* Sleep S4 */
PAD_CFG_NF(GPD8, NONE, DEEP, NF1), /* Bluetooth Suspend */
/* Touchpad */
PAD_CFG_NF(GPP_C16, NONE, DEEP, NF1), /* Data */
PAD_CFG_NF(GPP_C17, NONE, DEEP, NF1), /* Clock */
PAD_CFG_GPI_APIC_LOW(GPP_C8, NONE, PLTRST), /* Interrupt */
/* GPD2: LAN_WAKE# */
PAD_NC(GPD2, NONE),
/* GPD6: SIO_SLP_A# */
@ -139,8 +144,6 @@ const struct pad_config gpio_table[] = {
PAD_CFG_NF(GPP_C6, NONE, PWROK, NF1),
/* C7: SML1_DATA */
PAD_CFG_NF(GPP_C7, NONE, PWROK, NF1),
/* C8: CLICK_PAD_INT_N */
PAD_CFG_GPI_APIC_LOW(GPP_C8, NONE, PLTRST),
/* C9: Not Connected */
PAD_NC(GPP_C9, NONE),
/* C10: Not Connected */
@ -155,10 +158,6 @@ const struct pad_config gpio_table[] = {
PAD_CFG_NF(GPP_C14, NONE, DEEP, NF1),
/* C15: TPM_RST */
PAD_NC(GPP_C15, NONE),
/* C16: I2C0_SDA */
PAD_CFG_NF(GPP_C16, NONE, DEEP, NF1),
/* C17: I2C0_SCL */
PAD_CFG_NF(GPP_C17, NONE, DEEP, NF1),
/* C18: TOUCH_I2C_SDA */
PAD_CFG_NF(GPP_C18, NONE, DEEP, NF1),
/* C19: TOUCH_I2C_CLK */

View file

@ -44,6 +44,11 @@ const struct pad_config gpio_table[] = {
// PAD_CFG_NF_IOSTANDBY_IGNORE(GPP_A9, UP_20K, DEEP, NF1), /* eSPI Clk */
// PAD_CFG_NF_IOSTANDBY_IGNORE(GPP_A10, NONE, DEEP, NF1), /* eSPI Reset */
/* Touchpad */
PAD_CFG_NF(GPP_H4, NONE, DEEP, NF1), /* Data */
PAD_CFG_NF(GPP_H5, NONE, DEEP, NF1), /* Clock */
PAD_CFG_GPI_APIC_LOW(GPP_D11, NONE, PLTRST), /* Interrupt */
/* GPD2: LAN Wake */
PAD_NC(GPD2, NONE),
/* GPD6: Sleep A */
@ -220,8 +225,6 @@ const struct pad_config gpio_table[] = {
PAD_NC(GPP_D9, NONE),
/* D10: TPM_IRQ */
PAD_CFG_GPI_APIC_LOW(GPP_D10, NONE, PLTRST),
/* D11: TCHPAD_INT_N */
PAD_CFG_GPI_APIC_LOW(GPP_D11, NONE, PLTRST),
/* D12: ES8336_INT_N */
PAD_NC(GPP_D12, NONE),
/* D13: Wireless LAN Wake */
@ -341,10 +344,6 @@ const struct pad_config gpio_table[] = {
PAD_CFG_GPO(GPP_H2, 1, PLTRST),
/* H3: Not Connected */
PAD_NC(GPP_H3, NONE),
/* H4: I2C 0 SDA Touchpad */
PAD_CFG_NF(GPP_H4, NONE, DEEP, NF1),
/* H5: I2C 0 SDL Touchpad */
PAD_CFG_NF(GPP_H5, NONE, DEEP, NF1),
/* H6: Not Connected */
PAD_NC(GPP_H6, NONE),
/* H7: Not Connected */