GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

util/cbfstool/ifittool: Remove dead code 

The 'x' option is not set up in the getopt options.

Change-Id: Ib4aa10b0ea2a3f97e8d2439152b708613bcf43db
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50923
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Arthur Heymans 2021-02-17 14:57:02 +01:00
parent 63813f9d79
commit 9a5d6e958f
1 changed files with 0 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
util/cbfstool/ifittool.c
View File

@ -127,7 +127,6 @@ enum fit_operation {
NO_OP = 0,
ADD_CBFS_OP,
ADD_REGI_OP,
ADD_ADDR_OP,
DEL_OP
};
@ -182,15 +181,6 @@ int main(int argc, char *argv[])
}
op = ADD_REGI_OP;
break;
case 'x':
if (op != NO_OP) {
ERROR("specified multiple actions at once\n");
usage(argv[0]);
return 1;
}
op = ADD_ADDR_OP;
addr = atoll(optarg);
break;
case 'c':
clear_table = true;
break;
@ -262,17 +252,6 @@ int main(int argc, char *argv[])
return 1;
}
}
if (op == ADD_ADDR_OP) {
if (fit_type == 0) {
ERROR("Adding FIT entry, but no type given\n");
usage(argv[0]);
return 1;
} else if (max_table_size == 0) {
ERROR("Maximum table size not given\n");
usage(argv[0]);
return 1;
}
}
if (!region_name) {
ERROR("Region not given\n");
@ -373,15 +352,6 @@ int main(int argc, char *argv[])
}
break;
}
case ADD_ADDR_OP:
{
if (fit_add_entry(fit, addr, 0, fit_type, max_table_size)) {
partitioned_file_close(image_file);
ERROR("Adding type %u FIT entry\n", fit_type);
return 1;
}
}
break;
case DEL_OP:
{
if (fit_delete_entry(fit, table_entry)) {