1- I dont think we should be worried about names that are too long. I'd
much rather have a long name than a non-descriptive one (or in this
case a totally misleading one)<br>
2- Changing the attribute shouldn't be that big of a deal.<br>
3 - dimm_mask isn't exposed to the mainboard layer which is where we need it. <br>
<br>
<br>
As for 'why we need it', we have a post card which displays the DIMM population map on boot for our ops guys...<br>
<br>
<br><br><div><span class="gmail_quote">On 10/20/05, <b class="gmail_sendername">yhlu</b> <<a href="mailto:yinghailu@gmail.com">yinghailu@gmail.com</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<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><div><span class="e" id="q_1070f245bff50688_1"><span class="gmail_quote">On 10/20/05, <b class="gmail_sendername">San Mehat</b> <<a href="mailto:san@google.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
san@google.com</a>> wrote:</span>
</span></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0px 0px 0px 0.8ex; padding-left: 1ex;"><div><span class="e" id="q_1070f245bff50688_3">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><br>-San<br><br></span><br></span></div>--<br>LinuxBIOS mailing list<br><a href="mailto:LinuxBIOS@openbios.org" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

LinuxBIOS@openbios.org</a><br><a href="http://www.google.com/url?sa=D&q=http%3A%2F%2Fwww.openbios.org%2Fmailman%2Flistinfo%2Flinuxbios" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://www.openbios.org/mailman/listinfo/linuxbios
</a><br><br>
</blockquote></div><br>

</blockquote></div><br>