[coreboot] v2[PATCH]RCA RM4100 i82830 support

[Peter Stuge]

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.

Yes, exactly.


> I can put together another patch later, and put the code in ifdefs
> with a note to that effect.

Hm? ifdef what?


//Peter