[LinuxBIOS] Proposed change to amdk8 memory controller code

yhlu 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
ROMCC....
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,
is_Width128, is_opteron.....
 YH
 On 10/20/05, San Mehat <san at google.com> wrote:
>
> Greets,
>
> 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
> map.
>
> For us knowing the detected DIMM population map *at the mainboard level*
> is very usefull.
>
>
> Fire away ;)
>
> -San
>
>
> --
> LinuxBIOS mailing list
> LinuxBIOS at openbios.org
> http://www.openbios.org/mailman/listinfo/linuxbios
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20051020/973ca663/attachment.html>


More information about the coreboot mailing list