[coreboot] [coreboot-gerrit] Patch set updated for coreboot: 26eaf9d Family14: Add support for solder-down DDR memory
paulepanter at users.sourceforge.net
Sat Dec 14 13:08:08 CET 2013
Dear Dave, dear coreboot folks,
hopefully this can be discussed and agreed upon more quickly on the
Am Samstag, den 14.12.2013, 01:25 +0100 schrieb Dave Frodin:
> Dave Frodin (dave.frodin at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4532
> commit 26eaf9ded8eac82dff31dc546f5e1c84c5e63063
> Author: Dave Frodin <dave.frodin at se-eng.com>
> Date: Wed Dec 11 13:28:25 2013 -0700
> Family14: Add support for solder-down DDR memory
> Normally the memory configuration/settings would be
> read from a DIMM/SODIMMs SPD. This change adds support
> for solder-down memory that doesn't have a SPD. The
> data that would normally be read from the SPD instead
> comes from a file.
> Change-Id: Ic10b0c5ca16cbc6af9400ef706392adcb151ac97
> Signed-off-by: Dave Frodin <dave.frodin at se-eng.com>
> src/northbridge/amd/agesa/Kconfig | 12 +++++
> src/northbridge/amd/agesa/family14/dimmSpd.c | 75 ++++++++++++++++++++++++++++
> 2 files changed, 87 insertions(+)
Are there boards in the tree with hard-coded memory
> diff --git a/src/northbridge/amd/agesa/Kconfig b/src/northbridge/amd/agesa/Kconfig
> index d5c039f..5af88f7 100644
> --- a/src/northbridge/amd/agesa/Kconfig
> +++ b/src/northbridge/amd/agesa/Kconfig
> @@ -2,6 +2,7 @@
> # This file is part of the coreboot project.
> # Copyright (C) 2011 - 2012 Advanced Micro Devices, Inc.
> +# Copyright (C) 2013 Sage Electronic Engineering, LLC
> # This program is free software; you can redistribute it and/or modify
> # it under the terms of the GNU General Public License as published by
> @@ -31,6 +32,17 @@ config S3_VGA_ROM_RUN
> default n
> +config DDR3_SOLDERED_DOWN
> + bool
> + default n
In my opinion such an option should be introduced globally and not just
for AMD. Especially as the name does not included »AMD«.
> +config PATH_TO_DDR3_SPD
> + string
> + default ""
> + help
> + This is the path from where it is being used/built to where
> + it is stored.
I have no idea how much data such a file contains. If it is rather small
can it be put into `devicetree.cb` directly?
PS: Dave, Bruce, sorry for sending the message to you twice.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 198 bytes
Desc: This is a digitally signed message part
More information about the coreboot