GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

nb/amd_fam10/mct_ddr3: Use common function to compute crc16 checksum 

Change-Id: I730a8a150134cc1ef8fb3872728bb0586ac7b210
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/19732
Tested-by: Raptor Engineering Automated Test Stand <noreply@raptorengineeringinc.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
This commit is contained in:
Arthur Heymans 2017-05-17 10:42:39 +02:00 committed by Martin Roth
parent 4769cc3c0c
commit 0722613563
1 changed files with 6 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
View File

@ -42,6 +42,7 @@
#include <cpu/x86/msr.h>
#include <arch/acpi.h>
#include <string.h>
#include <device/dram/ddr3.h>
#include "s3utils.h"
static u8 ReconfigureDIMMspare_D(struct MCTStatStruc *pMCTstat,
@ -8214,30 +8215,12 @@ static void AfterDramInit_D(struct DCTStatStruc *pDCTstat, u8 dct) {
*/
uint8_t crcCheck(struct DCTStatStruc *pDCTstat, uint8_t dimm)
{
u8 byte_use;
u8 Index;
u16 CRC;
u8 byte, i;
u16 crc_calc = spd_ddr3_calc_crc(pDCTstat->spd_data.spd_bytes[dimm],
sizeof(pDCTstat->spd_data.spd_bytes[dimm]));
u16 checksum_spd = pDCTstat->spd_data.spd_bytes[dimm][SPD_byte_127] << 8
| pDCTstat->spd_data.spd_bytes[dimm][SPD_byte_126];
byte_use = pDCTstat->spd_data.spd_bytes[dimm][SPD_ByteUse];
if (byte_use & 0x80)
byte_use = 117;
else
byte_use = 126;
CRC = 0;
for (Index = 0; Index < byte_use; Index ++) {
byte = pDCTstat->spd_data.spd_bytes[dimm][Index];
CRC ^= byte << 8;
for (i = 0; i < 8; i++) {
if (CRC & 0x8000) {
CRC <<= 1;
CRC ^= 0x1021;
} else
CRC <<= 1;
}
}
return CRC == (pDCTstat->spd_data.spd_bytes[dimm][SPD_byte_127] << 8 | pDCTstat->spd_data.spd_bytes[dimm][SPD_byte_126]);
return crc_calc == checksum_spd;
}
int32_t abs(int32_t val)