Vladimir Serbinenko (phcoder(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7568
-gerrit
commit b17abbc4e57c58719aee6cba9037fdc4fdedd80e
Author: Vladimir Serbinenko <phcoder(a)gmail.com>
Date: Sat Nov 22 21:12:14 2014 +0100
i945: Disable check for 2-dimm support.
The check is wrong. On Acer Aspire One it returns 0 despite 2 DIMMs working
fine on the same channel if this check is disabled (tested by memtest).
On boards that have only 1 DIMM per channel, the code will simply find no
SPD and skip empty slot.
Change-Id: I5f2fdcd1d948ebf3eabebaea4441af4c19e47f8f
Signed-off-by: Vladimir Serbinenko <phcoder(a)gmail.com>
---
src/northbridge/intel/i945/raminit.c | 14 --------------
1 file changed, 14 deletions(-)
diff --git a/src/northbridge/intel/i945/raminit.c b/src/northbridge/intel/i945/raminit.c
index c858cb8..e9c6956 100644
--- a/src/northbridge/intel/i945/raminit.c
+++ b/src/northbridge/intel/i945/raminit.c
@@ -208,16 +208,6 @@ static int sdram_capabilities_enhanced_addressing_xor(void)
return (!reg8);
}
-static int sdram_capabilities_two_dimms_per_channel(void)
-{
- u8 reg8;
-
- reg8 = pci_read_config8(PCI_DEV(0, 0x00, 0), 0xe8); /* CAPID0 + 8 */
- reg8 &= (1 << 0);
-
- return (reg8 != 0);
-}
-
// TODO check if we ever need this function
#if 0
static int sdram_capabilities_MEM4G_disable(void)
@@ -382,10 +372,6 @@ static void sdram_get_dram_configuration(struct sys_info *sysinfo)
if (!sdram_capabilities_dual_channel() && (dimmno >> 1))
continue;
- /* Two DIMMs per channel not supported, but odd DIMM number? */
- if (!sdram_capabilities_two_dimms_per_channel() && (dimmno& 1))
- continue;
-
printk(BIOS_DEBUG, "DDR II Channel %d Socket %d: ", (dimmno >> 1), (dimmno & 1));
if (sysinfo->spd[dimmno][SPD_MEMORY_TYPE] != SPD_MEMORY_TYPE_SDRAM_DDR2) {
the following patch was just integrated into master:
commit 9d45d6975ccdae7253c75259e1885b68186ce83d
Author: Vladimir Serbinenko <phcoder(a)gmail.com>
Date: Mon Oct 20 19:16:44 2014 +0200
ibexpeak: Merge common NVS init
Change-Id: Ia5e26110928fa011305c13362f20fbe78ca9cf30
Signed-off-by: Vladimir Serbinenko <phcoder(a)gmail.com>
Reviewed-on: http://review.coreboot.org/7134
Tested-by: build bot (Jenkins)
Reviewed-by: Nicolas Reinecke <nr(a)das-labor.org>
See http://review.coreboot.org/7134 for details.
-gerrit
Vladimir Serbinenko (phcoder(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7375
-gerrit
commit e9bb4e59a2ce67f45fb76f33d2daa35fceabcd14
Author: Vladimir Serbinenko <phcoder(a)gmail.com>
Date: Sun Nov 9 13:44:36 2014 +0100
gm45: Link cstates.c rather than including it.
The comment about necessity of include isn't true anymore as get_cst_entries
is not weak anymore so if it's not found, the linking would fail.
Change-Id: I4bf88208d63ac3e625f464c3907e2e1ea575dd9f
Signed-off-by: Vladimir Serbinenko <phcoder(a)gmail.com>
---
src/mainboard/lenovo/x200/Makefile.inc | 1 +
src/mainboard/lenovo/x200/mainboard.c | 3 ---
src/mainboard/roda/rk9/Makefile.inc | 2 ++
src/mainboard/roda/rk9/mainboard.c | 3 ---
4 files changed, 3 insertions(+), 6 deletions(-)
diff --git a/src/mainboard/lenovo/x200/Makefile.inc b/src/mainboard/lenovo/x200/Makefile.inc
index 4eb1216..c3c8a0e 100644
--- a/src/mainboard/lenovo/x200/Makefile.inc
+++ b/src/mainboard/lenovo/x200/Makefile.inc
@@ -18,3 +18,4 @@
##
ramstage-y += dock.c
+ramstage-y += cstates.c
diff --git a/src/mainboard/lenovo/x200/mainboard.c b/src/mainboard/lenovo/x200/mainboard.c
index 5354834..8ba7a35 100644
--- a/src/mainboard/lenovo/x200/mainboard.c
+++ b/src/mainboard/lenovo/x200/mainboard.c
@@ -32,9 +32,6 @@
#include <ec/acpi/ec.h>
#include <ec/lenovo/h8/h8.h>
-#include "cstates.c" /* Include it, as the linker won't find
- the overloaded weak function in there. */
-
const char *smbios_mainboard_bios_version(void)
{
static char *s = NULL;
diff --git a/src/mainboard/roda/rk9/Makefile.inc b/src/mainboard/roda/rk9/Makefile.inc
index 1a6cb02..d69f715 100644
--- a/src/mainboard/roda/rk9/Makefile.inc
+++ b/src/mainboard/roda/rk9/Makefile.inc
@@ -18,3 +18,5 @@
##
ramstage-$(CONFIG_CARDBUS_PLUGIN_SUPPORT) += ti_pci7xx1.c
+ramstage-y += cstates.c
+
diff --git a/src/mainboard/roda/rk9/mainboard.c b/src/mainboard/roda/rk9/mainboard.c
index 4ae004b..1b607cc 100644
--- a/src/mainboard/roda/rk9/mainboard.c
+++ b/src/mainboard/roda/rk9/mainboard.c
@@ -26,9 +26,6 @@
#include <pc80/keyboard.h>
#include <ec/acpi/ec.h>
-#include "cstates.c" /* Include it, as the linker won't find
- the overloaded weak function in there. */
-
static void ec_setup(void)
{
/* Thermal limits? Values are from ectool's ram dump. */
Vladimir Serbinenko (phcoder(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7375
-gerrit
commit 8c0fd0c8de83362923017d786b61a7dba375e719
Author: Vladimir Serbinenko <phcoder(a)gmail.com>
Date: Sun Nov 9 13:44:36 2014 +0100
gm45: Link cstates.c rather than including it.
The comment about necessity of include isn't true anymore as get_cst_entries
is not weak anymore so if it's not found, the linking would fail.
Change-Id: I4bf88208d63ac3e625f464c3907e2e1ea575dd9f
Signed-off-by: Vladimir Serbinenko <phcoder(a)gmail.com>
---
src/mainboard/lenovo/x200/Makefile.inc | 1 +
src/mainboard/lenovo/x200/mainboard.c | 3 ---
src/mainboard/roda/rk9/Makefile.inc | 2 ++
src/mainboard/roda/rk9/mainboard.c | 3 ---
4 files changed, 3 insertions(+), 6 deletions(-)
diff --git a/src/mainboard/lenovo/x200/Makefile.inc b/src/mainboard/lenovo/x200/Makefile.inc
index 4eb1216..c3c8a0e 100644
--- a/src/mainboard/lenovo/x200/Makefile.inc
+++ b/src/mainboard/lenovo/x200/Makefile.inc
@@ -18,3 +18,4 @@
##
ramstage-y += dock.c
+ramstage-y += cstates.c
diff --git a/src/mainboard/lenovo/x200/mainboard.c b/src/mainboard/lenovo/x200/mainboard.c
index 5354834..8ba7a35 100644
--- a/src/mainboard/lenovo/x200/mainboard.c
+++ b/src/mainboard/lenovo/x200/mainboard.c
@@ -32,9 +32,6 @@
#include <ec/acpi/ec.h>
#include <ec/lenovo/h8/h8.h>
-#include "cstates.c" /* Include it, as the linker won't find
- the overloaded weak function in there. */
-
const char *smbios_mainboard_bios_version(void)
{
static char *s = NULL;
diff --git a/src/mainboard/roda/rk9/Makefile.inc b/src/mainboard/roda/rk9/Makefile.inc
index 1a6cb02..d69f715 100644
--- a/src/mainboard/roda/rk9/Makefile.inc
+++ b/src/mainboard/roda/rk9/Makefile.inc
@@ -18,3 +18,5 @@
##
ramstage-$(CONFIG_CARDBUS_PLUGIN_SUPPORT) += ti_pci7xx1.c
+ramstage-y += cstates.c
+
diff --git a/src/mainboard/roda/rk9/mainboard.c b/src/mainboard/roda/rk9/mainboard.c
index 4ae004b..1b607cc 100644
--- a/src/mainboard/roda/rk9/mainboard.c
+++ b/src/mainboard/roda/rk9/mainboard.c
@@ -26,9 +26,6 @@
#include <pc80/keyboard.h>
#include <ec/acpi/ec.h>
-#include "cstates.c" /* Include it, as the linker won't find
- the overloaded weak function in there. */
-
static void ec_setup(void)
{
/* Thermal limits? Values are from ectool's ram dump. */