[coreboot-gerrit] New patch to review for coreboot: 316a9d8 smbios: Fix type1 family setting

Marc Jones (marc.jones@se-eng.com) gerrit at coreboot.org
Wed Jun 10 05:24:24 CEST 2015


Marc Jones (marc.jones at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10498

-gerrit

commit 316a9d8b7d2a19235bf6a7bc2bcb065f2f7f5f79
Author: Marc Jones <marc.jones at se-eng.com>
Date:   Tue Jun 9 21:10:43 2015 -0600

    smbios: Fix type1 family setting
    
    The type1 family setting from chromium was mis-merged into the
    type2 function. Move it to the correct type1 function.
    Bad commit: 51bdc4781635b99d89e6b7a414a2172be8cb690c
    
    Change-Id: I72e6ef80bbf185a39fcf169c8247dc16462e6bc3
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
---
 src/arch/x86/boot/smbios.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/arch/x86/boot/smbios.c b/src/arch/x86/boot/smbios.c
index 11c2d1c..2f1226d 100644
--- a/src/arch/x86/boot/smbios.c
+++ b/src/arch/x86/boot/smbios.c
@@ -326,6 +326,9 @@ static int smbios_write_type1(unsigned long *current, int handle)
 	t->product_name = smbios_add_string(t->eos, smbios_mainboard_product_name());
 	t->serial_number = smbios_add_string(t->eos, smbios_mainboard_serial_number());
 	t->version = smbios_add_string(t->eos, smbios_mainboard_version());
+#ifdef CONFIG_MAINBOARD_FAMILY
+	t->family = smbios_add_string(t->eos, smbios_mainboard_family());
+#endif
 	smbios_mainboard_set_uuid(t->uuid);
 	len = t->length + smbios_string_table_len(t->eos);
 	*current += len;
@@ -345,9 +348,6 @@ static int smbios_write_type2(unsigned long *current, int handle)
 	t->product_name = smbios_add_string(t->eos, smbios_mainboard_product_name());
 	t->serial_number = smbios_add_string(t->eos, smbios_mainboard_serial_number());
 	t->version = smbios_add_string(t->eos, smbios_mainboard_version());
-#ifdef CONFIG_MAINBOARD_FAMILY
-	t->family = smbios_add_string(t->eos, smbios_mainboard_family());
-#endif
 	len = t->length + smbios_string_table_len(t->eos);
 	*current += len;
 	return len;



More information about the coreboot-gerrit mailing list