Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Arthur Heymans, Fred Reitberger, Felix Held. Hello Arthur Heymans,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/63053
to review the following change.
Change subject: soc/amd/pi/amd_late_init.c: Fix implicit enum conversion ......................................................................
soc/amd/pi/amd_late_init.c: Fix implicit enum conversion
This fixes building with clang.
Change-Id: Ifda9be8996703b06fe9ee30ffb5f56a91629e065 Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/soc/amd/common/pi/amd_late_init.c 1 file changed, 3 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/53/63053/1
diff --git a/src/soc/amd/common/pi/amd_late_init.c b/src/soc/amd/common/pi/amd_late_init.c index 4d2250b..ea16576 100644 --- a/src/soc/amd/common/pi/amd_late_init.c +++ b/src/soc/amd/common/pi/amd_late_init.c @@ -36,8 +36,9 @@
dimm->rank_per_dimm = dmi17->Attributes;
- dimm->mod_type = smbios_form_factor_to_spd_mod_type(dmi17->MemoryType, - dmi17->FormFactor); + dimm->mod_type = smbios_form_factor_to_spd_mod_type( + (smbios_memory_type)dmi17->MemoryType, + (smbios_memory_form_factor)dmi17->FormFactor);
dimm->bus_width = smbios_bus_width_to_spd_width(dmi17->MemoryType, dmi17->TotalWidth, dmi17->DataWidth);