diff options
author | Martin Roth <martinroth@chromium.org> | 2016-11-08 10:44:18 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2016-11-10 00:25:45 +0100 |
commit | 8806f7f6620d8156fddb8039cb5a9b276097a6e5 (patch) | |
tree | 6e85195fa81ae8d6dc95cb1082bc8199deacc91c /util/abuild | |
parent | 60f1551461a16cf1b6dc294f715a22748dcb501c (diff) |
util/amdfwtool: fix clang warnings
Various fixes for clang warnings:
warning: arithmetic on a pointer to void is a GNU extension
- change *rom from void* to char* and cast back to void* as needed
warning: implicit conversion changes signedness
- In ALIGN macro, pass in value as unsigned
warning: no previous prototype for function
- Change functions to static
warning: no previous extern declaration for non-static variable
- Change global variables to static
warning: comparison of integers of different signs
- Make loop variable 'i' unsigned
warning: variable 'output' may be uninitialized
- Make sure an output filename was specified
warning: implicit conversion loses integer precision
- cast fd_stat.st_size to the appropriate type
Change-Id: I0134a79c00938e121e63b52fd63bd502f4cb9e99
Signed-off-by: Martin Roth <martinroth@chromium.org>
Reviewed-on: https://review.coreboot.org/17322
Tested-by: build bot (Jenkins)
Reviewed-by: Marc Jones <marc@marcjonesconsulting.com>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Diffstat (limited to 'util/abuild')
0 files changed, 0 insertions, 0 deletions