summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVille Syrjälä <ville.syrjala@linux.intel.com>2017-08-16 17:02:10 +0300
committerVille Syrjälä <ville.syrjala@linux.intel.com>2017-08-16 17:53:24 +0300
commit5a17ee2c8f9013f5db852d27564b837f9f2c5a9f (patch)
tree27413e9f29bb0ae4ed64f14f4c1d81542d19b7c3
parent103af58478e4a8aa644449e469a1cec0cd57b160 (diff)
tools/intel_vbt_decode: Fix decoding of child device structure
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 fields 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@linux.intel.com> Reviewed-by: Jani Nikula <jani.nikula@intel.com>
-rw-r--r--tools/intel_bios.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/intel_bios.h b/tools/intel_bios.h
index ca0d2c587..f2ccb55ab 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 */