Re: [coreboot] Some work on the cn700 ram init

[Peter Stuge]

On Fri, Feb 22, 2008 at 05:25:51PM -0500, Corey Osgood wrote:
> > > +static int ram_speeds[6] = {100, 133, 166, 200, 266, 333};
> > > +/* Transform spd byte 40's tRFC value into a decimal */
> > > +static int byte_40_tRFC[6] = {0, 25, 33, 50, 66, 75 };
> >
> > Could this be something like:
> >
> > static int ram_speed_tRFC[][2] = {
> >        {100, 0}, {133, 25}, {166, 33}, {200, 50}, {266, 66}, {333, 75}
> > }
> >
> > instead?
>
> Yes, but I really don't like it. The two are used entirely
> independently, I don't like combining two tables that are
> completely unrelated to one another.

Hm. Maybe I misunderstood.

I thought one [] was used to find an index from a known/probed value,
and that index then used to look up a value in the other [].

Never mind my comment if that's not what's happening. :)

//Peter