>>>>> "Dan" == Dan Carpenter <dan.carpenter@xxxxxxxxxx> writes:
Dan> The callers expect this function to return zero on success or -EIO if it
Dan> times out. The type should be int instead of unsigned short.
Dan> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Acked-by: Peter Korsgaard <jacmet@xxxxxxxxxx>
Dan> diff --git a/drivers/usb/c67x00/c67x00-ll-hpi.c b/drivers/usb/c67x00/c67x00-ll-hpi.c
Dan> index a9636f4..3a1ca4d 100644
Dan> --- a/drivers/usb/c67x00/c67x00-ll-hpi.c
Dan> +++ b/drivers/usb/c67x00/c67x00-ll-hpi.c
Dan> @@ -237,7 +237,7 @@ void c67x00_ll_hpi_disable_sofeop(struct c67x00_sie *sie)
Dan> /* -------------------------------------------------------------------------- */
Dan> /* Transactions */
Dan> -static inline u16 ll_recv_msg(struct c67x00_device *dev)
Dan> +static inline int ll_recv_msg(struct c67x00_device *dev)
Dan> {
Dan> u16 res;
--
Bye, Peter Korsgaard
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
