Attention is currently required from: Anastasia Klimchuk, Stefan Reinauer, Subrata Banik.
3 comments:
Commit Message:
Patch Set #1, Line 12: * Incorporate missing region names from https://github.com/coreboot/coreboot/blob/main/util/ifdtool/ifdtool.c for completeness.
Just wanted to follow up on this, since the patch is waiting for a while. […]
Unfortunately, I don't have specific knowledge about 'flash descriptor V2' to help with this.
These patches were tested on MTL (Core Ultra) to ensure Region 9 access. We verified this through local testing, and the `flashrom -VV` command correctly displayed BIOS read/write access permissions for Region 9, which is critical to MTL.
File ich_descriptors.c:
Patch Set #7, Line 558: 10GbE
FYR: C620
File util/ich_descriptors_tool/ich_descriptors_tool.c:
Patch Set #5, Line 42: 10GbE0
This one occurrence that I noticed when the new name is different from the old one. […]
I could only find 2:
1. The one in ich_descriptors.c#442: This table seems to be a summary table, so only include the most frequently occurring names. (Update with 10GbE0, 10GbE1)
2. The one in ichspi.c#558: This table is for Intel C620. See https://www.intel.com/content/dam/www/public/us/en/documents/datasheets/c620-series-chipset-datasheet.pdf datasheet page 1904, they only have one 10GbE and they have OPROM (option ROM)
To view, visit change 81356. To unsubscribe, or for help writing mail filters, visit settings.