diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2020-07-23 19:37:17 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2020-07-24 23:55:57 +0000 |
| commit | 0b5a6143ea392ddd3656c2fef3be927b06c73300 (patch) | |
| tree | b680755175c72378e4a7366cc5ed06123394255b | |
| parent | d46c023a22e4765de86f5057d6cd02fe9c1f8f6c (diff) | |
| download | coreboot-0b5a6143ea392ddd3656c2fef3be927b06c73300.tar.xz | |
soc/amd/picasso: mark usb2_phy_tune struct as packed
Since the binary layout of this struct matters, it should be marked as
packed. Since all struct elements are uint8_t, this shouldn't result in
a different layout though.
BUG=b:161923068
Change-Id: I6a390c3a3f35eaf8a72928b4cef0e9f405770619
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43780
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/soc/amd/picasso/chip.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/amd/picasso/chip.h b/src/soc/amd/picasso/chip.h index 1950691c89..3bcd3cc5c4 100644 --- a/src/soc/amd/picasso/chip.h +++ b/src/soc/amd/picasso/chip.h @@ -16,7 +16,7 @@ /* USB 2.0 PHY Parameters */ -struct usb2_phy_tune { +struct __packed usb2_phy_tune { /* Disconnect Threshold Adjustment. Range 0 - 0x7 */ uint8_t com_pds_tune; /* Squelch Threshold Adjustment. Range 0 - 0x7 */ |