>>>>> "Dan" == Dan Carpenter <dan.carpen...@oracle.com> 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.carpen...@oracle.com>

Acked-by: Peter Korsgaard <jac...@sunsite.dk>


 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 linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to