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

HAOUAS Elyes (ehaouas@noos.fr) gerrit at coreboot.org
Wed Oct 19 08:33:42 CEST 2016


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 d3b7c86e9ae3172e5360b44d1057f8130ccec7ac
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 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/northbridge/intel/i5000/raminit.c b/src/northbridge/intel/i5000/raminit.c
index 1a802b0..07ef068 100644
--- a/src/northbridge/intel/i5000/raminit.c
+++ b/src/northbridge/intel/i5000/raminit.c
@@ -353,7 +353,7 @@ static int i5000_amb_smbus_write(struct i5000_fbdimm *d,  int byte1, int byte2)
 
 	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;
 }
 



More information about the coreboot-gerrit mailing list