build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32693 )
Change subject: google/mistral: Add LP5562 driver ......................................................................
Patch Set 15:
(10 comments)
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... File src/drivers/i2c/lp5562/led_lp5562_programs.c:
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... PS15, Line 86: }}; space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... PS15, Line 107: }}; space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... PS15, Line 152: }}; space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... PS15, Line 208: }}; space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... PS15, Line 252: }}; space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... PS15, Line 259: {LED_ALL_OFF, {&solid_000000_program}}, space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... PS15, Line 260: {LED_RECOVERY_PUSHED, {&solid_FFFFFF_program}}, space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... PS15, Line 261: {LED_WIPEOUT_REQUEST, {&fdr_press1_program}}, space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... PS15, Line 262: {LED_RECOVERY_REQUEST, {&blink_recovery1_program}}, space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/32693/15/src/drivers/i2c/lp5562/led... PS15, Line 263: {LED_NORMAL_BOOT, {&preboot1_program}}, space required after that close brace '}'