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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
