staging: usbip: reformat function stub_recv_cmd_unlink()
authorKurt Kanzenbach <ly80toro@cip.cs.fau.de>
Thu, 4 Apr 2013 14:03:03 +0000 (16:03 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 5 Apr 2013 21:22:50 +0000 (14:22 -0700)
Reformat function stub_recv_cmd_unlink() to improve readability.

Signed-off-by: Kurt Kanzenbach <ly80toro@cip.cs.fau.de>
Signed-off-by: Stefan Reif <ke42caxa@cip.cs.fau.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/usbip/stub_rx.c

index 715e8a79fb4e846895594cb5a3d3bdc4283dcce5..9fc62b35e80b32f6bdedbe8c3d68c2d94952cc74 100644 (file)
@@ -228,61 +228,61 @@ static void tweak_special_requests(struct urb *urb)
 static int stub_recv_cmd_unlink(struct stub_device *sdev,
                                struct usbip_header *pdu)
 {
+       int ret;
        unsigned long flags;
-
        struct stub_priv *priv;
 
        spin_lock_irqsave(&sdev->priv_lock, flags);
 
        list_for_each_entry(priv, &sdev->priv_init, list) {
-               if (priv->seqnum == pdu->u.cmd_unlink.seqnum) {
-                       int ret;
-
-                       dev_info(&priv->urb->dev->dev, "unlink urb %p\n",
-                                priv->urb);
-
-                       /*
-                        * This matched urb is not completed yet (i.e., be in
-                        * flight in usb hcd hardware/driver). Now we are
-                        * cancelling it. The unlinking flag means that we are
-                        * now not going to return the normal result pdu of a
-                        * submission request, but going to return a result pdu
-                        * of the unlink request.
-                        */
-                       priv->unlinking = 1;
-
-                       /*
-                        * In the case that unlinking flag is on, prev->seqnum
-                        * is changed from the seqnum of the cancelling urb to
-                        * the seqnum of the unlink request. This will be used
-                        * to make the result pdu of the unlink request.
-                        */
-                       priv->seqnum = pdu->base.seqnum;
-
-                       spin_unlock_irqrestore(&sdev->priv_lock, flags);
-
-                       /*
-                        * usb_unlink_urb() is now out of spinlocking to avoid
-                        * spinlock recursion since stub_complete() is
-                        * sometimes called in this context but not in the
-                        * interrupt context.  If stub_complete() is executed
-                        * before we call usb_unlink_urb(), usb_unlink_urb()
-                        * will return an error value. In this case, stub_tx
-                        * will return the result pdu of this unlink request
-                        * though submission is completed and actual unlinking
-                        * is not executed. OK?
-                        */
-                       /* In the above case, urb->status is not -ECONNRESET,
-                        * so a driver in a client host will know the failure
-                        * of the unlink request ?
-                        */
-                       ret = usb_unlink_urb(priv->urb);
-                       if (ret != -EINPROGRESS)
-                               dev_err(&priv->urb->dev->dev,
-                                       "failed to unlink a urb %p, ret %d\n",
-                                       priv->urb, ret);
-                       return 0;
-               }
+               if (priv->seqnum != pdu->u.cmd_unlink.seqnum)
+                       continue;
+
+               dev_info(&priv->urb->dev->dev, "unlink urb %p\n",
+                        priv->urb);
+
+               /*
+                * This matched urb is not completed yet (i.e., be in
+                * flight in usb hcd hardware/driver). Now we are
+                * cancelling it. The unlinking flag means that we are
+                * now not going to return the normal result pdu of a
+                * submission request, but going to return a result pdu
+                * of the unlink request.
+                */
+               priv->unlinking = 1;
+
+               /*
+                * In the case that unlinking flag is on, prev->seqnum
+                * is changed from the seqnum of the cancelling urb to
+                * the seqnum of the unlink request. This will be used
+                * to make the result pdu of the unlink request.
+                */
+               priv->seqnum = pdu->base.seqnum;
+
+               spin_unlock_irqrestore(&sdev->priv_lock, flags);
+
+               /*
+                * usb_unlink_urb() is now out of spinlocking to avoid
+                * spinlock recursion since stub_complete() is
+                * sometimes called in this context but not in the
+                * interrupt context.  If stub_complete() is executed
+                * before we call usb_unlink_urb(), usb_unlink_urb()
+                * will return an error value. In this case, stub_tx
+                * will return the result pdu of this unlink request
+                * though submission is completed and actual unlinking
+                * is not executed. OK?
+                */
+               /* In the above case, urb->status is not -ECONNRESET,
+                * so a driver in a client host will know the failure
+                * of the unlink request ?
+                */
+               ret = usb_unlink_urb(priv->urb);
+               if (ret != -EINPROGRESS)
+                       dev_err(&priv->urb->dev->dev,
+                               "failed to unlink a urb %p, ret %d\n",
+                               priv->urb, ret);
+
+               return 0;
        }
 
        usbip_dbg_stub_rx("seqnum %d is not pending\n",