Matt Parnell has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34890 )
Change subject: util/superiotool: default to using the first discovered chip and exit (let's fix false positives so this isn't necessary); put aspeed in the appropriate execution order to prevent false positives. ......................................................................
Patch Set 1:
Patch Set 1: Code-Review-1
there are systems with more than one super IO chip. I'd strongly prefer fixing the underlying issue than implementing some workaround to hide it, that causes other issues
In that case, currently for the IT8987 I'm getting false positives for these: Found SMSC SCH5027 (id=0x89, rev=0x87) at 0x2e Found Aspeed AST2400 (id=0x00) at 0x2e
And previously a Winbond...