tools: dumpimage: Provide more feedback on error
authorMartyn Welch <martyn.welch@collabora.com>
Sat, 26 Jan 2019 02:31:50 +0000 (02:31 +0000)
committerTom Rini <trini@konsulko.com>
Fri, 1 Feb 2019 19:13:45 +0000 (14:13 -0500)
The dumpimage utility errors out in a number of places without providing
sufficient feedback to allow the user to easily determine what they have
done wrong. Add addtional error messages to make the cause of the failure
more obvious.

Signed-off-by: Martyn Welch <martyn.welch@collabora.com>
tools/dumpimage.c

index 7115df04c120ee4a49b6626ffde62465b518f3b5..2847e6c0b470f5957166ce9ce2fc86f005c3571d 100644 (file)
@@ -80,6 +80,8 @@ int main(int argc, char **argv)
                case 'T':
                        params.type = genimg_get_type_id(optarg);
                        if (params.type < 0) {
+                               fprintf(stderr, "%s: Invalid type\n",
+                                       params.cmdname);
                                usage();
                        }
                        break;
@@ -101,8 +103,10 @@ int main(int argc, char **argv)
                }
        }
 
-       if (optind >= argc)
+       if (optind >= argc) {
+               fprintf(stderr, "%s: image file missing\n", params.cmdname);
                usage();
+       }
 
        /* set tparams as per input type_id */
        tparams = imagetool_get_type(params.type);
@@ -117,8 +121,11 @@ int main(int argc, char **argv)
         * as per image type to be generated/listed
         */
        if (tparams->check_params) {
-               if (tparams->check_params(&params))
+               if (tparams->check_params(&params)) {
+                       fprintf(stderr, "%s: Parameter check failed\n",
+                               params.cmdname);
                        usage();
+               }
        }
 
        if (params.iflag)