diff options
| author | Daniel Gröber <dxld@darkboxed.org> | 2020-11-30 13:30:10 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2020-12-02 10:38:25 +0000 |
| commit | 2aa15872d2468c6244d4c4a1dd0a325a2c9002e9 (patch) | |
| tree | 27217c89903fc9051b61a0c0ffc4564555c12630 /tests | |
| parent | 7c0f007cf589ac01e35d555d0c8d98f4a72396d4 (diff) | |
| download | coreboot-2aa15872d2468c6244d4c4a1dd0a325a2c9002e9.tar.gz coreboot-2aa15872d2468c6244d4c4a1dd0a325a2c9002e9.tar.bz2 coreboot-2aa15872d2468c6244d4c4a1dd0a325a2c9002e9.zip | |
edist-test: Fix _Static_assert missing message string
Older GCCs don't support _Static_assert without a message string as the
second argument. AFAICT _Static_assert with two arguments is in C11 but
omitting the message argument is an extension.
The tests appear to be built with the system gcc rather than our
crossgcc so that's probably why this was not cought by CI.
Change-Id: I41fd0ffc42ded8b6d145c3ec30cc7407a78b9a43
Signed-off-by: Daniel Gröber <dxld@darkboxed.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48151
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/include/lib/edid-test.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/include/lib/edid-test.h b/tests/include/lib/edid-test.h index 073905d5f0b8..8327748b66f1 100644 --- a/tests/include/lib/edid-test.h +++ b/tests/include/lib/edid-test.h @@ -43,7 +43,8 @@ struct edid_raw { uint8_t checksum; } __packed; -_Static_assert(sizeof(struct edid_raw) == 128); +_Static_assert(sizeof(struct edid_raw) == 128, + "assert failed: edid_raw size mismatch"); #define EDID_HEADER_RAW { 0x00, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0x00 } #define EDID_HEADER_INVALID_RAW { 0, 0, 0, 0, 0, 0, 0, 0 } |