diff options
author | Konrad Adamczyk <konrada@google.com> | 2023-06-28 12:23:08 +0000 |
---|---|---|
committer | Eric Lai <eric_lai@quanta.corp-partner.google.com> | 2023-07-21 07:30:08 +0000 |
commit | 86dfcb80ce7808103b014ad4cbfc80ecc7a5d355 (patch) | |
tree | 18cd14c47940ac4ed7832a19696aaf399184fb19 /src/soc/amd/common/fsp | |
parent | 0a96a1ca0613e5d7b624e5e608aaeb679ef56dbf (diff) |
vendorcode/amd/fsp/common: Refactor dmi_info.h
SoC family is able to provide SoC-specific information
via amd/fsp/<soc_family>/soc_dmi_info.h.
Use common amd/fsp/common/dmi_info.h for all AMD platforms.
This way, duplicated dmi_info.h files in
vendorcode/amd/fsp/<soc_family>/ can be removed.
BUG=b:288520486
TEST=Dump `dmidecode -t 17`.
Change-Id: I5e0109af51b78360f7038b20a2975aceb721a7d5
Signed-off-by: Konrad Adamczyk <konrada@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/76107
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/amd/common/fsp')
-rw-r--r-- | src/soc/amd/common/fsp/dmi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/soc/amd/common/fsp/dmi.c b/src/soc/amd/common/fsp/dmi.c index 408c0e4c2b..d0d98738e8 100644 --- a/src/soc/amd/common/fsp/dmi.c +++ b/src/soc/amd/common/fsp/dmi.c @@ -216,8 +216,8 @@ static void prepare_dmi_16_17(void *unused) } } - for (unsigned int channel = 0; channel < MAX_CHANNELS_PER_SOCKET; channel++) { - for (unsigned int dimm = 0; dimm < MAX_DIMMS_PER_CHANNEL; dimm++) { + for (unsigned int channel = 0; channel < AGESA_STRUCT_CHANNELS_PER_SOCKET; channel++) { + for (unsigned int dimm = 0; dimm < AGESA_STRUCT_DIMMS_PER_CHANNEL; dimm++) { type17_dmi_info = &dmi_table->T17[0][channel][dimm]; /* DIMMs that are present will have a non-zero handle. */ |