[coreboot-gerrit] New patch to review for coreboot: SMBIOS: Implement SKU field

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Mon May 30 16:21:10 CEST 2016


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14996

-gerrit

commit 9e8fcbd811ee1a8374a8485d36dde5d0aba138fb
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Mon May 30 16:52:31 2016 +0300

    SMBIOS: Implement SKU field
    
    Leave it for the platform to fill in the string.
    
    Change-Id: I7b4fe585f8d1efc8c9743f0d8b38de1f98124aab
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/arch/x86/smbios.c | 7 +++++++
 src/include/smbios.h  | 1 +
 2 files changed, 8 insertions(+)

diff --git a/src/arch/x86/smbios.c b/src/arch/x86/smbios.c
index 7c5497e..b590388 100644
--- a/src/arch/x86/smbios.c
+++ b/src/arch/x86/smbios.c
@@ -334,6 +334,12 @@ const char *__attribute__((weak)) smbios_mainboard_product_name(void)
 	return CONFIG_MAINBOARD_SMBIOS_PRODUCT_NAME;
 }
 
+const char *__attribute__((weak)) smbios_mainboard_sku(void)
+{
+	static const char empty_sku[] = "";
+	return empty_sku;
+}
+
 void __attribute__((weak)) smbios_mainboard_set_uuid(u8 *uuid)
 {
 	/* leave all zero */
@@ -359,6 +365,7 @@ static int smbios_write_type1(unsigned long *current, int handle)
 	t->manufacturer = smbios_add_string(t->eos, smbios_mainboard_manufacturer());
 	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->sku = smbios_add_string(t->eos, smbios_mainboard_sku());
 	t->version = smbios_add_string(t->eos, smbios_mainboard_version());
 #ifdef CONFIG_MAINBOARD_FAMILY
 	t->family = smbios_add_string(t->eos, smbios_mainboard_family());
diff --git a/src/include/smbios.h b/src/include/smbios.h
index 004bcc6..0400262 100644
--- a/src/include/smbios.h
+++ b/src/include/smbios.h
@@ -35,6 +35,7 @@ const char *smbios_mainboard_serial_number(void);
 const char *smbios_mainboard_version(void);
 void smbios_mainboard_set_uuid(u8 *uuid);
 const char *smbios_mainboard_bios_version(void);
+const char *smbios_mainboard_sku(void);
 u8 smbios_mainboard_enclosure_type(void);
 #ifdef CONFIG_MAINBOARD_FAMILY
 const char *smbios_mainboard_family(void);



More information about the coreboot-gerrit mailing list