Fix coding style issues detected by checkpatch.pl:
1. do not use assignment in if condition
2. line over 80 characters

Signed-off-by: Wahib Faizi <wahibfa...@gmail.com>
---
 .../usbip/userspace/libsrc/usbip_host_driver.c     |    7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/usbip/userspace/libsrc/usbip_host_driver.c 
b/drivers/staging/usbip/userspace/libsrc/usbip_host_driver.c
index 92caef7..bef08d5 100644
--- a/drivers/staging/usbip/userspace/libsrc/usbip_host_driver.c
+++ b/drivers/staging/usbip/userspace/libsrc/usbip_host_driver.c
@@ -47,7 +47,8 @@ static int32_t read_attr_usbip_status(struct usbip_usb_device 
*udev)
        snprintf(status_attr_path, SYSFS_PATH_MAX, "%s/usbip_status",
                 udev->path);
 
-       if ((fd = open(status_attr_path, O_RDONLY)) < 0) {
+       fd = open(status_attr_path, O_RDONLY);
+       if (fd < 0) {
                err("error opening attribute %s", status_attr_path);
                return -1;
        }
@@ -87,8 +88,8 @@ struct usbip_exported_device *usbip_exported_device_new(const 
char *sdevpath)
                goto err;
 
        /* reallocate buffer to include usb interface data */
-       size = sizeof(struct usbip_exported_device) + edev->udev.bNumInterfaces 
*
-               sizeof(struct usbip_usb_interface);
+       size = sizeof(struct usbip_exported_device) +
+               edev->udev.bNumInterfaces * sizeof(struct usbip_usb_interface);
 
        edev_old = edev;
        edev = realloc(edev, size);
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to