David Hendricks posted comments on this change.
Patch set 4:
New results: https://paste.flashrom.org/view.php?id=3047
(9 comments)
chi*ps*et
oops - done.
Patch Set #4, Line 292: desc->component.FLILL1 == 0) {
Just realized that you changed parentheses here. Which is wrong.
Darn! It felt a bit awkward since I also had to invert logic and a add set of parens.
I tried to make this a bit more idiot-proof by breaking apart the if-statements for FLILL/FLILL1 and the chipset. This way we only need to check the chipset once, and it will make it easier to use a switch statement later if we want. At the very least it should require fewer functioning brain cells when going thru the tedious effort of adding a new chipset.
Patch Set #4, Line 320: unknown
Region 6 (counting from 0) is the Secondary BIOS region, according to
I went ahead and named it BIOS2 as suggested (it was one of a few suggestions actually) on IRC.
IIRC, this is "BMC" for Lewisburg. Maybe name it "EC/BMC"?
Done
Patch Set #4, Line 371: if (cs == CHIPSET_100_SERIES_SUNRISE_POINT) {
missing C620 here
Done. I gave it its own special case since since we have a longer region name length (5 chars instead of 4). Here's how it looks (I'll post another example in case gerrit mangles this):
--- Details ---
FD BIOS ME GbE Pltf DE BIOS2 Reg7 BMC DE2 IE 10GbE OpROM Reg13 Reg14 Reg15
BIOS rw rw rw rw rw r
ME r rw rw r
GbE rw
DE rw rw rw
BMC rw
IE rw
We have to add a sixth region, "unknown", or we'd run into the
Yes, 4th region is DE.
Since I'm not sharing the Sunrise Point code, I guess this can just be "BMC"? Though in the future some PCH that calls it "EC" will re-use the Lewisburg codepath, in which case we can change it.
Patch Set #4, Line 381: msg_pdbg2(" FD BIOS ME GbE Pltf Reg5 Reg6 Reg7 EC Reg9\n");
More regions, sigh.
Done
I just remembered, here's a third list. Though, this one uses identi-
I went ahead and updated this list for what I would expect ifdtool to use, though I haven't gone thru all of ifdtool yet.
I thought access to unallocated space was generally forbidden. I'm not certain of that, though.
File util/ich_descriptors_tool/ich_descriptors_tool.c:
That's a lot of maintenance, maybe we should move the list from
Yes, in another patch :-)
To view, visit change 20922. To unsubscribe, visit settings.