USB: gadget rndis: send notifications
authorRichard Röjfors <[email protected]>
Sun, 16 Nov 2008 03:53:24 +0000 (19:53 -0800)
committerGreg Kroah-Hartman <[email protected]>
Thu, 20 Nov 2008 06:01:34 +0000 (22:01 -0800)
It turns out that atomic_inc_return() returns the *new* value
not the original one, so the logic in rndis_response_available()
kept the first RNDIS response notification from getting out.
This prevented interoperation with MS-Windows (but not Linux).

Fix this to make RNDIS behave again.

Signed-off-by: Richard Röjfors <[email protected]>
Signed-off-by: David Brownell <[email protected]>
Cc: stable <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
drivers/usb/gadget/f_rndis.c

index 8afb14a4a72f2bd20d51bf6f1eba357dee147a0f..428b5993575a9126104da3ec32ea695e7881c9e8 100644 (file)
@@ -302,7 +302,7 @@ static void rndis_response_available(void *_rndis)
        __le32                          *data = req->buf;
        int                             status;
 
-       if (atomic_inc_return(&rndis->notify_count))
+       if (atomic_inc_return(&rndis->notify_count) != 1)
                return;
 
        /* Send RNDIS RESPONSE_AVAILABLE notification; a