[coreboot-gerrit] New patch to review for coreboot: 82b47af mainboards/asus/kfsn4-dre: Ignore Coverity false positive

Timothy Pearson (tpearson@raptorengineeringinc.com) gerrit at coreboot.org
Fri Feb 6 23:12:01 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/8377

-gerrit

commit 82b47af67c0b3bf604633a94a349a2f6f1115485
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Fri Feb 6 16:11:16 2015 -0600

    mainboards/asus/kfsn4-dre: Ignore Coverity false positive
    
    Change-Id: I8c0b8f4bab7ac630d48924a40b2714e21b83b572
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/mainboard/asus/kfsn4-dre/romstage.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mainboard/asus/kfsn4-dre/romstage.c b/src/mainboard/asus/kfsn4-dre/romstage.c
index cd8b9de..f197373 100644
--- a/src/mainboard/asus/kfsn4-dre/romstage.c
+++ b/src/mainboard/asus/kfsn4-dre/romstage.c
@@ -232,7 +232,7 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
 	console_init();
 
 	if (CONFIG_MAX_PHYSICAL_CPUS != 2)
-		printk(BIOS_WARNING, "CONFIG_MAX_PHYSICAL_CPUS is %d, but this is a dual socket board!\n", CONFIG_MAX_PHYSICAL_CPUS);
+		/* coverity[CONSTANT_EXPRESSION_RESULT] */ printk(BIOS_WARNING, "CONFIG_MAX_PHYSICAL_CPUS is %d, but this is a dual socket board!\n", CONFIG_MAX_PHYSICAL_CPUS);
 
 	/* Halt if there was a built in self test failure */
 	report_bist_failure(bist);



More information about the coreboot-gerrit mailing list