usb: gadget: ether: Add \n to each attribute of ethernet functions
authorKrzysztof Opasiak <k.opasiak@samsung.com>
Wed, 21 Dec 2016 08:48:44 +0000 (09:48 +0100)
committerFelipe Balbi <felipe.balbi@linux.intel.com>
Tue, 24 Jan 2017 09:04:07 +0000 (11:04 +0200)
Generally in SysFS and ConfigFS files are new line terminated.
Also most of USB functions adds a trailing newline to each attribute.
Let's follow this convention also in ethernet functions.

Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
drivers/usb/gadget/function/u_ether.c
drivers/usb/gadget/function/u_ether_configfs.h

index b4e5d6dfd549501b29a95f8da35b3d32f29bfb45..c3cab77181d4c2677ea5214d6c9abc0b120d83e8 100644 (file)
@@ -913,9 +913,16 @@ EXPORT_SYMBOL_GPL(gether_set_dev_addr);
 int gether_get_dev_addr(struct net_device *net, char *dev_addr, int len)
 {
        struct eth_dev *dev;
+       int ret;
 
        dev = netdev_priv(net);
-       return get_ether_addr_str(dev->dev_mac, dev_addr, len);
+       ret = get_ether_addr_str(dev->dev_mac, dev_addr, len);
+       if (ret + 1 < len) {
+               dev_addr[ret++] = '\n';
+               dev_addr[ret] = '\0';
+       }
+
+       return ret;
 }
 EXPORT_SYMBOL_GPL(gether_get_dev_addr);
 
@@ -935,9 +942,16 @@ EXPORT_SYMBOL_GPL(gether_set_host_addr);
 int gether_get_host_addr(struct net_device *net, char *host_addr, int len)
 {
        struct eth_dev *dev;
+       int ret;
 
        dev = netdev_priv(net);
-       return get_ether_addr_str(dev->host_mac, host_addr, len);
+       ret = get_ether_addr_str(dev->host_mac, host_addr, len);
+       if (ret + 1 < len) {
+               host_addr[ret++] = '\n';
+               host_addr[ret] = '\0';
+       }
+
+       return ret;
 }
 EXPORT_SYMBOL_GPL(gether_get_host_addr);
 
@@ -984,10 +998,12 @@ EXPORT_SYMBOL_GPL(gether_get_qmult);
 
 int gether_get_ifname(struct net_device *net, char *name, int len)
 {
+       int ret;
+
        rtnl_lock();
-       strlcpy(name, netdev_name(net), len);
+       ret = snprintf(name, len, "%s\n", netdev_name(net));
        rtnl_unlock();
-       return strlen(name);
+       return ret < len ? ret : len;
 }
 EXPORT_SYMBOL_GPL(gether_get_ifname);
 
index 4f47289fcf7cee45e93e8f9171a19390f91ee49f..c71133de17e71a950f532a7ab599a1c25abad705 100644 (file)
                mutex_lock(&opts->lock);                                \
                qmult = gether_get_qmult(opts->net);                    \
                mutex_unlock(&opts->lock);                              \
-               return sprintf(page, "%d", qmult);                      \
+               return sprintf(page, "%d\n", qmult);                    \
        }                                                               \
                                                                        \
        static ssize_t _f_##_opts_qmult_store(struct config_item *item, \