Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33282 )
Change subject: spi_flash: Make .read() callback optional ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/33282/1/src/drivers/spi/spi_flash.c File src/drivers/spi/spi_flash.c:
https://review.coreboot.org/#/c/33282/1/src/drivers/spi/spi_flash.c@116 PS1, Line 116: sizeof(cmd) shouldn't this be cmd_len?