[coreboot] [PATCH 2/5] artecgroup/dbe61: Use dbe61c specific variables and macros instead of generic ones
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Thu Nov 13 04:27:59 CET 2008
On 13.11.2008 03:56, Mart Raudsepp wrote:
> On N, 2008-11-13 at 01:51 +0100, Carl-Daniel Hailfinger wrote:
>
>> On 13.11.2008 01:18, Mart Raudsepp wrote:
>>
>>> artecgroup/dbe61: Use dbe61c specific variables and macros instead of generic ones
>>>
>>> This is in preparation of dbe61a handling addition.
>>>
>>> Uses DIMM_DBE61C and DIMM_EMPTY instead of generic DIMM0/DIMM1, and
>>> spd_table_dbe61c instead of spd_table.
>>>
>>>
>> I somehow dislike using 0xA2 for DIMM_EMPTY. Sorry.
>>
>
> Note that 0xA2 is used by all other boards that have soldered on memory
> as one DIMM. It's just named perhaps less intuitively DIMM1.
> In case of DBE61 what really makes these values perhaps not ideal, is
> the addition of a third one that is an OR relationship with the first.
>
Hm yes. Could you convert all other boards with soldered DIMMS to the
DIMM_EMPTY scheme as well? That change would be
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
> What random numbers between 0x00 and 0xFF would you suggest for all the
> three then? :)
>
You gave a perfectly valid reason to keep it as is. Go for it. If we
ever want to move DIMM_EMPTY to 0x0 or something else, we can do so in a
separate patch.
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
More information about the coreboot
mailing list