On Fri, Feb 22, 2008 at 7:08 AM, Peter Stuge <<a href="mailto:peter@stuge.se">peter@stuge.se</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Fri, Feb 22, 2008 at 02:30:35AM -0500, Corey Osgood wrote:<br>
> Squash a major bug in the previously committed cn700 patch (spd<br>
> data only shifted 1 bit instead of 4), add Urbez Santana Roma's<br>
> fixes for the ram setup data, and add the start of dynamic ram<br>
> setup. This is an intermediate patch, there will be another to<br>
> finish the dynamic ram setup, but the fix is very necessary. Note<br>
> that this is not build tested.<br>
><br>
> Signed-off-by: Corey Osgood <<a href="mailto:corey.osgood@gmail.com">corey.osgood@gmail.com</a>><br>
<br>
I'd like to see Urbez' signoff in this commit too.</blockquote><div> <br><br>I agree.<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

> +/* Table to transform the ram speed register value into a MHz value */<br>
> +static int ram_speeds[6] = {100, 133, 166, 200, 266, 333};<br>
> +/* Transform spd byte 40's tRFC value into a decimal */<br>
> +static int byte_40_tRFC[6] = {0, 25, 33, 50, 66, 75 };<br>
<br>
Could this be something like:<br>
<br>
static int ram_speed_tRFC[][2] = {<br>
        {100, 0}, {133, 25}, {166, 33}, {200, 50}, {266, 66}, {333, 75}<br>
}<br>
<br>
instead?</blockquote><div><br>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.<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
 Other than that:<br>
<br>
Acked-by: Peter Stuge <<a href="mailto:peter@stuge.se">peter@stuge.se</a>></blockquote><div><br>Thanks, I'll commit once Urbez can sign off. <br><br>-Corey</div></div>