[LinuxBIOS] [PATCH] flashrom: prepare for upcoming massive SPI chip support

Ward Vandewege ward at gnu.org
Mon Dec 17 14:54:44 CET 2007


On Mon, Dec 17, 2007 at 01:14:35AM +0100, Carl-Daniel Hailfinger wrote:
> To make it easier to add new SPI chips to flashchips.c, rename functions
> with multiple possible opcodes from linear numbering at the end (_1, _2)
> to include the opcode at the end (_60, _c7). That way, you only have to
> take a short look at the data sheet and choose the right function by
> appending the opcode listed in the data sheet.
> No functional changes.

I like how this makes the code easier to understand.

> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

Acked-by: Ward Vandewege <ward at gnu.org>

Thanks,
Ward.

-- 
Ward Vandewege <ward at fsf.org>
Free Software Foundation - Senior System Administrator




More information about the coreboot mailing list