diff options
| author | Brian J. Nemec <bnemec@chromium.org> | 2019-09-12 23:44:39 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2019-09-17 20:08:37 +0000 |
| commit | 40e22bc78e72aad5b47d9c602d78fbc0abae56e2 (patch) | |
| tree | ade040df0f457056afa8acfd45ec5f4fd42513e6 /chip | |
| parent | 4284bf8c73aa210cb997d39a14851c5505a9c90b (diff) | |
| download | chrome-ec-40e22bc78e72aad5b47d9c602d78fbc0abae56e2.tar.gz | |
Sweetberry: Fix for struct declaration in power logging.
The struct declaration in Sweetberry was changed while supporting
the new 7-bit address declarations without updating the interface.
This reverses the change on the firmware side so the structs are
correctly unpacked. This structure is only used in the usb_power file
to load the value into a field used internally and during validation
of correctly formated packets.
BUG=b:140969461
BRANCH=None
TEST=Manual testing with powerlog using:
powerlog -b atlas_rev3.board -c atlas_rev3.scenario
Before: No INA output is returned, command fails validation.
After: INA output is returned and visible on console.
Signed-off-by: Brian Nemec <bnemec@chromium.org>
Change-Id: Ibddf4b8f8d48af0eea9e219ab3728a95e2cf66a8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1802578
Reviewed-by: Ruben Rodriguez Buchillon <coconutruben@chromium.org>
Reviewed-by: Nick Sanders <nsanders@chromium.org>
Diffstat (limited to 'chip')
| -rw-r--r-- | chip/stm32/usb_power.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chip/stm32/usb_power.h b/chip/stm32/usb_power.h index cbea9554a5..68b7f75ca2 100644 --- a/chip/stm32/usb_power.h +++ b/chip/stm32/usb_power.h @@ -243,7 +243,7 @@ struct __attribute__ ((__packed__)) usb_power_command_addina { uint16_t command; uint8_t port; uint8_t type; - uint16_t addr_flags; + uint8_t addr_flags; uint8_t extra; uint32_t rs; }; |