usb: gadget: error out if g_dnl registration fails
authorSjoerd Simons <sjoerd.simons@collabora.co.uk>
Tue, 4 Jun 2019 19:01:55 +0000 (21:01 +0200)
committerMarek Vasut <marex@denx.de>
Fri, 14 Jun 2019 10:39:50 +0000 (12:39 +0200)
If g_dnl_register fails return an error rather then stubornly
continuing onwards.

Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
cmd/usb_gadget_sdp.c
common/spl/spl_sdp.c

index 808ed974fa7537a367f3b8d5567fe1d0988f6224..2ead06be9f5acf351f2e24e450ecff683167f6d0 100644 (file)
@@ -13,7 +13,7 @@
 
 static int do_sdp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
-       int ret = CMD_RET_FAILURE;
+       int ret;
 
        if (argc < 2)
                return CMD_RET_USAGE;
@@ -23,7 +23,11 @@ static int do_sdp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        usb_gadget_initialize(controller_index);
 
        g_dnl_clear_detach();
-       g_dnl_register("usb_dnl_sdp");
+       ret = g_dnl_register("usb_dnl_sdp");
+       if (ret) {
+               pr_err("SDP dnl register failed: %d\n", ret);
+               goto exit_register;
+       }
 
        ret = sdp_init(controller_index);
        if (ret) {
@@ -37,9 +41,10 @@ static int do_sdp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 
 exit:
        g_dnl_unregister();
+exit_register:
        usb_gadget_release(controller_index);
 
-       return ret;
+       return CMD_RET_FAILURE;
 }
 
 U_BOOT_CMD(sdp, 2, 1, do_sdp,
index 807256e908c2cc43eefcced065f7cb25b822f1b6..7fc440497186f0ba25ea6f37375296497d1a7d6a 100644 (file)
@@ -17,7 +17,11 @@ static int spl_sdp_load_image(struct spl_image_info *spl_image,
        const int controller_index = 0;
 
        g_dnl_clear_detach();
-       g_dnl_register("usb_dnl_sdp");
+       ret = g_dnl_register("usb_dnl_sdp");
+       if (ret) {
+               pr_err("SDP dnl register failed: %d\n", ret);
+               return ret;
+       }
 
        ret = sdp_init(controller_index);
        if (ret) {