diff options
author | George Burgess IV <gbiv@google.com> | 2024-07-10 09:41:03 -0600 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2024-07-12 16:01:54 +0000 |
commit | 794934cbeebb586847c98cc3d7ea04fa09a7330b (patch) | |
tree | 1390e6a40d4a9441ebac4bcd8e99b03fe2c60931 /util/lint | |
parent | 04937a9a20e7cb9d020c74a942b9894c5bc5556a (diff) |
amdfwtool: make fields unsigned
The value stored in `gen` is only ever `1` or `0`. Storing `1` causes
Clang to warn, since the only valid values for a 1-bit int are -1 and 0:
```
amdfwtool.c:1487:27: error: implicit truncation from 'int' to a one-bit
wide bit-field changes value from 1 to -1
[-Werror,-Wsingle-bit-bitfield-constant-conversion]
1487 | amd_romsig->efs_gen.gen = EFS_BEFORE_SECOND_GEN;
```
TEST=Rebuilt coreboot; no warning was emitted.
Change-Id: Ibd83be8302e8a717db7e7dc86a403b5648976586
Signed-off-by: George Burgess IV <gbiv@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/83412
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-by: Matt DeVillier <matt.devillier@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Diffstat (limited to 'util/lint')
0 files changed, 0 insertions, 0 deletions