New patch to review for coreboot: 7ae18c2 Fix our CMOS checksum algorithm so it matches what /dev/nvram expects

[Stefan Reinauer]

Stefan Reinauer ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/279

-gerrit

commit 7ae18c212c2ccca0c7f4691db0bfb5c89d8ce0ac
Author: Stefan Reinauer 
Date:   Fri Oct 14 15:24:03 2011 -0700

Fix our CMOS checksum algorithm so it matches what /dev/nvram expects

Our cmos checksum is inverted to what the Linux /dev/nvram device expects (and
    BIOSes use). This makes it impossible to use /dev/nvram with coreboot. Fix it!

Change-Id: I239f7e3aca05d3691aee16490dd801df2ccaefd1
    Signed-off-by: Vadim Bendebury 
    Signed-off-by: Stefan Reinauer 
---
 src/pc80/mc146818rtc.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/src/pc80/mc146818rtc.c b/src/pc80/mc146818rtc.c
index f3bee9b..034957a 100644
--- a/src/pc80/mc146818rtc.c
+++ b/src/pc80/mc146818rtc.c
@@ -98,7 +98,6 @@ static void rtc_set_checksum(int range_start, int range_end, int cks_loc)
    for(i = range_start; i <= range_end; i++) {
    	sum += cmos_read(i);
    }
-	sum = ~(sum & 0x0ffff);
    cmos_write(((sum >> 8) & 0x0ff), cks_loc);
    cmos_write(((sum >> 0) & 0x0ff), cks_loc+1);
 }