[coreboot] v2[PATCH]RCA RM4100 i82830 support
peter at stuge.se
Tue Feb 26 03:03:28 CET 2008
On Mon, Feb 25, 2008 at 08:51:00PM -0500, Corey Osgood wrote:
> > > + /* Set the value for DRAM Timing Register */
> > > + /* TODO: Configure the value according to SPD values. */
> > > + pci_write_config32(ctrl->d0, DRT, 0x00000010);
> > How about these two TODOs?
> The memory on the rm4100 is onboard.
I guess this port could use the SPD emulation a la ALIX then?
> The only version with memory slots is the Thomson IP1001
Perhaps another port for that, then?
> without the ability to try other memory sticks or with real spd
> data, it might look right on the rm4100, but fail in practice.
The problem is that this is in (supposedly) generic 830 code.
> > > + /* TODO: Set offset according to DRT values */
> > > + PRINT_DEBUG("RAM Enable 4: Mode register set\r\n");
> > > + do_ram_command(ctrl, RAM_COMMAND_MRS, 0x1d0);
> > Can this be done?
> Same deal as above.
> I can put together another patch later, and put the code in ifdefs
> with a note to that effect.
Hm? ifdef what?
More information about the coreboot