[coreboot] [flashrom] [PATCH] superiotool: libsuperiodetect

Michael Karcher flashrom at mkarcher.dialup.fu-berlin.de
Wed Jun 30 16:38:22 CEST 2010

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?

  Michael Karcher

More information about the coreboot mailing list