[coreboot] [PATCH] [flashrom] initial numonyx support
Stefan Reinauer
stepan at coresystems.de
Fri Jun 27 18:25:35 CEST 2008
* Peter Stuge <peter at stuge.se> [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.
>
>
> > {"Macronix", "MX25L3205", MX_ID, MX_25L3205, 4096, 256, TEST_OK_PREW, probe_spi_rdid, spi_chip_erase_c7, spi_chip_write, spi_chip_read},
> > {"Macronix", "MX29F002", MX_ID, MX_29F002, 256, 64 * 1024, TEST_UNTESTED, probe_29f002, erase_29f002, write_29f002},
> > + {"Numonyx", "M25PE80", ST_ID, 0x8014, 1024, 256, TEST_UNTESTED, probe_spi_rdid, spi_chip_erase_c7, spi_chip_write, spi_chip_read},
>
> Does this indent just break in the patch but look good in your file?
Thanks for noticing. It's indeed broken. I will fix it in the commit if
I get an Acked.
> And, did you intentionally not make a #define for the product id?
Yes, that is on purpose. I think it is a bad bad idea to define every flash
chip in two different places in two different files in the tree. I think
to remember this was discussed quite a while ago, and to some extent
agreed upon, even.
The array is the only place in the code really using all those defines,
and the each line with the #defined ID also contains the human readable
name of the chip, so it really is a false friend of readability maing
the code uglier and harder to maintain (much like the PCI IDs #define
list)
Stefan
--
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
More information about the coreboot
mailing list