diff options
author | Felix Held <felix-coreboot@felixheld.de> | 2020-07-13 23:37:06 +0200 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2020-07-15 08:32:18 +0000 |
commit | 692dde599f35719d3e9e9a43800b9deea50c002a (patch) | |
tree | fa27b2b338b75880f1aaf0c5fafe23cf72c1e045 /src/drivers/ipmi/ipmi_fru.c | |
parent | de3fe84ca970068f0a077631bbdafc1266edf0aa (diff) |
vc/amd/fsp/picasso: replace #pragma pack(1) with __packed attribute
Forgetting to add the #pragma pack() at the end of the header file can
lead to hard to debug breakage, so get rid of the #pragma pack usage and
add a __packed to the structs that need to be packed which has less
possibly unwanted side effects.
Since commit d44221f9c8f3686e040ff9481829315068b321a3 coreboot always
includes commonlib/compiler.h which provides __packed.
TEST=Timeless build results in identical binary.
Change-Id: Icc53168f4fbc3a63a859f686b18e7023d225f8d2
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43433
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/drivers/ipmi/ipmi_fru.c')
0 files changed, 0 insertions, 0 deletions