diff options
| author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2017-08-16 17:02:10 +0300 |
|---|---|---|
| committer | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2017-08-16 17:53:24 +0300 |
| commit | 5a17ee2c8f9013f5db852d27564b837f9f2c5a9f (patch) | |
| tree | 27413e9f29bb0ae4ed64f14f4c1d81542d19b7c3 /tools/intel_bios.h | |
| parent | 103af58478e4a8aa644449e469a1cec0cd57b160 (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>
Diffstat (limited to 'tools/intel_bios.h')
| -rw-r--r-- | tools/intel_bios.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/intel_bios.h b/tools/intel_bios.h index ca0d2c58..f2ccb55a 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 */ |