diff options
| author | Martin Roth <martin@coreboot.org> | 2021-02-13 21:42:46 -0700 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2021-02-16 08:09:35 +0000 |
| commit | a8e31caee8ddb9563cdd3bac48ee3b9dc5cf6f48 (patch) | |
| tree | 4412f8c268acc9fc7a98179633547e884b46e0b2 /util/amdfwtool | |
| parent | 6d085446feb3d47154ff9641b66f583ddc218d40 (diff) | |
| download | coreboot-a8e31caee8ddb9563cdd3bac48ee3b9dc5cf6f48.tar.gz coreboot-a8e31caee8ddb9563cdd3bac48ee3b9dc5cf6f48.tar.bz2 coreboot-a8e31caee8ddb9563cdd3bac48ee3b9dc5cf6f48.zip | |
util/amdfwtool: Fix all warnings
Fixes these warnings:
warning: alignment 1 of 'struct _psp_directory_table' is less
than 16 [-Wpacked-not-aligned]
warning: alignment 1 of 'struct _psp_combo_directory' is less
than 16 [-Wpacked-not-aligned]
In function 'find_register_fw_filename_bios_dir':
warning: implicit conversion from 'enum _amd_fw_type' to
'amd_bios_type' {aka 'enum _amd_bios_type'} [-Wenum-conversion]
BUG=None
TEST=Build and verify binaries are identical.
Signed-off-by: Martin Roth <martin@coreboot.org>
Change-Id: I761d9893ac6737b42af96c4b2a57c5a4fc61ab05
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50643
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Diffstat (limited to 'util/amdfwtool')
| -rw-r--r-- | util/amdfwtool/amdfwtool.c | 4 | ||||
| -rw-r--r-- | util/amdfwtool/data_parse.c | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c index 3d33d704533b..b2b7c42219b6 100644 --- a/util/amdfwtool/amdfwtool.c +++ b/util/amdfwtool/amdfwtool.c @@ -396,7 +396,7 @@ typedef struct _psp_directory_entry { typedef struct _psp_directory_table { psp_directory_header header; psp_directory_entry entries[]; -} __attribute__((packed)) psp_directory_table; +} __attribute__((packed, aligned(16))) psp_directory_table; #define MAX_PSP_ENTRIES 0x1f @@ -417,7 +417,7 @@ typedef struct _psp_combo_entry { typedef struct _psp_combo_directory { psp_combo_header header; psp_combo_entry entries[]; -} __attribute__((packed)) psp_combo_directory; +} __attribute__((packed, aligned(16))) psp_combo_directory; #define MAX_COMBO_ENTRIES 1 diff --git a/util/amdfwtool/data_parse.c b/util/amdfwtool/data_parse.c index 80d8d01aa7b9..0c2e71d94f4a 100644 --- a/util/amdfwtool/data_parse.c +++ b/util/amdfwtool/data_parse.c @@ -321,7 +321,7 @@ static uint8_t find_register_fw_filename_bios_dir(char *fw_name, char *filename, fw_type = AMD_BIOS_MP2_CFG; subprog = 0; } else { - fw_type = AMD_FW_SKIP; + fw_type = AMD_BIOS_SKIP; } } else { fw_type = AMD_BIOS_INVALID; |