[coreboot] PATCH: Add Spansion S25FL016A to flashrom

Peter Stuge peter at stuge.se
Sun Jan 27 08:08:48 CET 2008


On Sun, Jan 27, 2008 at 03:15:58AM +0100, Stefan Reinauer wrote:
> > #define ID is well tested but I'm not adamant in any way. You're
> > basically asking for less information in source and more at run
> > time. I like more info at run time but I still think it would be
> > nice to have IDs in the code.
> 
> Not really less information in the source. Just a single place for
> that information. flash.h is the only place where the VENDOR_ID
> defines are defined and the array in flashchips.c is the only place
> where they are used. 
> 
> In fact i think it would add information if we replace the
> VENDOR_ID defines with strings and drop the DEVICE_ID defines
> completely.
> 
>  {"W29C011",     WINBOND_ID,     W_29C011,       128, 128, ...}
> 
> should rather read
> 
>  {"Winbond", "W29C011", 0xda, 0xc1, 128, 128, ...}

I like it.


> > I was also thinking that it would/should be simple to tell
> > flashrom about these flash chip parameters at run time so as to
> > not need a rebuild with as-yet unsupported flash.
> 
> yes, this would be really neat.

I'll think a bit more about this. If someone wants to hack on it go
for it.


//Peter




More information about the coreboot mailing list