the following patch was just integrated into master: commit d173962c6e951486a7c234e269333073584b77ac Author: Hung-Te Lin hungte@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@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
Build-Tested: build bot (Jenkins) at Mon Jan 28 15:27:11 2013, giving +1 Reviewed-By: Stefan Reinauer stefan.reinauer@coreboot.org at Mon Jan 28 19:59:40 2013, giving +2 See http://review.coreboot.org/2195 for details.
-gerrit