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

[PATCH] ALSA: usb-audio: add the profile name of Dell desktop
by shumingf＠realtek.com 11 Nov '21

11 Nov '21

From: Shuming Fan <shumingf(a)realtek.com> Add the profile name to let userspace pick correct UCM profile Signed-off-by: Shuming Fan <shumingf(a)realtek.com> --- sound/usb/card.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/sound/usb/card.c b/sound/usb/card.c index 1764b9302d46..9c67ecbe710e 100644 --- a/sound/usb/card.c +++ b/sound/usb/card.c @@ -398,6 +398,10 @@ static const struct usb_audio_device_name usb_audio_names[] = { PROFILE_NAME(0x0bda, 0x4014, "Dell", "WD15 Dock", "Dell-WD15-Dock"), /* Dell WD19 Dock */ PROFILE_NAME(0x0bda, 0x402e, "Dell", "WD19 Dock", "Dell-WD15-Dock"), + /* Dell Desktop - Front */ + PROFILE_NAME(0x0bda, 0x4c54, "Dell", "Desktop", "Dell-Desktop-Front"), + /* Dell Desktop - Rear */ + PROFILE_NAME(0x0bda, 0x4c55, "Dell", "Desktop", "Dell-Desktop-Rear"), DEVICE_NAME(0x0ccd, 0x0028, "TerraTec", "Aureon5.1MkII"), -- 2.33.0

5 8

[PATCH] ALSA: hda: Free card instance properly at probe errors
by Takashi Iwai 10 Nov '21

10 Nov '21

The recent change in hda-intel driver to allow repeated probes surfaced a problem that has been hidden until; the probe process in the work calls azx_free() at the error path, and this skips the card free process that eventually releases codec instances. As a result, we get a kernel WARNING like: snd_hda_intel 0000:00:1f.3: Cannot probe codecs, giving up ------------[ cut here ]------------ WARNING: CPU: 14 PID: 186 at sound/hda/hdac_bus.c:73 .... For fixing this, we need to call snd_card_free() instead of azx_free(). Additionally, the device drvdata has to be cleared, as the driver binding itself is still active. Then the PM and other driver callbacks will ignore the procedure. Fixes: c0f1886de7e1 ("ALSA: hda: intel: Allow repeatedly probing on codec configuration errors") Reported-and-tested-by: Scott Branden <scott.branden(a)broadcom.com> Cc: <stable(a)vger.kernel.org> Link: https://lore.kernel.org/r/063e2397-7edb-5f48-7b0d-618b938d9dd8@broadcom.com Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/pci/hda/hda_intel.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index 7762718cf429..90e9263ac0bd 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -2327,7 +2327,8 @@ static int azx_probe_continue(struct azx *chip) out_free: if (err < 0) { - azx_free(chip); + pci_set_drvdata(pci, NULL); + snd_card_free(chip->card); return err; } -- 2.26.2

1 0

5.15 Linux Regression in sound hda
by Scott Branden 10 Nov '21

10 Nov '21

I'm reporting a new WARN_ON in sound/hda/hdac_bus.c that was not present in 5.15. Sorry, I don't have time to bisect this right now but report is below. dmesg in 5.14 kernel was: snd_hda_intel 0000:00:1f.3: bound 0000:00:02.0 (ops 0xffffffffaa2ac520) GACT probability on Mirror/redirect action on Simple TC action Loaded u32 classifier Performance counters on input device check on Actions configured hdaudio hdaudioC0D0: Unable to bind the codec hdaudio hdaudioC0D2: Unable to bind the codec dmesg in 5.15 kernel produces WARN_ON in sound/hda/hdac_bus.c snd_hda_intel 0000:00:1f.3: bound 0000:00:02.0 (ops 0xffffffffad8b02a0) GACT probability on Mirror/redirect action on Simple TC action Loaded u32 classifier Performance counters on input device check on Actions configured snd_hda_intel 0000:00:1f.3: Cannot probe codecs, giving up Initializing XFRM netlink socket ------------[ cut here ]------------ NET: Registered PF_INET6 protocol family WARNING: CPU: 14 PID: 186 at sound/hda/hdac_bus.c:73 snd_hdac_bus_exit+0x40/0x50 Segment Routing with IPv6 Modules linked in: CPU: 14 PID: 186 Comm: kworker/14:2 Tainted: G I 5.15.0 #1 Hardware name: ASUSTeK COMPUTER INC. System Product Name/WS C246 PRO, BIOS 1401 08/13/2020 In-situ OAM (IOAM) with IPv6 Workqueue: events azx_probe_work sit: IPv6, IPv4 and MPLS over IPv4 tunneling driver RIP: 0010:snd_hdac_bus_exit+0x40/0x50 NET: Registered PF_PACKET protocol family Code: 75 19 48 8b 57 58 48 8d 47 58 48 39 c2 75 1b 48 81 c7 f8 02 00 00 e9 ff 97 57 ff 0f 0b 48 8b 57 58 48 8d 47 58 48 39 c2 74 e5 <0f> 0b 48 81 c7 f8 02 00 00 e9 e2 97 57 ff 66 90 0f 1f 44 00 00 41 RPC: Registered rdma transport module. RSP: 0000:ffffa545007b7e88 EFLAGS: 00010202 RPC: Registered rdma backchannel transport module. Key type dns_resolver registered RAX: ffff89f483078880 RBX: ffff89f483078828 RCX: 0000000080200010 RDX: ffff89f4830792f0 RSI: 0000000000000001 RDI: ffff89f483078828 RBP: ffff89f483078e70 R08: 0000000000000000 R09: ffffffffacf094a4 R10: 0000000000000004 R11: 0000000000000032 R12: ffff89f7dbda50c0 R13: ffff89f7dbda9100 R14: 0000000000000000 R15: ffff89f7dbda9105 FS: 0000000000000000(0000) GS:ffff89f7dbd80000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 CR2: 0000000000000000 CR3: 0000000194a0c001 CR4: 00000000003706e0 DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000 DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400 Call Trace: azx_free+0xba/0x170 process_one_work+0x1db/0x380 worker_thread+0x4d/0x3e0 ? rescuer_thread+0x370/0x370 kthread+0x124/0x150 ? set_kthread_struct+0x40/0x40 ret_from_fork+0x1f/0x30 ---[ end trace 9779b9e2ef53104c ]---

2 3

[PATCH] ALSA: hda/realtek: Add quirk for HP EliteBook 840 G7 mute LED
by Kai-Heng Feng 10 Nov '21

10 Nov '21

The mute and micmute LEDs don't work on HP EliteBook 840 G7. The same quirk for other HP laptops can let LEDs work, so apply it. Signed-off-by: Kai-Heng Feng <kai.heng.feng(a)canonical.com> --- 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 a6098c942aeb5..2f1727faec698 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -8636,6 +8636,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x103c, 0x8716, "HP Elite Dragonfly G2 Notebook PC", ALC285_FIXUP_HP_GPIO_AMP_INIT), SND_PCI_QUIRK(0x103c, 0x8720, "HP EliteBook x360 1040 G8 Notebook PC", ALC285_FIXUP_HP_GPIO_AMP_INIT), SND_PCI_QUIRK(0x103c, 0x8724, "HP EliteBook 850 G7", ALC285_FIXUP_HP_GPIO_LED), + SND_PCI_QUIRK(0x103c, 0x8728, "HP EliteBook 840 G7", ALC285_FIXUP_HP_GPIO_LED), SND_PCI_QUIRK(0x103c, 0x8729, "HP", ALC285_FIXUP_HP_GPIO_LED), SND_PCI_QUIRK(0x103c, 0x8730, "HP ProBook 445 G7", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF), SND_PCI_QUIRK(0x103c, 0x8736, "HP", ALC285_FIXUP_HP_GPIO_AMP_INIT), -- 2.32.0

2 1

ucm2: Add support for SC7180 Trogdor Lazor Chromebooks
by GitHub pull_request - edited 10 Nov '21

10 Nov '21

alsa-project/alsa-ucm-conf pull request #118 was edited from kholk: Almost all Trogdor Chromebooks are using RT5682 for headphones/headset on 3.5mm jack, internal microphone and headset microphone, and MAX98357A for the internal speakers (Lazor and Limozeen). This adds support for the sc7180-rt5682-max98357a-1mic sound card found on the aforementioned Chromebooks. `Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno(a)collabora.com>` Tested on sc7180-trogdor-lazor-limozeen-nots. Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/118 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/118.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

ucm2: Add support for SC7180 Trogdor Lazor Chromebooks
by GitHub pull_request - opened 10 Nov '21

10 Nov '21

alsa-project/alsa-ucm-conf pull request #118 was opened from kholk: Almost all Trogdor Chromebooks are using RT5682 for headphones/headset on 3.5mm jack, internal microphone and headset microphone, and MAX98357A for the internal speakers (Lazor and Limozeen). This adds support for the sc7180-rt5682-max98357a-1mic sound card found on the aforementioned Chromebooks. Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno(a)collabora.com> Tested on sc7180-trogdor-lazor-limozeen-nots. Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/118 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/118.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[PATCH] ALSA: memalloc: Remove a stale comment
by Takashi Iwai 10 Nov '21

10 Nov '21

The comment about the reused vmalloc helpers is no longer valid after the recent change for the noncontig allocator. Drop the stale comment. Fixes: ad4f93ca4138 ("ALSA: memalloc: Use proper SG helpers for noncontig allocations") Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/core/memalloc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/core/memalloc.c b/sound/core/memalloc.c index 8c362bb5b81a..9fc971a704a9 100644 --- a/sound/core/memalloc.c +++ b/sound/core/memalloc.c @@ -615,7 +615,6 @@ static const struct snd_malloc_ops snd_dma_noncontig_ops = { .free = snd_dma_noncontig_free, .mmap = snd_dma_noncontig_mmap, .sync = snd_dma_noncontig_sync, - /* re-use vmalloc helpers for get_* ops */ .get_addr = snd_dma_noncontig_get_addr, .get_page = snd_dma_noncontig_get_page, .get_chunk_size = snd_dma_noncontig_get_chunk_size, -- 2.26.2

1 0

[PATCH AUTOSEL 4.4 11/12] ALSA: gus: fix null pointer dereference on pointer block
by Sasha Levin 09 Nov '21

09 Nov '21

From: Chengfeng Ye <cyeaa(a)connect.ust.hk> [ Upstream commit a0d21bb3279476c777434c40d969ea88ca64f9aa ] The pointer block return from snd_gf1_dma_next_block could be null, so there is a potential null pointer dereference issue. Fix this by adding a null check before dereference. Signed-off-by: Chengfeng Ye <cyeaa(a)connect.ust.hk> Link: https://lore.kernel.org/r/20211024104611.9919-1-cyeaa@connect.ust.hk Signed-off-by: Takashi Iwai <tiwai(a)suse.de> Signed-off-by: Sasha Levin <sashal(a)kernel.org> --- sound/isa/gus/gus_dma.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/isa/gus/gus_dma.c b/sound/isa/gus/gus_dma.c index 36c27c8323601..2e27cd3427c87 100644 --- a/sound/isa/gus/gus_dma.c +++ b/sound/isa/gus/gus_dma.c @@ -141,6 +141,8 @@ static void snd_gf1_dma_interrupt(struct snd_gus_card * gus) } block = snd_gf1_dma_next_block(gus); spin_unlock(&gus->dma_lock); + if (!block) + return; snd_gf1_dma_program(gus, block->addr, block->buf_addr, block->count, (unsigned short) block->cmd); kfree(block); #if 0 -- 2.33.0

1 0

[PATCH AUTOSEL 4.9 12/13] ALSA: gus: fix null pointer dereference on pointer block
by Sasha Levin 09 Nov '21

09 Nov '21

From: Chengfeng Ye <cyeaa(a)connect.ust.hk> [ Upstream commit a0d21bb3279476c777434c40d969ea88ca64f9aa ] The pointer block return from snd_gf1_dma_next_block could be null, so there is a potential null pointer dereference issue. Fix this by adding a null check before dereference. Signed-off-by: Chengfeng Ye <cyeaa(a)connect.ust.hk> Link: https://lore.kernel.org/r/20211024104611.9919-1-cyeaa@connect.ust.hk Signed-off-by: Takashi Iwai <tiwai(a)suse.de> Signed-off-by: Sasha Levin <sashal(a)kernel.org> --- sound/isa/gus/gus_dma.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/isa/gus/gus_dma.c b/sound/isa/gus/gus_dma.c index 36c27c8323601..2e27cd3427c87 100644 --- a/sound/isa/gus/gus_dma.c +++ b/sound/isa/gus/gus_dma.c @@ -141,6 +141,8 @@ static void snd_gf1_dma_interrupt(struct snd_gus_card * gus) } block = snd_gf1_dma_next_block(gus); spin_unlock(&gus->dma_lock); + if (!block) + return; snd_gf1_dma_program(gus, block->addr, block->buf_addr, block->count, (unsigned short) block->cmd); kfree(block); #if 0 -- 2.33.0

1 0

[PATCH AUTOSEL 4.14 13/14] ALSA: gus: fix null pointer dereference on pointer block
by Sasha Levin 09 Nov '21

09 Nov '21

From: Chengfeng Ye <cyeaa(a)connect.ust.hk> [ Upstream commit a0d21bb3279476c777434c40d969ea88ca64f9aa ] The pointer block return from snd_gf1_dma_next_block could be null, so there is a potential null pointer dereference issue. Fix this by adding a null check before dereference. Signed-off-by: Chengfeng Ye <cyeaa(a)connect.ust.hk> Link: https://lore.kernel.org/r/20211024104611.9919-1-cyeaa@connect.ust.hk Signed-off-by: Takashi Iwai <tiwai(a)suse.de> Signed-off-by: Sasha Levin <sashal(a)kernel.org> --- sound/isa/gus/gus_dma.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/isa/gus/gus_dma.c b/sound/isa/gus/gus_dma.c index 36c27c8323601..2e27cd3427c87 100644 --- a/sound/isa/gus/gus_dma.c +++ b/sound/isa/gus/gus_dma.c @@ -141,6 +141,8 @@ static void snd_gf1_dma_interrupt(struct snd_gus_card * gus) } block = snd_gf1_dma_next_block(gus); spin_unlock(&gus->dma_lock); + if (!block) + return; snd_gf1_dma_program(gus, block->addr, block->buf_addr, block->count, (unsigned short) block->cmd); kfree(block); #if 0 -- 2.33.0

1 0