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

[Stefan Reinauer]

On 30.05.2009 6:26 Uhr, Peter Stuge wrote:
> Stefan Reinauer wrote:
> > > we need a better way of matching cmos data with the options table.
> >
> > Why?
>
> nvramtool can ask user for a decision.
>
> coreboot can more easily ignore invalid data.
>
> I like validation.

Sorry, I think I completely lost you.

What's wrong with our way of handling nvram again? And what's the
context of your statements above?

Afaict:
- coreboot does validate the nvram before using it.
- coreboot should not try to use nvram if it is not valid.
- nvramtool never _asks_ the user. It's not an interactive tool. Instead
it will fix checksums as soon as you start setting options.

Stefan