On 18-12-2017 21:56, Mauro Santos wrote: > On 18-12-2017 19:30, Takashi Iwai wrote: >> On Mon, 18 Dec 2017 20:10:44 +0100, >> Mauro Santos wrote: >>> >>> On 18-12-2017 17:50, Jaejoong Kim wrote: >>>> Mauro, >>>> >>>> Could you please try debug patch(I also attach the patch file)? >>> >>> With the attached patch I get the following when plugging in the usb dac >>> directly to a usb3 port: >>> [ 54.391539] usb 1-2: new full-speed USB device number 7 using xhci_hcd >>> [ 54.514996] usb 1-2: device descriptor read/64, error -71 >>> [ 54.849808] input: HiFimeDIY Audio HiFimeDIY DAC as >>> /devices/pci0000:00/0000:00:14.0/usb1/1-2/1-2:1.0/0003:1852:7022.0003/input/input20 >>> [ 54.850168] hid-generic 0003:1852:7022.0003: input,hidraw2: USB HID >>> v1.00 Device [HiFimeDIY Audio HiFimeDIY DAC] on usb-0000:00:14.0-2/input0 >>> [ 54.950421] usb 1-2: [DEBUG] nameid:0, len:0 >>> [ 54.950426] usb 1-2: [DEBUG] len:3, get_term_name:PCM >>> [ 54.950429] usb 1-2: [11] SU [PCM] items = 2 >>> [ 54.950985] usbcore: registered new interface driver snd-usb-audio >> >> Hmm, the driver get the supposedly correct name string here, so I see >> no flaw, so far. >> >> Could you put the similar debug prints after reverting the commit and >> compare? Or, at minimum, you can enable simply the kernel debug >> prints like below: >> >> % echo "file sound/usb/mixer.c +p" > /sys/kernel/debug/dynamic_debug_control >> >> and re-plug the device. >> >> Also, could you attach the output of "amixer contents" on both working >> and non-working kernels? >> > > I have compiled a new kernel where I have reverted the commit and I've > added the debug output based on your last debug patch. I attach the > patch that reverts the changes and adds the debug output just in case > anyone wants to do a sanity check on it (don't mind the indentation I > think I botched that). > > With the debug patches I get no extra output when echoing to the > dynamic_debug/control file, I guess that's expected. > Turns out there is some output, I can't echo and plug, I need to plug, echo, replug. Dmesg outputs are attached. > I attach the dmesg and amixer outputs for the case without revert plus > debug (bad) and revert plus debug (good). > > One change does jump out: > > bad: usb 1-2: [11] SU [PCM] items = 2 > good: usb 1-2: [11] SU [PCM Capture Source] items = 2 > -- Mauro Santos
[ 81.753703] usb 1-2: new full-speed USB device number 9 using xhci_hcd [ 81.877148] usb 1-2: device descriptor read/64, error -71 [ 82.210907] input: HiFimeDIY Audio HiFimeDIY DAC as /devices/pci0000:00/0000:00:14.0/usb1/1-2/1-2:1.0/0003:1852:7022.0005/input/input22 [ 82.211121] hid-generic 0003:1852:7022.0005: input,hidraw2: USB HID v1.00 Device [HiFimeDIY Audio HiFimeDIY DAC] on usb-0000:00:14.0-2/input0 [ 82.231042] usb 1-2: [16] FU [PCM Playback Switch] ch = 1, val = 0/1/1 [ 82.232211] usb 1-2: cannot set ctl value: req = 0x4, wValue = 0x201, wIndex = 0x1001, type = 4, data = 0x40/0x0 [ 82.232616] usb 1-2: [16] FU [PCM Playback Volume] ch = 2, val = -14080/0/128 [ 82.232626] usb 1-2: [14] FU [Line Capture Switch] ch = 1, val = 0/1/1 [ 82.233919] usb 1-2: cannot set ctl value: req = 0x4, wValue = 0x201, wIndex = 0xe01, type = 4, data = 0x40/0x0 [ 82.234301] usb 1-2: [14] FU [Line Capture Volume] ch = 2, val = -10240/3072/128 [ 82.234310] usb 1-2: [DEBUG] nameid:0, len:0 [ 82.234313] usb 1-2: [DEBUG] len:3, get_term_name:PCM [ 82.234315] usb 1-2: [11] SU [PCM] items = 2
[ 59.283418] usb 1-2: new full-speed USB device number 8 using xhci_hcd [ 59.407116] usb 1-2: device descriptor read/64, error -71 [ 59.740686] input: HiFimeDIY Audio HiFimeDIY DAC as /devices/pci0000:00/0000:00:14.0/usb1/1-2/1-2:1.0/0003:1852:7022.0004/input/input21 [ 59.740893] hid-generic 0003:1852:7022.0004: input,hidraw2: USB HID v1.00 Device [HiFimeDIY Audio HiFimeDIY DAC] on usb-0000:00:14.0-2/input0 [ 59.760971] usb 1-2: [16] FU [PCM Playback Switch] ch = 1, val = 0/1/1 [ 59.762056] usb 1-2: cannot set ctl value: req = 0x4, wValue = 0x201, wIndex = 0x1001, type = 4, data = 0x40/0x0 [ 59.762418] usb 1-2: [16] FU [PCM Playback Volume] ch = 2, val = -14080/0/128 [ 59.762423] usb 1-2: [14] FU [Line Capture Switch] ch = 1, val = 0/1/1 [ 59.763510] usb 1-2: cannot set ctl value: req = 0x4, wValue = 0x201, wIndex = 0xe01, type = 4, data = 0x40/0x0 [ 59.763845] usb 1-2: [14] FU [Line Capture Volume] ch = 2, val = -10240/3072/128 [ 59.763848] usb 1-2: [DEBUG] nameid:0, len:0 [ 59.763849] usb 1-2: [DEBUG] len:3, get_term_name:PCM [ 59.763850] usb 1-2: [11] SU [PCM Capture Source] items = 2
_______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
