[Intel-gfx] [PATCH i-g-t] tools/intel_vbt_decode: Fix decoding of child device structure

ville.syrjala at linux.intel.com ville.syrjala at linux.intel.com
Wed Aug 16 14:20:11 UTC 2017


From: Ville Syrjälä <ville.syrjala at linux.intel.com>

Fix decoding of the start of the child device structure. I had
accidentally duplicated the "device class/type" member and forgot to
include the add-in offset later. Fortunately both were two byte fields
so they effectively cancelled each other out and thus the remainder of
the child device structure was being decoded correctly. But of course
anything sitting between these two fieds was being decoded incorrectly.

Fixes: 86a546f6f798 ("tools/intel_bios_reader: Dump out more information from the child device structure")
Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
---
 tools/intel_bios.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/intel_bios.h b/tools/intel_bios.h
index ca0d2c587120..f2ccb55ab6c3 100644
--- a/tools/intel_bios.h
+++ b/tools/intel_bios.h
@@ -273,7 +273,6 @@ struct child_device_config {
 struct efp_child_device_config {
 	uint16_t handle;
 	uint16_t device_type;
-	uint16_t device_class;
 	uint8_t i2c_speed;
 	uint8_t dp_onboard_redriver; /* 158 */
 	uint8_t dp_ondock_redriver; /* 158 */
@@ -289,6 +288,7 @@ struct efp_child_device_config {
 	uint8_t skip1:4;
 	uint8_t slave_port; /*  202 */
 	uint8_t skip2;
+	uint16_t addin_offset;
 	uint8_t port;
 	uint8_t i2c_pin; /* for add-in card */
 	uint8_t slave_addr; /* for add-in card */
-- 
2.13.0



More information about the Intel-gfx mailing list