Patrick Rudolph (siro@das-labor.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13862
-gerrit
commit b3783c7e606d64fe0571318019b2843ac5589f0f Author: Patrick Rudolph siro@das-labor.org Date: Mon Feb 29 18:21:00 2016 +0100
src/device/dram/ddr3: Parse additional information
Parse manufacturer id and ASCII serial. Required for SMBIOS type 17 field.
Change-Id: I710de1a6822e4777c359d0bfecc6113cb2a5ed8e Signed-off-by: Patrick Rudolph siro@das-labor.org --- src/device/dram/ddr3.c | 8 ++++++++ src/include/device/dram/ddr3.h | 4 ++++ 2 files changed, 12 insertions(+)
diff --git a/src/device/dram/ddr3.c b/src/device/dram/ddr3.c index 6bfaabc..b5658ef 100644 --- a/src/device/dram/ddr3.c +++ b/src/device/dram/ddr3.c @@ -23,6 +23,7 @@ #include <console/console.h> #include <device/device.h> #include <device/dram/ddr3.h> +#include <string.h>
/*============================================================================== * = DDR3 SPD decoding helpers @@ -340,6 +341,13 @@ int spd_decode_ddr3(dimm_attr * dimm, spd_raw_data spd) dimm->reference_card = spd[62] & 0x1f; printram(" DIMM Reference card %c\n", 'A' + dimm->reference_card);
+ dimm->manufacturer_id = (spd[118] << 8) | spd[117]; + printram(" DIMM Manufacturer ID %x\n", dimm->manufacturer_id); + + dimm->part_number[16] = 0; + memcpy(dimm->part_number, &spd[128], 16); + printram(" DIMM Part number %s\n", dimm->part_number); + return ret; }
diff --git a/src/include/device/dram/ddr3.h b/src/include/device/dram/ddr3.h index 0520ead..455dc1c 100644 --- a/src/include/device/dram/ddr3.h +++ b/src/include/device/dram/ddr3.h @@ -165,6 +165,10 @@ typedef struct dimm_attr_st { u16 voltage; /* XMP: max DIMMs per channel supported (1-4) */ u8 dimms_per_channel; + /* Manufacturer ID */ + u16 manufacturer_id; + /* ASCII part number - NULL terminated */ + u8 part_number[17]; } dimm_attr;
/** Result of the SPD decoding process */