[coreboot] [PATCH] flashrom: Explicit read functions

Uwe Hermann uwe at hermann-uwe.de
Fri May 8 13:57:41 CEST 2009


On Fri, May 08, 2009 at 03:04:18AM +0200, Carl-Daniel Hailfinger wrote:
> Flashrom assumes that the flash chip contents are available via mmap if
> no read function is defined. This special case is handled in lots of
> places all over the code.
> 
> Remove the special case and use the read_memmapped function. Not only
> does this allow us to fix a read bug in flashrom I recently uncovered on
> ICH SPI, it also allows us to add support for Paraflasher to flashrom.
> 
> This depends on the external flasher support patch I sent a few minutes ago.

Please feel free to copy the read_memmapped() from your other patch into
flashrom.c to make this patch build, then it's:

Acked-by: Uwe Hermann <uwe at hermann-uwe.de>

As for the rest of the external flasher infrastructure we might want to
discuss the approach we take a little more (see my other mail).


Uwe.
-- 
http://www.hermann-uwe.de  | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org




More information about the coreboot mailing list