[coreboot] [PATCH] flashrom: ST M25P40 bug?

Peter Stuge peter at stuge.se
Thu May 15 05:06:25 CEST 2008


On Thu, May 15, 2008 at 04:37:18AM +0200, Carl-Daniel Hailfinger wrote:
> > Needs a commit message of course.
> 
> Add support for the JEDEC RES (Read Electronic Signature and Resume from
> Powerdown) SPI command to flashrom to identify older SPI chips which
> can't handle JEDEC RDID. Since RES gives a one-byte identifier which is
> shared among many different vendors and even different sizes, we want to
> match RES as a last resort if RDID returns 0xff 0xff 0xff.

Great!


> >> -	{"ST",		"M25P40",		ST_ID,		ST_M25P40,		512,	256,		TEST_UNTESTED,	probe_spi,		spi_chip_erase_c7,	spi_chip_write, spi_chip_read},
> >> +	{"ST",		"M25P40",		ST_ID,		ST_M25P40,		512,	256,		TEST_UNTESTED,	probe_spi_rdid,		spi_chip_erase_c7,	spi_chip_write, spi_chip_read},
> >> +	{"ST",		"M25P40(old)",		ST_ID,		ST_M25P40_RES,		512,	256,		TEST_UNTESTED,	probe_spi_res,		spi_chip_erase_c7,	spi_chip_write, spi_chip_read},
> >>     
> >
> > Hmm. probe_spi_res() calls spi_rdid() first. I don't like duplicating
> > chips? If multiple checks are needed for one chip I think they should
> > be in code?
> 
> To be honest, the "M25P40(old)" entry should more look like
> "STM25P40/SST25something/AT26something/dozens_of_other_chips". For
> more details, see the changelog.

With name change per IRC discussion:

Acked-by: Peter Stuge <peter at stuge.se>




More information about the coreboot mailing list