diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-01-18 10:41:38 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-01-18 15:49:00 -0800 |
commit | a29c408521b2b8969fead6aa3c7819800d6b473a (patch) | |
tree | 9f26e5681dbaa5c54f29610dc8c9b5ccc8cc2e6d /drivers/usb/c67x00/c67x00-ll-hpi.c | |
parent | f40d781554ef5413d70254e422bc59d6538333d2 (diff) | |
download | linux-a29c408521b2b8969fead6aa3c7819800d6b473a.tar.gz linux-a29c408521b2b8969fead6aa3c7819800d6b473a.tar.xz |
USB: c67x00-ll-hpi.c: signedness bug in ll_recv_msg()
The callers expect this function to return zero on success or -EIO if it
times out. The type should be int instead of unsigned short.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Peter Korsgaard <jacmet@sunsite.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/c67x00/c67x00-ll-hpi.c')
-rw-r--r-- | drivers/usb/c67x00/c67x00-ll-hpi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/c67x00/c67x00-ll-hpi.c b/drivers/usb/c67x00/c67x00-ll-hpi.c index a9636f43bca2..3a1ca4dfc83a 100644 --- a/drivers/usb/c67x00/c67x00-ll-hpi.c +++ b/drivers/usb/c67x00/c67x00-ll-hpi.c @@ -237,7 +237,7 @@ void c67x00_ll_hpi_disable_sofeop(struct c67x00_sie *sie) /* -------------------------------------------------------------------------- */ /* Transactions */ -static inline u16 ll_recv_msg(struct c67x00_device *dev) +static inline int ll_recv_msg(struct c67x00_device *dev) { u16 res; |