[coreboot-gerrit] Change in ...coreboot[master]: spi_flash: Make .read() callback optional

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33282 )

Change subject: spi_flash: Make .read() callback optional
......................................................................

Patch Set 4: Code-Review+2

(1 comment)

https://review.coreboot.org/#/c/33282/4/src/drivers/spi/spi_flash.c 
File src/drivers/spi/spi_flash.c:

https://review.coreboot.org/#/c/33282/4/src/drivers/spi/spi_flash.c@104 
PS4, Line 104: CONFIG(SPI_FLASH_NO_FAST_READ)
Outside the scope of this patch: Is the only use case for this Intel SPI controllers that are locked down and cannot use fast_read? If so a callback that bases the decision on spispeed and in the Intel case the opmenu + lockdown state could be better?