Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63053 )
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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/63053 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Raul Rangel rrangel@chromium.org Reviewed-by: Elyes Haouas ehaouas@noos.fr --- M src/soc/amd/common/pi/amd_late_init.c 1 file changed, 3 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Elyes Haouas: Looks good to me, approved Raul Rangel: Looks good to me, approved
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);