Attention is currently required from: Nico Huber, Angel Pons. Simon Buhrow has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/58971 )
Change subject: flashchips: Add W25Q64JV ......................................................................
Patch Set 2:
(3 comments)
File flashchips.c:
https://review.coreboot.org/c/flashrom/+/58971/comment/3321bb00_9c2e6158 PS1, Line 17879: */
nit: missing space before `*/`
Done
https://review.coreboot.org/c/flashrom/+/58971/comment/de8c970a_f84bc8b3 PS1, Line 17903: spi_prettyprint_status_register_plain
spi_prettyprint_status_register_bp2_tb_bpl
I wasn´t aware of this function(s)! Thanks!
https://review.coreboot.org/c/flashrom/+/58971/comment/3743869f_1d946532 PS1, Line 17904: spi_disable_blockprotect
spi_disable_blockprotect_bp2_srwd
Done
-
Simon Buhrow (Code Review)