Am Mittwoch, den 30.06.2010, 16:29 +0200 schrieb Carl-Daniel Hailfinger:
> Comments welcome.
You want to use that for flashrom. Please be advised that superiotool
not only contains IDs for Super I/O chips, but also for ECs with a Super
I/O-like config space. There are systems with both a Winbond EC and a
Winbond Super I/O, one on 0x2E, the other on 0x4E, so there might be
more than one match. This is something flashrom is not prepared to
handle. If it's just about Super I/O detection, we would need to throw
out the non-Super-I/O-containing ECs from the list.
> + {0x0581, "F8000", /* Fintek/ASUS F8000 */
> + LDNDEF({
> + {EOT}})},
It might be just me, But I don't like the extra line containing just
"boilerplate" stuff. Are you geting serious line-length problems with
putting LDNDEF also on the first line?
Regards,
Michael Karcher
_______________________________________________
flashrom mailing list
[email protected]
http://www.flashrom.org/mailman/listinfo/flashrom
