[coreboot-gerrit] Patch set updated for coreboot: northbridge/intel/i5000: Convert 'for(; ; )' to 'die'

[HAOUAS Elyes (ehaouas@noos.fr)]

HAOUAS Elyes (ehaouas at noos.fr) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17006

-gerrit

commit 28eb05f7e28ff571d68e349bedb1597b6aa74e1d
Author: Elyes HAOUAS <ehaouas at noos.fr>
Date:   Wed Oct 12 19:57:38 2016 +0200

    northbridge/intel/i5000: Convert 'for(;;)' to 'die'
    
    Change-Id: I1ceea759a40d740503bde725ad6d72fab4aa7971
    Signed-off-by: Elyes HAOUAS <ehaouas at noos.fr>
---
 src/northbridge/intel/i5000/raminit.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/intel/i5000/raminit.c b/src/northbridge/intel/i5000/raminit.c
index 1a802b0..03a95dc 100644
--- a/src/northbridge/intel/i5000/raminit.c
+++ b/src/northbridge/intel/i5000/raminit.c
@@ -351,9 +351,9 @@ static int i5000_amb_smbus_write(struct i5000_fbdimm *d,  int byte1, int byte2)
 	if (status & I5000_SPD_WOD && timeout)
 		return 0;
 
-	printk(BIOS_ERR, "SMBus write failed: %d/%d/%d, byte1 %02x, byte2 %02x status %04x\n",
+	printk(BIOS_ERR,("SMBus write failed: %d/%d/%d, byte1 %02x, byte2 %02x status %04x\n",
 	       d->branch->num, d->channel->num, d->num, byte1, byte2, status);
-	for (;;);
+	die("Error: SMBus write failed");
 	return -1;
 }