[U-Boot] [PATCH v2 03/27] mkimage: Sort the option processing code by option

Simon Glass sjg at chromium.org
Tue Feb 23 06:55:35 CET 2016


Adjust the code so that option alphabetical order matches the order in the
switch() statement. This makes it easier to find options.

Signed-off-by: Simon Glass <sjg at chromium.org>
---

Changes in v2: None

 tools/mkimage.c | 50 +++++++++++++++++++++++++-------------------------
 1 file changed, 25 insertions(+), 25 deletions(-)

diff --git a/tools/mkimage.c b/tools/mkimage.c
index 97ab05a..00391d0 100644
--- a/tools/mkimage.c
+++ b/tools/mkimage.c
@@ -75,8 +75,13 @@ static void process_args(int argc, char **argv)
 	while ((opt = getopt(argc, argv,
 			     "a:A:cC:d:D:e:f:Fk:K:ln:O:rR:sT:vVx")) != -1) {
 		switch (opt) {
-		case 'l':
-			params.lflag = 1;
+		case 'a':
+			params.addr = strtoul(optarg, &ptr, 16);
+			if (*ptr) {
+				fprintf(stderr, "%s: invalid load address %s\n",
+					params.cmdname, optarg);
+				exit(EXIT_FAILURE);
+			}
 			break;
 		case 'A':
 			params.arch = genimg_get_arch_id(optarg);
@@ -91,33 +96,13 @@ static void process_args(int argc, char **argv)
 			if (params.comp < 0)
 				usage();
 			break;
-		case 'D':
-			params.dtc = optarg;
-			break;
-		case 'O':
-			params.os = genimg_get_os_id(optarg);
-			if (params.os < 0)
-				usage();
-			break;
-		case 'T':
-			params.type = genimg_get_type_id(optarg);
-			if (params.type < 0) {
-				show_image_types();
-				usage();
-			}
-			break;
-		case 'a':
-			params.addr = strtoul(optarg, &ptr, 16);
-			if (*ptr) {
-				fprintf(stderr, "%s: invalid load address %s\n",
-					params.cmdname, optarg);
-				exit(EXIT_FAILURE);
-			}
-			break;
 		case 'd':
 			params.datafile = optarg;
 			params.dflag = 1;
 			break;
+		case 'D':
+			params.dtc = optarg;
+			break;
 		case 'e':
 			params.ep = strtoul(optarg, &ptr, 16);
 			if (*ptr) {
@@ -144,9 +129,17 @@ static void process_args(int argc, char **argv)
 		case 'K':
 			params.keydest = optarg;
 			break;
+		case 'l':
+			params.lflag = 1;
+			break;
 		case 'n':
 			params.imagename = optarg;
 			break;
+		case 'O':
+			params.os = genimg_get_os_id(optarg);
+			if (params.os < 0)
+				usage();
+			break;
 		case 'r':
 			params.require_keys = 1;
 			break;
@@ -160,6 +153,13 @@ static void process_args(int argc, char **argv)
 		case 's':
 			params.skipcpy = 1;
 			break;
+		case 'T':
+			params.type = genimg_get_type_id(optarg);
+			if (params.type < 0) {
+				show_image_types();
+				usage();
+			}
+			break;
 		case 'v':
 			params.vflag++;
 			break;
-- 
2.7.0.rc3.207.g0ac5344



More information about the U-Boot mailing list