[alsa-devel] Dell USB audio driver workaround
Hui Wang
hui.wang at canonical.com
Wed Jun 14 04:29:13 CEST 2017
In this thread, I added tanuk at iki.fi and svillar at igalia.com since they
reported or discussed similar bugs recently
(https://www.spinics.net/lists/alsa-devel/msg62460.html and
https://bugs.freedesktop.org/show_bug.cgi?id=100711).
Right now, we are trying to enable the USB audio for the Dell dock
stations TB16[1] and WD15[2]. For usb audio part, they have the same
design, they use realtek alc4020 USB bridge and alc3263 codec, and there
are one headset jack on the front panel and one speaker output jack on
the rear panel, but due to some reason (probably it is the firmware's
problem), the alsa mixers can't reflect there is speaker or headset on
them, as a result the pulseaudio can't handle the headset or speaker
correctly.
Right now, the pulseaudio can find a playback device and a record
device, it is the output and input device for that headset jack, but
pulseaudio does not think they are headphone and Mic, pulseaudio think
they are just USB analog output and input devices; for the speaker jack,
the pulseaudio can't find it at all. so through pulseaudio, we can't
play any sound to that speaker jack.
This is the output and input devices listed by alsa-utils:
!!Aplay/Arecord output
!!--------------------
APLAY
**** List of PLAYBACK Hardware Devices ****
...
card 1: Audio [USB Audio], device 0: USB Audio [USB Audio]
Subdevices: 1/1
Subdevice #0: subdevice #0
card 1: Audio [USB Audio], device 1: USB Audio [USB Audio #1]
Subdevices: 1/1
Subdevice #0: subdevice #0
ARECORD
**** List of CAPTURE Hardware Devices ****
...
card 1: Audio [USB Audio], device 0: USB Audio [USB Audio]
Subdevices: 1/1
Subdevice #0: subdevice #0
And we can use alsa-utils command to play sound to speaker jack:
$ aplay -D plughw:CARD=1,DEV=1 /usr/share/sounds/speech-dispatcher/test.wav
I don't know how to fix this problem in the kernel driver, if we can let
driver export standard mixers, this issue will be fixed. But it looks
like the mixers depend on the firmware, we can't change the firmware, so
probably we can't fix it in the kernel driver.
Another way to fix it is to add a conf to pulseaudio, in the conf, we
add one profile for headset(hw:%f,0,0) and one profile for
speaker(hw:%f,1,0), this is the draft patch I sent to pulseaudio-discuss
mail list:
https://lists.freedesktop.org/archives/pulseaudio-discuss/2017-May/028252.html
Do you have any idea to fix this problem? Thanks in advance. And below
are some logs I collected on the TB16, if you need more logs, just tell
me, I will collect them for you.
===================================================================
$ amixer -c1 contents > amixser.log
amixer: Control hw:1 element read error: Invalid argument
numid=4,iface=MIXER,name='Mic Capture Switch'
; type=BOOLEAN,access=rw------,values=1
: values=on
numid=5,iface=MIXER,name='Mic Capture Volume'
; type=INTEGER,access=rw---R--,values=2,min=0,max=127,step=0
: values=67,67
| dBminmax-min=-33.50dB,max=30.00dB
numid=7,iface=MIXER,name='Extension Unit Switch'
; type=BOOLEAN,access=rw------,values=1
numid=6,iface=MIXER,name='Loudness'
; type=BOOLEAN,access=rw------,values=1
: values=off
numid=1,iface=PCM,name='Capture Channel Map'
; type=INTEGER,access=r----R--,values=2,min=0,max=36,step=0
: values=0,0
| container
| chmap-fixed=FL,FR
numid=2,iface=PCM,name='Playback Channel Map'
; type=INTEGER,access=r----R--,values=2,min=0,max=36,step=0
: values=0,0
| container
| chmap-fixed=FL,FR
numid=3,iface=PCM,name='Playback Channel Map',device=1
; type=INTEGER,access=r----R--,values=2,min=0,max=36,step=0
: values=0,0
| container
| chmap-fixed=FL,FR
========================================================================
$ cat /proc/asound/card1/usbmixer
USB Mixer: usb_id=0x0bda4014, ctrlif=0, ctlerr=0
Card: Generic USB Audio at usb-0000:0e:00.0-1.5, high speed
Unit: 3
Control: name="Loudness", index=0
Info: id=3, control=10, cmask=0x0, channels=1, type="BOOLEAN"
Volume: min=0, max=1, dBmin=0, dBmax=0
Unit: 3
Control: name="Mic Capture Volume", index=0
Info: id=3, control=2, cmask=0x3, channels=2, type="S16"
Volume: min=-8576, max=7680, dBmin=-3350, dBmax=3000
Unit: 3
Control: name="Mic Capture Switch", index=0
Info: id=3, control=1, cmask=0x0, channels=1, type="INV_BOOLEAN"
Volume: min=0, max=1, dBmin=0, dBmax=0
Unit: 4
Control: name="Extension Unit Switch", index=0
Info: id=4, control=1, cmask=0x0, channels=1, type="BOOLEAN"
Volume: min=0, max=1, dBmin=0, dBmax=0
===========================================================================
$ usb-devices
...
T: Bus=03 Lev=02 Prnt=02 Port=04 Cnt=01 Dev#= 4 Spd=480 MxCh= 0
D: Ver= 2.00 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs= 1
P: Vendor=0bda ProdID=4014 Rev=00.01
S: Manufacturer=Generic
S: Product=USB Audio
S: SerialNumber=200901010001
C: #Ifs= 4 Cfg#= 1 Atr=80 MxPwr=500mA
I: If#= 0 Alt= 0 #EPs= 1 Cls=01(audio) Sub=01 Prot=00 Driver=snd-usb-audio
I: If#= 1 Alt= 0 #EPs= 0 Cls=01(audio) Sub=02 Prot=00 Driver=snd-usb-audio
I: If#= 2 Alt= 0 #EPs= 0 Cls=01(audio) Sub=02 Prot=00 Driver=snd-usb-audio
I: If#= 3 Alt= 0 #EPs= 0 Cls=01(audio) Sub=02 Prot=00 Driver=snd-usb-audio
....
=================================================================================
With these mixers, the pulseaudio supports this USB sound card as below:
$pactl list cards
Card #0
Name: alsa_card.usb-Generic_USB_Audio_200901010001-00
Driver: module-alsa-card.c
Owner Module: 6
Properties:
alsa.card = "1"
alsa.card_name = "USB Audio"
alsa.long_card_name = "Generic USB Audio at
usb-0000:09:00.0-1.5, high speed"
alsa.driver_name = "snd_usb_audio"
device.bus_path = "pci-0000:09:00.0-usb-0:1.5:1.0"
sysfs.path =
"/devices/pci0000:00/0000:00:1d.6/0000:05:00.0/0000:06:02.0/0000:09:00.0/usb3/3-1/3-1.5/3-1.5:1.0/sound/card1"
udev.id = "usb-Generic_USB_Audio_200901010001-00"
device.bus = "usb"
device.vendor.id = "0bda"
device.vendor.name = "Realtek Semiconductor Corp."
device.product.id = "4014"
device.product.name = "USB Audio"
device.serial = "Generic_USB_Audio_200901010001"
device.string = "1"
device.description = "USB Audio"
module-udev-detect.discovered = "1"
device.icon_name = "audio-card-usb"
Profiles:
input:analog-stereo: Analog Stereo Input (sinks: 0, sources: 1,
priority: 60, available: yes)
input:iec958-stereo: Digital Stereo (IEC958) Input (sinks: 0,
sources: 1, priority: 55, available: yes)
output:analog-stereo: Analog Stereo Output (sinks: 1, sources:
0, priority: 6000, available: yes)
output:analog-stereo+input:analog-stereo: Analog Stereo Duplex
(sinks: 1, sources: 1, priority: 6060, available: yes)
output:analog-stereo+input:iec958-stereo: Analog Stereo Output
+ Digital Stereo (IEC958) Input (sinks: 1, sources: 1, priority: 6055,
available: yes)
output:iec958-stereo: Digital Stereo (IEC958) Output (sinks: 1,
sources: 0, priority: 5500, available: yes)
output:iec958-stereo+input:analog-stereo: Digital Stereo
(IEC958) Output + Analog Stereo Input (sinks: 1, sources: 1, priority:
5560, available: yes)
output:iec958-stereo+input:iec958-stereo: Digital Stereo Duplex
(IEC958) (sinks: 1, sources: 1, priority: 5555, available: yes)
off: Off (sinks: 0, sources: 0, priority: 0, available: yes)
Active Profile: output:analog-stereo+input:analog-stereo
[1]
http://www.dell.com/support/article/us/en/19/SLN301105/dell-thunderbolt-dock--tb16--information--specifications--frequently-asked-questions?lang=EN
[2]
http://www.dell.com/support/article/us/en/19/SLN304627/dell-dock-wd15--usb-type-c---information--capability-and-specifications?lang=EN
On 04/12/2016 05:26 PM, Takashi Iwai wrote:
> On Tue, 12 Apr 2016 05:02:08 +0200,
> Kailang wrote:
>>
>> Hi Takashi,
>>
>> Attach is new patch witch follow your method.
>> Many Thanks for you. ^^
> OK, I applied it with a few modification / correction in the comments
> and the changelog. Also added Cc to stable.
>
>
> thanks,
>
> Takashi
>
>
>> BR,
>> Kailang
>>
>>> -----Original Message-----
>>> From: Takashi Iwai [mailto:tiwai at suse.de]
>>> Sent: Thursday, April 07, 2016 6:18 PM
>>> To: Kailang
>>> Cc: (alsa-devel at alsa-project.org)
>>> Subject: Re: Dell USB audio driver workaround
>>>
>>> On Thu, 07 Apr 2016 12:07:25 +0200,
>>> Kailang wrote:
>>>>
>>>>
>>>>> -----Original Message-----
>>>>> From: Takashi Iwai [mailto:tiwai at suse.de]
>>>>> Sent: Thursday, April 07, 2016 5:27 PM
>>>>> To: Kailang
>>>>> Cc: (alsa-devel at alsa-project.org)
>>>>> Subject: Re: Dell USB audio driver workaround
>>>>>
>>>>> On Thu, 07 Apr 2016 09:05:52 +0200,
>>>>> Kailang wrote:
>>>>>>
>>>>>>
>>>>>>> -----Original Message-----
>>>>>>> From: Takashi Iwai [mailto:tiwai at suse.de]
>>>>>>> Sent: Wednesday, April 06, 2016 5:37 PM
>>>>>>> To: Kailang
>>>>>>> Cc: (alsa-devel at alsa-project.org)
>>>>>>> Subject: Re: Dell USB audio driver workaround
>>>>>>>
>>>>>>> On Wed, 06 Apr 2016 11:22:43 +0200, Kailang wrote:
>>>>>>>>
>>>>>>>>
>>>>>>>>> -----Original Message-----
>>>>>>>>> From: Takashi Iwai [mailto:tiwai at suse.de]
>>>>>>>>> Sent: Wednesday, April 06, 2016 4:57 PM
>>>>>>>>> To: Kailang
>>>>>>>>> Cc: (alsa-devel at alsa-project.org)
>>>>>>>>> Subject: Re: Dell USB audio driver workaround
>>>>>>>>>
>>>>>>>>> On Wed, 06 Apr 2016 08:29:12 +0200, Kailang wrote:
>>>>>>>>>> Hi Takashi,
>>>>>>>>>>
>>>>>>>>>> Dell had USB dock.
>>>>>>>>>> Maybe firmware issue, the master volume always
>>> keep at 0x57.
>>>>>>>>>> Attach patch will fix it to keep at Max volume.
>>>>>>>>>>
>>>>>>>>>> Could you help us to update it?
>>>>>>>>> Well, I don't get how this patch works..
>>>>>>>>>
>>>>>>>>>> diff --git a/sound/usb/mixer.c
>>> b/sound/usb/mixer.c index
>>>>>>>>>> 4f85757..84d9a4e 100644
>>>>>>>>>> --- a/sound/usb/mixer.c
>>>>>>>>>> +++ b/sound/usb/mixer.c
>>>>>>>>>> @@ -846,6 +846,12 @@ static void
>>>>> volume_control_quirks(struct
>>>>>>>>>> usb_mixer_elem_info *cval, {
>>>>>>>>>> struct snd_usb_audio *chip =
>>> cval->head.mixer->chip;
>>>>>>>>>> switch (chip->usb_id) {
>>>>>>>>>> + case USB_ID(0x0bda, 0x4014): /* Dell
>>> workaround */
>>>>>>>>>> + if (strstr(kctl->id.name, "Playback
>>>>>>> Volume") != NULL) {
>>>>>>>>>> + cval->min = 0x0000;
>>>>>>>>>> + cval->max = 0x0000;
>>>>>>>>>> + }
>>>>>>>>>> + break;
>>>>>>>>> So, this means that this value has to be fixed to 0?
>>>>>>>>> If yes, can't we simply remove this volume instead?
>>>>>>>> Yes, but it need to set Max volume to 0. This is test
>>>>> on my site.
>>>>>>>> Because it will rewrite 0x57 value to codec register when
>>>>>>> driver is loaded.
>>>>>>>> And keep 0x57 volume value all the time. So, Master volume
>>>>>>> will lower than windows machine.
>>>>>>>> This case which driver fail to build the alsa mixer(This is
>>>>>>> firmware cause it). So, no hardware volume control on this.
>>>>>>>> If the Max volume value is 0, system will not set default
>>>>>>> value 0x57 for first time loading driver.
>>>>>>>
>>>>>>> Which values (min/max) does this FU take and which values
>>>>> are buggy?
>>>>>> Max value.
>>>>> Sorry, my question wasn't clear. "What" min and max are they?
>>>>> min=0 and max=0x57?
>>>> Oh! Sorry. It's my wrong. I forgot.
>>>>
>>>> Unit: 16
>>>> Control: name="PCM Playback Volume", index=0
>>>> Info: id=16, control=2, cmask=0x3, channels=2, type="S16"
>>>> Volume: min=-16256, max=0, dBmin=-6350, dBmax=0
>>>>
>>>> 0x0 (codec reg) = (-16256) (USB vloume level) 0x57(codec reg) =
>>>> 0xEC00(USB vloume level) 0x7F(codec reg) = 0x0000 (USB vloume level)
>>>>
>>>>> And, your patch changes both min and max to be 0. That is, it
>>>>> accepts only 0. Does it mean that this FU isn't any
>>> proper volume
>>>>> control?
>>>> cval->max original is 0.
>>>> cval->min => I set it to 0. It will be not rewrite 0x57
>>> value to codec.
>>>
>>> OK, now it's clear, thanks.
>>>
>>> Though, I still don't like to set min = max. It basically
>>> makes this control useless, and then there is no reason to
>>> keep this exposed.
>>>
>>> Can we just skip parsing this control, then? You can create
>>> an own usbmix_name_map[] to indicate the ID to skip building
>>> a control by passing NULL to its name. That is, a patch like
>>> below. (You need to fill the right ID number in the patch,
>>> of course.)
>>>
>>>
>>> Takashi
>>>
>>> ---
>>> diff --git a/sound/usb/mixer_maps.c b/sound/usb/mixer_maps.c
>>> --- a/sound/usb/mixer_maps.c
>>> +++ b/sound/usb/mixer_maps.c
>>> @@ -349,6 +349,16 @@ static struct usbmix_name_map
>>> bose_companion5_map[] = { };
>>>
>>> /*
>>> + * Dell XXX with ALC4020 codec had a firmware problem where it got
>>> + * screwed up when zero volume is passed; just skip it as a
>>> workaround
>>> +*/ static const struct usbmix_name_map dell_alc4020_map[] = {
>>> + { 10 /* XXX fill this right ID XXX */, NULL },
>>> + { 0 }
>>> +};
>>> +
>>> +
>>> +/*
>>> * Control map entries
>>> */
>>>
>>> @@ -423,6 +433,10 @@ static struct usbmix_ctl_map
>>> usbmix_ctl_maps[] = {
>>> .map = maya44_map,
>>> },
>>> {
>>> + .id = USB_ID(0x0bda, 0x4014),
>>> + .map = dell_alc4020_map,
>>> + },
>>> + {
>>> .id = USB_ID(0x0c45, 0x1158),
>>> .map = justlink_map,
>>> },
>>>
>>> ------Please consider the environment before printing this e-mail.
>>>
>> [2 0001-Dell-usb-dock-vloume-issue.patch <application/octet-stream (base64)>]
>>
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>
More information about the Alsa-devel
mailing list