* Peter Stuge <[EMAIL PROTECTED]> [080627 17:48]:
> > * drop erroneous "unknown flash chip" entries. They badly clash with the
> >   "multiple flash chip support" and are no longer required since it's
> >    possible to force a flash chip with Peter's last patch.
> 
> Thumbs up on this. I thought on doing this in my patch, but I wanted
> a flashrom mode that was completely equivalent - ie. it would scan
> for known vendors. But I am fine with removing this right now.

Great. This has been giving me the weirdest errors and outputs over the
time, nearly rendering the tool unusable for production use.

In /dev/bios I had an extra array with { VENDOR_ID, "vendorname" }
that I could use to match an unknown flash chip against a vendor.

One idea I had a long time ago was also to always print an vendor/device
ID in case the probe command changed the chip's state (ie the read ID is
different from the memory read before in non-spi mode)

Then we could identify the "you have a flashchip in your system, but we
don't fully know it" case much better.

Stefan



-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: [EMAIL PROTECTED]  • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866

-- 
coreboot mailing list
[email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to