summaryrefslogtreecommitdiff
path: root/util/cbfstool/cbfstool.c
diff options
context:
space:
mode:
authorVadim Bendebury <vbendeb@chromium.org>2014-12-23 15:59:57 -0800
committerPatrick Georgi <pgeorgi@google.com>2015-04-18 08:50:38 +0200
commit45e5997c61b7decdaebf48a47dc15040c96de3dc (patch)
tree335837f25756313752fd853c4017823cdce73cb6 /util/cbfstool/cbfstool.c
parent1161473ecb65844ee24359b0b943a33a696eea24 (diff)
downloadcoreboot-45e5997c61b7decdaebf48a47dc15040c96de3dc.tar.xz
cbfstool: clean up source code
The following changes were made: - order commands and options definitions alphabetically - do not report errors at cbfs_image_from_file() call sites - the error is reported by the function itself - remove the unused parameter in cbfs_create_empty_entry() prototype BRANCH=storm BUG=none TEST=compiled cbfstool, built a storm image, observed that the image still boots Change-Id: I31b15fab0a63749c6f2d351901ed545de531eb39 Signed-off-by: Patrick Georgi <pgeorgi@chromium.org> Original-Commit-Id: a909a50e03be77f972b1a497198fe758661aa9f8 Original-Change-Id: I4b8898dbd44eeb2c6b388a485366e4e22b1bed16 Original-Signed-off-by: Vadim Bendebury <vbendeb@chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/237560 Original-Reviewed-by: David Hendricks <dhendrix@chromium.org> Reviewed-on: http://review.coreboot.org/9746 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'util/cbfstool/cbfstool.c')
-rw-r--r--util/cbfstool/cbfstool.c78
1 files changed, 29 insertions, 49 deletions
diff --git a/util/cbfstool/cbfstool.c b/util/cbfstool/cbfstool.c
index b60f199f55..3554cfa2bc 100644
--- a/util/cbfstool/cbfstool.c
+++ b/util/cbfstool/cbfstool.c
@@ -144,10 +144,8 @@ static int cbfs_add_component(const char *cbfs_name,
return 1;
}
- if (cbfs_image_from_file(&image, cbfs_name, headeroffset) != 0) {
- ERROR("Could not load ROM image '%s'.\n", cbfs_name);
+ if (cbfs_image_from_file(&image, cbfs_name, headeroffset))
return 1;
- }
if (buffer_from_file(&buffer, filename) != 0) {
ERROR("Could not load file '%s'.\n", filename);
@@ -319,12 +317,8 @@ static int cbfs_remove(void)
return 1;
}
- if (cbfs_image_from_file(&image, param.cbfs_name,
- param.headeroffset) != 0) {
- ERROR("Could not load ROM image '%s'.\n",
- param.cbfs_name);
+ if (cbfs_image_from_file(&image, param.cbfs_name, param.headeroffset))
return 1;
- }
if (cbfs_remove_entry(&image, param.name) != 0) {
ERROR("Removing file '%s' failed.\n",
@@ -444,11 +438,8 @@ static int cbfs_locate(void)
return 1;
}
- if (cbfs_image_from_file(&image, param.cbfs_name,
- param.headeroffset) != 0) {
- ERROR("Failed to load %s.\n", param.cbfs_name);
+ if (cbfs_image_from_file(&image, param.cbfs_name, param.headeroffset))
return 1;
- }
if (cbfs_get_entry(&image, param.name))
WARN("'%s' already in CBFS.\n", param.name);
@@ -481,12 +472,9 @@ static int cbfs_locate(void)
static int cbfs_print(void)
{
struct cbfs_image image;
- if (cbfs_image_from_file(&image, param.cbfs_name,
- param.headeroffset) != 0) {
- ERROR("Could not load ROM image '%s'.\n",
- param.cbfs_name);
+ if (cbfs_image_from_file(&image, param.cbfs_name, param.headeroffset))
return 1;
- }
+
cbfs_print_directory(&image);
cbfs_image_delete(&image);
return 0;
@@ -507,15 +495,11 @@ static int cbfs_extract(void)
return 1;
}
- if (cbfs_image_from_file(&image, param.cbfs_name,
- param.headeroffset) != 0) {
- ERROR("Could not load ROM image '%s'.\n",
- param.cbfs_name);
+ if (cbfs_image_from_file(&image, param.cbfs_name, param.headeroffset))
result = 1;
- } else if (cbfs_export_entry(&image, param.name,
- param.filename) != 0) {
+ else if (cbfs_export_entry(&image, param.name,
+ param.filename))
result = 1;
- }
cbfs_image_delete(&image);
return result;
@@ -537,12 +521,8 @@ static int cbfs_update_fit(void)
return 1;
}
- if (cbfs_image_from_file(&image, param.cbfs_name,
- param.headeroffset) != 0) {
- ERROR("Could not load ROM image '%s'.\n",
- param.cbfs_name);
+ if (cbfs_image_from_file(&image, param.cbfs_name, param.headeroffset))
return 1;
- }
ret = fit_update_table(&image, param.fit_empty_entries, param.name);
if (!ret)
@@ -580,43 +560,43 @@ static int cbfs_copy(void)
static const struct command commands[] = {
{"add", "H;f:n:t:b:vh?", cbfs_add},
+ {"add-flat-binary", "H:f:n:l:e:c:b:vh?", cbfs_add_flat_binary},
{"add-payload", "H:f:n:t:c:b:vh?C:I:", cbfs_add_payload},
{"add-stage", "H:f:n:t:c:b:S:vh?", cbfs_add_stage},
- {"add-flat-binary", "H:f:n:l:e:c:b:vh?", cbfs_add_flat_binary},
{"add-int", "H:i:n:b:vh?", cbfs_add_integer},
- {"remove", "H:n:vh?", cbfs_remove},
- {"copy", "H:D:s:", cbfs_copy},
{"create", "s:B:b:H:a:o:m:vh?", cbfs_create},
+ {"copy", "H:D:s:", cbfs_copy},
+ {"extract", "H:n:f:vh?", cbfs_extract},
{"locate", "H:f:n:P:a:Tvh?", cbfs_locate},
{"print", "H:vh?", cbfs_print},
- {"extract", "H:n:f:vh?", cbfs_extract},
+ {"remove", "H:n:vh?", cbfs_remove},
{"update-fit", "H:n:x:vh?", cbfs_update_fit},
};
static struct option long_options[] = {
- {"name", required_argument, 0, 'n' },
- {"type", required_argument, 0, 't' },
- {"compression", required_argument, 0, 'c' },
+ {"alignment", required_argument, 0, 'a' },
{"base-address", required_argument, 0, 'b' },
- {"load-address", required_argument, 0, 'l' },
- {"top-aligned", required_argument, 0, 'T' },
+ {"bootblock", required_argument, 0, 'B' },
+ {"cmdline", required_argument, 0, 'C' },
+ {"compression", required_argument, 0, 'c' },
{"copy-offset", required_argument, 0, 'D' },
+ {"empty-fits", required_argument, 0, 'x' },
{"entry-point", required_argument, 0, 'e' },
- {"size", required_argument, 0, 's' },
- {"bootblock", required_argument, 0, 'B' },
- {"header-offset", required_argument, 0, 'H' },
- {"alignment", required_argument, 0, 'a' },
- {"page-size", required_argument, 0, 'P' },
- {"offset", required_argument, 0, 'o' },
{"file", required_argument, 0, 'f' },
+ {"header-offset", required_argument, 0, 'H' },
+ {"help", no_argument, 0, 'h' },
+ {"ignore-sec", required_argument, 0, 'S' },
+ {"initrd", required_argument, 0, 'I' },
{"int", required_argument, 0, 'i' },
+ {"load-address", required_argument, 0, 'l' },
{"machine", required_argument, 0, 'm' },
- {"empty-fits", required_argument, 0, 'x' },
- {"initrd", required_argument, 0, 'I' },
- {"cmdline", required_argument, 0, 'C' },
- {"ignore-sec", required_argument, 0, 'S' },
+ {"name", required_argument, 0, 'n' },
+ {"offset", required_argument, 0, 'o' },
+ {"page-size", required_argument, 0, 'P' },
+ {"size", required_argument, 0, 's' },
+ {"top-aligned", required_argument, 0, 'T' },
+ {"type", required_argument, 0, 't' },
{"verbose", no_argument, 0, 'v' },
- {"help", no_argument, 0, 'h' },
{NULL, 0, 0, 0 }
};