diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2022-11-17 12:09:12 +0100 |
|---|---|---|
| committer | Martin L Roth <gaumless@gmail.com> | 2022-11-30 16:34:33 +0000 |
| commit | 8d9cce1e096fa9d35aae7a17bc5acb1adf44e18f (patch) | |
| tree | ec0b506785e00ceac7b2e653935260e1adda4b85 | |
| parent | 54a6b1f281e95300fc37be515da04f00925c30d6 (diff) | |
ec/google/chromeec: Add packed attribute to structs in union
Clang warns about structs inside a union also needing the packed
attribute.
This files is copied from the chromeec project, so it adds comment next
to the coreboot specific changes as a reference.
TEST: google/vilboz remains the same with BUILD_TIMELESS=1 and gcc.
Change-Id: I8b5233618081db86caedcb2d14870974e109ed9b
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/69742
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin L Roth <gaumless@gmail.com>
| -rw-r--r-- | src/ec/google/chromeec/ec_commands.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/ec/google/chromeec/ec_commands.h b/src/ec/google/chromeec/ec_commands.h index 0bb5db6081..7c311a1f5a 100644 --- a/src/ec/google/chromeec/ec_commands.h +++ b/src/ec/google/chromeec/ec_commands.h @@ -4636,12 +4636,12 @@ struct ec_params_charge_state { struct __ec_todo_unpacked { uint32_t param; /* enum charge_state_param */ - } get_param; + } __ec_todo_packed get_param; /* coreboot change */ struct __ec_todo_unpacked { uint32_t param; /* param to set */ uint32_t value; /* value to set */ - } set_param; + } __ec_todo_packed set_param; /* coreboot change */ }; uint8_t chgnum; /* Version 1 supports chgnum */ } __ec_todo_packed; |