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 <s...@chromium.org>
---

 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

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to