[LinuxBIOS] Proposed change to amdk8 memory controller code
yinghailu at gmail.com
Thu Oct 20 19:47:06 CEST 2005
1. that would be too long, do we need DIMM_SOCKETS_PER_CHANNEL_PER_NODE?
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
dimm_mask already did popppp for you.
for rev f.. I have made one CAR only raminit.c, it keep the dimmask and
needed dimm spd info in struct mem_info...
So don't need to read pci_conf or smbus to get info....
it include dimm_mask, x4_mask, x16_mask, ecc_mask, registered_mask,
On 10/20/05, San Mehat <san at google.com> wrote:
> 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:
> 1 - Renamed DIMM_SOCKETS define to DIMM_SOCKETS_PER_CHANNEL since thats
> what it actually is.
> 2 - Added 'channel0_pop' and 'channel1_pop' to 'struct mem_controller' and
> modified 'raminit.c' to fill this in with the physical DIMM population
> For us knowing the detected DIMM population map *at the mainboard level*
> is very usefull.
> Fire away ;)
> LinuxBIOS mailing list
> LinuxBIOS at openbios.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the coreboot