build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38378 )
Change subject: drivers/spi/spi_flash: introduce common spi_flash_part_id object ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/38378/1/src/drivers/spi/spansion.c File src/drivers/spi/spansion.c:
https://review.coreboot.org/c/coreboot/+/38378/1/src/drivers/spi/spansion.c@... PS1, Line 156: if (i == num_parts) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/38378/1/src/drivers/spi/sst.c File src/drivers/spi/sst.c:
https://review.coreboot.org/c/coreboot/+/38378/1/src/drivers/spi/sst.c@285 PS1, Line 285: if (i == num_parts) { braces {} are not necessary for single statement blocks