Attention is currently required from: ChrisEric1 CECL, Thomas Heijligen.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/72020 )
Change subject: Add AMD Ryzen Support & VIA VL805 Support ......................................................................
Patch Set 6:
(3 comments)
Patchset:
PS6: Please drop the AMD changes and just add in VL805 in one patch.
File sb600spi.c:
https://review.coreboot.org/c/flashrom/+/72020/comment/7ad4595c_2f5a0f66 PS6, Line 56: uint8_t RZN32BM = 0; Accessing larger than 16M is indeed a problem. https://review.coreboot.org/c/flashrom/+/71565 fixes it.
https://review.coreboot.org/c/flashrom/+/72020/comment/de025556_ec0d118f PS6, Line 128: (rev == 0x51 || rev == 0x59 || rev == 0x61) Those cases are covered below...