build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41750 )
Change subject: spi/macronix: Add support for boot media protection ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41750/2/src/drivers/spi/macronix.c File src/drivers/spi/macronix.c:
https://review.coreboot.org/c/coreboot/+/41750/2/src/drivers/spi/macronix.c@... PS2, Line 197: } else { else is not generally useful after a break or return
https://review.coreboot.org/c/coreboot/+/41750/2/src/drivers/spi/macronix.c@... PS2, Line 212: } else { else is not generally useful after a break or return