On Tue, 2010-03-02 at 11:40 +0100, Oliver Neukum wrote:
> Am Dienstag, 2. März 2010 11:28:45 schrieb Libin Yang:
> > @@ -827,9 +864,18 @@ int xhci_urb_dequeue(struct usb_hcd *hcd, struct urb *urb, int status)
> > ep_ring = ep->ring;
> > xhci_dbg(xhci, "Endpoint ring:\n");
> > xhci_debug_ring(xhci, ep_ring);
> > - td = (struct xhci_td *) urb->hcpriv;
> >
> > - list_add_tail(&td->cancelled_td_list, &ep->cancelled_td_list);
> > + urb_priv = urb->hcpriv;
> > +
> > + if (urb_priv) {
>
> How can this be NULL?
>
> Regards
> Oliver
>
Thanks for the comment. Will remove unnecessary if-clauses.
--
Thanks,
Andiry
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
