Re: [coreboot] [PATCH] More Kconfig changes

[Myles Watson]

> > We should add a MEM_TRAIN_SEQ comment or help text, it's unclear to me
>
> what it is
> > supposed to do. Is it a per-chipset, per-cpu, or per-board option? What
> > do the values of the variable mean?
> >
> > Regardless of that, it seems like it should be a user-visible option in
> > menuconfig
>
> Shouldn't we clarify what it does _before_ deciding it should be user
> visible? ;-)

Agreed.
> Maybe we can even drop the option completely?
> Anyone knows more details what the option is good for?

Unfortunately, I don't.  I was just trying to match newconfig.

Thanks,
Myles