build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/27420 )
Change subject: google/cave: Add as a variant of glados ......................................................................
Patch Set 2:
(13 comments)
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... File src/mainboard/google/glados/variants/cave/include/variant/gpio.h:
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 97: /* SRCCLKREQ0# */ PAD_CFG_GPI_ACPI_SCI(GPP_B5, NONE, DEEP, YES), /* TRACKPAD WAKE */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 108: /* GSPI0_CLK */ PAD_CFG_GPI_ACPI_SCI(GPP_B16, NONE, DEEP, YES), /* WLAN WAKE */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 122: /* SM1CLK */ PAD_CFG_GPI_GPIO_DRIVER(GPP_C6, 20K_PU, DEEP), /* EC_IN_RW */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 128: /* UART1_RXD */ PAD_CFG_GPI_GPIO_DRIVER(GPP_C12, NONE, DEEP), /* MEM_CONFIG[0] */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 129: /* UART1_TXD */ PAD_CFG_GPI_GPIO_DRIVER(GPP_C13, NONE, DEEP), /* MEM_CONFIG[1] */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 130: /* UART1_RTS# */ PAD_CFG_GPI_GPIO_DRIVER(GPP_C14, NONE, DEEP), /* MEM_CONFIG[2] */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 131: /* UART1_CTS# */ PAD_CFG_GPI_GPIO_DRIVER(GPP_C15, NONE, DEEP), /* MEM_CONFIG[3] */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 139: /* UART2_CTS# */ PAD_CFG_GPI_GPIO_DRIVER(GPP_C23, 20K_PU, DEEP), /* PCH_WP */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 167: /* CPU_GP0 */ PAD_CFG_GPI_GPIO_DRIVER(GPP_E3, NONE, DEEP), /* AUDIO_DB_ID */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 171: /* CPU_GP1 */ PAD_CFG_GPI_APIC(GPP_E7, NONE, PLTRST), /* TOUCHSCREEN */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 179: /* DDPD_HPD2 */ PAD_CFG_GPI_ACPI_SMI(GPP_E15, NONE, DEEP, YES), /* EC_SMI_L */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 180: /* DDPE_HPD3 */ PAD_CFG_GPI_ACPI_SCI(GPP_E16, NONE, DEEP, YES), /* EC_SCI_L */ line over 80 characters
https://review.coreboot.org/#/c/27420/2/src/mainboard/google/glados/variants... PS2, Line 241: /* UART2_CTS# */ PAD_CFG_GPI_GPIO_DRIVER(GPP_C23, 20K_PU, DEEP), /* PCH_WP */ line over 80 characters