usb: gadget: configfs: Fix set but not used variable warning
authorDaniel Scally <dan.scally@ideasonboard.com>
Thu, 9 Feb 2023 09:43:59 +0000 (09:43 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 9 Feb 2023 11:12:47 +0000 (12:12 +0100)
Fix a -Wunused-but-set-variable warning in gadget_string_s_store()

Fixes: 15a7cf8caabe ("usb: gadget: configfs: Support arbitrary string descriptors")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Daniel Scally <dan.scally@ideasonboard.com>
Link: https://lore.kernel.org/r/20230209094359.1549629-1-dan.scally@ideasonboard.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/gadget/configfs.c

index 06a0b73..b9f1136 100644 (file)
@@ -821,13 +821,11 @@ static ssize_t gadget_string_s_store(struct config_item *item, const char *page,
 {
        struct gadget_string *string = to_gadget_string(item);
        int size = min(sizeof(string->string), len + 1);
-       int ret;
 
        if (len > USB_MAX_STRING_LEN)
                return -EINVAL;
 
-       ret = strscpy(string->string, page, size);
-       return len;
+       return strscpy(string->string, page, size);
 }
 CONFIGFS_ATTR(gadget_string_, s);