[coreboot] [PATCH] Add support for Asus M4A785-M, with build instructions

Myles Watson mylesgw at gmail.com
Fri Sep 10 17:49:39 CEST 2010


On Fri, Sep 10, 2010 at 9:30 AM, Marc Jones <marcj303 at gmail.com> wrote:
> On Thu, Sep 9, 2010 at 1:39 PM, Myles Watson <mylesgw at gmail.com> wrote:
>>> There are now three smaller patches attached:
>> Much nicer!  Thanks.
>>
>>> (1) Changes specific to this board. Fairly small changes from AMD
>>> Tilapia. Requires (2) to work.
>>
>> From your mainboard's Kconfig:
>>
>> +config DIMM_SUPPORT
>> +       hex
>> +       default 0x0004
>> +       depends on CPU_AMD_SOCKET_AM3
>>
>> This really worries me.  You shouldn't need to change the type of memory on
>> the Socket.  I looked at your board online, and they suggest that your board
>> supports socket AM2, AM2+, and AM3.  That seems like it breaks our model.  I
>> thought AM2 was DDR2 and AM3 was DDR3.
>
> That used to be true, but you can have AM3(AM2+) with DDR2, as with
> this board. Yeah, thanks AMD..... :) Connecting the socket to the
> memory type was convenient, but I was never really comfortable with
> it. It may be time to disconnect them.

Thanks Marc.  I didn't realize that we'd gotten ourselves into a mess there.

Thanks,
Myles




More information about the coreboot mailing list