summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorLukasz Siudut <lsiudut@fb.com>2019-02-19 10:06:09 +0000
committerNico Huber <nico.h@gmx.de>2019-02-26 21:41:20 +0000
commit2352a507aff7a319fc2c5c1d2a48a7545dfec235 (patch)
tree3d35b9b3a2f075e4c67297827e811698e8b99aa4 /src
parent3986d39471fd0b7dca0c53a48ac6697afd779819 (diff)
downloadcoreboot-2352a507aff7a319fc2c5c1d2a48a7545dfec235.tar.xz
Add missing u8 eos[2] declaration to struct smbios_type38
Each smbios entry should be followed with two null bytes. In other structures it's done by adding `u8 eos[2]` extra bytes at the end, it was omitted in type38 (IPMI) though. This change fixes this - tables decodes nicely: ``` IPMI Device Information Interface Type: KCS (Keyboard Control Style) Specification Version: 2.0 I2C Slave Address: 0x10 NV Storage Device: Not Present Base Address: 0x0000000000000CA2 (I/O) Register Spacing: 32-bit Boundaries ``` Signed-off-by: Lukasz Siudut <lsiudut@fb.com> Change-Id: I8efea9612448f48e23e7b2226aea2a9f3bc21824 Reviewed-on: https://review.coreboot.org/c/31482 Reviewed-by: Patrick Rudolph <siro@das-labor.org> Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr> Reviewed-by: Nico Huber <nico.h@gmx.de> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src')
-rw-r--r--src/include/smbios.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/include/smbios.h b/src/include/smbios.h
index 38ebcdcab0..c046b1a2a9 100644
--- a/src/include/smbios.h
+++ b/src/include/smbios.h
@@ -493,6 +493,7 @@ struct smbios_type38 {
u64 base_address;
u8 base_address_modifier;
u8 irq;
+ u8 eos[2];
} __packed;
enum smbios_bmc_interface_type {