build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/27421 )
Change subject: google/asuka: Add as a variant of glados ......................................................................
Patch Set 1:
(22 comments)
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... File src/mainboard/google/glados/variants/asuka/include/variant/gpio.h:
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 81: /* SD_PWR_EN */ PAD_CFG_NC(GPP_A17), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 84: /* GYRO_DRDY */ PAD_CFG_NC(GPP_A20), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 88: /* CORE_VID0 */ PAD_CFG_NC(GPP_B0), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 89: /* CORE_VID1 */ PAD_CFG_NC(GPP_B1), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 93: /* SRCCLKREQ0# */ PAD_CFG_GPI_ACPI_SCI(GPP_B5, NONE, DEEP, YES), /* TOUCHPAD WAKE */ line over 80 characters
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 100: /* PM_SLP_S0 */ PAD_CFG_NF(GPP_B12, NONE, DEEP, NF1), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 103: /* GSPI0_CS# */ PAD_CFG_NC(GPP_B15), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 111: /* SM1ALERT# */ PAD_CFG_NC(GPP_B23), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 114: /* SMBALERT# */ PAD_CFG_GPO(GPP_C2, 0, DEEP), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 145: /* ISH_SPI_CSB */ PAD_CFG_NC(GPP_D9), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 162: /* SSD_PEDET */ PAD_CFG_NC(GPP_E2), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 169: /* USB2_OC_0 */ PAD_CFG_NF(GPP_E9, NONE, DEEP, NF1), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 170: /* USB2_OC_1 */ PAD_CFG_NF(GPP_E10, NONE, DEEP, NF1), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 171: /* USB2_OC_2 */ PAD_CFG_NF(GPP_E11, NONE, DEEP, NF1), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 172: /* USB2_OC_3 */ PAD_CFG_NF(GPP_E12, NONE, DEEP, NF1), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 185: /* I2S2_SCLK */ PAD_CFG_NC(GPP_F0), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 186: /* I2S2_SFRM */ PAD_CFG_NC(GPP_F1), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 195: /* AUDIO_IRQ */ PAD_CFG_GPI_APIC(GPP_F10, NONE, PLTRST), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 196: /* AUDIO_IRQ */ PAD_CFG_GPI_ACPI_SCI(GPP_F11, NONE, DEEP, YES), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 225: /* PM_SUSCLK */ PAD_CFG_NF(GPD8, NONE, DEEP, NF1), please, no space before tabs
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... File src/mainboard/google/glados/variants/asuka/variant.c:
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 27: { 0x0F, 0xF0, 0x00, 0xF0, 0x0F, 0xF0 , space prohibited before that ',' (ctx:WxE)
https://review.coreboot.org/#/c/27421/1/src/mainboard/google/glados/variants... PS1, Line 29: { 0x0F, 0xF0, 0x00, 0xF0, 0x0F, 0xF0 , space prohibited before that ',' (ctx:WxE)