Łukasz Siudut has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31482
Change subject: Add missing u8 eos[2] declaration to struct smbios_type38 ......................................................................
Add missing u8 eos[2] declaration to struct smbios_type38
Each smbios 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 ```
This is follow up of change #31435
Signed-off-by: Lukasz Siudut lsiudut@fb.com Change-Id: I8efea9612448f48e23e7b2226aea2a9f3bc21824 --- M src/include/smbios.h 1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/82/31482/1
diff --git a/src/include/smbios.h b/src/include/smbios.h index 38ebcdc..c046b1a 100644 --- a/src/include/smbios.h +++ b/src/include/smbios.h @@ -493,6 +493,7 @@ u64 base_address; u8 base_address_modifier; u8 irq; + u8 eos[2]; } __packed;
enum smbios_bmc_interface_type {