<div dir="ltr">Really nice (and CLEAN...) patch :)<br><br>i do have two questions - <br><br>1. in sdram_set_spd_registers(),  what is the correct order of the 3 following commands:<br><br>spd_set_dram_size();<br>set_dram_buffer_strength();<br>
set_dram_timing();<br><br>since ive noticed its not the same as in other related intel northbridges (not that it counts for something...).<br>i would assume timing & strength should be called before setting the size. no?<br>
<br>2. also, in do_ram_command(), i have no idea how you got those addr_offset from... can you please elaborate or link me to the proper page on the datasheet (or similar)?<br><br>thanks again,<br><br>Elia.<br><br><div class="gmail_quote">
On Fri, Nov 21, 2008 at 7:15 PM, Uwe Hermann <span dir="ltr"><<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Fri, Nov 21, 2008 at 05:04:26PM +0100, Peter Stuge wrote:<br>
> Uwe Hermann wrote:<br>
> > i810: Add support for multiple DIMMs, both single-sided and double-sided,<br>
> > as well as most (all?) combinations thereof.<br>
> ><br>
> > Drop some unused code, the unused row_offset variable, and obsolete comments.<br>
> > Also, fix a typo (thanks to Stefan Reinauer for noticing).<br>
> ><br>
> > This is tested on the MSI MS-6178 with a number of different DIMM<br>
> > combinations and so far all of them worked fine.<br>
> ><br>
> > Signed-off-by: Elia Yehuda <<a href="mailto:z4ziggy@gmail.com">z4ziggy@gmail.com</a>><br>
> > Signed-off-by: Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>><br>
><br>
> Acked-by: Peter Stuge <<a href="mailto:peter@stuge.se">peter@stuge.se</a>><br>
<br>
</div>Thanks, 3765.<br>
<div class="Ih2E3d"><br>
<br>
Uwe.<br>
--<br>
<a href="http://www.hermann-uwe.de" target="_blank">http://www.hermann-uwe.de</a>  | <a href="http://www.holsham-traders.de" target="_blank">http://www.holsham-traders.de</a><br>
<a href="http://www.crazy-hacks.org" target="_blank">http://www.crazy-hacks.org</a> | <a href="http://www.unmaintained-free-software.org" target="_blank">http://www.unmaintained-free-software.org</a><br>
<br>
</div><div><div></div><div class="Wj3C7c">--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a><br>
</div></div></blockquote></div><br></div>