* Re: Regression in sound/usb/ [not found] <CADYPuQ75GAxGyZgiUwygvnQMmDcL9+NMHNo432kg1RYC7vbfQA@mail.gmail.com> @ 2012-07-02 12:10 ` Daniel Mack [not found] ` <CADYPuQ5RFJAaFDpT+4XM-JDWzDO5czVAU3A++_HVHjhN=VBhvQ@mail.gmail.com> 0 siblings, 1 reply; 13+ messages in thread From: Daniel Mack @ 2012-07-02 12:10 UTC (permalink / raw) To: Philipp Dreimann; +Cc: Takashi Iwai, alsa-devel, Clemens Ladisch On 01.07.2012 13:40, Philipp Dreimann wrote: > Hello, > > my usb webcam, which happens to have a microphone too, stopped working > using usb2 since I tried kernel 3.5-rc2. (It works using usb3.) Please try 3.5-rc5. A number of regressions were fixed just recently. Let me know if that version still causes problems. Daniel ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <CADYPuQ5RFJAaFDpT+4XM-JDWzDO5czVAU3A++_HVHjhN=VBhvQ@mail.gmail.com>]
* Re: Regression in sound/usb/ [not found] ` <CADYPuQ5RFJAaFDpT+4XM-JDWzDO5czVAU3A++_HVHjhN=VBhvQ@mail.gmail.com> @ 2012-07-03 8:00 ` Takashi Iwai [not found] ` <CADYPuQ4Uj8pq9nZOJemY9kZtSrG-WfG4s90b+a00_KteJXU_Sg@mail.gmail.com> 0 siblings, 1 reply; 13+ messages in thread From: Takashi Iwai @ 2012-07-03 8:00 UTC (permalink / raw) To: Philipp Dreimann; +Cc: alsa-devel, Clemens Ladisch, Daniel Mack At Tue, 3 Jul 2012 09:41:09 +0200, Philipp Dreimann wrote: > > On 2 July 2012 14:10, Daniel Mack <zonque@gmail.com> wrote: > > On 01.07.2012 13:40, Philipp Dreimann wrote: > >> Hello, > >> > >> my usb webcam, which happens to have a microphone too, stopped working > >> using usb2 since I tried kernel 3.5-rc2. (It works using usb3.) > > > > Please try 3.5-rc5. A number of regressions were fixed just recently. > > > > Let me know if that version still causes problems. > > It still does. Try to pass ignore_ctl_error=1 to snd-usb-audio module. If this works and no other problem happens, we can add a similar quirk like some Logitech webcams. Takashi ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <CADYPuQ4Uj8pq9nZOJemY9kZtSrG-WfG4s90b+a00_KteJXU_Sg@mail.gmail.com>]
* Re: Regression in sound/usb/ [not found] ` <CADYPuQ4Uj8pq9nZOJemY9kZtSrG-WfG4s90b+a00_KteJXU_Sg@mail.gmail.com> @ 2012-07-03 8:27 ` Takashi Iwai [not found] ` <CADYPuQ6DT4=NW2tzCLrKYnUHQ6p5qONZjY8PeKq-Z4HAk68tpw@mail.gmail.com> 0 siblings, 1 reply; 13+ messages in thread From: Takashi Iwai @ 2012-07-03 8:27 UTC (permalink / raw) To: Philipp Dreimann; +Cc: alsa-devel, Clemens Ladisch, Daniel Mack At Tue, 3 Jul 2012 10:20:13 +0200, Philipp Dreimann wrote: > > On 3 July 2012 10:00, Takashi Iwai <tiwai@suse.de> wrote: > > At Tue, 3 Jul 2012 09:41:09 +0200, > > Philipp Dreimann wrote: > >> > >> On 2 July 2012 14:10, Daniel Mack <zonque@gmail.com> wrote: > >> > On 01.07.2012 13:40, Philipp Dreimann wrote: > >> >> Hello, > >> >> > >> >> my usb webcam, which happens to have a microphone too, stopped working > >> >> using usb2 since I tried kernel 3.5-rc2. (It works using usb3.) > >> > > >> > Please try 3.5-rc5. A number of regressions were fixed just recently. > >> > > >> > Let me know if that version still causes problems. > >> > >> It still does. > > > > Try to pass ignore_ctl_error=1 to snd-usb-audio module. > > If this works and no other problem happens, we can add a similar quirk > > like some Logitech webcams. > > This does not seem to help. And webcam itself is working with our without snd-usb-audio module? Which device at all? Too little information to analyze. Takashi ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <CADYPuQ6DT4=NW2tzCLrKYnUHQ6p5qONZjY8PeKq-Z4HAk68tpw@mail.gmail.com>]
* Re: Regression in sound/usb/ [not found] ` <CADYPuQ6DT4=NW2tzCLrKYnUHQ6p5qONZjY8PeKq-Z4HAk68tpw@mail.gmail.com> @ 2012-07-03 9:01 ` Takashi Iwai 2012-07-05 14:41 ` Takashi Iwai 0 siblings, 1 reply; 13+ messages in thread From: Takashi Iwai @ 2012-07-03 9:01 UTC (permalink / raw) To: Philipp Dreimann; +Cc: alsa-devel, Clemens Ladisch, Daniel Mack At Tue, 3 Jul 2012 10:40:51 +0200, Philipp Dreimann wrote: > > On 3 July 2012 10:27, Takashi Iwai <tiwai@suse.de> wrote: > > At Tue, 3 Jul 2012 10:20:13 +0200, > > Philipp Dreimann wrote: > >> > >> On 3 July 2012 10:00, Takashi Iwai <tiwai@suse.de> wrote: > >> > At Tue, 3 Jul 2012 09:41:09 +0200, > >> > Philipp Dreimann wrote: > >> >> > >> >> On 2 July 2012 14:10, Daniel Mack <zonque@gmail.com> wrote: > >> >> > On 01.07.2012 13:40, Philipp Dreimann wrote: > >> >> >> Hello, > >> >> >> > >> >> >> my usb webcam, which happens to have a microphone too, stopped working > >> >> >> using usb2 since I tried kernel 3.5-rc2. (It works using usb3.) > >> >> > > >> >> > Please try 3.5-rc5. A number of regressions were fixed just recently. > >> >> > > >> >> > Let me know if that version still causes problems. > >> >> > >> >> It still does. > >> > > >> > Try to pass ignore_ctl_error=1 to snd-usb-audio module. > >> > If this works and no other problem happens, we can add a similar quirk > >> > like some Logitech webcams. > >> > >> This does not seem to help. > > > > And webcam itself is working with our without snd-usb-audio module? > Yes, if I blacklist the module, the webcam works fine. > > > Which device at all? > logitech c910 > > > Too little information to analyze. > You guys tell me what you need to know. Try the patch below. If it still doesn't work, give the kernel message again and lsusb -v output, too. thanks, Takashi --- diff --git a/sound/usb/pcm.c b/sound/usb/pcm.c index 54607f8..f0ede13 100644 --- a/sound/usb/pcm.c +++ b/sound/usb/pcm.c @@ -387,7 +387,7 @@ add_sync_ep: subs->data_endpoint->sync_master = subs->sync_endpoint; } - if ((err = snd_usb_init_pitch(subs->stream->chip, subs->interface, alts, fmt)) < 0) + if ((err = snd_usb_init_pitch(subs->stream->chip, fmt->iface, alts, fmt)) < 0) return err; subs->cur_audiofmt = fmt; @@ -450,7 +450,7 @@ static int snd_usb_hw_params(struct snd_pcm_substream *substream, struct usb_interface *iface; iface = usb_ifnum_to_if(subs->dev, fmt->iface); alts = &iface->altsetting[fmt->altset_idx]; - ret = snd_usb_init_sample_rate(subs->stream->chip, subs->interface, alts, fmt, rate); + ret = snd_usb_init_sample_rate(subs->stream->chip, fmt->iface, alts, fmt, rate); if (ret < 0) return ret; subs->cur_rate = rate; ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: Regression in sound/usb/ 2012-07-03 9:01 ` Takashi Iwai @ 2012-07-05 14:41 ` Takashi Iwai [not found] ` <CADYPuQ7yWpnPSvoVQDV+EyJfC39MDxhP9r5Jiqk_Jh8g65Xd4g@mail.gmail.com> 0 siblings, 1 reply; 13+ messages in thread From: Takashi Iwai @ 2012-07-05 14:41 UTC (permalink / raw) To: Philipp Dreimann; +Cc: alsa-devel, Clemens Ladisch, Daniel Mack At Tue, 03 Jul 2012 11:01:42 +0200, Takashi Iwai wrote: > > At Tue, 3 Jul 2012 10:40:51 +0200, > Philipp Dreimann wrote: > > > > On 3 July 2012 10:27, Takashi Iwai <tiwai@suse.de> wrote: > > > At Tue, 3 Jul 2012 10:20:13 +0200, > > > Philipp Dreimann wrote: > > >> > > >> On 3 July 2012 10:00, Takashi Iwai <tiwai@suse.de> wrote: > > >> > At Tue, 3 Jul 2012 09:41:09 +0200, > > >> > Philipp Dreimann wrote: > > >> >> > > >> >> On 2 July 2012 14:10, Daniel Mack <zonque@gmail.com> wrote: > > >> >> > On 01.07.2012 13:40, Philipp Dreimann wrote: > > >> >> >> Hello, > > >> >> >> > > >> >> >> my usb webcam, which happens to have a microphone too, stopped working > > >> >> >> using usb2 since I tried kernel 3.5-rc2. (It works using usb3.) > > >> >> > > > >> >> > Please try 3.5-rc5. A number of regressions were fixed just recently. > > >> >> > > > >> >> > Let me know if that version still causes problems. > > >> >> > > >> >> It still does. > > >> > > > >> > Try to pass ignore_ctl_error=1 to snd-usb-audio module. > > >> > If this works and no other problem happens, we can add a similar quirk > > >> > like some Logitech webcams. > > >> > > >> This does not seem to help. > > > > > > And webcam itself is working with our without snd-usb-audio module? > > Yes, if I blacklist the module, the webcam works fine. > > > > > Which device at all? > > logitech c910 > > > > > Too little information to analyze. > > You guys tell me what you need to know. > > Try the patch below. If it still doesn't work, give the kernel > message again and lsusb -v output, too. Did it make any difference? thanks, Takashi > > > thanks, > > Takashi > > --- > diff --git a/sound/usb/pcm.c b/sound/usb/pcm.c > index 54607f8..f0ede13 100644 > --- a/sound/usb/pcm.c > +++ b/sound/usb/pcm.c > @@ -387,7 +387,7 @@ add_sync_ep: > subs->data_endpoint->sync_master = subs->sync_endpoint; > } > > - if ((err = snd_usb_init_pitch(subs->stream->chip, subs->interface, alts, fmt)) < 0) > + if ((err = snd_usb_init_pitch(subs->stream->chip, fmt->iface, alts, fmt)) < 0) > return err; > > subs->cur_audiofmt = fmt; > @@ -450,7 +450,7 @@ static int snd_usb_hw_params(struct snd_pcm_substream *substream, > struct usb_interface *iface; > iface = usb_ifnum_to_if(subs->dev, fmt->iface); > alts = &iface->altsetting[fmt->altset_idx]; > - ret = snd_usb_init_sample_rate(subs->stream->chip, subs->interface, alts, fmt, rate); > + ret = snd_usb_init_sample_rate(subs->stream->chip, fmt->iface, alts, fmt, rate); > if (ret < 0) > return ret; > subs->cur_rate = rate; ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <CADYPuQ7yWpnPSvoVQDV+EyJfC39MDxhP9r5Jiqk_Jh8g65Xd4g@mail.gmail.com>]
* Re: Regression in sound/usb/ [not found] ` <CADYPuQ7yWpnPSvoVQDV+EyJfC39MDxhP9r5Jiqk_Jh8g65Xd4g@mail.gmail.com> @ 2012-07-05 15:36 ` Takashi Iwai 2012-07-10 21:14 ` Daniel Mack 2012-07-05 15:57 ` Daniel Mack 1 sibling, 1 reply; 13+ messages in thread From: Takashi Iwai @ 2012-07-05 15:36 UTC (permalink / raw) To: Philipp Dreimann; +Cc: alsa-devel, Clemens Ladisch, Daniel Mack At Thu, 5 Jul 2012 17:32:30 +0200, Philipp Dreimann wrote: > > On 5 July 2012 16:41, Takashi Iwai <tiwai@suse.de> wrote: > >> Try the patch below. If it still doesn't work, give the kernel > >> message again and lsusb -v output, too. > > > > Did it make any difference? > > No, it unfortunately did not. > > lsusb is attached and the parts of the kernel log are in the mail: > > [ 41.243211] usb 2-1.2: new high-speed USB device number 3 using ehci_hcd > [ 41.605666] uvcvideo: Found UVC 1.00 device <unnamed> (046d:0821) > [ 42.002275] input: UVC Camera (046d:0821) as > /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.2/2-1.2:1.2/input/input12 > [ 42.085880] usbcore: registered new interface driver snd-usb-audio > > [ 52.790139] uvcvideo: Failed to query (GET_DEF) UVC control 2 on > unit 2: -71 (exp. 2). > [ 52.794363] uvcvideo: Failed to query (GET_DEF) UVC control 2 on > unit 2: -71 (exp. 2). > [ 52.798375] uvcvideo: Failed to query (GET_DEF) UVC control 3 on > unit 2: -71 (exp. 2). > [ 52.802499] uvcvideo: Failed to query (GET_DEF) UVC control 7 on > unit 2: -71 (exp. 2). > [ 52.806617] uvcvideo: Failed to query (GET_DEF) UVC control 11 on > unit 2: -71 (exp. 1). > [ 52.810734] uvcvideo: Failed to query (GET_DEF) UVC control 4 on > unit 2: -71 (exp. 2). > [ 52.814855] uvcvideo: Failed to query (GET_DEF) UVC control 5 on > unit 2: -71 (exp. 1). > [ 52.818976] uvcvideo: Failed to query (GET_CUR) UVC control 11 on > unit 2: -71 (exp. 1). > [ 52.823096] uvcvideo: Failed to query (GET_DEF) UVC control 8 on > unit 2: -71 (exp. 2). > [ 52.827215] uvcvideo: Failed to query (GET_DEF) UVC control 1 on > unit 2: -71 (exp. 2). > [ 52.831331] uvcvideo: Failed to set UVC probe control : -71 (exp. 26). > [ 52.835459] uvcvideo: Failed to set UVC probe control : -71 (exp. 26). > The last line repeats a few times, then the Failed to query block > repeats again, and so on. But only if I try to use the webcam. > > [ 82.363599] 3:1:3: cannot set freq 32000 to ep 0x86 > Is showing up without using the webcam. Hm, it looks better than before (it was '3:-1:3' or such), but there is still something strange there. thanks, Takashi ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Regression in sound/usb/ 2012-07-05 15:36 ` Takashi Iwai @ 2012-07-10 21:14 ` Daniel Mack [not found] ` <CADYPuQ6XRSAxKvctCr3FrJ71smk-xOEMF_5VAv-6F5gdM7dwuQ@mail.gmail.com> 0 siblings, 1 reply; 13+ messages in thread From: Daniel Mack @ 2012-07-10 21:14 UTC (permalink / raw) To: Takashi Iwai Cc: alsa-devel, Clemens Ladisch, Philipp Dreimann, Joseph Salisbury [-- Attachment #1: Type: text/plain, Size: 2541 bytes --] On 05.07.2012 17:36, Takashi Iwai wrote: > At Thu, 5 Jul 2012 17:32:30 +0200, > Philipp Dreimann wrote: >> >> On 5 July 2012 16:41, Takashi Iwai <tiwai@suse.de> wrote: >>>> Try the patch below. If it still doesn't work, give the kernel >>>> message again and lsusb -v output, too. >>> >>> Did it make any difference? >> >> No, it unfortunately did not. >> >> lsusb is attached and the parts of the kernel log are in the mail: >> >> [ 41.243211] usb 2-1.2: new high-speed USB device number 3 using ehci_hcd >> [ 41.605666] uvcvideo: Found UVC 1.00 device <unnamed> (046d:0821) >> [ 42.002275] input: UVC Camera (046d:0821) as >> /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.2/2-1.2:1.2/input/input12 >> [ 42.085880] usbcore: registered new interface driver snd-usb-audio >> >> [ 52.790139] uvcvideo: Failed to query (GET_DEF) UVC control 2 on >> unit 2: -71 (exp. 2). >> [ 52.794363] uvcvideo: Failed to query (GET_DEF) UVC control 2 on >> unit 2: -71 (exp. 2). >> [ 52.798375] uvcvideo: Failed to query (GET_DEF) UVC control 3 on >> unit 2: -71 (exp. 2). >> [ 52.802499] uvcvideo: Failed to query (GET_DEF) UVC control 7 on >> unit 2: -71 (exp. 2). >> [ 52.806617] uvcvideo: Failed to query (GET_DEF) UVC control 11 on >> unit 2: -71 (exp. 1). >> [ 52.810734] uvcvideo: Failed to query (GET_DEF) UVC control 4 on >> unit 2: -71 (exp. 2). >> [ 52.814855] uvcvideo: Failed to query (GET_DEF) UVC control 5 on >> unit 2: -71 (exp. 1). >> [ 52.818976] uvcvideo: Failed to query (GET_CUR) UVC control 11 on >> unit 2: -71 (exp. 1). >> [ 52.823096] uvcvideo: Failed to query (GET_DEF) UVC control 8 on >> unit 2: -71 (exp. 2). >> [ 52.827215] uvcvideo: Failed to query (GET_DEF) UVC control 1 on >> unit 2: -71 (exp. 2). >> [ 52.831331] uvcvideo: Failed to set UVC probe control : -71 (exp. 26). >> [ 52.835459] uvcvideo: Failed to set UVC probe control : -71 (exp. 26). >> The last line repeats a few times, then the Failed to query block >> repeats again, and so on. But only if I try to use the webcam. >> >> [ 82.363599] 3:1:3: cannot set freq 32000 to ep 0x86 >> Is showing up without using the webcam. > > Hm, it looks better than before (it was '3:-1:3' or such), but there > is still something strange there. Sorry for the long delay on this. I finally found some time to look into this and it seems that I got the calls to usb_set_interface wrong. Attached is a patch to fix this - could everybody who is affect please give that a quick try? More explanation is in the commit log. Thanks, Daniel [-- Attachment #2: 0001-ALSA-snd-usb-fix-calls-to-usb_set_interface.patch --] [-- Type: text/x-patch, Size: 3644 bytes --] >From 1b8ebc9974c16b8587bdce3ca9a4435c8ef8141e Mon Sep 17 00:00:00 2001 From: Daniel Mack <zonque@gmail.com> Date: Tue, 10 Jul 2012 22:52:56 +0200 Subject: [PATCH] ALSA: snd-usb: fix calls to usb_set_interface Don't call usb_set_interface() on snd_usb_endpoint creation and also drop it from snd_usb_endpoint_deactivate(). Rather unselect the the active interface before choosing a new alt setting upon the next stream start. Signed-off-by: Daniel Mack <zonque@gmail.com> --- sound/usb/endpoint.c | 45 ++++++++++++++------------------------------- 1 file changed, 14 insertions(+), 31 deletions(-) diff --git a/sound/usb/endpoint.c b/sound/usb/endpoint.c index e690690..2226769 100644 --- a/sound/usb/endpoint.c +++ b/sound/usb/endpoint.c @@ -414,7 +414,7 @@ struct snd_usb_endpoint *snd_usb_add_endpoint(struct snd_usb_audio *chip, { struct list_head *p; struct snd_usb_endpoint *ep; - int ret, is_playback = direction == SNDRV_PCM_STREAM_PLAYBACK; + int is_playback = direction == SNDRV_PCM_STREAM_PLAYBACK; mutex_lock(&chip->mutex); @@ -434,16 +434,6 @@ struct snd_usb_endpoint *snd_usb_add_endpoint(struct snd_usb_audio *chip, type == SND_USB_ENDPOINT_TYPE_DATA ? "data" : "sync", ep_num); - /* select the alt setting once so the endpoints become valid */ - ret = usb_set_interface(chip->dev, alts->desc.bInterfaceNumber, - alts->desc.bAlternateSetting); - if (ret < 0) { - snd_printk(KERN_ERR "%s(): usb_set_interface() failed, ret = %d\n", - __func__, ret); - ep = NULL; - goto __exit_unlock; - } - ep = kzalloc(sizeof(*ep), GFP_KERNEL); if (!ep) goto __exit_unlock; @@ -936,8 +926,8 @@ void snd_usb_endpoint_stop(struct snd_usb_endpoint *ep, * * In case of any active users, this functions does nothing. * - * Returns an error if usb_set_interface() failed, 0 in all other - * cases. + * Returns an error if any of the usb_set_interface() calls failed, 0 in + * all other cases. */ int snd_usb_endpoint_activate(struct snd_usb_endpoint *ep) { @@ -948,6 +938,14 @@ int snd_usb_endpoint_activate(struct snd_usb_endpoint *ep) !test_and_set_bit(EP_FLAG_ACTIVATED, &ep->flags)) { int ret; + ret = usb_set_interface(ep->chip->dev, ep->iface, 0); + if (ret < 0) { + snd_printk(KERN_ERR "%s() usb_set_interface() failed, ret = %d\n", + __func__, ret); + clear_bit(EP_FLAG_ACTIVATED, &ep->flags); + return ret; + } + ret = usb_set_interface(ep->chip->dev, ep->iface, ep->alt_idx); if (ret < 0) { snd_printk(KERN_ERR "%s() usb_set_interface() failed, ret = %d\n", @@ -967,13 +965,8 @@ int snd_usb_endpoint_activate(struct snd_usb_endpoint *ep) * * @ep: the endpoint to deactivate * - * If the endpoint is not currently in use, this functions will select the - * alternate interface setting 0 for the interface of this endpoint. - * - * In case of any active users, this functions does nothing. - * - * Returns an error if usb_set_interface() failed, 0 in all other - * cases. + * Returns 0 on success, -EINVAL if ep was NULL and -EBUSY if the endpoint + * is still in use. */ int snd_usb_endpoint_deactivate(struct snd_usb_endpoint *ep) { @@ -984,18 +977,8 @@ int snd_usb_endpoint_deactivate(struct snd_usb_endpoint *ep) return 0; if (!ep->chip->shutdown && - test_and_clear_bit(EP_FLAG_ACTIVATED, &ep->flags)) { - int ret; - - ret = usb_set_interface(ep->chip->dev, ep->iface, 0); - if (ret < 0) { - snd_printk(KERN_ERR "%s(): usb_set_interface() failed, ret = %d\n", - __func__, ret); - return ret; - } - + test_and_clear_bit(EP_FLAG_ACTIVATED, &ep->flags)) return 0; - } return -EBUSY; } -- 1.7.10.4 [-- Attachment #3: Type: text/plain, Size: 0 bytes --] ^ permalink raw reply related [flat|nested] 13+ messages in thread
[parent not found: <CADYPuQ6XRSAxKvctCr3FrJ71smk-xOEMF_5VAv-6F5gdM7dwuQ@mail.gmail.com>]
* Re: Regression in sound/usb/ [not found] ` <CADYPuQ6XRSAxKvctCr3FrJ71smk-xOEMF_5VAv-6F5gdM7dwuQ@mail.gmail.com> @ 2012-07-11 8:31 ` Daniel Mack [not found] ` <4FFD9B29.1020403@canonical.com> 2012-07-11 23:10 ` Daniel Mack 1 sibling, 1 reply; 13+ messages in thread From: Daniel Mack @ 2012-07-11 8:31 UTC (permalink / raw) To: Philipp Dreimann Cc: Takashi Iwai, alsa-devel, Clemens Ladisch, Joseph Salisbury On 11.07.2012 09:30, Philipp Dreimann wrote: > On 10 July 2012 23:14, Daniel Mack <zonque@gmail.com> wrote: >> On 05.07.2012 17:36, Takashi Iwai wrote: >>> At Thu, 5 Jul 2012 17:32:30 +0200, >>> Philipp Dreimann wrote: >>>> >>>> On 5 July 2012 16:41, Takashi Iwai <tiwai@suse.de> wrote: >>>>>> Try the patch below. If it still doesn't work, give the kernel >>>>>> message again and lsusb -v output, too. >>>>> >>>>> Did it make any difference? >>>> >>>> No, it unfortunately did not. >>>> >>>> lsusb is attached and the parts of the kernel log are in the mail: >>>> >>>> [ 41.243211] usb 2-1.2: new high-speed USB device number 3 using ehci_hcd >>>> [ 41.605666] uvcvideo: Found UVC 1.00 device <unnamed> (046d:0821) >>>> [ 42.002275] input: UVC Camera (046d:0821) as >>>> /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.2/2-1.2:1.2/input/input12 >>>> [ 42.085880] usbcore: registered new interface driver snd-usb-audio >>>> >>>> [ 52.790139] uvcvideo: Failed to query (GET_DEF) UVC control 2 on >>>> unit 2: -71 (exp. 2). >>>> [ 52.794363] uvcvideo: Failed to query (GET_DEF) UVC control 2 on >>>> unit 2: -71 (exp. 2). >>>> [ 52.798375] uvcvideo: Failed to query (GET_DEF) UVC control 3 on >>>> unit 2: -71 (exp. 2). >>>> [ 52.802499] uvcvideo: Failed to query (GET_DEF) UVC control 7 on >>>> unit 2: -71 (exp. 2). >>>> [ 52.806617] uvcvideo: Failed to query (GET_DEF) UVC control 11 on >>>> unit 2: -71 (exp. 1). >>>> [ 52.810734] uvcvideo: Failed to query (GET_DEF) UVC control 4 on >>>> unit 2: -71 (exp. 2). >>>> [ 52.814855] uvcvideo: Failed to query (GET_DEF) UVC control 5 on >>>> unit 2: -71 (exp. 1). >>>> [ 52.818976] uvcvideo: Failed to query (GET_CUR) UVC control 11 on >>>> unit 2: -71 (exp. 1). >>>> [ 52.823096] uvcvideo: Failed to query (GET_DEF) UVC control 8 on >>>> unit 2: -71 (exp. 2). >>>> [ 52.827215] uvcvideo: Failed to query (GET_DEF) UVC control 1 on >>>> unit 2: -71 (exp. 2). >>>> [ 52.831331] uvcvideo: Failed to set UVC probe control : -71 (exp. 26). >>>> [ 52.835459] uvcvideo: Failed to set UVC probe control : -71 (exp. 26). >>>> The last line repeats a few times, then the Failed to query block >>>> repeats again, and so on. But only if I try to use the webcam. >>>> >>>> [ 82.363599] 3:1:3: cannot set freq 32000 to ep 0x86 >>>> Is showing up without using the webcam. >>> >>> Hm, it looks better than before (it was '3:-1:3' or such), but there >>> is still something strange there. >> >> Sorry for the long delay on this. I finally found some time to look into >> this and it seems that I got the calls to usb_set_interface wrong. >> Attached is a patch to fix this - could everybody who is affect please >> give that a quick try? More explanation is in the commit log. > > The camera is now working for a short time: > [ 56.421399] usb 2-1.2: new high-speed USB device number 4 using ehci_hcd > [ 57.224234] uvcvideo: Found UVC 1.00 device <unnamed> (046d:0821) > [ 57.237221] input: UVC Camera (046d:0821) as > /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.2/2-1.2:1.2/input/input15 > Camera and microphone are working. Some time later we're back to the old issue: Ok, I see. Drop this, I'm working on a new one. I will find some time later today to look into this. Thanks for your patience! Daniel ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <4FFD9B29.1020403@canonical.com>]
* Re: Regression in sound/usb/ [not found] ` <4FFD9B29.1020403@canonical.com> @ 2012-07-11 15:28 ` Daniel Mack 0 siblings, 0 replies; 13+ messages in thread From: Daniel Mack @ 2012-07-11 15:28 UTC (permalink / raw) To: Joseph Salisbury Cc: Takashi Iwai, alsa-devel, Clemens Ladisch, Philipp Dreimann On 11.07.2012 17:26, Joseph Salisbury wrote: > On 07/11/2012 04:31 AM, Daniel Mack wrote: >> On 11.07.2012 09:30, Philipp Dreimann wrote: >>> On 10 July 2012 23:14, Daniel Mack <zonque@gmail.com> wrote: >>>> On 05.07.2012 17:36, Takashi Iwai wrote: >>>>> At Thu, 5 Jul 2012 17:32:30 +0200, >>>>> Philipp Dreimann wrote: >>>>>> >>>>>> On 5 July 2012 16:41, Takashi Iwai <tiwai@suse.de> wrote: >>>>>>>> Try the patch below. If it still doesn't work, give the kernel >>>>>>>> message again and lsusb -v output, too. >>>>>>> >>>>>>> Did it make any difference? >>>>>> >>>>>> No, it unfortunately did not. >>>>>> >>>>>> lsusb is attached and the parts of the kernel log are in the mail: >>>>>> >>>>>> [ 41.243211] usb 2-1.2: new high-speed USB device number 3 using ehci_hcd >>>>>> [ 41.605666] uvcvideo: Found UVC 1.00 device <unnamed> (046d:0821) >>>>>> [ 42.002275] input: UVC Camera (046d:0821) as >>>>>> /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.2/2-1.2:1.2/input/input12 >>>>>> [ 42.085880] usbcore: registered new interface driver snd-usb-audio >>>>>> >>>>>> [ 52.790139] uvcvideo: Failed to query (GET_DEF) UVC control 2 on >>>>>> unit 2: -71 (exp. 2). >>>>>> [ 52.794363] uvcvideo: Failed to query (GET_DEF) UVC control 2 on >>>>>> unit 2: -71 (exp. 2). >>>>>> [ 52.798375] uvcvideo: Failed to query (GET_DEF) UVC control 3 on >>>>>> unit 2: -71 (exp. 2). >>>>>> [ 52.802499] uvcvideo: Failed to query (GET_DEF) UVC control 7 on >>>>>> unit 2: -71 (exp. 2). >>>>>> [ 52.806617] uvcvideo: Failed to query (GET_DEF) UVC control 11 on >>>>>> unit 2: -71 (exp. 1). >>>>>> [ 52.810734] uvcvideo: Failed to query (GET_DEF) UVC control 4 on >>>>>> unit 2: -71 (exp. 2). >>>>>> [ 52.814855] uvcvideo: Failed to query (GET_DEF) UVC control 5 on >>>>>> unit 2: -71 (exp. 1). >>>>>> [ 52.818976] uvcvideo: Failed to query (GET_CUR) UVC control 11 on >>>>>> unit 2: -71 (exp. 1). >>>>>> [ 52.823096] uvcvideo: Failed to query (GET_DEF) UVC control 8 on >>>>>> unit 2: -71 (exp. 2). >>>>>> [ 52.827215] uvcvideo: Failed to query (GET_DEF) UVC control 1 on >>>>>> unit 2: -71 (exp. 2). >>>>>> [ 52.831331] uvcvideo: Failed to set UVC probe control : -71 (exp. 26). >>>>>> [ 52.835459] uvcvideo: Failed to set UVC probe control : -71 (exp. 26). >>>>>> The last line repeats a few times, then the Failed to query block >>>>>> repeats again, and so on. But only if I try to use the webcam. >>>>>> >>>>>> [ 82.363599] 3:1:3: cannot set freq 32000 to ep 0x86 >>>>>> Is showing up without using the webcam. >>>>> >>>>> Hm, it looks better than before (it was '3:-1:3' or such), but there >>>>> is still something strange there. >>>> >>>> Sorry for the long delay on this. I finally found some time to look into >>>> this and it seems that I got the calls to usb_set_interface wrong. >>>> Attached is a patch to fix this - could everybody who is affect please >>>> give that a quick try? More explanation is in the commit log. >>> >>> The camera is now working for a short time: >>> [ 56.421399] usb 2-1.2: new high-speed USB device number 4 using ehci_hcd >>> [ 57.224234] uvcvideo: Found UVC 1.00 device <unnamed> (046d:0821) >>> [ 57.237221] input: UVC Camera (046d:0821) as >>> /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.2/2-1.2:1.2/input/input15 >>> Camera and microphone are working. Some time later we're back to the old issue: >> >> Ok, I see. Drop this, I'm working on a new one. I will find some time >> later today to look into this. Thanks for your patience! >> >> >> Daniel > > I just sent an email to Philipp that I've been unable to reproduce the > failure on my hardware. However, a user testing your patch has also run > into the same failure as Philipp: > > https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1018020/comments/38 > > That users webcam will work up to 15 minutes, than fail. I'll pick up an USB hw analyzer later today and then try with a Logitech camera. Will keep you posted. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Regression in sound/usb/ [not found] ` <CADYPuQ6XRSAxKvctCr3FrJ71smk-xOEMF_5VAv-6F5gdM7dwuQ@mail.gmail.com> 2012-07-11 8:31 ` Daniel Mack @ 2012-07-11 23:10 ` Daniel Mack 2012-07-11 23:41 ` Daniel Mack [not found] ` <CADYPuQ5sE28JRXLaBUYzjqvuJyWY9cWUCM4mYFByYQUXppDOQA@mail.gmail.com> 1 sibling, 2 replies; 13+ messages in thread From: Daniel Mack @ 2012-07-11 23:10 UTC (permalink / raw) To: Philipp Dreimann Cc: Takashi Iwai, alsa-devel, Clemens Ladisch, Joseph Salisbury [-- Attachment #1: Type: text/plain, Size: 874 bytes --] On 11.07.2012 09:30, Philipp Dreimann wrote: > The camera is now working for a short time: > [ 56.421399] usb 2-1.2: new high-speed USB device number 4 using ehci_hcd > [ 57.224234] uvcvideo: Found UVC 1.00 device <unnamed> (046d:0821) > [ 57.237221] input: UVC Camera (046d:0821) as > /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.2/2-1.2:1.2/input/input15 > Camera and microphone are working. Some time later we're back to the old issue: So I found the reason for this breakage now. It's about orphaned packets that are sent over the wire after alt setting 0 has been selected. The chipset doesn't like these at and bails out with protocol errors. Fixing this involves some rework of the logic. The patch is unfortunately a little big, but I see no option than getting it merged for 3.6. Everyone, please test - and sorry for the trouble caused. Thanks, Daniel [-- Attachment #2: snd-usb-endpoint-fixups.diff --] [-- Type: text/x-patch, Size: 7788 bytes --] diff --git a/sound/usb/endpoint.c b/sound/usb/endpoint.c index e690690..4bc5778 100644 --- a/sound/usb/endpoint.c +++ b/sound/usb/endpoint.c @@ -350,7 +350,8 @@ static void snd_complete_urb(struct urb *urb) urb->status == -ENODEV || /* device removed */ urb->status == -ECONNRESET || /* unlinked */ urb->status == -ESHUTDOWN || /* device disabled */ - ep->chip->shutdown)) /* device disconnected */ + ep->chip->shutdown) || /* device disconnected */ + !test_bit(EP_FLAG_RUNNING, &ep->flags)) goto exit_clear; if (usb_pipeout(ep->pipe)) { @@ -414,7 +415,7 @@ struct snd_usb_endpoint *snd_usb_add_endpoint(struct snd_usb_audio *chip, { struct list_head *p; struct snd_usb_endpoint *ep; - int ret, is_playback = direction == SNDRV_PCM_STREAM_PLAYBACK; + int is_playback = direction == SNDRV_PCM_STREAM_PLAYBACK; mutex_lock(&chip->mutex); @@ -434,16 +435,6 @@ struct snd_usb_endpoint *snd_usb_add_endpoint(struct snd_usb_audio *chip, type == SND_USB_ENDPOINT_TYPE_DATA ? "data" : "sync", ep_num); - /* select the alt setting once so the endpoints become valid */ - ret = usb_set_interface(chip->dev, alts->desc.bInterfaceNumber, - alts->desc.bAlternateSetting); - if (ret < 0) { - snd_printk(KERN_ERR "%s(): usb_set_interface() failed, ret = %d\n", - __func__, ret); - ep = NULL; - goto __exit_unlock; - } - ep = kzalloc(sizeof(*ep), GFP_KERNEL); if (!ep) goto __exit_unlock; @@ -522,13 +513,13 @@ static int deactivate_urbs(struct snd_usb_endpoint *ep, int force, int can_sleep unsigned int i; int async; + clear_bit(EP_FLAG_RUNNING, &ep->flags); + if (!force && ep->chip->shutdown) /* to be sure... */ return -EBADFD; async = !can_sleep && ep->chip->async_unlink; - clear_bit(EP_FLAG_RUNNING, &ep->flags); - INIT_LIST_HEAD(&ep->ready_playback_urbs); ep->next_packet_read_pos = 0; ep->next_packet_write_pos = 0; @@ -831,9 +822,6 @@ int snd_usb_endpoint_start(struct snd_usb_endpoint *ep) if (++ep->use_count != 1) return 0; - if (snd_BUG_ON(!test_bit(EP_FLAG_ACTIVATED, &ep->flags))) - return -EINVAL; - /* just to be sure */ deactivate_urbs(ep, 0, 1); wait_clear_urbs(ep); @@ -911,9 +899,6 @@ void snd_usb_endpoint_stop(struct snd_usb_endpoint *ep, if (snd_BUG_ON(ep->use_count == 0)) return; - if (snd_BUG_ON(!test_bit(EP_FLAG_ACTIVATED, &ep->flags))) - return; - if (--ep->use_count == 0) { deactivate_urbs(ep, force, can_sleep); ep->data_subs = NULL; @@ -927,42 +912,6 @@ void snd_usb_endpoint_stop(struct snd_usb_endpoint *ep, } /** - * snd_usb_endpoint_activate: activate an snd_usb_endpoint - * - * @ep: the endpoint to activate - * - * If the endpoint is not currently in use, this functions will select the - * correct alternate interface setting for the interface of this endpoint. - * - * In case of any active users, this functions does nothing. - * - * Returns an error if usb_set_interface() failed, 0 in all other - * cases. - */ -int snd_usb_endpoint_activate(struct snd_usb_endpoint *ep) -{ - if (ep->use_count != 0) - return 0; - - if (!ep->chip->shutdown && - !test_and_set_bit(EP_FLAG_ACTIVATED, &ep->flags)) { - int ret; - - ret = usb_set_interface(ep->chip->dev, ep->iface, ep->alt_idx); - if (ret < 0) { - snd_printk(KERN_ERR "%s() usb_set_interface() failed, ret = %d\n", - __func__, ret); - clear_bit(EP_FLAG_ACTIVATED, &ep->flags); - return ret; - } - - return 0; - } - - return -EBUSY; -} - -/** * snd_usb_endpoint_deactivate: deactivate an snd_usb_endpoint * * @ep: the endpoint to deactivate @@ -980,24 +929,15 @@ int snd_usb_endpoint_deactivate(struct snd_usb_endpoint *ep) if (!ep) return -EINVAL; + deactivate_urbs(ep, 1, 1); + wait_clear_urbs(ep); + if (ep->use_count != 0) return 0; - if (!ep->chip->shutdown && - test_and_clear_bit(EP_FLAG_ACTIVATED, &ep->flags)) { - int ret; - - ret = usb_set_interface(ep->chip->dev, ep->iface, 0); - if (ret < 0) { - snd_printk(KERN_ERR "%s(): usb_set_interface() failed, ret = %d\n", - __func__, ret); - return ret; - } + clear_bit(EP_FLAG_ACTIVATED, &ep->flags); - return 0; - } - - return -EBUSY; + return 0; } /** diff --git a/sound/usb/pcm.c b/sound/usb/pcm.c index 54607f8..536b706 100644 --- a/sound/usb/pcm.c +++ b/sound/usb/pcm.c @@ -261,19 +261,6 @@ static void stop_endpoints(struct snd_usb_substream *subs, force, can_sleep, wait); } -static int activate_endpoints(struct snd_usb_substream *subs) -{ - if (subs->sync_endpoint) { - int ret; - - ret = snd_usb_endpoint_activate(subs->sync_endpoint); - if (ret < 0) - return ret; - } - - return snd_usb_endpoint_activate(subs->data_endpoint); -} - static int deactivate_endpoints(struct snd_usb_substream *subs) { int reta, retb; @@ -314,6 +301,31 @@ static int set_format(struct snd_usb_substream *subs, struct audioformat *fmt) if (fmt == subs->cur_audiofmt) return 0; + /* close the old interface */ + if (subs->interface >= 0 && subs->interface != fmt->iface) { + err = usb_set_interface(subs->dev, subs->interface, 0); + if (err < 0) { + snd_printk(KERN_ERR "%d:%d:%d: return to setting 0 failed (%d)\n", + dev->devnum, fmt->iface, fmt->altsetting, err); + return -EIO; + } + subs->interface = -1; + subs->altset_idx = 0; + } + + /* set interface */ + if (subs->interface != fmt->iface || subs->altset_idx != fmt->altset_idx) { + err = usb_set_interface(dev, fmt->iface, fmt->altsetting); + if (err < 0) { + snd_printk(KERN_ERR "%d:%d:%d: usb_set_interface failed (%d)\n", + dev->devnum, fmt->iface, fmt->altsetting, err); + return -EIO; + } + snd_printdd(KERN_INFO "setting usb interface %d:%d\n", fmt->iface, fmt->altsetting); + subs->interface = fmt->iface; + subs->altset_idx = fmt->altset_idx; + } + subs->data_endpoint = snd_usb_add_endpoint(subs->stream->chip, alts, fmt->endpoint, subs->direction, SND_USB_ENDPOINT_TYPE_DATA); @@ -460,12 +472,6 @@ static int snd_usb_hw_params(struct snd_pcm_substream *substream, mutex_lock(&subs->stream->chip->shutdown_mutex); /* format changed */ stop_endpoints(subs, 0, 0, 0); - deactivate_endpoints(subs); - - ret = activate_endpoints(subs); - if (ret < 0) - goto unlock; - ret = snd_usb_endpoint_set_params(subs->data_endpoint, hw_params, fmt, subs->sync_endpoint); if (ret < 0) @@ -500,6 +506,7 @@ static int snd_usb_hw_free(struct snd_pcm_substream *substream) subs->period_bytes = 0; mutex_lock(&subs->stream->chip->shutdown_mutex); stop_endpoints(subs, 0, 1, 1); + deactivate_endpoints(subs); mutex_unlock(&subs->stream->chip->shutdown_mutex); return snd_pcm_lib_free_vmalloc_buffer(substream); } @@ -938,16 +945,20 @@ static int snd_usb_pcm_open(struct snd_pcm_substream *substream, int direction) static int snd_usb_pcm_close(struct snd_pcm_substream *substream, int direction) { - int ret; struct snd_usb_stream *as = snd_pcm_substream_chip(substream); struct snd_usb_substream *subs = &as->substream[direction]; stop_endpoints(subs, 0, 0, 0); - ret = deactivate_endpoints(subs); + + if (!as->chip->shutdown && subs->interface >= 0) { + usb_set_interface(subs->dev, subs->interface, 0); + subs->interface = -1; + } + subs->pcm_substream = NULL; snd_usb_autosuspend(subs->stream->chip); - return ret; + return 0; } /* Since a URB can handle only a single linear buffer, we must use double [-- Attachment #3: Type: text/plain, Size: 0 bytes --] ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: Regression in sound/usb/ 2012-07-11 23:10 ` Daniel Mack @ 2012-07-11 23:41 ` Daniel Mack [not found] ` <CADYPuQ5sE28JRXLaBUYzjqvuJyWY9cWUCM4mYFByYQUXppDOQA@mail.gmail.com> 1 sibling, 0 replies; 13+ messages in thread From: Daniel Mack @ 2012-07-11 23:41 UTC (permalink / raw) To: Philipp Dreimann Cc: Takashi Iwai, alsa-devel, Clemens Ladisch, Joseph Salisbury On 12.07.2012 01:10, Daniel Mack wrote: > On 11.07.2012 09:30, Philipp Dreimann wrote: > >> The camera is now working for a short time: >> [ 56.421399] usb 2-1.2: new high-speed USB device number 4 using ehci_hcd >> [ 57.224234] uvcvideo: Found UVC 1.00 device <unnamed> (046d:0821) >> [ 57.237221] input: UVC Camera (046d:0821) as >> /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.2/2-1.2:1.2/input/input15 >> Camera and microphone are working. Some time later we're back to the old issue: > > So I found the reason for this breakage now. It's about orphaned packets > that are sent over the wire after alt setting 0 has been selected. The > chipset doesn't like these at and bails out with protocol errors. Fixing > this involves some rework of the logic. The patch is unfortunately a > little big, but I see no option than getting it merged for 3.6. I meant 3.5, of course. > Everyone, please test - and sorry for the trouble caused. To avoid confusion: this patch replaces the other one I sent. Daniel ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <CADYPuQ5sE28JRXLaBUYzjqvuJyWY9cWUCM4mYFByYQUXppDOQA@mail.gmail.com>]
* Re: Regression in sound/usb/ [not found] ` <CADYPuQ5sE28JRXLaBUYzjqvuJyWY9cWUCM4mYFByYQUXppDOQA@mail.gmail.com> @ 2012-07-12 13:20 ` Daniel Mack 0 siblings, 0 replies; 13+ messages in thread From: Daniel Mack @ 2012-07-12 13:20 UTC (permalink / raw) To: Philipp Dreimann Cc: Takashi Iwai, alsa-devel, Clemens Ladisch, Joseph Salisbury On 12.07.2012 11:18, Philipp Dreimann wrote: > On 12 July 2012 01:10, Daniel Mack <zonque@gmail.com> wrote: >> On 11.07.2012 09:30, Philipp Dreimann wrote: >> >>> The camera is now working for a short time: >>> [ 56.421399] usb 2-1.2: new high-speed USB device number 4 using ehci_hcd >>> [ 57.224234] uvcvideo: Found UVC 1.00 device <unnamed> (046d:0821) >>> [ 57.237221] input: UVC Camera (046d:0821) as >>> /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.2/2-1.2:1.2/input/input15 >>> Camera and microphone are working. Some time later we're back to the old issue: >> >> So I found the reason for this breakage now. It's about orphaned packets >> that are sent over the wire after alt setting 0 has been selected. The >> chipset doesn't like these at and bails out with protocol errors. Fixing >> this involves some rework of the logic. The patch is unfortunately a >> little big, but I see no option than getting it merged for 3.6. > > The issue seems to be fixed for me. Let's hope that it'll make it in > the next v3.5-rc. Thanks for testing. I just sent two patches. Combined, they are identical to what you tested. Anyone else wants to add a Tested-by? Thanks, Daniel ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Regression in sound/usb/ [not found] ` <CADYPuQ7yWpnPSvoVQDV+EyJfC39MDxhP9r5Jiqk_Jh8g65Xd4g@mail.gmail.com> 2012-07-05 15:36 ` Takashi Iwai @ 2012-07-05 15:57 ` Daniel Mack 1 sibling, 0 replies; 13+ messages in thread From: Daniel Mack @ 2012-07-05 15:57 UTC (permalink / raw) To: Philipp Dreimann; +Cc: Takashi Iwai, alsa-devel, Clemens Ladisch On 05.07.2012 17:32, Philipp Dreimann wrote: > On 5 July 2012 16:41, Takashi Iwai <tiwai@suse.de> wrote: >>> Try the patch below. If it still doesn't work, give the kernel >>> message again and lsusb -v output, too. >> >> Did it make any difference? > > No, it unfortunately did not. > > lsusb is attached and the parts of the kernel log are in the mail: > > [ 41.243211] usb 2-1.2: new high-speed USB device number 3 using ehci_hcd > [ 41.605666] uvcvideo: Found UVC 1.00 device <unnamed> (046d:0821) > [ 42.002275] input: UVC Camera (046d:0821) as > /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.2/2-1.2:1.2/input/input12 > [ 42.085880] usbcore: registered new interface driver snd-usb-audio > > [ 52.790139] uvcvideo: Failed to query (GET_DEF) UVC control 2 on > unit 2: -71 (exp. 2). > [ 52.794363] uvcvideo: Failed to query (GET_DEF) UVC control 2 on > unit 2: -71 (exp. 2). > [ 52.798375] uvcvideo: Failed to query (GET_DEF) UVC control 3 on > unit 2: -71 (exp. 2). > [ 52.802499] uvcvideo: Failed to query (GET_DEF) UVC control 7 on > unit 2: -71 (exp. 2). > [ 52.806617] uvcvideo: Failed to query (GET_DEF) UVC control 11 on > unit 2: -71 (exp. 1). > [ 52.810734] uvcvideo: Failed to query (GET_DEF) UVC control 4 on > unit 2: -71 (exp. 2). > [ 52.814855] uvcvideo: Failed to query (GET_DEF) UVC control 5 on > unit 2: -71 (exp. 1). > [ 52.818976] uvcvideo: Failed to query (GET_CUR) UVC control 11 on > unit 2: -71 (exp. 1). > [ 52.823096] uvcvideo: Failed to query (GET_DEF) UVC control 8 on > unit 2: -71 (exp. 2). > [ 52.827215] uvcvideo: Failed to query (GET_DEF) UVC control 1 on > unit 2: -71 (exp. 2). > [ 52.831331] uvcvideo: Failed to set UVC probe control : -71 (exp. 26). > [ 52.835459] uvcvideo: Failed to set UVC probe control : -71 (exp. 26). > The last line repeats a few times, then the Failed to query block > repeats again, and so on. But only if I try to use the webcam. > > [ 82.363599] 3:1:3: cannot set freq 32000 to ep 0x86 > Is showing up without using the webcam. > What really puzzles me is that the uvcvideo code fails as well, and even before the sound driver bails out. If you wouldn't have bisected the problem, I would clearly blame some other change for that. It seems something confuses the device quite badly so requests to it return -EPROTO. From a quick grep, this error can only be originated from low-level EHCI functions. I need to ponder on that a little ... Daniel ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2012-07-12 13:27 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CADYPuQ75GAxGyZgiUwygvnQMmDcL9+NMHNo432kg1RYC7vbfQA@mail.gmail.com>
2012-07-02 12:10 ` Regression in sound/usb/ Daniel Mack
[not found] ` <CADYPuQ5RFJAaFDpT+4XM-JDWzDO5czVAU3A++_HVHjhN=VBhvQ@mail.gmail.com>
2012-07-03 8:00 ` Takashi Iwai
[not found] ` <CADYPuQ4Uj8pq9nZOJemY9kZtSrG-WfG4s90b+a00_KteJXU_Sg@mail.gmail.com>
2012-07-03 8:27 ` Takashi Iwai
[not found] ` <CADYPuQ6DT4=NW2tzCLrKYnUHQ6p5qONZjY8PeKq-Z4HAk68tpw@mail.gmail.com>
2012-07-03 9:01 ` Takashi Iwai
2012-07-05 14:41 ` Takashi Iwai
[not found] ` <CADYPuQ7yWpnPSvoVQDV+EyJfC39MDxhP9r5Jiqk_Jh8g65Xd4g@mail.gmail.com>
2012-07-05 15:36 ` Takashi Iwai
2012-07-10 21:14 ` Daniel Mack
[not found] ` <CADYPuQ6XRSAxKvctCr3FrJ71smk-xOEMF_5VAv-6F5gdM7dwuQ@mail.gmail.com>
2012-07-11 8:31 ` Daniel Mack
[not found] ` <4FFD9B29.1020403@canonical.com>
2012-07-11 15:28 ` Daniel Mack
2012-07-11 23:10 ` Daniel Mack
2012-07-11 23:41 ` Daniel Mack
[not found] ` <CADYPuQ5sE28JRXLaBUYzjqvuJyWY9cWUCM4mYFByYQUXppDOQA@mail.gmail.com>
2012-07-12 13:20 ` Daniel Mack
2012-07-05 15:57 ` Daniel Mack
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.