August 2019 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH v2] ALSA: hda/ca0132 - Add new SBZ quirk
by Paweł Rekowski 20 Aug '19

20 Aug '19

This patch adds a new PCI subsys ID for the SBZ, as found and tested by me and some reddit users. Signed-off-by: Paweł Rekowski <p.rekowski(a)gmail.com> --- sound/pci/hda/patch_ca0132.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/pci/hda/patch_ca0132.c b/sound/pci/hda/patch_ca0132.c index c3096796ee05..c41865e1222c 100644 --- a/sound/pci/hda/patch_ca0132.c +++ b/sound/pci/hda/patch_ca0132.c @@ -1175,6 +1175,7 @@ static const struct snd_pci_quirk ca0132_quirks[] = { SND_PCI_QUIRK(0x1028, 0x0708, "Alienware 15 R2 2016", QUIRK_ALIENWARE), SND_PCI_QUIRK(0x1102, 0x0010, "Sound Blaster Z", QUIRK_SBZ), SND_PCI_QUIRK(0x1102, 0x0023, "Sound Blaster Z", QUIRK_SBZ), + SND_PCI_QUIRK(0x1102, 0x0027, "Sound Blaster Z", QUIRK_SBZ), SND_PCI_QUIRK(0x1102, 0x0033, "Sound Blaster ZxR", QUIRK_SBZ), SND_PCI_QUIRK(0x1458, 0xA016, "Recon3Di", QUIRK_R3DI), SND_PCI_QUIRK(0x1458, 0xA026, "Gigabyte G1.Sniper Z97", QUIRK_R3DI), -- 2.23.0

2 1

[alsa-devel] [PATCH] ALSA: usb-audio: Add implicit fb quirk for Behringer UFX1604
by Takashi Iwai 20 Aug '19

20 Aug '19

Behringer UFX1604 requires the similar quirk to apply implicit fb like another Behringer model UFX1204 in order to fix the noisy playback. BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=204631 Cc: <stable(a)vger.kernel.org> Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/usb/pcm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/usb/pcm.c b/sound/usb/pcm.c index 75b96929f76c..e4bbf79de956 100644 --- a/sound/usb/pcm.c +++ b/sound/usb/pcm.c @@ -339,6 +339,7 @@ static int set_sync_ep_implicit_fb_quirk(struct snd_usb_substream *subs, ep = 0x81; ifnum = 2; goto add_sync_ep_from_ifnum; + case USB_ID(0x1397, 0x0001): /* Behringer UFX1604 */ case USB_ID(0x1397, 0x0002): /* Behringer UFX1204 */ ep = 0x81; ifnum = 1; -- 2.16.4

1 0

[alsa-devel] [PATCH 00/15] ASoC: use modern style for aux_dev
by Kuninori Morimoto 20 Aug '19

20 Aug '19

Hi Mark, aux_dev users I'm sorry to bother you. I know I'm posting soc-core.c cleanup patches, and these are still under reviewing, but I want to post next patches before my summer vacation. These patches support modern style for aux_dev. I think it should work, but I can't test, because my platform doesn't use aux_dev. Please test and review these. These exchange aux_dev "style" only, do nothing from "technical" point of view. Kuninori Morimoto (15): ASoC: soc-core: support snd_soc_dai_link_component for aux_dev ASoC: simple-card: use snd_soc_dai_link_component for aux_dev ASoC: intel: cht_bsw_max98090_ti: use snd_soc_dai_link_component for aux_dev ASoC: mediatek: mt8183-da7219-max98357: use snd_soc_dai_link_component for aux_dev ASoC: mediatek: mt8183-mt6358-ts3a227-max98357: use snd_soc_dai_link_component for aux_dev ASoC: meson: axg-card: use snd_soc_dai_link_component for aux_dev ASoC: rockchip: rockchip_max98090: use snd_soc_dai_link_component for aux_dev ASoC: samsung: neo1973_wm8753: use snd_soc_dai_link_component for aux_dev ASoC: samsung: speyside: use snd_soc_dai_link_component for aux_dev ASoC: samsung: tm2_wm5110: use snd_soc_dai_link_component for aux_dev ASoC: sunxi: sun4i-codec: use snd_soc_dai_link_component for aux_dev ASoC: ti: rx51: use snd_soc_dai_link_component for aux_dev ASoC: soc-core: remove legacy style of aux_dev ASoC: soc-core: add for_each_xxx macro for aux_dev ASoC: soc-core: add soc_unbind_aux_dev() include/sound/soc.h | 16 +++++-- sound/soc/generic/simple-card.c | 2 +- sound/soc/intel/boards/cht_bsw_max98090_ti.c | 3 +- sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c | 6 +-- .../mt8183/mt8183-mt6358-ts3a227-max98357.c | 6 +-- sound/soc/meson/axg-card.c | 11 ++--- sound/soc/rockchip/rockchip_max98090.c | 6 +-- sound/soc/samsung/neo1973_wm8753.c | 3 +- sound/soc/samsung/speyside.c | 3 +- sound/soc/samsung/tm2_wm5110.c | 10 ++--- sound/soc/soc-core.c | 52 ++++++++-------------- sound/soc/sunxi/sun4i-codec.c | 14 +++--- sound/soc/ti/rx51.c | 14 +++--- 13 files changed, 66 insertions(+), 80 deletions(-) -- 2.7.4

7 41

[alsa-devel] [PATCH] Fix an OOB bug in uac_mixer_unit_bmControls
by Hui Peng 20 Aug '19

20 Aug '19

`uac_mixer_unit_get_channels` calls `uac_mixer_unit_bmControls` to get pointer to bmControls field. The current implementation of `uac_mixer_unit_get_channels` does properly check the size of uac_mixer_unit_descriptor descriptor and may allow OOB access in `uac_mixer_unit_bmControls`. Reported-by: Hui Peng <benquike(a)gmail.com> Reported-by: Mathias Payer <mathias.payer(a)nebelwelt.net> Signed-off-by: Hui Peng <benquike(a)gmail.com> --- sound/usb/mixer.c | 25 ++++++++++++++++++------- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c index b5927c3d5bc0..00e6274a63c3 100644 --- a/sound/usb/mixer.c +++ b/sound/usb/mixer.c @@ -738,28 +738,39 @@ static int get_cluster_channels_v3(struct mixer_build *state, unsigned int clust static int uac_mixer_unit_get_channels(struct mixer_build *state, struct uac_mixer_unit_descriptor *desc) { - int mu_channels; + int mu_channels = 0; void *c; - if (desc->bLength < sizeof(*desc)) - return -EINVAL; if (!desc->bNrInPins) return -EINVAL; - if (desc->bLength < sizeof(*desc) + desc->bNrInPins) - return -EINVAL; switch (state->mixer->protocol) { case UAC_VERSION_1: + // limit derived from uac_mixer_unit_bmControls + if (desc->bLength < sizeof(*desc) + desc->bNrInPins + 4) + return 0; + + mu_channels = uac_mixer_unit_bNrChannels(desc); + break; + case UAC_VERSION_2: - default: - if (desc->bLength < sizeof(*desc) + desc->bNrInPins + 1) + // limit derived from uac_mixer_unit_bmControls + if (desc->bLength < sizeof(*desc) + desc->bNrInPins + 6) return 0; /* no bmControls -> skip */ + mu_channels = uac_mixer_unit_bNrChannels(desc); break; case UAC_VERSION_3: + // limit derived from uac_mixer_unit_bmControls + if (desc->bLength < sizeof(*desc) + desc->bNrInPins + 2) + return 0; /* no bmControls -> skip */ + mu_channels = get_cluster_channels_v3(state, uac3_mixer_unit_wClusterDescrID(desc)); break; + + default: + break; } if (!mu_channels) -- 2.22.1

1 1

[alsa-devel] [PATCH 0/3] ASoC: SOF: initial support for i.MX8
by Pierre-Louis Bossart 20 Aug '19

20 Aug '19

The reviews for these patches took place already on mailing lists and GitHub, and the code is already integrated in the SOF tree (along with dependencies already accepted in the i.MX tree) As agreed with Daniel, I am sending these patches as part of my weekly updates so that we avoid conflicts with other contributions (the next series adds Intel ALH/SoundWire suport which changes the same files) Daniel Baluta (3): ASoC: SOF: Add OF DSP device support ASoC: SOF: imx: Add i.MX8 HW support ASoC: SOF: topology: Add dummy support for i.MX8 DAIs include/sound/sof/dai.h | 2 + include/uapi/sound/sof/tokens.h | 8 + sound/soc/sof/Kconfig | 11 + sound/soc/sof/Makefile | 4 + sound/soc/sof/imx/Kconfig | 22 ++ sound/soc/sof/imx/Makefile | 4 + sound/soc/sof/imx/imx8.c | 394 ++++++++++++++++++++++++++++++++ sound/soc/sof/sof-of-dev.c | 143 ++++++++++++ sound/soc/sof/topology.c | 30 +++ 9 files changed, 618 insertions(+) create mode 100644 sound/soc/sof/imx/Kconfig create mode 100644 sound/soc/sof/imx/Makefile create mode 100644 sound/soc/sof/imx/imx8.c create mode 100644 sound/soc/sof/sof-of-dev.c -- 2.20.1

5 13

[alsa-devel] [PATCH] ALSA: hda - Fixes inverted Conexant GPIO mic mute led
by jeronimo＠borque.com.ar 19 Aug '19

19 Aug '19

From: Jeronimo Borque <jeronimo(a)borque.com.ar> "enabled" parameter historically referred to the device input or output, not to the led indicator. After the changes added with the led helper functions the mic mute led logic refers to the led and not to the mic input which caused led indicator to be negated (Mic mute led was on when the input enabled) Fixing it in the call to cxt_update_gpio_led at the cxt_gpio_micmute_update hook. Maybe more changes are required to be consistent everywhere. Signed-off-by: Jeronimo Borque <jeronimo(a)borque.com.ar> --- sound/pci/hda/patch_conexant.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/sound/pci/hda/patch_conexant.c b/sound/pci/hda/patch_conexant.c index f299f137eaea..8edf0d1290b5 100644 --- a/sound/pci/hda/patch_conexant.c +++ b/sound/pci/hda/patch_conexant.c @@ -636,6 +636,10 @@ static void cxt_update_gpio_led(struct hda_codec *codec, unsigned int mask, spec->gpio_led &= ~mask; else spec->gpio_led |= mask; + + codec_dbg(codec, "mask:%d enabled:%d gpio_led:%d\n", + mask, enabled, spec->gpio_led); + if (spec->gpio_led != oldval) snd_hda_codec_write(codec, 0x01, 0, AC_VERB_SET_GPIO_DATA, spec->gpio_led); @@ -656,7 +660,7 @@ static void cxt_gpio_micmute_update(struct hda_codec *codec) struct conexant_spec *spec = codec->spec; cxt_update_gpio_led(codec, spec->gpio_mic_led_mask, - spec->gen.micmute_led.led_value); + !spec->gen.micmute_led.led_value); } @@ -669,7 +673,6 @@ static void cxt_fixup_mute_led_gpio(struct hda_codec *codec, { 0x01, AC_VERB_SET_GPIO_DIRECTION, 0x03 }, {} }; - codec_info(codec, "action: %d gpio_led: %d\n", action, spec->gpio_led); if (action == HDA_FIXUP_ACT_PRE_PROBE) { spec->gen.vmaster_mute.hook = cxt_fixup_gpio_mute_hook; -- 2.21.0

3 9

[alsa-devel] [bug report] ASoC: SOF: topology: use set_get_data in process load
by Dan Carpenter 19 Aug '19

19 Aug '19

Hello Jaska Uimonen, The patch cac974a51ebb: "ASoC: SOF: topology: use set_get_data in process load" from Aug 9, 2019, leads to the following static checker warning: sound/soc/sof/topology.c:1857 sof_process_load() warn: 'ipc_data_size' unsigned <= 0 sound/soc/sof/topology.c 1825 static int sof_process_load(struct snd_soc_component *scomp, int index, 1826 struct snd_sof_widget *swidget, 1827 struct snd_soc_tplg_dapm_widget *tw, 1828 struct sof_ipc_comp_reply *r, 1829 int type) 1830 { 1831 struct snd_sof_dev *sdev = snd_soc_component_get_drvdata(scomp); 1832 struct snd_soc_dapm_widget *widget = swidget->widget; 1833 struct snd_soc_tplg_private *private = &tw->priv; 1834 struct sof_ipc_comp_process *process = NULL; 1835 struct sof_widget_data *wdata = NULL; 1836 size_t ipc_data_size = 0; ^^^^^^^^^^^^^^^^^^^^ Unsigned. 1837 size_t ipc_size; 1838 int offset = 0; 1839 int ret = 0; 1840 int i; 1841 1842 if (type == SOF_COMP_NONE) { 1843 dev_err(sdev->dev, "error: invalid process comp type %d\n", 1844 type); 1845 return -EINVAL; 1846 } 1847 1848 /* allocate struct for widget control data sizes and types */ 1849 if (widget->num_kcontrols) { 1850 wdata = kcalloc(widget->num_kcontrols, 1851 sizeof(*wdata), 1852 GFP_KERNEL); 1853 1854 if (!wdata) 1855 return -ENOMEM; 1856 1857 /* get possible component controls and get size of all pdata */ 1858 ipc_data_size = sof_get_control_data(sdev, widget, wdata); 1859 1860 if (ipc_data_size <= 0) { The sof_get_control_data() returns negative error codes, but it's also declared as unsigned. But the main reason I'm reporting this instead of fixing it myself (besides laziness) is that I was wondering if it was really handling zero returns correctly. Should we really return success in that situation? 1861 ret = ipc_data_size; 1862 goto out; 1863 } 1864 } 1865 regards, dan carpenter

1 0

[alsa-devel] es8316 multiple issues
by H. C. Dornelas 19 Aug '19

19 Aug '19

Hi. Using only ALSA on my Positivo 2 em 1 Q432A (motherboard CHT12CP, es8316 sound card, kernel 5.2.9), speakers only work with headphones plugged in. Using Pulseaudio, speakers work without headphones, but many other problems remain: no microphones work; speakers are muted every boot and anytime headphones are unplugged, have no volume control and are only a mute/unmute switch; there are more than 160 controls in alsamixer's bytcht-es8316 card, most having no effect on sound output; and I still couldn't test hdmi audio. Here's the alsa-info file: http://alsa-project.org/db/?f=ee6dd9bdcf11a192fcccba9ab1b7abb2f3999b5d UCM files might solve these issues, but I have no experience with those, so any help would be dearly appreciated. If any other info is needed, just tell me and I'll add it. Thank you very much.

1 0

[alsa-devel] [PATCH] Fix an OOB bug in uac_mixer_unit_bmControls
by Hui Peng 17 Aug '19

17 Aug '19

`uac_mixer_unit_get_channels` calls `uac_mixer_unit_bmControls` to get pointer to bmControls field. The current implementation of `uac_mixer_unit_get_channels` does properly check the size of uac_mixer_unit_descriptor descriptor and may allow OOB access in `uac_mixer_unit_bmControls`. Reported-by: Hui Peng <benquike(a)gmail.com> Reported-by: Mathias Payer <mathias.payer(a)nebelwelt.net> Signed-off-by: Hui Peng <benquike(a)gmail.com> --- sound/usb/mixer.c | 25 ++++++++++++++++++------- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c index b5927c3d5bc0..00e6274a63c3 100644 --- a/sound/usb/mixer.c +++ b/sound/usb/mixer.c @@ -738,28 +738,39 @@ static int get_cluster_channels_v3(struct mixer_build *state, unsigned int clust static int uac_mixer_unit_get_channels(struct mixer_build *state, struct uac_mixer_unit_descriptor *desc) { - int mu_channels; + int mu_channels = 0; void *c; - if (desc->bLength < sizeof(*desc)) - return -EINVAL; if (!desc->bNrInPins) return -EINVAL; - if (desc->bLength < sizeof(*desc) + desc->bNrInPins) - return -EINVAL; switch (state->mixer->protocol) { case UAC_VERSION_1: + // limit derived from uac_mixer_unit_bmControls + if (desc->bLength < sizeof(*desc) + desc->bNrInPins + 4) + return 0; + + mu_channels = uac_mixer_unit_bNrChannels(desc); + break; + case UAC_VERSION_2: - default: - if (desc->bLength < sizeof(*desc) + desc->bNrInPins + 1) + // limit derived from uac_mixer_unit_bmControls + if (desc->bLength < sizeof(*desc) + desc->bNrInPins + 6) return 0; /* no bmControls -> skip */ + mu_channels = uac_mixer_unit_bNrChannels(desc); break; case UAC_VERSION_3: + // limit derived from uac_mixer_unit_bmControls + if (desc->bLength < sizeof(*desc) + desc->bNrInPins + 2) + return 0; /* no bmControls -> skip */ + mu_channels = get_cluster_channels_v3(state, uac3_mixer_unit_wClusterDescrID(desc)); break; + + default: + break; } if (!mu_channels) -- 2.22.1

2 5

Re: [alsa-devel] INFO: rcu detected stall in __do_softirq
by Srinivas Kandagatla 17 Aug '19

17 Aug '19

On 16/08/2019 01:10, syzbot wrote: > syzbot has bisected this bug to: > > commit 2aeac95d1a4cc85aae57ab842d5c3340df0f817f > Author: Srinivas Kandagatla <srinivas.kandagatla(a)linaro.org> > Date: Tue Jun 11 10:40:41 2019 +0000 > > soundwire: add module_sdw_driver helper macro Not sure how adding a macro with no users triggers this rcu stall. BTW this was in mainline since rc1. --srini > > bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=114b45ee600000 > start commit: 882e8691 Add linux-next specific files for 20190801 > git tree: linux-next > final crash: https://syzkaller.appspot.com/x/report.txt?x=134b45ee600000 > console output: https://syzkaller.appspot.com/x/log.txt?x=154b45ee600000 > kernel config: https://syzkaller.appspot.com/x/.config?x=466b331af3f34e94 > dashboard link: > https://syzkaller.appspot.com/bug?extid=6593c6b8c8b66a07cd98 > syz repro: https://syzkaller.appspot.com/x/repro.syz?x=16b216b2600000 > > Reported-by: syzbot+6593c6b8c8b66a07cd98(a)syzkaller.appspotmail.com > Fixes: 2aeac95d1a4c ("soundwire: add module_sdw_driver helper macro") > > For information about bisection process see: > https://goo.gl/tpsmEJ#bisection

2 1