build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33283 )
Change subject: spi_flash: Add Dual SPI support ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/33283/1/src/drivers/spi/spi_flash.c File src/drivers/spi/spi_flash.c:
https://review.coreboot.org/#/c/33283/1/src/drivers/spi/spi_flash.c@76 PS1, Line 76: if ((ret = spi_claim_bus(spi))) do not use assignment in if condition
https://review.coreboot.org/#/c/33283/1/src/drivers/spi/spi_flash.c@79 PS1, Line 79: if ((ret = spi_xfer_vector(spi, &vector, 1))) do not use assignment in if condition