[coreboot] [PATCH] flashrom: Use chipaddr_t instead of volatile uint8_t *

Uwe Hermann uwe at hermann-uwe.de
Sat May 16 20:06:40 CEST 2009


On Sat, May 16, 2009 at 04:29:14AM +0200, Carl-Daniel Hailfinger wrote:
> chipaddr is my favourite, though, because our accessor functions are
> named chip_read* and chip_write* and this would give us some naming
> consistency. Of course, said consistency can also be achieved by
> renaming the chip accessor functions to flash_read* and flash_write*.
> 
> The name is a not important for me as long as we can kill volatile
> uint8_t * with it.
> :-)
> 
> Do you want me to resend, and with which name?

I'd say let's use flashaddr for now, but IMHO either name is fine.

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


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