[coreboot] [PATCH 0/4] Geode GX2 auto DRAM detect patch V2
njacobs8 at hetnet.nl
Mon Nov 1 20:28:27 CET 2010
Thanks for the fast review and commit!
>Heh, fun. This was a bug anyway (0x2000001d vs. 0x2000201d).
>Thanks, r6009. I took the freedom to do a few more whitespace cleanups.
Thanks, i must have overlooked them.
>Thanks, r6011 with some small whitespace changes and s/Mhz/MHz/.
>Maybe we could make a GX2_PROCESSOR_MHZ_300, GX2_PROCESSOR_MHZ_366 etc.
>here, and let each board "select" the correct one in its Kconfig file?
This sounds good, i actually thought about something like that but i didn’t
quite know how.
And because at the moment there are only boards with 366 Mhz so i skipped it.
I will study it some more and try to make a patch for it.
>I'd personally drop this (also in the LX code), let's just use printk(),
>there's no reason to add such a mini-wrapper.
I copied it from LX.
I will make a patch for that when i find some time.
>Shouldn't there be a "while (1)" around the hlt instruction?
?? i don't know should it?
The code seems to work, but if it is preferred/needed i will add it.
Can you point me to some example code or could you supply some code snipped i
More information about the coreboot