On Wed, 6 Oct 2010 20:35:53 +0200 Jean-Francois Moine <moinejf@xxxxxxx> wrote: > On Wed, 6 Oct 2010 16:04:41 +0200 > Antonio Ospite <ospite@xxxxxxxxxxxxxxxxx> wrote: > > > Thanks, the following change fixes it, was this what you had in mind? > > > > diff --git a/drivers/media/video/gspca/gspca.c [...] > > Yes, but, after thought, as there is only one alternate setting, the > tests could be: > if (gspca_dev->audio && i < gspca_dev->nbalt - 1) > and > if (gspca_dev->audio && i > 0) > > This should work also for isochronous transfers. JF this change as is does not work for me, if I change the second check to if (gspca_dev->audio && i > 1) it does, but I don't know if this breaks anything else. In my case I have: get_ep: gspca_dev->cam.reverse_alts: 0 get_ep: gspca_dev->alt: 1 get_ep: gspca_dev->nbalt: 1 get_ep: gspca_dev->audio: 1 get_ep: gspca_dev->cam.bulk: 1 Thanks, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why is top-posting such a bad thing?
Attachment:
pgpS7FIEPnlSb.pgp
Description: PGP signature
