Lijian Zhao has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32837
Change subject: SMBIOS: Fix SPD manufacture ID decoder ......................................................................
SMBIOS: Fix SPD manufacture ID decoder
According to latest JEP106 from JEDEC, modify manufacture ID accordingly.
Signed-off-by: Lijian Zhao lijian.zhao@intel.com Change-Id: I10a268a7f3bde405b95bd3a16d5d121be623c7ed --- M src/arch/x86/smbios.c 1 file changed, 6 insertions(+), 10 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/37/32837/1
diff --git a/src/arch/x86/smbios.c b/src/arch/x86/smbios.c index 589f4f0..f411bc7 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 0x9b05: t->manufacturer = smbios_add_string(t->eos, "Crucial"); break; @@ -148,7 +148,7 @@ t->manufacturer = smbios_add_string(t->eos, "Ramaxel"); break; - case 0x4f01: + case 0x4f80: t->manufacturer = smbios_add_string(t->eos, "Transcend"); break; @@ -156,10 +156,6 @@ t->manufacturer = smbios_add_string(t->eos, "Kingston"); break; - case 0x987f: - t->manufacturer = smbios_add_string(t->eos, - "Hynix"); - break; case 0x9e02: t->manufacturer = smbios_add_string(t->eos, "Corsair"); @@ -170,11 +166,11 @@ break; case 0xad80: t->manufacturer = smbios_add_string(t->eos, - "Hynix/Hyundai"); + "SK Hynix"); 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 +184,7 @@ t->manufacturer = smbios_add_string(t->eos, "Elpida"); break; - case 0xff2c: + case 0x2c80: t->manufacturer = smbios_add_string(t->eos, "Micron"); break;