Re: [coreboot] [PATCH] Geode GX2 auto DRAM detect patch

[Myles Watson]

> Op donderdag 14 oktober 2010 23:23:39 schreef u:
> > Maybe we should define it to be 0xFF.  In the future, I would think
> > that these values will move into Kconfig, and I think it will make
> > that conversion easier if we already know which values are bogus.
>
> That sounds reasonable and builds ok so its fine with me.
> Should i redo the patch for that?

No.  I was just thinking that it would be an easy change while you were
splitting the patch.  I'm not familiar enough with the Geode code to feel
comfortable committing it as is.

Thanks,
Myles