[coreboot] [Patch] CMOS: Add set_option and rework get_option.

Peter Stuge peter at stuge.se
Fri May 29 16:16:33 CEST 2009


Luc Verhaegen wrote:
> That was mighty fast :)

I read it through and I liked it.


> Since this changes "known" behaviour of cmos options, i would
> prefer at least a second Ack though, especially since the above
> reasoning might not be agreed with fully by all.

Sure.


> A quick suggestion would be this:

Or store a 32 bit CRC in NVRAM, calculated from a canonicalized form
of the cmos.layout contents.


> Minor number should be bumped on additional entries or additional
> values, so that only newer option tables (from cb tables or from
> cmos.layout) are allowed by nvram_tool.

Versioning is nice - but do we need it? I think this nvram stuff is
the most infrequent changing part of all of coreboot.


//Peter




More information about the coreboot mailing list