February 2021 - Alsa-devel - mailman.alsa-project.org

Re: [PATCH 1/2] dt-bindings: mediatek: mt6359: add ASoC mt6359 accdet document
by Argus Lin 03 Feb '21

03 Feb '21

Dear maintainers: Can you reply to me if any opinion? thanks B.R. Argus On Wed, 2021-01-06 at 20:19 +0800, Argus Lin wrote: > This patch adds MediaTek MT6359 accdet document. > > Signed-off-by: Argus Lin <argus.lin(a)mediatek.com> > --- > .../devicetree/bindings/sound/mt6359-accdet.yaml | 142 +++++++++++++++++++++ > 1 file changed, 142 insertions(+) > create mode 100644 Documentation/devicetree/bindings/sound/mt6359-accdet.yaml > > diff --git a/Documentation/devicetree/bindings/sound/mt6359-accdet.yaml b/Documentation/devicetree/bindings/sound/mt6359-accdet.yaml > new file mode 100644 > index 0000000..fde0d29 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/mt6359-accdet.yaml > @@ -0,0 +1,142 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/sound/mt6359-accdet.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Mediatek Technologies Inc. MT6359 ASoC accdet driver bindings > + > +maintainers: > + - Chipeng Chang <chipeng.chang(a)mediatek.com> > + > +description: | > + This binding describes the Mediatek Technologies MT6359 ASoC > + accdet driver. > + > +properties: > + compatible: > + enum: > + - mediatek,mt6359-accdet > + - mediatek,pmic-accdet > + > + accdet-name: > + maxItems: 1 > + description: named as "mt6359-accdet" for snd_soc_card_jack. > + > + accdet-mic-vol: > + maxItems: 1 > + description: | > + accdet micbias1 voltage. > + > + enum: > + - 0x00 # 1.7v - micbias1 volage is 1.7v. > + - 0x01 # 1.8v - micbias1 volage is 1.8v. > + - 0x02 # 1.9v - micbias1 volage is 1.9v. > + - 0x03 # 2.0v - micbias1 volage is 2.0v. > + - 0x04 # 2.1v - micbias1 volage is 2.1v. > + - 0x05 # 2.5v - micbias1 volage is 2.5v. > + - 0x06 # 2.6v - micbias1 volage is 2.6v. > + - 0x07 # 2.7v - micbias1 volage is 2.7v. > + - 0x08 # 2.8v - micbias1 volage is 2.8v. > + - 0x09 # 2.85v - micbias1 volage is 2.85v. > + > + accdet-plugout-debounce: > + maxItems: 1 > + description: For using ap gpio eint only, it is sw debounce time. > + > + accdet-mic-mode: > + maxItems: 1 > + description: | > + value as 1, 2, 6 to indicate ACC/DCC mode. default is DCC mode 2. > + > + enum: > + - 0x01 # ACC - ACC mode. > + - 0x02 # DCC - low cost without in bias. > + - 0x06 # DCC - low cost with in bias. > + > + eint_use_ext_res: > + maxItems: 1 > + description: select HP_EINT pull up resistance, default 0 use internal resistance. > + > + headset-mode-setting: > + maxItems: 15 > + description: | > + headset-mode-setting : Indicates customized pwm, debounce setting > + accdet pwm_width, pwm_thresh, fall_delay, rise_delay > + debounce0, debounce1, debounce3, debounce4 > + eint pwm_width, eint pwm_thresh > + eint deb(debounce0, debounce1, debounce2, debounce3), inv_debounce. > + > + headset-use-ap-eint: > + maxItems: 1 > + description: indicates to use ap gpio. > + > + headset-eint-num: > + maxItems: 1 > + description: | > + incicates pmic eint usage. > + > + enum: > + - 0x0 # PMIC_EINT0 - use pmic eint0 to trigger plug interrupt. > + - 0x1 # PMIC_EINT1 - use pmic eint1 to trigger plug interrupt. > + - 0x2 # PMIC_BI_EINT - use pmic eint0/1 to trigger plug interrupt. > + > + headset-eint-trig-mode: > + maxItems: 1 > + description: | > + incicates pmic eint trigger mode. > + > + enum: > + - 0x0 # PMIC_GPIO - use pmic gpio to detect plug status by setting polarity to high or low. > + - 0x1 # PMIC_INVERTER - use pmic inverter to detect plug status. > + > + headset-key-mode: > + maxItems: 1 > + description: | > + incicates key mode type. > + > + enum: > + - 0x0 # THREE_KEY - support 3-key key mode. > + - 0x1 # FOUR_KEY - support 4-key key mode. > + - 0x1 # TRI_KEY_CDD - support 3-key google CDD key mode. > + > + headset-three-key-threshold: > + maxItems: 4 > + description: | > + 3 key device detection threshold: 0--MD_MAX--UP_MAX--DW_MAX. > + > + headset-three-key-threshold-CDD: > + maxItems: 4 > + description: | > + 3 key CDD device detection threshold: 0--MD_MAX--UP_MAX--DW_MAX. > + > + headset-four-key-threshold: > + maxItems: 4 > + description: | > + 4 key device detection threshold: 0--MD_MAX--VOICE_MAX--UP_MAX--DW_MAX. > + > +additionalProperties: false > + > +examples: > + - | > + accdet: accdet { > + compatible = "mediatek,mt6359-accdet"; > + accdet-name = "mt6359-accdet"; > + accdet-mic-vol = <8>; > + accdet-plugout-debounce = <1>; > + accdet-mic-mode = <2>; > + eint_use_ext_res = <0>; > + headset-mode-setting = <0x500 0x500 1 0x1f0 > + 0x800 0x800 0x20 0x44 > + 0x4 0x1 > + 0x5 0x3 0x3 0x5 0xe>; > + headset-use-ap-eint = <0>; > + headset-eint-num = <0>; > + headset-eint-trig-mode = <1>; > + headset-key-mode = <0>; > + headset-three-key-threshold = <0 80 220 400>; > + headset-three-key-threshold-CDD = <0 121 192 600>; > + headset-four-key-threshold = <0 58 121 192 400>; > + status = "okay"; > + }; > +... > -- > 1.8.1.1.dirty >

1 0

Oops with "ALSA: jack: implement software jack injection via debugfs"
by Chris Wilson 03 Feb '21

03 Feb '21

commit 2d670ea2bd53 ("ALSA: jack: implement software jack injection via debugfs") is causing issues for our CI as we see a use-after-free on module unload (on all machines): https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9715/fi-skl-6700k2/pstore0-… <4> [241.294412] general protection fault, probably for non-canonical address 0x6b6b6b6b6b6b6c13: 0000 [#1] PREEMPT SMP PTI <4> [241.294424] CPU: 7 PID: 5820 Comm: i915_module_loa Not tainted 5.11.0-rc6-CI-CI_DRM_9715+ #1 <4> [241.294432] Hardware name: System manufacturer System Product Name/Z170 PRO GAMING, BIOS 0802 09/02/2015 <4> [241.294438] RIP: 0010:__lock_acquire+0x61a/0x25a0 <4> [241.294444] Code: 00 00 83 f8 2f 0f 87 a0 00 00 00 3b 05 97 61 07 02 c7 44 24 18 01 00 00 00 0f 86 d4 00 00 00 89 05 83 61 07 02 e9 c9 00 00 00 <48> 81 3f 80 14 d9 82 41 bc 00 00 00 00 45 0f 45 e0 83 fe 01 0f 87 <4> [241.294451] RSP: 0018:ffffc90000e77b78 EFLAGS: 00010002 <4> [241.294454] RAX: 0000000000000000 RBX: ffff888137d50040 RCX: 0000000000000000 <4> [241.294458] RDX: 0000000000000000 RSI: 0000000000000000 RDI: 6b6b6b6b6b6b6c13 <4> [241.294461] RBP: 0000000000000001 R08: 0000000000000001 R09: 0000000000000000 <4> [241.294464] R10: 0000000000000001 R11: 00000000d2337ff6 R12: 0000000000000001 <4> [241.294467] R13: 0000000000000000 R14: 0000000000000000 R15: 6b6b6b6b6b6b6c13 <4> [241.294470] FS: 00007f0e00616e40(0000) GS:ffff88824db80000(0000) knlGS:0000000000000000 <4> [241.294474] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 <4> [241.294477] CR2: 00007f375bda2cb0 CR3: 000000010ddd6001 CR4: 00000000003706e0 <4> [241.294480] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000 <4> [241.294483] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400 <4> [241.294486] Call Trace: <4> [241.294489] lock_acquire+0xd0/0x3b0 <4> [241.294493] ? lockref_get+0x9/0x20 <4> [241.294498] _raw_spin_lock+0x2a/0x40 <4> [241.294501] ? lockref_get+0x9/0x20 <4> [241.294505] lockref_get+0x9/0x20 <4> [241.294508] simple_recursive_removal+0x31/0x2a0 <4> [241.294511] ? debugfs_remove+0x50/0x50 <4> [241.294515] ? _raw_spin_unlock+0x24/0x40 <4> [241.294519] debugfs_remove+0x3b/0x50 <4> [241.294522] snd_card_free+0x76/0xa0 <4> [241.294527] pci_device_remove+0x36/0xb0 <4> [241.294531] device_release_driver_internal+0xf2/0x1d0 <4> [241.294820] unbind_store+0xeb/0x120 <4> [241.294824] kernfs_fop_write_iter+0x11d/0x1c0 <4> [241.294828] new_sync_write+0x11d/0x1b0 <4> [241.294832] vfs_write+0x260/0x390 <4> [241.294835] ksys_write+0x5a/0xd0 <4> [241.294838] do_syscall_64+0x33/0x80 <4> [241.294842] entry_SYSCALL_64_after_hwframe+0x44/0xa9 <4> [241.294845] RIP: 0033:0x7f0dffd80281 <4> [241.294848] Code: c3 0f 1f 84 00 00 00 00 00 48 8b 05 59 8d 20 00 c3 0f 1f 84 00 00 00 00 00 8b 05 8a d1 20 00 85 c0 75 16 b8 01 00 00 00 0f 05 <48> 3d 00 f0 ff ff 77 57 f3 c3 0f 1f 44 00 00 41 54 55 49 89 d4 53 <4> [241.294855] RSP: 002b:00007fff5a3a9c88 EFLAGS: 00000246 ORIG_RAX: 0000000000000001 <4> [241.294860] RAX: ffffffffffffffda RBX: 0000555708dc2cf0 RCX: 00007f0dffd80281 <4> [241.294863] RDX: 000000000000000c RSI: 0000555708dc2ddb RDI: 0000000000000006 <4> [241.294866] RBP: 00007fff5a3a9d20 R08: 0000555708dc2ddb R09: 000000000000000c <4> [241.294869] R10: 00000000fffffff4 R11: 0000000000000246 R12: 00007fff5a3a9c90 <4> [241.294872] R13: 00007f0e00007b18 R14: 0000000000000006 R15: 0000555708dc2ddb <4> [241.294877] Modules linked in: vgem snd_hda_codec_hdmi snd_hda_codec_realtek snd_hda_codec_generic ledtrig_audio i915 mei_hdcp x86_pkg_temp_thermal coretemp crct10dif_pclmul crc32_pclmul snd_hda_intel snd_intel_dspcfg ghash_clmulni_intel snd_hda_codec snd_hwdep snd_hda_core e1000e mei_me snd_pcm mei ptp pps_core prime_numbers [last unloaded: vgem] <4> [241.294901] ---[ end trace 98e116c0344cf275 ]--- <4> [242.399691] RIP: 0010:__lock_acquire+0x61a/0x25a0 <4> [242.399698] Code: 00 00 83 f8 2f 0f 87 a0 00 00 00 3b 05 97 61 07 02 c7 44 24 18 01 00 00 00 0f 86 d4 00 00 00 89 05 83 61 07 02 e9 c9 00 00 00 <48> 81 3f 80 14 d9 82 41 bc 00 00 00 00 45 0f 45 e0 83 fe 01 0f 87 <4> [242.399705] RSP: 0018:ffffc90000e77b78 EFLAGS: 00010002 <4> [242.399709] RAX: 0000000000000000 RBX: ffff888137d50040 RCX: 0000000000000000 <4> [242.399713] RDX: 0000000000000000 RSI: 0000000000000000 RDI: 6b6b6b6b6b6b6c13 <4> [242.399716] RBP: 0000000000000001 R08: 0000000000000001 R09: 0000000000000000 <4> [242.399719] R10: 0000000000000001 R11: 00000000d2337ff6 R12: 0000000000000001 <4> [242.399722] R13: 0000000000000000 R14: 0000000000000000 R15: 6b6b6b6b6b6b6c13 <4> [242.399725] FS: 00007f0e00616e40(0000) GS:ffff88824db80000(0000) knlGS:0000000000000000 <4> [242.399729] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 <4> [242.399732] CR2: 00007f375bda2cb0 CR3: 000000010ddd6001 CR4: 00000000003706e0 <4> [242.399735] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000 <4> [242.399739] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400 <6> [242.399742] note: i915_module_loa[5820] exited with preempt_count 1 <3> [242.399745] BUG: sleeping function called from invalid context at include/linux/percpu-rwsem.h:49 <3> [242.399749] in_atomic(): 0, irqs_disabled(): 1, non_block: 0, pid: 5820, name: i915_module_loa <4> [242.399753] INFO: lockdep is turned off. <4> [242.399755] irq event stamp: 125679 <4> [242.399757] hardirqs last enabled at (125679): [<ffffffff81a9e36f>] _raw_spin_unlock_irq+0x1f/0x40 <4> [242.399763] hardirqs last disabled at (125678): [<ffffffff81a9e151>] _raw_spin_lock_irq+0x41/0x50 <4> [242.399768] softirqs last enabled at (125054): [<ffffffff81e00342>] __do_softirq+0x342/0x48e <4> [242.399773] softirqs last disabled at (124021): [<ffffffff81c00f52>] asm_call_irq_on_stack+0x12/0x20 <4> [242.399779] CPU: 7 PID: 5820 Comm: i915_module_loa Tainted: G D 5.11.0-rc6-CI-CI_DRM_9715+ #1 <4> [242.399784] Hardware name: System manufacturer System Product Name/Z170 PRO GAMING, BIOS 0802 09/02/2015 <4> [242.399787] Call Trace: <4> [242.399790] dump_stack+0x77/0x97 <4> [242.399794] ___might_sleep.cold.120+0xf2/0x106 <4> [242.399799] exit_signals+0x2b/0x350 <4> [242.399803] do_exit+0xc8/0xcb0 <4> [242.399807] ? ksys_write+0x5a/0xd0 <4> [242.399811] rewind_stack_do_exit+0x17/0x17 <4> [242.399814] RIP: 0033:0x7f0dffd80281 <4> [242.399817] Code: c3 0f 1f 84 00 00 00 00 00 48 8b 05 59 8d 20 00 c3 0f 1f 84 00 00 00 00 00 8b 05 8a d1 20 00 85 c0 75 16 b8 01 00 00 00 0f 05 <48> 3d 00 f0 ff ff 77 57 f3 c3 0f 1f 44 00 00 41 54 55 49 89 d4 53 <4> [242.399824] RSP: 002b:00007fff5a3a9c88 EFLAGS: 00000246 ORIG_RAX: 0000000000000001 <4> [242.399828] RAX: ffffffffffffffda RBX: 0000555708dc2cf0 RCX: 00007f0dffd80281 <4> [242.399831] RDX: 000000000000000c RSI: 0000555708dc2ddb RDI: 0000000000000006 <4> [242.399834] RBP: 00007fff5a3a9d20 R08: 0000555708dc2ddb R09: 000000000000000c <4> [242.399838] R10: 00000000fffffff4 R11: 0000000000000246 R12: 00007fff5a3a9c90 <4> [242.399841] R13: 00007f0e00007b18 R14: 0000000000000006 R15: 0000555708dc2ddb Created at 2021-02-02 17:16:13

3 4

[PATCH] ALSA: core: Fix the debugfs removal at snd_card_free()
by Takashi Iwai 02 Feb '21

02 Feb '21

The commit 2d670ea2bd53 ("ALSA: jack: implement software jack injection via debugfs") introduced a debugfs root for each sound card object. The debugfs entry gets removed at the card removal, too, but it turned out that debugfs_remove() is called at a wrong place; it's after the card object gets freed, hence it leads to use-after-free. Fix it by moving the debugfs_remove() at the right place, the destructor of the card device. Fixes: 2d670ea2bd53 ("ALSA: jack: implement software jack injection via debugfs") Reported-and-tested-by: Chris Wilson <chris(a)chris-wilson.co.uk> Link: https://lore.kernel.org/r/161228343605.1150.8862281636043446562@build.alpor… Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/core/init.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/sound/core/init.c b/sound/core/init.c index d4e78b176793..84b573e9c1f9 100644 --- a/sound/core/init.c +++ b/sound/core/init.c @@ -487,6 +487,10 @@ static int snd_card_do_free(struct snd_card *card) dev_warn(card->dev, "unable to free card info\n"); /* Not fatal error */ } +#ifdef CONFIG_SND_DEBUG + debugfs_remove(card->debugfs_root); + card->debugfs_root = NULL; +#endif if (card->release_completion) complete(card->release_completion); kfree(card); @@ -537,11 +541,6 @@ int snd_card_free(struct snd_card *card) /* wait, until all devices are ready for the free operation */ wait_for_completion(&released); -#ifdef CONFIG_SND_DEBUG - debugfs_remove(card->debugfs_root); - card->debugfs_root = NULL; -#endif - return 0; } EXPORT_SYMBOL(snd_card_free); -- 2.26.2

1 0

Nux Mighty Plug USB - no clock source
by Peter Allen 02 Feb '21

02 Feb '21

I have a NUX Mighty Plug guitar effects which has a USB Audio interface. Unfortunately it doesn't work in Linux, and I'd love to fix it. It gets registered with Alsa, except it ends up without any valid sample rates (by the look of it due to a buggy clock source descriptor). This is on Ubuntu-Studio 20.04 LTS _Focal Fossa with kernel 5.4.0-62-lowlatency on a HP Omen 15 with i5-7300HQ I don't know what I'm doing (but did a little module development rather a long time ago in the 2.2 days :-)) Help is greatly appreciated. When plugged in I get: [12476.413792] usb 1-1: New USB device found, idVendor=1fc9, idProduct=8260, bcdDevice= 1.00 [12476.413797] usb 1-1: New USB device strings: Mfr=1, Product=2, SerialNumber=3 [12476.413800] usb 1-1: Product: MIGHTY PLUG USB [12476.413804] usb 1-1: Manufacturer: NUX [12476.413807] usb 1-1: SerialNumber: 2007151403 [12476.444988] usb 1-1: __uac_clock_find_source(): selector reported illegal value, id 40, ret 0 [12476.444992] usb 1-1: parse_audio_format_rates_v2v3(): unable to find clock source (clock -22) [12476.445413] usb 1-1: __uac_clock_find_source(): selector reported illegal value, id 40, ret 0 [12476.445479] usb 1-1: __uac_clock_find_source(): selector reported illegal value, id 40, ret 0 [12476.447095] usb 1-1: __uac_clock_find_source(): selector reported illegal value, id 40, ret 0 [12476.447098] usb 1-1: parse_audio_format_rates_v2v3(): unable to find clock source (clock -22) [12476.447459] usb 1-1: __uac_clock_find_source(): selector reported illegal value, id 40, ret 0 [12476.447684] usb 1-1: __uac_clock_find_source(): selector reported illegal value, id 40, ret 0 [12476.450397] usbcore: registered new interface driver snd-usb-audio ===== /proc/asound/cards give ====== 0 [PCH ]: HDA-Intel - HDA Intel PCH HDA Intel PCH at 0xb4428000 irq 134 1 [NVidia ]: HDA-Intel - HDA NVidia HDA NVidia at 0xb4000000 irq 17 2 [USB ]: USB-Audio - MIGHTY PLUG USB NUX MIGHTY PLUG USB at usb-0000:00:14.0-1, high speed ==== /proc/asound/stream======== NUX MIGHTY PLUG USB at usb-0000:00:14.0-1, high speed : USB Audio Playback: Status: Stop Interface 1 Altset 1 Format: S32_LE Channels: 2 Endpoint: 1 OUT (ASYNC) Rates: Data packet interval: 1000 us Bits: 24 Capture: Status: Stop Interface 2 Altset 1 Format: S32_LE Channels: 2 Endpoint: 2 IN (ASYNC) Rates: Data packet interval: 1000 us Bits: 24 ======== arecord =========== arecord -D "hw:2" -c 2 -f s24 -r 44100 -d 2 -v test.wav Recording WAVE 'xxxc' : Signed 24 bit Little Endian, Rate 44100 Hz, Stereo arecord: set_params:1314: Broken configuration for this PCM: no configuration available ========lsusb -v ========== iSerial 2 200901010001 bNumConfigurations 1 Bus 001 Device 005: ID 1fc9:8260 NXP Semiconductors MIGHTY PLUG USB Device Descriptor: bLength 18 bDescriptorType 1 bcdUSB 2.00 bDeviceClass 239 Miscellaneous Device bDeviceSubClass 2 bDeviceProtocol 1 Interface Association bMaxPacketSize0 64 idVendor 0x1fc9 NXP Semiconductors idProduct 0x8260 bcdDevice 1.00 iManufacturer 1 NUX iProduct 2 MIGHTY PLUG USB iSerial 3 2007151403 bNumConfigurations 1 Configuration Descriptor: bLength 9 bDescriptorType 2 wTotalLength 0x013e bNumInterfaces 4 bConfigurationValue 1 iConfiguration 0 bmAttributes 0xc0 Self Powered MaxPower 2mA Interface Association: bLength 8 bDescriptorType 11 bFirstInterface 0 bInterfaceCount 3 bFunctionClass 1 Audio bFunctionSubClass 0 bFunctionProtocol 32 iFunction 0 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 0 bAlternateSetting 0 bNumEndpoints 0 bInterfaceClass 1 Audio bInterfaceSubClass 1 Control Device bInterfaceProtocol 32 iInterface 0 AudioControl Interface Descriptor: bLength 9 bDescriptorType 36 bDescriptorSubtype 1 (HEADER) bcdADC 2.00 bCategory 8 wTotalLength 0x0065 bmControls 0x00 AudioControl Interface Descriptor: bLength 8 bDescriptorType 36 bDescriptorSubtype 10 (CLOCK_SOURCE) bClockID 41 bmAttributes 3 Internal programmable clock bmControls 0x07 Clock Frequency Control (read/write) Clock Validity Control (read-only) bAssocTerminal 0 iClockSource 0 AudioControl Interface Descriptor: bLength 8 bDescriptorType 36 bDescriptorSubtype 11 (CLOCK_SELECTOR) bClockID 40 bNrInPins 1 baCSourceID(0) 41 bmControls 0x03 Clock Selector Control (read/write) iClockSelector 0 AudioControl Interface Descriptor: bLength 17 bDescriptorType 36 bDescriptorSubtype 2 (INPUT_TERMINAL) bTerminalID 2 wTerminalType 0x0101 USB Streaming bAssocTerminal 0 bCSourceID 40 bNrChannels 2 bmChannelConfig 0x00000000 iChannelNames 0 bmControls 0x0000 iTerminal 0 AudioControl Interface Descriptor: bLength 18 bDescriptorType 36 bDescriptorSubtype 6 (FEATURE_UNIT) bUnitID 10 bSourceID 2 bmaControls(0) 0x0000000f Mute Control (read/write) Volume Control (read/write) bmaControls(1) 0x0000000f Mute Control (read/write) Volume Control (read/write) bmaControls(2) 0x0000000f Mute Control (read/write) Volume Control (read/write) iFeature 0 AudioControl Interface Descriptor: bLength 12 bDescriptorType 36 bDescriptorSubtype 3 (OUTPUT_TERMINAL) bTerminalID 20 wTerminalType 0x0301 Speaker bAssocTerminal 0 bSourceID 10 bCSourceID 40 bmControls 0x0000 iTerminal 0 AudioControl Interface Descriptor: bLength 17 bDescriptorType 36 bDescriptorSubtype 2 (INPUT_TERMINAL) bTerminalID 1 wTerminalType 0x0201 Microphone bAssocTerminal 0 bCSourceID 40 bNrChannels 2 bmChannelConfig 0x00000000 iChannelNames 0 bmControls 0x0000 iTerminal 0 AudioControl Interface Descriptor: bLength 12 bDescriptorType 36 bDescriptorSubtype 3 (OUTPUT_TERMINAL) bTerminalID 22 wTerminalType 0x0101 USB Streaming bAssocTerminal 0 bSourceID 2 bCSourceID 40 bmControls 0x0000 iTerminal 0 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 1 bAlternateSetting 0 bNumEndpoints 0 bInterfaceClass 1 Audio bInterfaceSubClass 2 Streaming bInterfaceProtocol 32 iInterface 0 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 1 bAlternateSetting 1 bNumEndpoints 2 bInterfaceClass 1 Audio bInterfaceSubClass 2 Streaming bInterfaceProtocol 32 iInterface 0 AudioStreaming Interface Descriptor: bLength 16 bDescriptorType 36 bDescriptorSubtype 1 (AS_GENERAL) bTerminalLink 2 bmControls 0x00 bFormatType 1 bmFormats 0x00000001 PCM bNrChannels 2 bmChannelConfig 0x00000000 iChannelNames 0 AudioStreaming Interface Descriptor: bLength 6 bDescriptorType 36 bDescriptorSubtype 2 (FORMAT_TYPE) bFormatType 1 (FORMAT_TYPE_I) bSubslotSize 4 bBitResolution 24 Endpoint Descriptor: bLength 7 bDescriptorType 5 bEndpointAddress 0x01 EP 1 OUT bmAttributes 5 Transfer Type Isochronous Synch Type Asynchronous Usage Type Data wMaxPacketSize 0x0400 1x 1024 bytes bInterval 4 AudioStreaming Endpoint Descriptor: bLength 8 bDescriptorType 37 bDescriptorSubtype 1 (EP_GENERAL) bmAttributes 0x00 bmControls 0x00 bLockDelayUnits 0 Undefined wLockDelay 0x0000 Endpoint Descriptor: bLength 7 bDescriptorType 5 bEndpointAddress 0x81 EP 1 IN bmAttributes 17 Transfer Type Isochronous Synch Type None Usage Type Feedback wMaxPacketSize 0x0004 1x 4 bytes bInterval 4 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 2 bAlternateSetting 0 bNumEndpoints 0 bInterfaceClass 1 Audio bInterfaceSubClass 2 Streaming bInterfaceProtocol 32 iInterface 0 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 2 bAlternateSetting 1 bNumEndpoints 1 bInterfaceClass 1 Audio bInterfaceSubClass 2 Streaming bInterfaceProtocol 32 iInterface 0 AudioStreaming Interface Descriptor: bLength 16 bDescriptorType 36 bDescriptorSubtype 1 (AS_GENERAL) bTerminalLink 22 bmControls 0x00 bFormatType 1 bmFormats 0x00000001 PCM bNrChannels 2 bmChannelConfig 0x00000000 iChannelNames 0 AudioStreaming Interface Descriptor: bLength 6 bDescriptorType 36 bDescriptorSubtype 2 (FORMAT_TYPE) bFormatType 1 (FORMAT_TYPE_I) bSubslotSize 4 bBitResolution 24 Endpoint Descriptor: bLength 7 bDescriptorType 5 bEndpointAddress 0x82 EP 2 IN bmAttributes 5 Transfer Type Isochronous Synch Type Asynchronous Usage Type Data wMaxPacketSize 0x0400 1x 1024 bytes bInterval 4 AudioStreaming Endpoint Descriptor: bLength 8 bDescriptorType 37 bDescriptorSubtype 1 (EP_GENERAL) bmAttributes 0x00 bmControls 0x00 bLockDelayUnits 0 Undefined wLockDelay 0x0000 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 3 bAlternateSetting 0 bNumEndpoints 2 bInterfaceClass 1 Audio bInterfaceSubClass 3 MIDI Streaming bInterfaceProtocol 0 iInterface 4 MIGHTY PLUG USB MIDI MIDIStreaming Interface Descriptor: bLength 7 bDescriptorType 36 bDescriptorSubtype 1 (HEADER) bcdADC 1.00 wTotalLength 0x0041 MIDIStreaming Interface Descriptor: bLength 6 bDescriptorType 36 bDescriptorSubtype 2 (MIDI_IN_JACK) bJackType 1 Embedded bJackID 1 iJack 0 MIDIStreaming Interface Descriptor: bLength 6 bDescriptorType 36 bDescriptorSubtype 2 (MIDI_IN_JACK) bJackType 2 External bJackID 2 iJack 0 MIDIStreaming Interface Descriptor: bLength 9 bDescriptorType 36 bDescriptorSubtype 3 (MIDI_OUT_JACK) bJackType 1 Embedded bJackID 3 bNrInputPins 1 baSourceID( 0) 2 BaSourcePin( 0) 1 iJack 0 MIDIStreaming Interface Descriptor: bLength 9 bDescriptorType 36 bDescriptorSubtype 3 (MIDI_OUT_JACK) bJackType 2 External bJackID 4 bNrInputPins 1 baSourceID( 0) 1 BaSourcePin( 0) 1 iJack 0 Endpoint Descriptor: bLength 9 bDescriptorType 5 bEndpointAddress 0x04 EP 4 OUT bmAttributes 2 Transfer Type Bulk Synch Type None Usage Type Data wMaxPacketSize 0x0100 1x 256 bytes bInterval 0 bRefresh 0 bSynchAddress 0 MIDIStreaming Endpoint Descriptor: bLength 5 bDescriptorType 37 bDescriptorSubtype 1 (GENERAL) bNumEmbMIDIJack 1 baAssocJackID( 0) 1 Endpoint Descriptor: bLength 9 bDescriptorType 5 bEndpointAddress 0x83 EP 3 IN bmAttributes 2 Transfer Type Bulk Synch Type None Usage Type Data wMaxPacketSize 0x0100 1x 256 bytes bInterval 0 bRefresh 0 bSynchAddress 0 MIDIStreaming Endpoint Descriptor: bLength 5 bDescriptorType 37 bDescriptorSubtype 1 (GENERAL) bNumEmbMIDIJack 1 baAssocJackID( 0) 3 Device Qualifier (for other device speed): bLength 10 bDescriptorType 6 bcdUSB 2.00 bDeviceClass 0 bDeviceSubClass 0 bDeviceProtocol 0 bMaxPacketSize0 64 bNumConfigurations 1 can't get debug descriptor: Resource temporarily unavailable Device Status: 0x0001 Self Powered Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Device Descriptor: bLength 18 bDescriptorType 1 bcdUSB 2.00 bDeviceClass 9 Hub bDeviceSubClass 0 bDeviceProtocol 1 Single TT bMaxPacketSize0 64 idVendor 0x1d6b Linux Foundation idProduct 0x0002 2.0 root hub bcdDevice 5.04 iManufacturer 3 Linux 5.4.0-58-lowlatency xhci-hcd iProduct 2 xHCI Host Controller iSerial 1 0000:00:14.0 bNumConfigurations 1 Configuration Descriptor: bLength 9 bDescriptorType 2 wTotalLength 0x0019 bNumInterfaces 1 bConfigurationValue 1 iConfiguration 0 bmAttributes 0xe0 Self Powered Remote Wakeup MaxPower 0mA Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 0 bAlternateSetting 0 bNumEndpoints 1 bInterfaceClass 9 Hub bInterfaceSubClass 0 bInterfaceProtocol 0 Full speed (or root) hub iInterface 0 Endpoint Descriptor: bLength 7 bDescriptorType 5 bEndpointAddress 0x81 EP 1 IN bmAttributes 3 Transfer Type Interrupt Synch Type None Usage Type Data wMaxPacketSize 0x0004 1x 4 bytes bInterval 12 Hub Descriptor: bLength 13 bDescriptorType 41 nNbrPorts 16 wHubCharacteristic 0x000a No power switching (usb 1.0) Per-port overcurrent protection TT think time 8 FS bits bPwrOn2PwrGood 10 * 2 milli seconds bHubContrCurrent 0 milli Ampere DeviceRemovable 0x90 0x01 0x00 PortPwrCtrlMask 0xff 0xff 0xff Hub Port Status: Port 1: 0000.0503 highspeed power enable connect Port 2: 0000.0100 power Port 3: 0000.0100 power Port 4: 0000.0503 highspeed power enable connect Port 5: 0000.0100 power Port 6: 0000.0100 power Port 7: 0000.0103 power enable connect Port 8: 0000.0100 power Port 9: 0000.0100 power Port 10: 0000.0100 power Port 11: 0000.0100 power Port 12: 0000.0100 power Port 13: 0000.0100 power Port 14: 0000.0100 power Port 15: 0000.0100 power Port 16: 0000.0100 power can't get device qualifier: Resource temporarily unavailable can't get debug descriptor: Resource temporarily unavailable Device Status: 0x0001 Self Powered

2 3

[PATCH v2 0/2] ALSA: usb-audio: Add Pioneer DJM-450 support
by Olivia Mackintosh 02 Feb '21

02 Feb '21

Re-roll of this patchset to fix the following: 1. Missing 'Signed-of-by:' line 2. Spaces around operators 3. Test that I'm threading submissions correctly Apologies for the clumsiness on my part. Olivia Olivia Mackintosh (2): ALSA: usb-audio: Add DJM450 to Pioneer format quirk ALSA: usb-audio: Add DJM-450 to the quirks table sound/usb/quirks-table.h | 57 ++++++++++++++++++++++++++++++++++++++++ sound/usb/quirks.c | 20 ++++++++++++++ 2 files changed, 77 insertions(+) -- 2.30.0

2 3

[PATCH] ASoC: topology: KUnit: Convert from cpu to data format
by Amadeusz Sławiński 02 Feb '21

02 Feb '21

When creating topology templates and overriding data in specific test cases it should be done with cpu_to_le32 macro, so we operate on correct data on all architectures, as topology parser use le32_to_cpu to parse data from structures. Reported-by: kernel test robot <lkp(a)intel.com> Signed-off-by: Amadeusz Sławiński <amadeuszx.slawinski(a)linux.intel.com> --- sound/soc/soc-topology-test.c | 66 +++++++++++++++++------------------ 1 file changed, 33 insertions(+), 33 deletions(-) diff --git a/sound/soc/soc-topology-test.c b/sound/soc/soc-topology-test.c index 36e2a3486dbf..ae3968161509 100644 --- a/sound/soc/soc-topology-test.c +++ b/sound/soc/soc-topology-test.c @@ -121,19 +121,19 @@ struct tplg_tmpl_001 { static struct tplg_tmpl_001 tplg_tmpl_empty = { .header = { - .magic = SND_SOC_TPLG_MAGIC, - .abi = 5, + .magic = cpu_to_le32(SND_SOC_TPLG_MAGIC), + .abi = cpu_to_le32(5), .version = 0, - .type = SND_SOC_TPLG_TYPE_MANIFEST, - .size = sizeof(struct snd_soc_tplg_hdr), + .type = cpu_to_le32(SND_SOC_TPLG_TYPE_MANIFEST), + .size = cpu_to_le32(sizeof(struct snd_soc_tplg_hdr)), .vendor_type = 0, - .payload_size = sizeof(struct snd_soc_tplg_manifest), + .payload_size = cpu_to_le32(sizeof(struct snd_soc_tplg_manifest)), .index = 0, - .count = 1, + .count = cpu_to_le32(1), }, .manifest = { - .size = sizeof(struct snd_soc_tplg_manifest), + .size = cpu_to_le32(sizeof(struct snd_soc_tplg_manifest)), /* rest of fields is 0 */ }, }; @@ -149,60 +149,60 @@ struct tplg_tmpl_002 { static struct tplg_tmpl_002 tplg_tmpl_with_pcm = { .header = { - .magic = SND_SOC_TPLG_MAGIC, - .abi = 5, + .magic = cpu_to_le32(SND_SOC_TPLG_MAGIC), + .abi = cpu_to_le32(5), .version = 0, - .type = SND_SOC_TPLG_TYPE_MANIFEST, - .size = sizeof(struct snd_soc_tplg_hdr), + .type = cpu_to_le32(SND_SOC_TPLG_TYPE_MANIFEST), + .size = cpu_to_le32(sizeof(struct snd_soc_tplg_hdr)), .vendor_type = 0, - .payload_size = sizeof(struct snd_soc_tplg_manifest), + .payload_size = cpu_to_le32(sizeof(struct snd_soc_tplg_manifest)), .index = 0, - .count = 1, + .count = cpu_to_le32(1), }, .manifest = { - .size = sizeof(struct snd_soc_tplg_manifest), - .pcm_elems = 1, + .size = cpu_to_le32(sizeof(struct snd_soc_tplg_manifest)), + .pcm_elems = cpu_to_le32(1), /* rest of fields is 0 */ }, .pcm_header = { - .magic = SND_SOC_TPLG_MAGIC, - .abi = 5, + .magic = cpu_to_le32(SND_SOC_TPLG_MAGIC), + .abi = cpu_to_le32(5), .version = 0, - .type = SND_SOC_TPLG_TYPE_PCM, - .size = sizeof(struct snd_soc_tplg_hdr), + .type = cpu_to_le32(SND_SOC_TPLG_TYPE_PCM), + .size = cpu_to_le32(sizeof(struct snd_soc_tplg_hdr)), .vendor_type = 0, - .payload_size = sizeof(struct snd_soc_tplg_pcm), + .payload_size = cpu_to_le32(sizeof(struct snd_soc_tplg_pcm)), .index = 0, - .count = 1, + .count = cpu_to_le32(1), }, .pcm = { - .size = sizeof(struct snd_soc_tplg_pcm), + .size = cpu_to_le32(sizeof(struct snd_soc_tplg_pcm)), .pcm_name = "KUNIT Audio", .dai_name = "kunit-audio-dai", .pcm_id = 0, .dai_id = 0, - .playback = 1, - .capture = 1, + .playback = cpu_to_le32(1), + .capture = cpu_to_le32(1), .compress = 0, .stream = { [0] = { - .channels = 2, + .channels = cpu_to_le32(2), }, [1] = { - .channels = 2, + .channels = cpu_to_le32(2), }, }, .num_streams = 0, .caps = { [0] = { .name = "kunit-audio-playback", - .channels_min = 2, - .channels_max = 2, + .channels_min = cpu_to_le32(2), + .channels_max = cpu_to_le32(2), }, [1] = { .name = "kunit-audio-capture", - .channels_min = 2, - .channels_max = 2, + .channels_min = cpu_to_le32(2), + .channels_max = cpu_to_le32(2), }, }, .flag_mask = 0, @@ -460,7 +460,7 @@ static void snd_soc_tplg_test_load_empty_tplg_bad_magic(struct kunit *test) * override abi * any value != magic number is wrong */ - data->header.magic = SND_SOC_TPLG_MAGIC + 1; + data->header.magic = cpu_to_le32(SND_SOC_TPLG_MAGIC + 1); kunit_comp->fw.data = (u8 *)data; kunit_comp->fw.size = size; @@ -516,7 +516,7 @@ static void snd_soc_tplg_test_load_empty_tplg_bad_abi(struct kunit *test) * override abi * any value != accepted range is wrong */ - data->header.abi = SND_SOC_TPLG_ABI_VERSION + 1; + data->header.abi = cpu_to_le32(SND_SOC_TPLG_ABI_VERSION + 1); kunit_comp->fw.data = (u8 *)data; kunit_comp->fw.size = size; @@ -572,7 +572,7 @@ static void snd_soc_tplg_test_load_empty_tplg_bad_size(struct kunit *test) * override size * any value != struct size is wrong */ - data->header.size = sizeof(struct snd_soc_tplg_hdr) + 1; + data->header.size = cpu_to_le32(sizeof(struct snd_soc_tplg_hdr) + 1); kunit_comp->fw.data = (u8 *)data; kunit_comp->fw.size = size; -- 2.25.1

2 1

[PATCH] ASoC: fsl_spdif: Utilize the defined parameter to clear code
by Tang Bin 02 Feb '21

02 Feb '21

Utilize the defined parameter 'dev' to make the code cleaner. Signed-off-by: Tang Bin <tangbin(a)cmss.chinamobile.com> --- sound/soc/fsl/fsl_spdif.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/sound/soc/fsl/fsl_spdif.c b/sound/soc/fsl/fsl_spdif.c index 455f96908..b6d5563df 100644 --- a/sound/soc/fsl/fsl_spdif.c +++ b/sound/soc/fsl/fsl_spdif.c @@ -1215,7 +1215,7 @@ static int fsl_spdif_probe_txclk(struct fsl_spdif_priv *spdif_priv, for (i = 0; i < STC_TXCLK_SRC_MAX; i++) { sprintf(tmp, "rxtx%d", i); - clk = devm_clk_get(&pdev->dev, tmp); + clk = devm_clk_get(dev, tmp); if (IS_ERR(clk)) { dev_err(dev, "no rxtx%d clock in devicetree\n", i); return PTR_ERR(clk); @@ -1237,14 +1237,14 @@ static int fsl_spdif_probe_txclk(struct fsl_spdif_priv *spdif_priv, break; } - dev_dbg(&pdev->dev, "use rxtx%d as tx clock source for %dHz sample rate\n", + dev_dbg(dev, "use rxtx%d as tx clock source for %dHz sample rate\n", spdif_priv->txclk_src[index], rate[index]); - dev_dbg(&pdev->dev, "use txclk df %d for %dHz sample rate\n", + dev_dbg(dev, "use txclk df %d for %dHz sample rate\n", spdif_priv->txclk_df[index], rate[index]); if (clk_is_match(spdif_priv->txclk[index], spdif_priv->sysclk)) - dev_dbg(&pdev->dev, "use sysclk df %d for %dHz sample rate\n", + dev_dbg(dev, "use sysclk df %d for %dHz sample rate\n", spdif_priv->sysclk_df[index], rate[index]); - dev_dbg(&pdev->dev, "the best rate for %dHz sample rate is %dHz\n", + dev_dbg(dev, "the best rate for %dHz sample rate is %dHz\n", rate[index], spdif_priv->txrate[index]); return 0; -- 2.20.1.windows.1

3 2

[PATCH] ASoC: qcom: Fix typo error in HDMI regmap config callbacks
by Srinivasa Rao Mandadapu 02 Feb '21

02 Feb '21

Had a typo in lpass platform driver that resulted in crash during suspend/resume with an HDMI dongle connected. The regmap read/write/volatile regesters validation callbacks in lpass-cpu were using MI2S rdma_channels count instead of hdmi_rdma_channels. This typo error causing to read registers from the regmap beyond the length of the mapping created by ioremap(). This fix avoids the need for reducing number hdmi_rdma_channels, which is done in commit 7dfe20ee92f6 ("ASoC: qcom: Fix number of HDMI RDMA channels on sc7180"). So reverting the same. Fixes: 7cb37b7bd0d3c ("ASoC: qcom: Add support for lpass hdmi driver") Signed-off-by: Srinivasa Rao Mandadapu <srivasam(a)codeaurora.org> --- sound/soc/qcom/lpass-cpu.c | 8 ++++---- sound/soc/qcom/lpass-sc7180.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/sound/soc/qcom/lpass-cpu.c b/sound/soc/qcom/lpass-cpu.c index a669202e0001..c642e5f8f28c 100644 --- a/sound/soc/qcom/lpass-cpu.c +++ b/sound/soc/qcom/lpass-cpu.c @@ -595,7 +595,7 @@ static bool lpass_hdmi_regmap_writeable(struct device *dev, unsigned int reg) return true; } - for (i = 0; i < v->rdma_channels; ++i) { + for (i = 0; i < v->hdmi_rdma_channels; ++i) { if (reg == LPAIF_HDMI_RDMACTL_REG(v, i)) return true; if (reg == LPAIF_HDMI_RDMABASE_REG(v, i)) @@ -641,7 +641,7 @@ static bool lpass_hdmi_regmap_readable(struct device *dev, unsigned int reg) if (reg == LPASS_HDMITX_APP_IRQSTAT_REG(v)) return true; - for (i = 0; i < v->rdma_channels; ++i) { + for (i = 0; i < v->hdmi_rdma_channels; ++i) { if (reg == LPAIF_HDMI_RDMACTL_REG(v, i)) return true; if (reg == LPAIF_HDMI_RDMABASE_REG(v, i)) @@ -668,7 +668,7 @@ static bool lpass_hdmi_regmap_volatile(struct device *dev, unsigned int reg) if (reg == LPASS_HDMI_TX_LEGACY_ADDR(v)) return true; - for (i = 0; i < v->rdma_channels; ++i) { + for (i = 0; i < v->hdmi_rdma_channels; ++i) { if (reg == LPAIF_HDMI_RDMACURR_REG(v, i)) return true; } @@ -812,7 +812,7 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev) return PTR_ERR(drvdata->hdmiif); lpass_hdmi_regmap_config.max_register = LPAIF_HDMI_RDMAPER_REG(variant, - variant->hdmi_rdma_channels); + variant->hdmi_rdma_channels - 1); drvdata->hdmiif_map = devm_regmap_init_mmio(dev, drvdata->hdmiif, &lpass_hdmi_regmap_config); if (IS_ERR(drvdata->hdmiif_map)) { diff --git a/sound/soc/qcom/lpass-sc7180.c b/sound/soc/qcom/lpass-sc7180.c index 735c9dac28f2..8c168d3c589e 100644 --- a/sound/soc/qcom/lpass-sc7180.c +++ b/sound/soc/qcom/lpass-sc7180.c @@ -171,7 +171,7 @@ static struct lpass_variant sc7180_data = { .rdma_channels = 5, .hdmi_rdma_reg_base = 0x64000, .hdmi_rdma_reg_stride = 0x1000, - .hdmi_rdma_channels = 3, + .hdmi_rdma_channels = 4, .dmactl_audif_start = 1, .wrdma_reg_base = 0x18000, .wrdma_reg_stride = 0x1000, -- Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc., is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.

3 2

[PATCH 0/6] soundwire: qcom: various improvements
by Srinivas Kandagatla 02 Feb '21

02 Feb '21

During testing SoundWire controller on SM8250 MTP, we found few issues like all the interrupts are not handled, all transport parameters are not read from device tree. Other major issue was register read/writes which was interrupt based was an overhead and puts lot of limitation on context it can be used from. So this patchset add various improvements to the existing driver to address above issues. Tested it on SM8250 MTP with 2x WSA881x speakers, HeadPhones on WCD938x via lpass-rx-macro and Analog MICs via lpass-tx-macro. Also tested on DragonBoard DB845c with 2xWSA881x speakers. Srinivas Kandagatla (6): soundwire: qcom: add support to missing transport params soundwire: qcom: extract version field soundwire: qcom: set continue execution flag for ignored commands soundwire: qcom: start the clock during initialization soundwire: qcom: update register read/write routine soundwire: qcom: add support to new interrupts drivers/soundwire/qcom.c | 471 ++++++++++++++++++++++++++++++--------- 1 file changed, 366 insertions(+), 105 deletions(-) -- 2.21.0

3 29

hdspmixer: Hardware output loopback toggle buttons
by GitHub pull_request - edited 02 Feb '21

02 Feb '21

alsa-project/alsa-tools pull request #4 was edited from pfandl: Hi, as promised in the [sound/pci/rme9652 patch](https://github.com/tiwai/sound/commit/da2a040ee7cfe1dd57d5bec7906cb9… here are the user land changes which provide the user the ability to toggle the hardware output loopback functionality per output channel. I tried to hide the buttons for devices which do not expose the snd controls yet, but due to probably unnecessary struggle i cancelled that. Please let me know though should you want them removed for not yet enabled devices. Request URL : https://github.com/alsa-project/alsa-tools/pull/4 Patch URL : https://github.com/alsa-project/alsa-tools/pull/4.patch Repository URL: https://github.com/alsa-project/alsa-tools

1 0