diff options
author | Stewart Smith <stewart@linux.ibm.com> | 2019-06-18 15:48:48 +1000 |
---|---|---|
committer | Stewart Smith <stewart@linux.ibm.com> | 2019-06-19 09:54:46 +1000 |
commit | ef691db3533742d9dd6eed1a311472a7c52be94b (patch) | |
tree | b835296d5d5a1adf77acd013f09d91e40f801beb /include | |
parent | 9c89e53d235cf1513f7693ee6e4900fc06402f73 (diff) | |
download | skiboot-ef691db3533742d9dd6eed1a311472a7c52be94b.zip skiboot-ef691db3533742d9dd6eed1a311472a7c52be94b.tar.gz skiboot-ef691db3533742d9dd6eed1a311472a7c52be94b.tar.bz2 |
struct p9_sbe_msg doesn't need to be packed
Only the reg member is sent anywhere (via xscom_write), so the structure
does not need to be packed.
Fixes GCC9 build problem:
hw/sbe-p9.c: In function ‘p9_sbe_msg_send’:
hw/sbe-p9.c:270:9: error: taking address of packed member of ‘struct p9_sbe_msg’ may result in an unaligned pointer value [-Werror=address-of-packed-member]
270 | data = &msg->reg[0];
| ^~~~~~~~~~~~
Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
Diffstat (limited to 'include')
-rw-r--r-- | include/sbe-p9.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/sbe-p9.h b/include/sbe-p9.h index 4b839d8..5ad0422 100644 --- a/include/sbe-p9.h +++ b/include/sbe-p9.h @@ -205,7 +205,7 @@ struct p9_sbe_msg { /* Internal queuing */ struct list_node link; -} __packed; +}; /* Allocate and populate p9_sbe_msg structure */ |