Attention is currently required from: Felix Singer, Daniel Maslowski, Paul Menzel. Sophie van Soest has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/56055 )
Change subject: Enable probing for IT8620 allowing to use the `dualbiosindex` parameter. ......................................................................
Patch Set 6:
(5 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/56055/comment/34452638_32274c4e PS1, Line 7: Enabled
Enable
Done
https://review.coreboot.org/c/flashrom/+/56055/comment/b2b4abe6_ff2551fd PS1, Line 9: Added missing address: 0x8620 for the it97 to use the parameter dualbootbios. : Marked Z97 as OK. : : Dumped and verified both bios.
Nit: I’d use present tense.
Done
Commit Message:
https://review.coreboot.org/c/flashrom/+/56055/comment/579b0f9b_f53b5073 PS4, Line 7: Enable chipset it97spi: 0x8620 Gigabyte-Z97-HD3
I find the summary a little bit irritating. […]
I squashed them into one commit to have a clean patch. I see there no benefit to split the commits up again.
https://review.coreboot.org/c/flashrom/+/56055/comment/0b5f7f9a_48b9f017 PS4, Line 9: Add missing address: 0x8620 for the it97 to use the parameter dualbootbios. : Mark Z97 as OK.
It's not an address, but the ID number of the controller. […]
Done
https://review.coreboot.org/c/flashrom/+/56055/comment/62da7d70_5eb84769 PS4, Line 12: Dumped and verified both bios.
Dumped and verified both firmwares. Tested on GIGABYTE Z97-HD3.
Done