[coreboot-gerrit] New patch to review for coreboot: 213f3ab northbridge/amd/amdht/h3finit.c: Fix boot failure

Timothy Pearson (tpearson@raptorengineeringinc.com) gerrit at coreboot.org
Sun Mar 15 02:02:15 CET 2015


Timothy Pearson (tpearson at raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8685

-gerrit

commit 213f3abf49a05eb83454d20cf75cd34b5cb5fb70
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Sat Mar 14 19:59:54 2015 -0500

    northbridge/amd/amdht/h3finit.c: Fix boot failure
    
    GIT hash 586d6e introduced a regression that causes boot failure
    with an f0011449 AMD stop code.
    
    Change-Id: Ieced9088b79bc89d55117b7240b82a086eff9d21
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/northbridge/amd/amdht/h3finit.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/amd/amdht/h3finit.c b/src/northbridge/amd/amdht/h3finit.c
index 47bf809..1f94b44 100644
--- a/src/northbridge/amd/amdht/h3finit.c
+++ b/src/northbridge/amd/amdht/h3finit.c
@@ -77,7 +77,7 @@ static const uint16_t ht_speed_limit[16] =
 
 static const struct ht_speed_limit_map_t {
 	uint16_t mhz;
-	uint8_t config;
+	uint8_t nvram;
 } ht_speed_limit_map[] = {
 	{0, NVRAM_LIMIT_HT_SPEED_AUTO},
 	{200, NVRAM_LIMIT_HT_SPEED_200},
@@ -102,7 +102,7 @@ static const uint16_t ht_speed_mhz_to_hw(uint16_t mhz)
 	size_t i;
 	for (i = 0; i < ARRAY_SIZE(ht_speed_limit_map); i++)
 		if (ht_speed_limit_map[i].mhz == mhz)
-			return ht_speed_limit_map[i].config;
+			return ht_speed_limit[ht_speed_limit_map[i].nvram];
 
 	printk(BIOS_WARNING,
 		"WARNING: Invalid HT link limit frequency %d specified, ignoring...\n",



More information about the coreboot-gerrit mailing list