include/device/dram: Add SPD lengths for DDR3 to DDR5

DDR2 already had a define to specify the SPD length, but other memory
types did not. This led to the value being coded into other locations.
Unify the definition for DDR2 to DDR5 and put the value at the top of
the respective header file.

Signed-off-by: Martin Roth <gaumless@gmail.com>
Change-Id: Id13b9c5d311984d4a98b831a8746d1659724aa96
Reviewed-on: https://review.coreboot.org/c/coreboot/+/78601
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Keith Hui <buurin@gmail.com>
Reviewed-by: Felix Singer <service+coreboot-gerrit@felixsinger.de>
This commit is contained in:
Martin Roth 2023-10-23 09:59:09 -06:00 committed by Felix Held
parent 7039edd2da
commit 58964ff02c
4 changed files with 12 additions and 3 deletions

View file

@ -18,6 +18,9 @@
#include <spd.h>
#include <device/dram/common.h>
/** Maximum SPD size supported */
#define SPD_SIZE_MAX_DDR2 128
/* Byte 20 [5:0]: DDR2 Module type information */
enum spd_dimm_type_ddr2 {
SPD_DDR2_DIMM_TYPE_UNDEFINED = 0x00,
@ -151,9 +154,6 @@ struct dimm_attr_ddr2_st {
u32 serial;
};
/** Maximum SPD size supported */
#define SPD_SIZE_MAX_DDR2 128
int spd_dimm_is_registered_ddr2(enum spd_dimm_type_ddr2 type);
u8 spd_ddr2_calc_checksum(u8 *spd, int len);
u32 spd_decode_spd_size_ddr2(u8 byte0);

View file

@ -19,6 +19,9 @@
#include <device/dram/common.h>
#include <types.h>
/** Maximum SPD size supported */
#define SPD_SIZE_MAX_DDR3 256
/**
* Convenience definitions for SPD offsets
*

View file

@ -18,6 +18,9 @@
#include <device/dram/common.h>
#include <types.h>
/** Maximum SPD size supported */
#define SPD_SIZE_MAX_DDR4 512
#define SPD_DDR4_PART_OFF 329
#define SPD_DDR4_PART_LEN 20

View file

@ -12,6 +12,9 @@
#include <device/dram/common.h>
#include <types.h>
/** Maximum SPD size supported */
#define SPD_SIZE_MAX_DDR5 1024
/**
* Converts DDR5 clock speed in MHz to the standard reported speed in MT/s
*/