build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48518 )
Change subject: mb/google/volteer: Assert BT_DISABLE_L (GPP_A13) in early_gpio_table ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/48518/1/src/mainboard/google/voltee... File src/mainboard/google/volteer/variants/baseboard/gpio.c:
https://review.coreboot.org/c/coreboot/+/48518/1/src/mainboard/google/voltee... PS1, Line 403: /* A13 : PMC_I2C_SCL ==> BT_DISABLE_L */ code indent should use tabs where possible
https://review.coreboot.org/c/coreboot/+/48518/1/src/mainboard/google/voltee... PS1, Line 405: PAD_CFG_GPO(GPP_A13, 0, DEEP), code indent should use tabs where possible
https://review.coreboot.org/c/coreboot/+/48518/1/src/mainboard/google/voltee... PS1, Line 405: PAD_CFG_GPO(GPP_A13, 0, DEEP), please, no spaces at the start of a line