[coreboot] Patch merged into coreboot/master: d173962 cbfstool: Store global variables into struct.
gerrit at coreboot.org
gerrit at coreboot.org
Mon Jan 28 19:59:41 CET 2013
the following patch was just integrated into master:
commit d173962c6e951486a7c234e269333073584b77ac
Author: Hung-Te Lin <hungte at chromium.org>
Date: Mon Jan 28 14:23:49 2013 +0800
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 at chromium.org>
Reviewed-on: http://review.coreboot.org/2195
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>
Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
Build-Tested: build bot (Jenkins) at Mon Jan 28 15:27:11 2013, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer at coreboot.org> at Mon Jan 28 19:59:40 2013, giving +2
See http://review.coreboot.org/2195 for details.
-gerrit
More information about the coreboot
mailing list