diff options
| author | Konrad Adamczyk <konrada@google.com> | 2023-06-27 12:44:43 +0000 |
|---|---|---|
| committer | Matt DeVillier <matt.devillier@amd.corp-partner.google.com> | 2023-06-29 17:09:41 +0000 |
| commit | 2910a541f809fda288f78476967441b5ba649b9c (patch) | |
| tree | 23eff75db295139df350c2d5d53950f591656fc8 /src/soc/amd/common/fsp | |
| parent | 3ee699abee9289133a73757011a315fc72e66cf9 (diff) | |
| download | coreboot-2910a541f809fda288f78476967441b5ba649b9c.tar.gz coreboot-2910a541f809fda288f78476967441b5ba649b9c.tar.bz2 coreboot-2910a541f809fda288f78476967441b5ba649b9c.zip | |
soc/amd/common/fsp: Fix dimm_num assignment
The dimm_num shall be dimm, not channel.
BUG=b:288520486
TEST=In kernel, see output from `dmidecode -t 17`.
Observe that Locator reflects proper location of the module.
Change-Id: Id876a5c245ed1a145c930b3456830d7b42780b74
Signed-off-by: Konrad Adamczyk <konrada@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/76108
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>
Diffstat (limited to 'src/soc/amd/common/fsp')
| -rw-r--r-- | src/soc/amd/common/fsp/dmi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/amd/common/fsp/dmi.c b/src/soc/amd/common/fsp/dmi.c index a671476c9dc0..408c0e4c2bae 100644 --- a/src/soc/amd/common/fsp/dmi.c +++ b/src/soc/amd/common/fsp/dmi.c @@ -226,7 +226,7 @@ static void prepare_dmi_16_17(void *unused) print_dmi_info(type17_dmi_info); dimm_info = &mem_info->dimm[dimm_cnt]; dimm_info->channel_num = channel; - dimm_info->dimm_num = channel; + dimm_info->dimm_num = dimm; transfer_memory_info(type17_dmi_info, dimm_info); if (use_cbi_part_number) { /* mem_info is memset to 0 above, so it's |