[coreboot] Patch set updated for coreboot: c14199f smbios: show CONFIG_LOCALVERSION in DMI bios_version

Christian Gmeiner (christian.gmeiner@gmail.com) gerrit at coreboot.org
Mon Feb 4 16:03:14 CET 2013


Christian Gmeiner (christian.gmeiner at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2279

-gerrit

commit c14199f085099b3376e79b69104da43d85bf80d4
Author: Christian Gmeiner <christian.gmeiner at gmail.com>
Date:   Mon Feb 4 16:22:46 2013 +0100

    smbios: show CONFIG_LOCALVERSION in DMI bios_version
    
    If somebody makes use of CONFIG_LOCALVERSION show this
    user provided config string for DMI bios_version.
    
    As requested I have attached example output.
    
    CONFIG_LOCALVERSION=""
    CONFIG_CBFS_PREFIX="fallback"
    CONFIG_COMPILER_GCC=y
    ...
    
    root at OT:~# cat /sys/class/dmi/id/bios_version
    4.0-3360-g5be6673-dirty
    
    CONFIG_LOCALVERSION="V1.01.02 Beta"
    CONFIG_CBFS_PREFIX="fallback"
    CONFIG_COMPILER_GCC=y
    ...
    
    root at OT:~# cat /sys/class/dmi/id/bios_version
    V1.01.02 Beta
    
    Change-Id: I5640b72b56887ddf85113efa9ff23df9d4c7eb86
    Signed-off-by: Christian Gmeiner <christian.gmeiner at gmail.com>
---
 src/arch/x86/boot/smbios.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/arch/x86/boot/smbios.c b/src/arch/x86/boot/smbios.c
index f65ced2..308336a 100644
--- a/src/arch/x86/boot/smbios.c
+++ b/src/arch/x86/boot/smbios.c
@@ -131,7 +131,11 @@ static int smbios_write_type0(unsigned long *current, int handle)
 	t->vendor = smbios_add_string(t->eos, "coreboot");
 #if !CONFIG_CHROMEOS
 	t->bios_release_date = smbios_add_string(t->eos, COREBOOT_DMI_DATE);
-	t->bios_version = smbios_add_string(t->eos, COREBOOT_VERSION);
+
+	if (strlen(CONFIG_LOCALVERSION))
+		t->bios_version = smbios_add_string(t->eos, CONFIG_LOCALVERSION);
+	else
+		t->bios_version = smbios_add_string(t->eos, COREBOOT_VERSION);
 #else
 #define SPACES \
 	"                                                                  "



More information about the coreboot mailing list