[coreboot] New patch to review for coreboot: d59cb75 nvramtool: Fix CMOS checksum to match coreboot (and /dev/nvram)

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Mon Oct 17 17:59:28 CEST 2011


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/287

-gerrit

commit d59cb7508b77d9f8cc9ed75320c74842674224a0
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Mon Oct 17 08:58:27 2011 -0700

    nvramtool: Fix CMOS checksum to match coreboot (and /dev/nvram)
    
    Change-Id: I28b0dbad36403a31be83581107f40b3ca1332dcc
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
---
 util/nvramtool/cmos_ops.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/util/nvramtool/cmos_ops.c b/util/nvramtool/cmos_ops.c
index 2bbc8a1..91c9f45 100644
--- a/util/nvramtool/cmos_ops.c
+++ b/util/nvramtool/cmos_ops.c
@@ -210,7 +210,7 @@ uint16_t cmos_checksum_compute(void)
 	for (i = cmos_checksum_start; i <= cmos_checksum_end; i++)
 		sum += cmos_read_byte(i);
 
-	return ~((uint16_t) (sum & 0xffff));
+	return (uint16_t)(sum & 0xffff);
 }
 
 /****************************************************************************




More information about the coreboot mailing list