Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33067 )
Change subject: soc/intel/common: Add SMBIOS type 131 fields
......................................................................
Patch Set 3:
tested on google/nami (w/heci_enabled). Can confirm the SMBIOS table 131 is added, don't have an easy way to decode it. And with this patch, dmidecode reports a mismatch between the reported table length and actual length (actual is 6 bytes shorter)
--
To view, visit
https://review.coreboot.org/c/coreboot/+/33067
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ida4e1a95e20289fbab01e773c18e00ce74aa8223
Gerrit-Change-Number: 33067
Gerrit-PatchSet: 3
Gerrit-Owner: Francois Toguo Fotso
francois.toguo.fotso@intel.com
Gerrit-Reviewer: Matt DeVillier
matt.devillier@gmail.com
Gerrit-Reviewer: Patrick Rudolph
patrick.rudolph@9elements.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Francois Toguo Fotso
francois.toguo.fotso@intel.corp-partner.google.com
Gerrit-CC: Pratikkumar V Prajapati
pratikkumar.v.prajapati@intel.com
Gerrit-Comment-Date: Tue, 11 Jun 2019 17:32:59 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment