David Hendricks has posted comments on this change. ( https://review.coreboot.org/22384 )
Change subject: spi25: Integrate 4BA support ......................................................................
Patch Set 2: Code-Review+2
(1 comment)
Small nit, otherwise LGTM.
https://review.coreboot.org/#/c/22384/2/flash.h File flash.h:
https://review.coreboot.org/#/c/22384/2/flash.h@241 PS2, Line 241: int address_high_byte; : bool in_4ba_mode; A comment above these should be added to clarify how they affect the way the high address byte is communicated to the chip. Basically just restate here what you have in the commit message.