[alsa-devel] [PATCH] ALSA: usb: Parse UAC2 extension unit like for UAC1

[Takashi Iwai]

At Tue, 19 Mar 2013 20:26:22 +0100,
Daniel Mack wrote:
> 
> On 19.03.2013 19:10, Torstein Hegge wrote:
> > On Tue, Mar 19, 2013 at 06:08:15PM +0100, Takashi Iwai wrote:
> >> At Tue, 19 Mar 2013 17:12:14 +0100,
> >> Torstein Hegge wrote:
> >>>
> >>> UAC2_EXTENSION_UNIT_V2 differs from UAC1_EXTENSION_UNIT, but can be handled in
> >>> the same way when parsing the unit. Otherwise parse_audio_unit() fails when it
> >>> sees an extension unit on a UAC2 device.
> >>>
> >>> UAC2_EXTENSION_UNIT_V2 is outside the range allocated by UAC1.
> >>>
> >>> Signed-off-by: Torstein Hegge <hegge at resisty.net>
> >>
> >> Does this bug hit on any existing device with 3.9-rc kernel?
> >> If it does, it'd be nice to have a bit actual description.
> > 
> > The bug was reported on a Focusrite Scarlett 8i6 USB with a 3.5.0 kernel [1],
> > where the unhandled subtype causes the card creation to fail:
> > 
> > [ 1144.857836] usb 2-1.2: new high-speed USB device number 4 using ehci_hcd
> > [ 1144.951520] usb 2-1.2: New USB device found, idVendor=1235, idProduct=8002
> > [ 1144.951531] usb 2-1.2: New USB device strings: Mfr=1, Product=2, SerialNumber=3
> > [ 1144.951538] usb 2-1.2: Product: Scarlett 8i6 USB 
> > [ 1144.951543] usb 2-1.2: Manufacturer: Focusrite
> > [ 1144.951547] usb 2-1.2: SerialNumber: 00006B8B
> > [ 1144.956590] usbaudio: unit 51: unexpected type 0x09
> > [ 1144.956640] snd-usb-audio: probe of 2-1.2:1.0 failed with error -5
> > 
> > Still waiting for him to report back with a 3.9-rc kernel.
> > 
> > [1] http://thread.gmane.org/gmane.linux.alsa.user/37289/focus=37294
> 
> The 3.9-rc kernel doesn't change anything about that, but two error path
> cleanup patches in the mixer code seem to have at least some effect.
> I'll post them in a while.
> 
> I frankly think we should merge both this and my patches to 3.9, and
> maybe even mark the for @stable.

OK, that's fine.  I just wanted to know the situation.


Takashi