diff options
| author | Martin Roth <martin@coreboot.org> | 2021-02-13 23:19:51 -0700 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2021-02-16 08:12:15 +0000 |
| commit | db4719f0396854dfca4018487a6ed1580426f7ff (patch) | |
| tree | 5098fdea434dfad510f6e05dfdc70b4cde91ac79 /util/bincfg/bincfg.l | |
| parent | 264e14b143aa4073c6c10df84cffabc5065b8a58 (diff) | |
util/bincfg: Fix all issues
This fixes the following issues:
bincfg.l: In function ‘parsehex’:
error: declaration of ‘val’ shadows a global declaration
bincfg.y: In function ‘generate_binary_with_gbe_checksum’:
error: comparison of integer expressions of different signedness
bincfg.y: In function ‘yyerror’:
bincfg.y:408:28: error: unused parameter ‘fp’
bincfg.y: In function ‘main’:
bincfg.y:452:15: error: unused variable ‘pos’
bincfg.y:451:16: error: unused variable ‘c’
BUG=None
TEST=Build outputs and make sure they're identical.
Signed-off-by: Martin Roth <martin@coreboot.org>
Change-Id: I60039b741c226a6b6ba53306f6fa293da89e5355
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50653
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Diffstat (limited to 'util/bincfg/bincfg.l')
| -rw-r--r-- | util/bincfg/bincfg.l | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/util/bincfg/bincfg.l b/util/bincfg/bincfg.l index ca93547648..e55d6c609a 100644 --- a/util/bincfg/bincfg.l +++ b/util/bincfg/bincfg.l @@ -11,7 +11,7 @@ extern struct blob binary; unsigned int parsehex (char *s) { - unsigned int i, nib, val = 0; + unsigned int i, nib, retval = 0; unsigned int nibs = strlen(s) - 2; for (i = 2; i < nibs + 2; i++) { @@ -24,9 +24,9 @@ unsigned int parsehex (char *s) } else { return 0; } - val |= nib << (((nibs - 1) - (i - 2)) * 4); + retval |= nib << (((nibs - 1) - (i - 2)) * 4); } - return val; + return retval; } char* stripquotes (char *string) |