<div>1. that would be too long, do we need DIMM_SOCKETS_PER_CHANNEL_PER_NODE?</div>
<div>2. that is controller array is "static const", you need to change the attribute so you can change the value....that would break the code for ROMCC....</div>
<div>dimm_mask already did popppp for you.</div>
<div> </div>
<div>for rev f.. I have made one CAR only raminit.c, it keep the dimmask and needed dimm spd info in struct mem_info...</div>
<div>So don't need to read pci_conf or smbus to get info....</div>
<div>it include dimm_mask, x4_mask, x16_mask, ecc_mask, registered_mask, is_Width128, is_opteron.....</div>
<div> </div>
<div>YH<br> </div>
<div><span class="gmail_quote">On 10/20/05, <b class="gmail_sendername">San Mehat</b> <<a href="mailto:san@google.com">san@google.com</a>> wrote:</span>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">Greets,<br><br>I've made some modifications locally here to the amdk8 memory code and wanted to just chuck them out there to see what you guys think:
<br><br>1 - Renamed DIMM_SOCKETS define to DIMM_SOCKETS_PER_CHANNEL since thats what it actually is.<br>2 - Added 'channel0_pop' and 'channel1_pop' to 'struct mem_controller' and modified 'raminit.c' to fill this in with the physical DIMM population map.
<br><br>For us knowing the detected DIMM population map *at the mainboard level* is very usefull.<br><br><br>Fire away ;)<br><span class="sg"><br>-San<br><br></span><br>--<br>LinuxBIOS mailing list<br><a onclick="return top.js.OpenExtLink(window,event,this)" href="mailto:LinuxBIOS@openbios.org">
LinuxBIOS@openbios.org</a><br><a onclick="return top.js.OpenExtLink(window,event,this)" href="http://www.openbios.org/mailman/listinfo/linuxbios" target="_blank">http://www.openbios.org/mailman/listinfo/linuxbios</a><br><br>
</blockquote></div><br>