aboutsummaryrefslogtreecommitdiffstats
path: root/tools/firmware-utils/src/imagetag_cmdline.c
diff options
context:
space:
mode:
authorDaniel Dickinson <daniel@cshore.neomailbox.net>2011-04-15 12:18:25 +0000
committerDaniel Dickinson <daniel@cshore.neomailbox.net>2011-04-15 12:18:25 +0000
commitb2ae50ed2791c7ca9ee1d4592259829e0941f9c3 (patch)
treebe3eed32a0f0f4abc6895b3890ee1276d15d3a42 /tools/firmware-utils/src/imagetag_cmdline.c
parent9c505aa1f616514d031341e6fb1d2262b506a9d3 (diff)
downloadmaster-187ad058-b2ae50ed2791c7ca9ee1d4592259829e0941f9c3.tar.gz
master-187ad058-b2ae50ed2791c7ca9ee1d4592259829e0941f9c3.tar.bz2
master-187ad058-b2ae50ed2791c7ca9ee1d4592259829e0941f9c3.zip
bcm63xx: Move the OpenWrt rootfs length field
Dual image capable CFEs store an image sequence at the same place as currently OpenWrt stores the actual rootfs length, so it will get overwritten when flashing through such a CFE. To prevent this from happening, move the rootfs length field to the next four bytes, thus completely using the reserved1 field. Since the reserved1 field is now completely in use, it does not make sense to allow it to be set from the imagetag utility, so remove the option. Signed-off-by: Jonas Gorski <jonas.gorski+openwrt@gmail.com> git-svn-id: svn://svn.openwrt.org/openwrt/trunk@26680 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'tools/firmware-utils/src/imagetag_cmdline.c')
-rw-r--r--tools/firmware-utils/src/imagetag_cmdline.c28
1 files changed, 2 insertions, 26 deletions
diff --git a/tools/firmware-utils/src/imagetag_cmdline.c b/tools/firmware-utils/src/imagetag_cmdline.c
index 85f214bc6c..91ac90b09d 100644
--- a/tools/firmware-utils/src/imagetag_cmdline.c
+++ b/tools/firmware-utils/src/imagetag_cmdline.c
@@ -56,7 +56,6 @@ const char *gengetopt_args_info_help[] = {
" -r, --rsa-signature=STRING String for RSA Signature section.",
" --second-image-flag=flag-value\n Dual Image Flag (2=not-specified). (possible \n values=\"0\", \"1\", \"2\" default=`2')",
" --inactive=flag-value Inactive Flag (2=not-specified). (possible \n values=\"0\", \"1\", \"2\" default=`2')",
- " --reserved1=STRING String for first reserved section.",
" --reserved2=STRING String for second reserved section.",
" --kernel-file-has-header Indicates that the kernel file includes the \n kernel header with correct load address and \n entry point, so no changes are needed \n (default=off)",
0
@@ -112,7 +111,6 @@ void clear_given (struct gengetopt_args_info *args_info)
args_info->rsa_signature_given = 0 ;
args_info->second_image_flag_given = 0 ;
args_info->inactive_given = 0 ;
- args_info->reserved1_given = 0 ;
args_info->reserved2_given = 0 ;
args_info->kernel_file_has_header_given = 0 ;
}
@@ -164,8 +162,6 @@ void clear_args (struct gengetopt_args_info *args_info)
args_info->second_image_flag_orig = NULL;
args_info->inactive_arg = gengetopt_strdup ("2");
args_info->inactive_orig = NULL;
- args_info->reserved1_arg = NULL;
- args_info->reserved1_orig = NULL;
args_info->reserved2_arg = NULL;
args_info->reserved2_orig = NULL;
args_info->kernel_file_has_header_flag = 0;
@@ -201,9 +197,8 @@ void init_args_info(struct gengetopt_args_info *args_info)
args_info->rsa_signature_help = gengetopt_args_info_help[21] ;
args_info->second_image_flag_help = gengetopt_args_info_help[22] ;
args_info->inactive_help = gengetopt_args_info_help[23] ;
- args_info->reserved1_help = gengetopt_args_info_help[24] ;
- args_info->reserved2_help = gengetopt_args_info_help[25] ;
- args_info->kernel_file_has_header_help = gengetopt_args_info_help[26] ;
+ args_info->reserved2_help = gengetopt_args_info_help[24] ;
+ args_info->kernel_file_has_header_help = gengetopt_args_info_help[25] ;
}
@@ -326,8 +321,6 @@ cmdline_parser_release (struct gengetopt_args_info *args_info)
free_string_field (&(args_info->second_image_flag_orig));
free_string_field (&(args_info->inactive_arg));
free_string_field (&(args_info->inactive_orig));
- free_string_field (&(args_info->reserved1_arg));
- free_string_field (&(args_info->reserved1_orig));
free_string_field (&(args_info->reserved2_arg));
free_string_field (&(args_info->reserved2_orig));
@@ -449,8 +442,6 @@ cmdline_parser_dump(FILE *outfile, struct gengetopt_args_info *args_info)
write_into_file(outfile, "second-image-flag", args_info->second_image_flag_orig, cmdline_parser_second_image_flag_values);
if (args_info->inactive_given)
write_into_file(outfile, "inactive", args_info->inactive_orig, cmdline_parser_inactive_values);
- if (args_info->reserved1_given)
- write_into_file(outfile, "reserved1", args_info->reserved1_orig, 0);
if (args_info->reserved2_given)
write_into_file(outfile, "reserved2", args_info->reserved2_orig, 0);
if (args_info->kernel_file_has_header_given)
@@ -794,7 +785,6 @@ cmdline_parser_internal (
{ "rsa-signature", 1, NULL, 'r' },
{ "second-image-flag", 1, NULL, 0 },
{ "inactive", 1, NULL, 0 },
- { "reserved1", 1, NULL, 0 },
{ "reserved2", 1, NULL, 0 },
{ "kernel-file-has-header", 0, NULL, 0 },
{ 0, 0, 0, 0 }
@@ -1090,20 +1080,6 @@ cmdline_parser_internal (
goto failure;
}
- /* String for first reserved section.. */
- else if (strcmp (long_options[option_index].name, "reserved1") == 0)
- {
-
-
- if (update_arg( (void *)&(args_info->reserved1_arg),
- &(args_info->reserved1_orig), &(args_info->reserved1_given),
- &(local_args_info.reserved1_given), optarg, 0, 0, ARG_STRING,
- check_ambiguity, override, 0, 0,
- "reserved1", '-',
- additional_error))
- goto failure;
-
- }
/* String for second reserved section.. */
else if (strcmp (long_options[option_index].name, "reserved2") == 0)
{