build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/28571 )
Change subject: soc/amd/stoneyridge/gpio.c: Create I2C slave reset code ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/28571/1/src/soc/amd/stoneyridge/gpio.c File src/soc/amd/stoneyridge/gpio.c:
https://review.coreboot.org/#/c/28571/1/src/soc/amd/stoneyridge/gpio.c@352 PS1, Line 352: while(remain) { space required before the open parenthesis '('
https://review.coreboot.org/#/c/28571/1/src/soc/amd/stoneyridge/include/soc/... File src/soc/amd/stoneyridge/include/soc/gpio.h:
https://review.coreboot.org/#/c/28571/1/src/soc/amd/stoneyridge/include/soc/... PS1, Line 39: struct soc_amd_i2c_save{ missing space after struct definition
-
build bot (Jenkins) (Code Review)