[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:22:59 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 0ccb05d521df3cc5da8cf0d8e16ee73498f80b56
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 | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/northbridge/intel/i5000/raminit.c b/src/northbridge/intel/i5000/raminit.c
index 1a802b0..3b06a49 100644
--- a/src/northbridge/intel/i5000/raminit.c
+++ b/src/northbridge/intel/i5000/raminit.c
@@ -351,9 +351,8 @@ 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",
+	die("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 (;;);
 	return -1;
 }
 



More information about the coreboot-gerrit mailing list