[coreboot-gerrit] New patch to review for coreboot: AGESA f14: Fix infinite loop

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Wed Mar 15 07:48:04 CET 2017


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18822

-gerrit

commit 71248b237001624a462d4eeea05e52adce9d9f99
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Wed Mar 15 08:45:00 2017 +0200

    AGESA f14: Fix infinite loop
    
    Fix regression after commit:
      22f32c7 cpu/amd/agesa: Unify init files
    
    Change-Id: I36fb7369084c68577df69abc251c84dad64f7015
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/cpu/amd/agesa/family14/model_14_init.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/cpu/amd/agesa/family14/model_14_init.c b/src/cpu/amd/agesa/family14/model_14_init.c
index ecf1cfb..3103dd5 100644
--- a/src/cpu/amd/agesa/family14/model_14_init.c
+++ b/src/cpu/amd/agesa/family14/model_14_init.c
@@ -62,7 +62,7 @@ static void model_14_init(device_t dev)
 	msr.lo = msr.hi = 0x1e1e1e1e;
 	wrmsr(0x250, msr);
 	wrmsr(0x258, msr);
-	for (msrno = 0x268; msrno <= 0x26f; i++)
+	for (msrno = 0x268; msrno <= 0x26f; msrno++)
 		wrmsr(msrno, msr);
 
 	msr = rdmsr(SYSCFG_MSR);



More information about the coreboot-gerrit mailing list