Hi,
I think I reviewed and committed most of the stuff now, except for the microcode updates, IIRC there will be some bigger patch handling those in the near future anyway.
See some more notes below.
On Tue, Mar 02, 2010 at 11:19:24PM -0500, Keith Hui wrote:
- Adds Asus P2B-LS mainboard
Merged with some small cosmetic changes, thanks.
- Adds RAM detection for i440bx (based on i82830 code). We're no longer hard
coded for 64MB on one row!
Very nice! I did various smaller changes here, should be functionally equivalent mostly though, except for these:
- In i440bx.h: I dropped the generic PCI register definitions, those are not 440BX specific and already defined in pci_def.h.
- Dropped MLT / PCI_LATENCY_TIMER for now, should not be needed, and raminit.c is probably the wrong place for this stuff anyway, I guess.
- Not sure about this change, omitted it for now (need to re-check datasheets):
static const uint32_t refresh_rate_map[] = { - 1, 5, 5, 2, 3, 4 + 1, 1, 1, 2, 3, 4 };
Probably also some more stuff I forgot about. One thing I'd like to do next is to factor out some stuff from set_dram_row_attributes(), that's way too long and too nested for my taste.
- Adds a proper Slot 1 cpu under src/cpu/intel/slot_1. It's a stub copied
from slot_2 but addresses a few FIXMEs. My P2B-LS code refers to this.
Yep, thanks! I'll fixup the other slot 1 boards in a few minutes or so.
- Adds microcode for Intel Tualatin CPUs, cpuid 6B1 and 6B4.* Actually
loading them is pending.
Postponed for now, see above.
Will test the code on a few 440BX boards soonish, and fix the 3 vs. 4 DIMM slots issue.
Thanks, Uwe.