Duncan Laurie has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32837 )
Change subject: SMBIOS: Fix SPD manufacture ID decoder ......................................................................
SMBIOS: Fix SPD manufacture ID decoder
According to JEP106 from JEDEC, fix manufacture ID of Crucial, Super Talnet and Micron.
Signed-off-by: Lijian Zhao lijian.zhao@intel.com Change-Id: I10a268a7f3bde405b95bd3a16d5d121be623c7ed Reviewed-on: https://review.coreboot.org/c/coreboot/+/32837 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: HAOUAS Elyes ehaouas@noos.fr Reviewed-by: Duncan Laurie dlaurie@chromium.org Reviewed-by: Patrick Rudolph siro@das-labor.org --- M src/arch/x86/smbios.c 1 file changed, 4 insertions(+), 4 deletions(-)
Approvals: build bot (Jenkins): Verified Duncan Laurie: Looks good to me, approved HAOUAS Elyes: Looks good to me, approved Patrick Rudolph: Looks good to me, approved
diff --git a/src/arch/x86/smbios.c b/src/arch/x86/smbios.c index 589f4f0..90cd674 100644 --- a/src/arch/x86/smbios.c +++ b/src/arch/x86/smbios.c @@ -140,7 +140,7 @@ struct smbios_type17 *t) { switch (mod_id) { - case 0x2c80: + case 0x9b85: t->manufacturer = smbios_add_string(t->eos, "Crucial"); break; @@ -172,9 +172,9 @@ t->manufacturer = smbios_add_string(t->eos, "Hynix/Hyundai"); break; - case 0xb502: + case 0x3486: t->manufacturer = smbios_add_string(t->eos, - "SuperTalent"); + "Super Talent"); break; case 0xcd04: t->manufacturer = smbios_add_string(t->eos, @@ -188,7 +188,7 @@ t->manufacturer = smbios_add_string(t->eos, "Elpida"); break; - case 0xff2c: + case 0x2c80: t->manufacturer = smbios_add_string(t->eos, "Micron"); break;