diff options
author | Hung-Te Lin <hungte@chromium.org> | 2013-01-28 14:23:49 +0800 |
---|---|---|
committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2013-01-28 19:59:40 +0100 |
commit | d173962c6e951486a7c234e269333073584b77ac (patch) | |
tree | 6a138997da7e251beb7f8cda333c816662d77d51 /src/mainboard | |
parent | dd678a2e16743629e883287e11a53ad359fe7970 (diff) | |
download | coreboot-d173962c6e951486a7c234e269333073584b77ac.tar.xz |
cbfstool: Store global variables into struct.
cbfstool.c uses lots of global variables for command line options and all named
as "rom*". This may be confusing when other global variables also start with
rom, ex: int size = rom_size + romsize;
(rom_size is from command line and romsize is the size of last loaded ROM image).
If we pack all rom_* into a struct it may be more clear, ex:
do_something(param.cbfs_name, param.size, &romsize);
Change-Id: I5a298f4d67e712f90e998bcb70f2a68b8c0db6ac
Signed-off-by: Hung-Te Lin <hungte@chromium.org>
Reviewed-on: http://review.coreboot.org/2195
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/mainboard')
0 files changed, 0 insertions, 0 deletions