[coreboot-gerrit] Patch set updated for coreboot: arch/x86/smbios: Correct manufacturer ID

[HAOUAS Elyes (ehaouas@noos.fr)]

HAOUAS Elyes (ehaouas at noos.fr) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15271

-gerrit

commit 145e6116a19f2d24fd88e8d3d6ba14170bb9a484
Author: Elyes HAOUAS <ehaouas at noos.fr>
Date:   Mon Jun 20 21:19:55 2016 +0200

    arch/x86/smbios: Correct manufacturer ID
    
    Correct standard manufacturer's identification code.
    
    Change-Id: I273711e121a61a91176c15cd4cab75420f1f5a39
    Signed-off-by: Elyes HAOUAS <ehaouas at noos.fr>
---
 src/arch/x86/smbios.c | 23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/src/arch/x86/smbios.c b/src/arch/x86/smbios.c
index 5784beb..e57467a 100644
--- a/src/arch/x86/smbios.c
+++ b/src/arch/x86/smbios.c
@@ -15,6 +15,13 @@
  * GNU General Public License for more details.
  */
 
+/*
+ * Standard Manufacturer’s Identification Code
+ * JEP106AS (Revision of JEP106AR, October 2015)
+ * MAY 2016
+ * http://www.jedec.org/standards-documents/results/JEP106AS
+ */
+
 #include <stdlib.h>
 #include <string.h>
 #include <smbios.h>
@@ -126,8 +133,8 @@ static int smbios_processor_name(char *start)
 /* this function will fill the corresponding manufacturer */
 void smbios_fill_dimm_manufacturer_from_id(uint16_t mod_id, struct smbios_type17 *t)
 {
-	switch (mod_id) {
-		case 0x2c80:
+		switch (mod_id) {
+		case 0x9b05:
 			t->manufacturer = smbios_add_string(t->eos,
 							    "Crucial");
 			break;
@@ -139,7 +146,7 @@ void smbios_fill_dimm_manufacturer_from_id(uint16_t mod_id, struct smbios_type17
 			t->manufacturer = smbios_add_string(t->eos,
 							    "Kingston");
 			break;
-		case 0x987f:
+		case 0xad00:
 			t->manufacturer = smbios_add_string(t->eos,
 							    "Hynix");
 			break;
@@ -151,11 +158,7 @@ void smbios_fill_dimm_manufacturer_from_id(uint16_t mod_id, struct smbios_type17
 			t->manufacturer = smbios_add_string(t->eos,
 							    "OCZ");
 			break;
-		case 0xad80:
-			t->manufacturer = smbios_add_string(t->eos,
-							    "Hynix/Hyundai");
-			break;
-		case 0xb502:
+		case 0x3406:
 			t->manufacturer = smbios_add_string(t->eos,
 							    "SuperTalent");
 			break;
@@ -163,7 +166,7 @@ void smbios_fill_dimm_manufacturer_from_id(uint16_t mod_id, struct smbios_type17
 			t->manufacturer = smbios_add_string(t->eos,
 							    "GSkill");
 			break;
-		case 0xce80:
+		case 0xce00:
 			t->manufacturer = smbios_add_string(t->eos,
 							    "Samsung");
 			break;
@@ -171,7 +174,7 @@ void smbios_fill_dimm_manufacturer_from_id(uint16_t mod_id, struct smbios_type17
 			t->manufacturer = smbios_add_string(t->eos,
 							    "Elpida");
 			break;
-		case 0xff2c:
+		case 0x2c00:
 			t->manufacturer = smbios_add_string(t->eos,
 							    "Micron");
 			break;