Add -Werror to help us keep the code clean.
Change sizes from unsigned int to int.
Clean up some usage and parameter checking.
Signed-off-by: Myles Watson <mylesgw@gmail.com>
Acked-by: Ronald G. Minnich <rminnich@gmail.com>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@4262 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
diff --git a/util/cbfstool/Makefile b/util/cbfstool/Makefile
index 39af38b..a779eab 100644
--- a/util/cbfstool/Makefile
+++ b/util/cbfstool/Makefile
@@ -9,7 +9,7 @@
INC=cbfstool.h cbfs.h
CC=gcc
-CFLAGS=-g -Wall # -W -Werror
+CFLAGS=-g -Wall -W -Werror
DESTDIR ?= /usr/local/bin
diff --git a/util/cbfstool/add.c b/util/cbfstool/add.c
index aadb68d..83f8d29 100644
--- a/util/cbfstool/add.c
+++ b/util/cbfstool/add.c
@@ -205,25 +205,25 @@
void add_usage(void)
{
- printf("add [FILE] [NAME] [TYPE]\tAdd a component\n");
+ printf("add FILE NAME TYPE\tAdd a component\n");
}
void add_stage_usage(void)
{
- printf("add-stage [FILE] [NAME] [OPTIONS]\tAdd a stage to the ROM\n");
+ printf("add-stage FILE NAME [OPTIONS]\tAdd a stage to the ROM\n");
}
void add_payload_usage(void)
{
printf
- ("add-payload [FILE] [NAME] [OPTIONS]\tAdd a payload to the ROM\n");
+ ("add-payload FILE NAME [OPTIONS]\tAdd a payload to the ROM\n");
}
int add_handler(struct rom *rom, int argc, char **argv)
{
unsigned int type = CBFS_COMPONENT_NULL;
- if (argc < 2) {
+ if (argc != 3) {
add_usage();
return -1;
}
@@ -235,15 +235,13 @@
/* There are two ways to specify the type - a string or a number */
- if (argc == 3) {
- if (isdigit(*(argv[2])))
- type = strtoul(argv[2], 0, 0);
+ if (isdigit(*(argv[2])))
+ type = strtoul(argv[2], 0, 0);
+ else {
+ ERROR("String types (%s) aren't implemented yet.\n", argv[2]);
+ return -1;
}
- if (type == CBFS_COMPONENT_NULL)
- WARN("No file type was given for %s - using default\n",
- argv[0]);
-
return add_blob(rom, argv[0], argv[1], type);
}
diff --git a/util/cbfstool/cbfstool.c b/util/cbfstool/cbfstool.c
index cbf4481..7760a56 100644
--- a/util/cbfstool/cbfstool.c
+++ b/util/cbfstool/cbfstool.c
@@ -66,7 +66,7 @@
"extract", extract_handler, extract_usage}, {
"print", print_handler, print_usage}, {
"resize", resize_handler, resize_usage}, {
-"", NULL},};
+"", NULL, NULL},};
static struct rom rom;
diff --git a/util/cbfstool/cbfstool.h b/util/cbfstool/cbfstool.h
index 51abd29..dc52bfb 100644
--- a/util/cbfstool/cbfstool.h
+++ b/util/cbfstool/cbfstool.h
@@ -67,15 +67,15 @@
/* fs.c */
-struct cbfs_file *rom_find(struct rom *rom, unsigned int offset);
+struct cbfs_file *rom_find(struct rom *rom, int offset);
struct cbfs_file *rom_find_first(struct rom *);
struct cbfs_file *rom_find_next(struct rom *, struct cbfs_file *);
int rom_add(struct rom *rom, const char *name, void *, int size, int type);
int rom_set_header(struct rom *rom, struct cbfs_file *c,
const char*name, int size, int type);
-int rom_extract(struct rom *rom, const char *name, void **buf, unsigned long *size);
+int rom_extract(struct rom *rom, const char *name, void **buf, int *size);
int rom_remove(struct rom *rom, const char *name);
-unsigned int rom_used_space(struct rom *rom);
+int rom_used_space(struct rom *rom);
int rom_exists(struct rom *rom);
#endif
diff --git a/util/cbfstool/create.c b/util/cbfstool/create.c
index ecfb21c..e83758c 100644
--- a/util/cbfstool/create.c
+++ b/util/cbfstool/create.c
@@ -24,7 +24,7 @@
void create_usage(void)
{
- printf("create SIZE BOOTBLOCKSIZE [ALIGN] [BOOTBLOCK]\tCreate a ROM file\n");
+ printf("create SIZE BOOTBLOCKSIZE BOOTBLOCK [ALIGN]\tCreate a ROM file\n");
}
int create_handler(struct rom *rom, int argc, char **argv)
@@ -33,7 +33,7 @@
char *bootblock = NULL;
int bootblocksize;
- if (argc < 2) {
+ if (argc < 3) {
create_usage();
return -1;
}
@@ -42,11 +42,10 @@
bootblocksize = get_size(argv[1]);
- if (argc == 3) {
- bootblock = argv[2];
- } else if (argc >= 4) {
- align = strtoul(argv[2], NULL, 0);
- bootblock = argv[3];
+ bootblock = argv[2];
+
+ if (argc >= 4) {
+ align = strtoul(argv[3], NULL, 0);
}
if (size < bootblocksize) {
diff --git a/util/cbfstool/extract.c b/util/cbfstool/extract.c
index d694476..5a30553 100644
--- a/util/cbfstool/extract.c
+++ b/util/cbfstool/extract.c
@@ -28,7 +28,7 @@
{
void *buf;
int fd, ret;
- unsigned long size;
+ int size;
ret = rom_extract(rom, name, &buf, &size);
@@ -43,7 +43,7 @@
}
if (write(fd, buf, size) != size) {
- ERROR("Couldn't write %ld bytes!\n", size);
+ ERROR("Couldn't write %d bytes!\n", size);
ret = -1;
}
diff --git a/util/cbfstool/fs.c b/util/cbfstool/fs.c
index 8292163..d724dda 100644
--- a/util/cbfstool/fs.c
+++ b/util/cbfstool/fs.c
@@ -152,7 +152,7 @@
return ((struct cbfs_file *)ROM_PTR(rom, ret));
}
-struct cbfs_file *rom_find(struct rom *rom, unsigned int offset)
+struct cbfs_file *rom_find(struct rom *rom, int offset)
{
while (offset < rom->fssize) {
struct cbfs_file *c =
@@ -195,7 +195,7 @@
return NULL;
}
-unsigned int rom_used_space(struct rom *rom)
+int rom_used_space(struct rom *rom)
{
struct cbfs_file *c = rom_find_first(rom);
unsigned int ret = 0;
@@ -235,7 +235,7 @@
return 0;
}
-int rom_extract(struct rom *rom, const char *name, void** buf, unsigned long *size )
+int rom_extract(struct rom *rom, const char *name, void** buf, int *size )
{
struct cbfs_file *c = rom_find_by_name(rom, name);
unsigned int csize;
@@ -265,8 +265,8 @@
int rom_add(struct rom *rom, const char *name, void *buffer, int size, int type)
{
struct cbfs_file *c = rom_alloc(rom, size);
- unsigned int offset;
- unsigned int csize;
+ int offset;
+ int csize;
if (rom_find_by_name(rom, name)) {
ERROR("Component %s already exists in this rom\n", name);
diff --git a/util/cbfstool/print.c b/util/cbfstool/print.c
index 7eb9ee4..b23f949 100644
--- a/util/cbfstool/print.c
+++ b/util/cbfstool/print.c
@@ -27,6 +27,9 @@
int print_handler(struct rom *rom, int argc, char **argv)
{
+ if (argc > 0 || argv[1] != NULL)
+ printf("print\t\t\t\tShow the contents of the ROM\n");
+
printf("%s: %d kB, bootblocksize %d, romsize %d, offset 0x%x\n", rom->name, rom->size / 1024,
ntohl(rom->header->bootblocksize), ntohl(rom->header->romsize), ntohl(rom->header->offset));
printf("Alignment: %d bytes\n\n", ntohl(rom->header->align));
@@ -48,6 +51,9 @@
case CBFS_COMPONENT_OPTIONROM:
strcpy(type, "optionrom");
break;
+ case CBFS_COMPONENT_NULL:
+ strcpy(type, "free");
+ break;
default:
sprintf(type, "0x%8.8x", htonl(c->type));
break;
diff --git a/util/cbfstool/resize.c b/util/cbfstool/resize.c
index 1575156..d70c6d0 100644
--- a/util/cbfstool/resize.c
+++ b/util/cbfstool/resize.c
@@ -32,7 +32,8 @@
int resize_handler(struct rom *rom, int argc, char **argv)
{
- unsigned int size, align, offset;
+ unsigned int align, offset;
+ int size;
char null = '\0';
int bootblocksize = ntohl(rom->header->bootblocksize);
diff --git a/util/cbfstool/tools/Makefile b/util/cbfstool/tools/Makefile
index f5b0e04..65ba375 100644
--- a/util/cbfstool/tools/Makefile
+++ b/util/cbfstool/tools/Makefile
@@ -1,6 +1,8 @@
tobj ?= $(shell pwd)
tsrc ?= $(shell pwd)
+CFLAGS=-g -Wall
+
TARGETS += $(tobj)/cbfs-mkstage $(tobj)/cbfs-mkpayload
tools: $(tobj)/cbfs-mkstage $(tobj)/cbfs-mkpayload