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

[PATCH v5 0/1] ASoC: dpcm: acquire dpcm_lock in dpcm_do_trigger()
by Gyeongtaek Lee 03 Mar '21

03 Mar '21

On Wed, 2021-02-17 16:29, Takashi Iwai wrote: >On Wed, 17 Feb 2021 05:31:49 +0100, >Gyeongtaek Lee wrote: >> >> If stop by underrun and DPCM BE disconnection is run simultaneously, >> data abort can be occurred by the sequence below. >> >> CPU0 CPU1 >> dpcm_be_dai_trigger(): dpcm_be_disconnect(): >> >> for_each_dpcm_be(fe, stream, dpcm) { >> >> spin_lock_irqsave(&fe->card->dpcm_lock, flags); >> list_del(&dpcm->list_be); >> list_del(&dpcm->list_fe); >> spin_unlock_irqrestore(&fe->card->dpcm_lock, flags); >> kfree(dpcm); >> >> struct snd_soc_pcm_runtime *be = dpcm->be; <-- Accessing freed memory >> >> To prevent this situation, dpcm_lock should be acquired during >> iteration of dpcm list in dpcm_be_dai_trigger(). > >I don't think we can apply spin lock there blindly. There is >non-atomic PCM that must not take a spin lock there, too. Hi, Takashi I fixed the patch to hold dpcm_lock during accessing dpcm related data only, not on the trigger callback of driver. Thank you. Gyeongtaek Lee > > >thanks, > >Takashi > Gyeongtaek Lee (1): ASoC: dpcm: acquire dpcm_lock in dpcm_do_trigger() include/sound/soc-dpcm.h | 5 ++++ sound/soc/soc-pcm.c | 59 +++++++++++++++++++++++++++++++++------- 2 files changed, 54 insertions(+), 10 deletions(-) base-commit: fe07bfda2fb9cdef8a4d4008a409bb02f35f1bd8 -- 2.21.0

1 0

Re: [PATCH] ASoC: wm8960: Remove bitclk relax condition
by S.j. Wang 03 Mar '21

03 Mar '21

> > > > Using a higher bitclk then expected doesn't always work. > > Here is an example: > > > > aplay -Dhw:0,0 -d 5 -r 48000 -f S24_LE -c 2 audio48k24b2c.wav > > > > In this case, the required bitclk is 48000 * 24 * 2 = 2304000 but the > > closest bitclk that can be derived is 3072000. Since the clock is > > faster than expected, it will start to send bytes from the next > > channel so the sound will be corrupted. > > > > Fixes: 82bab88910ee ("ASoC: codec: wm8960: Relax bit clock computation > > when using PLL") > > Fixes: 3c01b9ee2ab9 ("ASoC: codec: wm8960: Relax bit clock > > computation") > > Signed-off-by: Daniel Baluta <daniel.baluta(a)nxp.com> > > Signed-off-by: Shengjiu Wang <shengjiu.wang(a)nxp.com> > > --- > > I think this is probably going to need a much more involved fix. > The problem is that there are systems that depend on this behaviour, so you > can't just flat out revert it. And to be fair the I2S specification says that bit > clock can run at a higher rate than required for the data, so the behaviour is > correct as well. > > Probably the best solution here is to add additional contraints from the > machine driver on which rates/bit depths/channels are supported. > Just double check the issue, the real reason is in below: The call sequence in wm8960_configure_clocking is ret = wm8960_configure_sysclk(); if (ret >= 0) goto configure_clock; .... ret = wm8960_configure_pll(); configure_clock: ... wm8960_configure_sysclk is called before wm8960_configure_pll, as there is bitclk relax on both functions, so wm8960_configure_sysclk always return success, then wm8960_configure_pll() never be called. So bitclk relax condition should be removed in wm8960_configure_sysclk, That wm8960_configure_pll can be called, and there is also bitclk relax function in wm8960_configure_pll. I will update and resend this patch. Best regards Wang shengjiu

1 0

Fiio M3K: Failed to find a working profile
by GitHub issues - edited 03 Mar '21

03 Mar '21

alsa-project/alsa-lib issue #123 was edited from x1unix: Hello, I have a [Fiio M3K](https://fiio.com/m3k) player which I often use as an external USB DAC. It worked well with Pipewire, but I had some other issues with Pipewire so I had to rollback to PulseAudio. Unfortunately, PulseAudio and module-alsa-card cannot setup this device, so it doesn't work. Here is a small portion of PulseAudio log (full log is available at link below): ``` E: [pulseaudio] module-alsa-card.c: Failed to find a working profile. E: [pulseaudio] module.c: Failed to load module "module-alsa-card" (argument: "device_id="2" name="usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00" card_name="alsa_card.usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00" namereg_fail=false tsched=yes fixed_latency_range=no ignore_dB=no deferred_volume=yes use_ucm=yes avoid_resampling=no card_properties="module-udev-detect.discovered=1""): initialization failed. I: [pulseaudio] module-udev-detect.c: Card /devices/pci0000:00/0000:00:08.1/0000:0b:00.3/usb5/5-1/5-1:1.0/sound/card2 (alsa_card.usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00) failed to load module. D: [pulseaudio] module-udev-detect.c: /dev/snd/controlC2 is accessible: yes D: [pulseaudio] module-udev-detect.c: /devices/pci0000:00/0000:00:08.1/0000:0b:00.3/usb5/5-1/5-1:1.0/sound/card2 is busy: no ``` I'm using ArchLinux with Linux 5.11.2 and PulseAudio 14.2. **Logs and debug info** USB device in `lsusb` output: ``` Bus 005 Device 022: ID 2972:0044 FiiO Electronics Technology FiiO M3K ``` aplay, pulseaudio debug log and other info I've posted as gist: https://gist.github.com/x1unix/dae3fb810ac499cc63362a805220968f Issue URL : https://github.com/alsa-project/alsa-lib/issues/123 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

Fiio M3K: Failed to find a working profile
by GitHub issues - edited 03 Mar '21

03 Mar '21

alsa-project/alsa-lib issue #123 was edited from x1unix: Hello, I have a [Fiio M3K](https://fiio.com/m3k) player which I often use as an external USB DAC. It worked well with Pipewire, but I had some other issues with Pipewire so I had to rollback to PulseAudio. Unfortunately, PulseAudio and module-alsa-card cannot setup this device, so it doesn't work. Here is a small portion of PulseAudio log (full log is available at link below): ``` E: [pulseaudio] module-alsa-card.c: Failed to find a working profile. E: [pulseaudio] module.c: Failed to load module "module-alsa-card" (argument: "device_id="2" name="usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00" card_name="alsa_card.usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00" namereg_fail=false tsched=yes fixed_latency_range=no ignore_dB=no deferred_volume=yes use_ucm=yes avoid_resampling=no card_properties="module-udev-detect.discovered=1""): initialization failed. I: [pulseaudio] module-udev-detect.c: Card /devices/pci0000:00/0000:00:08.1/0000:0b:00.3/usb5/5-1/5-1:1.0/sound/card2 (alsa_card.usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00) failed to load module. D: [pulseaudio] module-udev-detect.c: /dev/snd/controlC2 is accessible: yes D: [pulseaudio] module-udev-detect.c: /devices/pci0000:00/0000:00:08.1/0000:0b:00.3/usb5/5-1/5-1:1.0/sound/card2 is busy: no ``` I'm using ArchLinux with Linux 5.11.2 and PulseAudio 14.2. **Logs and debug info** aplay, pulseaudio debug log and other info I've posted as gist: https://gist.github.com/x1unix/dae3fb810ac499cc63362a805220968f Issue URL : https://github.com/alsa-project/alsa-lib/issues/123 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

Fiio M3K: Failed to find a working profile
by GitHub issues - edited 03 Mar '21

03 Mar '21

alsa-project/alsa-lib issue #123 was edited from x1unix: Hello, I have a [Fiio M3K](https://fiio.com/m3k) player which I often use as an external USB DAC. It worked well with Pipewire, but I had some other issues with Pipewire so I had to rollback to PulseAudio. Unfortunately, PulseAudio and module-alsa-card cannot setup this device, so it doesn't work. Here is a small portion of PulseAudio log (full log is available at link below): ``` E: [pulseaudio] module-alsa-card.c: Failed to find a working profile. E: [pulseaudio] module.c: Failed to load module "module-alsa-card" (argument: "device_id="2" name="usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00" card_name="alsa_card.usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00" namereg_fail=false tsched=yes fixed_latency_range=no ignore_dB=no deferred_volume=yes use_ucm=yes avoid_resampling=no card_properties="module-udev-detect.discovered=1""): initialization failed. I: [pulseaudio] module-udev-detect.c: Card /devices/pci0000:00/0000:00:08.1/0000:0b:00.3/usb5/5-1/5-1:1.0/sound/card2 (alsa_card.usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00) failed to load module. D: [pulseaudio] module-udev-detect.c: /dev/snd/controlC2 is accessible: yes D: [pulseaudio] module-udev-detect.c: /devices/pci0000:00/0000:00:08.1/0000:0b:00.3/usb5/5-1/5-1:1.0/sound/card2 is busy: no ``` I'm using ArchLinux and Linux 5.11.2 and PulseAudio 14.2. **Logs and debug info** aplay, pulseaudio debug log and other info I've posted as gist: https://gist.github.com/x1unix/dae3fb810ac499cc63362a805220968f Issue URL : https://github.com/alsa-project/alsa-lib/issues/123 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

Fiio M3K: Failed to find a working profile
by GitHub issues - opened 03 Mar '21

03 Mar '21

alsa-project/alsa-lib issue #123 was opened from x1unix: Hello, I have a [Fiio M3K](https://fiio.com/m3k) player which I often use as an external USB DAC. It worked well with Pipewire, but I had some other issues with Pipewire so I had to rollback to PulseAudio. Unfortunately, PulseAudio and module-alsa-card cannot setup this device, so it doesn't work. Here is a small portion of PulseAudio log (full log is available at link below): ``` E: [pulseaudio] module-alsa-card.c: Failed to find a working profile. E: [pulseaudio] module.c: Failed to load module "module-alsa-card" (argument: "device_id="2" name="usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00" card_name="alsa_card.usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00" namereg_fail=false tsched=yes fixed_latency_range=no ignore_dB=no deferred_volume=yes use_ucm=yes avoid_resampling=no card_properties="module-udev-detect.discovered=1""): initialization failed. I: [pulseaudio] module-udev-detect.c: Card /devices/pci0000:00/0000:00:08.1/0000:0b:00.3/usb5/5-1/5-1:1.0/sound/card2 (alsa_card.usb-GuangZhou_FiiO_Electronics_Technology_Co.__Ltd._FiiO_M3K_FiiO_M3K-00) failed to load module. D: [pulseaudio] module-udev-detect.c: /dev/snd/controlC2 is accessible: yes D: [pulseaudio] module-udev-detect.c: /devices/pci0000:00/0000:00:08.1/0000:0b:00.3/usb5/5-1/5-1:1.0/sound/card2 is busy: no ``` I'm using ArchLinux and Linux 5.11.2. **Logs and debug info** aplay, pulseaudio debug log and other info I've posted as gist: https://gist.github.com/x1unix/dae3fb810ac499cc63362a805220968f Issue URL : https://github.com/alsa-project/alsa-lib/issues/123 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[PATCH 00/15] Report jack and button detection + Capture Support
by Lucas Tanure 03 Mar '21

03 Mar '21

Hi All, Here is a patch series for reporting to user space jack and button events and add the support for Capture. With some cleanups and fixes along the way. Regards, Lucas Tanure Lucas Tanure (12): ASoC: cs42l42: Don't enable/disable regulator at Bias Level ASoC: cs42l42: Always wait at least 3ms after reset ASoC: cs42l42: Remove power if the driver is being removed ASoC: cs42l42: Disable regulators if probe fails ASoC: cs42l42: Fix Bitclock polarity inversion ASoC: cs42l42: Provide finer control on playback path ASoC: cs42l42: Set clock source for both ways of stream ASoC: cs42l42: Fix channel width support ASoC: cs42l42: Add Capture Support ASoC: cs42l42: Report jack and button detection ASoC: cs42l42: Use bclk from hw_params if set_sysclk was not called ASoC: cs42l42: Fix mixer volume control Richard Fitzgerald (3): ASoC: cs42l42: Wait at least 150us after writing SCLK_PRESENT ASoC: cs42l42: Only start PLL if it is needed ASoC: cs42l42: Wait for PLL to lock before switching to it sound/soc/codecs/cs42l42.c | 497 +++++++++++++++++++++---------------- sound/soc/codecs/cs42l42.h | 42 +++- 2 files changed, 317 insertions(+), 222 deletions(-) -- 2.30.1

3 24

[PATCH] ALSA: hda/realtek: Add quirk for Intel NUC 10
by Werner Sembach 02 Mar '21

02 Mar '21

ALSA: hda/realtek: Add quirk for Intel NUC 10 This adds a new SND_PCI_QUIRK(...) and applies it to the Intel NUC 10 devices. This fixes the issue of the devices not having audio input and output on the headset jack because the kernel does not recognize when something is plugged in. The new quirk was inspired by the quirk for the Intel NUC 8 devices, but it turned out that the NUC 10 uses another pin. This information was acquired by black box testing likely pins. Co-developed-by: Eckhart Mohr <e.mohr(a)tuxedocomputers.com> Signed-off-by: Eckhart Mohr <e.mohr(a)tuxedocomputers.com> Signed-off-by: Werner Sembach <wse(a)tuxedocomputers.com> Cc: <stable(a)vger.kernel.org> --- Forgot to add the "From"-line >From d281364b8ca6c054a0e5ce20caa599bf7d08160d Mon Sep 17 00:00:00 2001 From: Werner Sembach <wse(a)tuxedocomputers.com> Date: Fri, 26 Feb 2021 13:54:30 +0100 Subject: [PATCH] Fix Intel NUC10 no output and input on headset jack --- sound/pci/hda/patch_realtek.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 290645516313..c14d624dbaf1 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -6362,6 +6362,7 @@ enum { ALC269_FIXUP_LEMOTE_A1802, ALC269_FIXUP_LEMOTE_A190X, ALC256_FIXUP_INTEL_NUC8_RUGGED, + ALC256_FIXUP_INTEL_NUC10, ALC255_FIXUP_XIAOMI_HEADSET_MIC, ALC274_FIXUP_HP_MIC, ALC274_FIXUP_HP_HEADSET_MIC, @@ -7744,6 +7745,15 @@ static const struct hda_fixup alc269_fixups[] = { .chained = true, .chain_id = ALC269_FIXUP_HEADSET_MODE }, + [ALC256_FIXUP_INTEL_NUC10] = { + .type = HDA_FIXUP_PINS, + .v.pins = (const struct hda_pintbl[]) { + { 0x19, 0x01a1913c }, /* use as headset mic, without its own jack detect */ + { } + }, + .chained = true, + .chain_id = ALC269_FIXUP_HEADSET_MODE + }, [ALC255_FIXUP_XIAOMI_HEADSET_MIC] = { .type = HDA_FIXUP_VERBS, .v.verbs = (const struct hda_verb[]) { @@ -8183,6 +8193,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x1c06, 0x2013, "Lemote A1802", ALC269_FIXUP_LEMOTE_A1802), SND_PCI_QUIRK(0x1c06, 0x2015, "Lemote A190X", ALC269_FIXUP_LEMOTE_A190X), SND_PCI_QUIRK(0x8086, 0x2080, "Intel NUC 8 Rugged", ALC256_FIXUP_INTEL_NUC8_RUGGED), + SND_PCI_QUIRK(0x8086, 0x2081, "Intel NUC 10", ALC256_FIXUP_INTEL_NUC10), #if 0 /* Below is a quirk table taken from the old code. -- 2.25.1

1 0

[PATCH] ALSA: hda/realtek: Add quirk for Clevo NH55RZQ
by Werner Sembach 02 Mar '21

02 Mar '21

From: Eckhart Mohr <e.mohr(a)tuxedocomputers.com> ALSA: hda/realtek: Add quirk for Clevo NH55RZQ This applies a SND_PCI_QUIRK(...) to the Clevo NH55RZQ barebone. This fixes the issue of the device not recognizing a pluged in microphone. The device has both, a microphone only jack, and a speaker + microphone combo jack. The combo jack already works. The microphone-only jack does not recognize when a device is pluged in without this patch. Signed-off-by: Eckhart Mohr <e.mohr(a)tuxedocomputers.com> Co-developed-by: Werner Sembach <wse(a)tuxedocomputers.com> Signed-off-by: Werner Sembach <wse(a)tuxedocomputers.com> --- This is a resend of the patch because I missed that the editor I used to write the commit message replaced tabs with spaces. >From 2835edd753fd19c72a644dccb7e941cfc0ecdf8e Mon Sep 17 00:00:00 2001 From: Werner Sembach <wse(a)tuxedocomputers.com> Date: Fri, 26 Feb 2021 13:50:15 +0100 Subject: [PATCH] Fix device detection on microphone jack of Clevo NH55RZQ --- sound/pci/hda/patch_realtek.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 290645516313..8014e80d72c3 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -8089,6 +8089,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x1558, 0x8551, "System76 Gazelle (gaze14)", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE), SND_PCI_QUIRK(0x1558, 0x8560, "System76 Gazelle (gaze14)", ALC269_FIXUP_HEADSET_MIC), SND_PCI_QUIRK(0x1558, 0x8561, "System76 Gazelle (gaze14)", ALC269_FIXUP_HEADSET_MIC), + SND_PCI_QUIRK(0x1558, 0x8562, "Clevo NH[5|7][0-9]RZ[Q]", ALC269_FIXUP_DMIC), SND_PCI_QUIRK(0x1558, 0x8668, "Clevo NP50B[BE]", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE), SND_PCI_QUIRK(0x1558, 0x8680, "Clevo NJ50LU", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE), SND_PCI_QUIRK(0x1558, 0x8686, "Clevo NH50[CZ]U", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE), -- 2.25.1

2 2

[PATCH] ALSA: hda/realtek: Add quirk for Clevo NH55RZQ
by Werner Sembach 02 Mar '21

02 Mar '21

From: Eckhart Mohr <e.mohr(a)tuxedocomputers.com> ALSA: hda/realtek: Add quirk for Clevo NH55RZQ This applies a SND_PCI_QUIRK(...) to the Clevo NH55RZQ barebone. This fixes the issue of the device not recognizing a pluged in microphone. The device has both, a microphone only jack, and a speaker + microphone combo jack. The combo jack already works. The microphone-only jack does not recognize when a device is pluged in without this patch. Signed-off-by: Eckhart Mohr <e.mohr(a)tuxedocomputers.com> Co-developed-by: Werner Sembach <wse(a)tuxedocomputers.com> Signed-off-by: Werner Sembach <wse(a)tuxedocomputers.com> Cc: <stable(a)vger.kernel.org> --- Third time's the charm, now using git send-email, I'm really sorry for the spam. >From 2835edd753fd19c72a644dccb7e941cfc0ecdf8e Mon Sep 17 00:00:00 2001 From: Werner Sembach <wse(a)tuxedocomputers.com> Date: Fri, 26 Feb 2021 13:50:15 +0100 Subject: [PATCH] Fix device detection on microphone jack of Clevo NH55RZQ --- sound/pci/hda/patch_realtek.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 290645516313..8014e80d72c3 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -8089,6 +8089,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x1558, 0x8551, "System76 Gazelle (gaze14)", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE), SND_PCI_QUIRK(0x1558, 0x8560, "System76 Gazelle (gaze14)", ALC269_FIXUP_HEADSET_MIC), SND_PCI_QUIRK(0x1558, 0x8561, "System76 Gazelle (gaze14)", ALC269_FIXUP_HEADSET_MIC), + SND_PCI_QUIRK(0x1558, 0x8562, "Clevo NH[5|7][0-9]RZ[Q]", ALC269_FIXUP_DMIC), SND_PCI_QUIRK(0x1558, 0x8668, "Clevo NP50B[BE]", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE), SND_PCI_QUIRK(0x1558, 0x8680, "Clevo NJ50LU", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE), SND_PCI_QUIRK(0x1558, 0x8686, "Clevo NH50[CZ]U", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE), -- 2.25.1

1 0