diff options
| author | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2011-10-27 18:42:53 +0200 |
|---|---|---|
| committer | Patrick Georgi <patrick@georgi-clan.de> | 2011-10-28 09:09:40 +0200 |
| commit | ea5c2b62caec8a2acd5298777b825b799e2b9c15 (patch) | |
| tree | 293aa3b0d6c9a986f3a0cd416ce41e9961f55acc /src/pc80/mc146818rtc_early.c | |
| parent | 113c3497201a28fd58335788da5e206ea8902b90 (diff) | |
Fix checksum calculation both in romstage and ramstage.
The earlier fix for CMOS checksums only fixed the function rtc_set_checksum,
which would fix the checksum, but then coreboot would no longer honor the
settings because it assumed the checksum is wrong after this.
This change fixes the remaining functions.
Change-Id: I3f52d074df29fc29ae1d940b3dcec3aa2cfc96a5
Signed-off-by: Stefan Reinauer <reinauer@google.com>
Reviewed-on: http://review.coreboot.org/342
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
Diffstat (limited to 'src/pc80/mc146818rtc_early.c')
| -rw-r--r-- | src/pc80/mc146818rtc_early.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/pc80/mc146818rtc_early.c b/src/pc80/mc146818rtc_early.c index abddf87661..0652f27c47 100644 --- a/src/pc80/mc146818rtc_early.c +++ b/src/pc80/mc146818rtc_early.c @@ -1,3 +1,4 @@ +#include <stdint.h> #include <pc80/mc146818rtc.h> #include <fallback.h> #if CONFIG_USE_OPTION_TABLE @@ -23,13 +24,12 @@ static int cmos_chksum_valid(void) { #if CONFIG_USE_OPTION_TABLE unsigned char addr; - unsigned long sum, old_sum; + u16 sum, old_sum; sum = 0; - /* Comput the cmos checksum */ + /* Compute the cmos checksum */ for(addr = LB_CKS_RANGE_START; addr <= LB_CKS_RANGE_END; addr++) { sum += cmos_read(addr); } - sum = (sum & 0xffff) ^ 0xffff; /* Read the stored checksum */ old_sum = cmos_read(LB_CKS_LOC) << 8; |