On 12/13/2013 02:46 PM, Andrzej Pietrasiewicz wrote:
> diff --git a/drivers/usb/gadget/nokia.c b/drivers/usb/gadget/nokia.c
> index 0a8099a..3ab3861 100644
> --- a/drivers/usb/gadget/nokia.c
> +++ b/drivers/usb/gadget/nokia.c
> @@ -126,9 +126,9 @@ static int __init nokia_bind_config(struct 
> usb_configuration *c)
>       struct usb_function *f_ecm;
>       struct usb_function *f_obex2 = NULL;
>       int status = 0;
> -     int obex1_stat = 0;
> -     int obex2_stat = 0;
> -     int phonet_stat = 0;
> +     int obex1_stat = -1;
> +     int obex2_stat = -1;
> +     int phonet_stat = -1;

I haven't look at the whole context but please do -ENODEV or something
like that but don't do -1.

>       if (!IS_ERR(fi_phonet)) {
>               f_phonet = usb_get_function(fi_phonet);
> 

Sebastian
--
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