Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33282 )
Change subject: spi_flash: Make .read() callback optional ......................................................................
Patch Set 4:
(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 […]
I don't know tbh, I just didn't want to kill the existing functionality in case anyone needed it. It looks like nothing in the tree is selecting this, so I don't think any platform really depends on it. It's probably just meant as an out for cases where you have to work around some weird bug.