Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/45748 )
Change subject: spi25: Add 'spi_read_status_register_2()' helper ......................................................................
Patch Set 3: Code-Review+1
(1 comment)
https://review.coreboot.org/c/flashrom/+/45748/3/spi25_statusreg.c File spi25_statusreg.c:
https://review.coreboot.org/c/flashrom/+/45748/3/spi25_statusreg.c@118 PS3, Line 118: return spi_read_status_register_generic(flash, 0x35); /* Unclear if this is a std JEDEC opcode. */ Looks like not, as other vendors use it to enter QPI:
Macronix: https://www.macronix.com/Lists/ApplicationNote/Attachments/1899/AN0251V1%20-... ISSI: http://www.issi.com/WW/pdf/25WP064A.pdf
Others use it differently:
Cypress: https://docs.rs-online.com/e0e8/A700000006483477.pdf
Others use it for some status register:
SST: https://www.tme.eu/Document/84d3bb0d3f6d37c285de97bf579cab66/SST25VF020B.pdf Adesto: https://www.adestotech.com/wp-content/uploads/doc8715.pdf Zbit: https://datasheet.lcsc.com/szlcsc/2003141212_Zbit-ZB25VQ80ATIG_C495747.pdf