From 0b5a6143ea392ddd3656c2fef3be927b06c73300 Mon Sep 17 00:00:00 2001
From: Felix Held <felix-coreboot@felixheld.de>
Date: Thu, 23 Jul 2020 19:37:17 +0200
Subject: 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>
---
 src/soc/amd/picasso/chip.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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 */
-- 
cgit v1.2.3