[PATCH] USB: serial: Correct return value on read

From: Gimcuan Hui
Date: Sun Nov 26 2017 - 11:19:10 EST


It's meaningless to return buf[0] on read. Because the caller of this
interface checks the return value negative or not. Instead, we should
return the result variable.

Signed-off-by: Gimcuan Hui <gimcuan@xxxxxxxxx>
---
drivers/usb/serial/ark3116.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/usb/serial/ark3116.c b/drivers/usb/serial/ark3116.c
index 3c544782f60b..bfdbc7164e7b 100644
--- a/drivers/usb/serial/ark3116.c
+++ b/drivers/usb/serial/ark3116.c
@@ -101,11 +101,9 @@ static int ark3116_read_reg(struct usb_serial *serial,
reg, result);
if (result >= 0)
result = -EIO;
-
- return result;
}

- return buf[0];
+ return result;
}

static inline int calc_divisor(int bps)
--
2.11.0