<br><br><div class="gmail_quote">On Tue, Oct 28, 2008 at 10:36 AM, Marc Jones <span dir="ltr"><<a href="mailto:Marc.Jones@amd.com">Marc.Jones@amd.com</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">ron minnich wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Tue, Oct 28, 2008 at 8:03 AM, Myles Watson <<a href="mailto:mylesgw@gmail.com" target="_blank">mylesgw@gmail.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Here's a list of the #defines that I think we should look at.  I would like<br>
to detect as many as possible, and add the rest to the dts or Kconfig.<br>
</blockquote></blockquote></div>
...<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"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

./raminit.c:#if CPU_SOCKET_TYPE == SOCKET_L1<br>
./raminit.c:#if CPU_SOCKET_TYPE == SOCKET_AM2<br>
</blockquote>
<br>
yes, these are trivial to detect at runtime.<br>
<br>
</div><div class="Ih2E3d"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
./raminit.c:#if DIMM_SUPPORT==0x0104   /* DDR2 and REG */<br>
</blockquote>
<br>
This ought to be determined somehow but not sure how.<br>
</div></blockquote>
<br>
These two are related. The socket type defines the package and by extension what type of dimms are supported. It is better to define the socket for the platform than trying to detect it since it can't change.</blockquote>
<div><br>So it should be in the dts, right?  I think the more things we put in one place the easier later ports will be.<br><br>Myles <a href="http://www.amd.com/embeddedprocessors" target="_blank"></a></div></div>