Re: [coreboot] [PATCH]flashrom fix for M50FW080

[Carl-Daniel Hailfinger]

On 14.03.2008 19:55, Peter Stuge wrote:
> On Fri, Mar 14, 2008 at 11:49:07AM -0400, [e-mail] wrote:
> > I verified this is now working.
> >
> > Signed-off-by: Joseph Smith
> > Index: flashrom/flashchips.c
> >
> > --- flashrom/flashchips.c	(revision 3143)
> > +++ flashrom/flashchips.c	(working copy)
> > @@ -165,7 +165,7 @@
> >    {"M50FLW080B",	ST_ID,		ST_M50FLW080B,	1024,	64 * 1024,
> >     probe_jedec,	erase_chip_jedec,	write_jedec},
> >    {"M50FW080",	ST_ID,		ST_M50FW080,	1024,	64 * 1024,
> > probe_jedec,	erase_chip_jedec,	write_jedec},
> > probe_82802ab,	erase_82802ab,		write_82802ab},
> > {"M50FW016",	ST_ID,		ST_M50FW016,	2048,	64 * 1024,
> >   probe_jedec,	erase_chip_jedec,	write_jedec},
> >  {"M50LPW116",	ST_ID,		ST_M50LPW116,	2048,	64 * 1024,
>
> And why is this needed for M50FW080 but not others in the same
> family?

To be honest, it is probably needed for all of them, but I don't have
the time to read all data sheets nor do I have the hardware. We really
need that "supported" flag in flashrom.

Regards,
Carl-Daniel