[coreboot] v3 HT
rminnich at gmail.com
Tue Oct 28 18:18:07 CET 2008
On Tue, Oct 28, 2008 at 10:05 AM, Myles Watson <mylesgw at gmail.com> wrote:
> On Tue, Oct 28, 2008 at 10:36 AM, Marc Jones <Marc.Jones at amd.com> wrote:
>> ron minnich wrote:
>>> On Tue, Oct 28, 2008 at 8:03 AM, Myles Watson <mylesgw at gmail.com> wrote:
>>>> Here's a list of the #defines that I think we should look at. I would
>>>> to detect as many as possible, and add the rest to the dts or Kconfig.
>>>> ./raminit.c:#if CPU_SOCKET_TYPE == SOCKET_L1
>>>> ./raminit.c:#if CPU_SOCKET_TYPE == SOCKET_AM2
>>> yes, these are trivial to detect at runtime.
>>>> ./raminit.c:#if DIMM_SUPPORT==0x0104 /* DDR2 and REG */
>>> This ought to be determined somehow but not sure how.
>> These two are related. The socket type defines the package and by
>> extension what type of dimms are supported. It is better to define the
>> socket for the platform than trying to detect it since it can't change.
> So it should be in the dts, right? I think the more things we put in one
> place the easier later ports will be.
once we get dts available in initram, which it is not. Leave it a cpp
define for now.
More information about the coreboot