diff options
| author | Mathias Krause <minipli@googlemail.com> | 2012-07-17 21:17:15 +0200 |
|---|---|---|
| committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2012-07-18 00:15:35 +0200 |
| commit | 41c229c02944834a55fc0b17bed5950facc9eae6 (patch) | |
| tree | 50bbaa901bef4ba42d2bc6f2d3439c226fee0629 /util/cbfstool/common.c | |
| parent | 5c581c4d6c2c32bcfe67acaafd9aa513a8cad328 (diff) | |
cbfstool: signed vs. unsigned fixes
Use the right data types to fix compiler warnings.
Change-Id: Id23739421ba9e4a35599355fac9a17300ae4bda9
Signed-off-by: Mathias Krause <minipli@googlemail.com>
Reviewed-on: http://review.coreboot.org/1236
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Tested-by: build bot (Jenkins)
Diffstat (limited to 'util/cbfstool/common.c')
| -rw-r--r-- | util/cbfstool/common.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/util/cbfstool/common.c b/util/cbfstool/common.c index 7a6e2c2d58..648bfa849d 100644 --- a/util/cbfstool/common.c +++ b/util/cbfstool/common.c @@ -164,7 +164,7 @@ void print_supported_filetypes(void) const char *strfiletype(uint32_t number) { - int i; + size_t i; for (i = 0; i < (sizeof(filetypes) / sizeof(struct filetypes_t)); i++) if (filetypes[i].type == number) return filetypes[i].name; @@ -173,7 +173,7 @@ const char *strfiletype(uint32_t number) uint64_t intfiletype(const char *name) { - int i; + size_t i; for (i = 0; i < (sizeof(filetypes) / sizeof(struct filetypes_t)); i++) if (strcmp(filetypes[i].name, name) == 0) return filetypes[i].type; @@ -497,12 +497,12 @@ uint32_t cbfs_find_location(const char *romfile, uint32_t filesize, const char *filename, uint32_t alignment) { loadrom(romfile); - int filename_size = strlen(filename); + size_t filename_size = strlen(filename); - int headersize = + size_t headersize = sizeof(struct cbfs_file) + ALIGN(filename_size + 1, 16) + sizeof(struct cbfs_stage); - int totalsize = headersize + filesize; + size_t totalsize = headersize + filesize; uint32_t current = phys_start; while (current < phys_end) { |