summaryrefslogtreecommitdiff
path: root/src/superio
diff options
context:
space:
mode:
authorStefan Reinauer <stepan@coresystems.de>2009-09-22 15:58:19 +0000
committerStefan Reinauer <stepan@openbios.org>2009-09-22 15:58:19 +0000
commit3fec29cc963c95d44ff45151a1bc64d8988073ab (patch)
tree7dfdc9f802911f736efa6f2f743d1f62235ea59b /src/superio
parent853270af390f9e200f4a165ec55d3b2fa5246c0b (diff)
This separates the code for each command in cbfstool. For the good and for the
bad: It brings a certain amount of code duplication (some of which can be cleaned up again, or get rid of by proper refactoring). On the other hand now there's a very simple code flow for each command, rather than for each operation. ie. adding a file to a cbfs means: - open the cbfs - add the file - close the cbfs rather than open the cbfs: - do this for add, remove, but not for create create a new lar - if we don't have an open one yet add a file: - if we didn't bail out before close the file: - if we didn't bail out before The short term benefit is that this fixes a problem where cbfstool was trying to add a file if you gave a non-existing command because it bailed out on known, not on unknown commands. Signed-off-by: Stefan Reinauer <stepan@coresystems.de> Acked-by: Peter Stuge <peter@stuge.se> git-svn-id: svn://svn.coreboot.org/coreboot/trunk@4654 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'src/superio')
0 files changed, 0 insertions, 0 deletions