summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2016-11-18 18:41:17 +0200
committerKyösti Mälkki <kyosti.malkki@gmail.com>2016-11-20 21:21:59 +0100
commit7dc4b84d8c61f233cb33f2adb7962f2f8a95badb (patch)
treec151610d382009fe0b7617c7c93375889346e3a5
parentc13d65c29b6219d4b765f40e661548eb389524b5 (diff)
downloadcoreboot-7dc4b84d8c61f233cb33f2adb7962f2f8a95badb.tar.xz
device/dram/ddr3: Calculate CRC16 of SPD unique identifier
Specification allows for the unique identifier bytes 117..125 to be excluded of CRC calculation. For such SPD, the CRC would not identify replacement between two identical DIMM parts, while memory training needs to be redone. Change-Id: I8e830018b15c344d9f72f921ab84893f633f7654 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: https://review.coreboot.org/17486 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Rudolph <siro@das-labor.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
-rw-r--r--src/device/dram/ddr3.c53
-rw-r--r--src/include/device/dram/ddr3.h1
2 files changed, 38 insertions, 16 deletions
diff --git a/src/device/dram/ddr3.c b/src/device/dram/ddr3.c
index cb5b685d10..b3bcd680fe 100644
--- a/src/device/dram/ddr3.c
+++ b/src/device/dram/ddr3.c
@@ -46,6 +46,24 @@ int dimm_is_registered(enum spd_dimm_type type)
return 0;
}
+static u16 crc16(const u8 *ptr, int n_crc)
+{
+ int i;
+ u16 crc = 0;
+
+ while (--n_crc >= 0) {
+ crc = (crc ^ (int)*ptr++) << 8;
+ for (i = 0; i < 8; ++i)
+ if (crc & 0x8000) {
+ crc = (crc << 1) ^ 0x1021;
+ } else {
+ crc = crc << 1;
+ }
+ }
+
+ return crc;
+}
+
/**
* \brief Calculate the CRC of a DDR3 SPD
*
@@ -56,9 +74,7 @@ int dimm_is_registered(enum spd_dimm_type type)
*/
u16 spd_ddr3_calc_crc(u8 *spd, int len)
{
- int n_crc, i;
- u8 *ptr;
- u16 crc;
+ int n_crc;
/* Find the number of bytes covered by CRC */
if (spd[0] & 0x80) {
@@ -71,19 +87,24 @@ u16 spd_ddr3_calc_crc(u8 *spd, int len)
/* Not enough bytes available to get the CRC */
return 0;
- /* Compute the CRC */
- crc = 0;
- ptr = spd;
- while (--n_crc >= 0) {
- crc = crc ^ (int)*ptr++ << 8;
- for (i = 0; i < 8; ++i)
- if (crc & 0x8000) {
- crc = crc << 1 ^ 0x1021;
- } else {
- crc = crc << 1;
- }
- }
- return crc;
+ return crc16(spd, n_crc);
+}
+
+/**
+ * \brief Calculate the CRC of a DDR3 SPD unique identifier
+ *
+ * @param spd pointer to raw SPD data
+ * @param len length of data in SPD
+ *
+ * @return the CRC of SPD data bytes 117..127, or 0 when spd data is truncated.
+ */
+u16 spd_ddr3_calc_unique_crc(u8 *spd, int len)
+{
+ if (len < (117 + 11))
+ /* Not enough bytes available to get the CRC */
+ return 0;
+
+ return crc16(&spd[117], 11);
}
/**
diff --git a/src/include/device/dram/ddr3.h b/src/include/device/dram/ddr3.h
index 7388012f86..905aa8407a 100644
--- a/src/include/device/dram/ddr3.h
+++ b/src/include/device/dram/ddr3.h
@@ -199,6 +199,7 @@ enum ddr3_xmp_profile {
typedef u8 spd_raw_data[256];
u16 spd_ddr3_calc_crc(u8 *spd, int len);
+u16 spd_ddr3_calc_unique_crc(u8 *spd, int len);
int spd_decode_ddr3(dimm_attr * dimm, spd_raw_data spd_data);
int dimm_is_registered(enum spd_dimm_type type);
void dram_print_spd_ddr3(const dimm_attr * dimm);