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

[v4 0/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s
by Jiaxin Yu 14 Mar '22

14 Mar '22

The series reuses mt8192-mt6359-rt10150rt5682.c for supporting machine driver with rt1015p speaker amplifier and rt5682s headset codec. Changes from v3: - fix build error: too many arguments for format [-Werror-format-extra-args] Changes from v2: - fix build warnings such as "data argument not used by format string" Changes from v1: - uses the snd_soc_of_get_dai_link_codecs to complete the configuration of dai_link's codecs - uses definitions to simplifies card name and compatible name Jiaxin Yu (2): ASoC: dt-bindings: mt8192-mt6359: add new compatible for using rt1015p and rt5682 ASoC: mediatek: mt8192: support rt1015p_rt5682s .../sound/mt8192-mt6359-rt1015-rt5682.yaml | 1 + sound/soc/mediatek/Kconfig | 1 + .../mt8192/mt8192-mt6359-rt1015-rt5682.c | 204 +++++++++++------- 3 files changed, 129 insertions(+), 77 deletions(-) -- 2.18.0

2 3

USB audio not detected with kernels 5.15.02 and 5.16.0; snd_usb_audio and snd_usbmidi_lib not loaded.
by GitHub issues - opened 14 Mar '22

14 Mar '22

alsa-project/alsa-lib issue #216 was opened from stevesr0: System running antiX linux Sid version (based on antiX-19). Using Pipewire on top of ALSA, both internal card and usb headset detected and play with kernel 5.10.88 (antiX nosystemd kernel). With Debian 5.15.5 and 5.16.0 kernels, the usb card is not noted in aplay -l or with /proc/asound/cards. Comparison of loaded modules for working kernel and non working kernel: --Working kernel has both snd_usb_audio and snd_usbmidi_lib loaded. --Non working kernel has NEITHER of these. Instead usbcore and usb_common are loaded. I am seeking advice about fixing the nonworking kernels. Not sure if this is a bug in the kernels, but both nonworking kernels have identical outputs in /proc/asound/cards and pactl list cards. Thanks. Happy to provide files as suggested; not sure what data would be useful. Steve Rinsler Issue URL : https://github.com/alsa-project/alsa-lib/issues/216 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

Sound automatically switch from USB Speaker to HeadSet
by GitHub issues - opened 13 Mar '22

13 Mar '22

alsa-project/alsa-lib issue #215 was opened from modulusmath: Ubuntu 21.10 / Codename: impish Running LXDE if it matters I watch youtube listening on my external usb speakers and sounds plays just fine. All of a sudden they go silent and sound is played on the plugged in headset (Plantronics, Inc. DA40 Adapter) without my intervention...Even if I unplug the headset it will switch over to the 'default usb audio analog stereo'. I do see some logs via journalctl near the time of the switch: Mar 01 10:04:57 wilddog pulseaudio[1678]: ALSA woke us up to write new data to the device, but there was actually nothing to write. Mar 01 10:04:57 wilddog pulseaudio[1678]: Most likely this is a bug in the ALSA driver 'snd_usb_audio'. Please report this issue to the ALSA developers. Mar 01 10:04:57 wilddog pulseaudio[1678]: We were woken up with POLLOUT set -- however a subsequent snd_pcm_avail() returned 0 or another value < min_avail. This is a pretty fresh install. How can I keep it stable on the external speakers? Here's my also info: http://alsa-project.org/db/?f=3589356420b912e19608eaee1960f19b3f0a4e22 Issue URL : https://github.com/alsa-project/alsa-lib/issues/215 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[RESEND V2] ASoC: max98390: Add reset gpio control
by Steve Lee 13 Mar '22

13 Mar '22

Add reset gpio control to support RESET PIN connected to gpio. Signed-off-by: Steve Lee <steve.lee.analog(a)gmail.com> --- sound/soc/codecs/max98390.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/sound/soc/codecs/max98390.c b/sound/soc/codecs/max98390.c index 40fd6f363f35..9a9299e5cc5a 100644 --- a/sound/soc/codecs/max98390.c +++ b/sound/soc/codecs/max98390.c @@ -1022,6 +1022,7 @@ static int max98390_i2c_probe(struct i2c_client *i2c, struct max98390_priv *max98390 = NULL; struct i2c_adapter *adapter = i2c->adapter; + struct gpio_desc *reset_gpio; ret = i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE @@ -1073,6 +1074,17 @@ static int max98390_i2c_probe(struct i2c_client *i2c, return ret; } + reset_gpio = devm_gpiod_get_optional(&i2c->dev, + "maxim,reset-gpios", GPIOD_OUT_LOW); + + /* Power on device */ + if (reset_gpio) { + usleep_range(1000, 2000); + /* bring out of reset */ + gpiod_set_value_cansleep(reset_gpio, 1); + usleep_range(1000, 2000); + } + /* Check Revision ID */ ret = regmap_read(max98390->regmap, MAX98390_R24FF_REV_ID, &reg); -- 2.17.1

4 4

[PATCH] ALSA: usb-audio: Add mute TLV for playback volumes on RODE NT-USB
by Lars-Peter Clausen 12 Mar '22

12 Mar '22

For the RODE NT-USB the lowest Playback mixer volume setting mutes the audio output. But it is not reported as such causing e.g. PulseAudio to accidentally mute the device when selecting a low volume. Fix this by applying the existing quirk for this kind of issue when the device is detected. Signed-off-by: Lars-Peter Clausen <lars(a)metafoo.de> --- sound/usb/mixer_quirks.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/sound/usb/mixer_quirks.c b/sound/usb/mixer_quirks.c index e447ddd6854c..d35cf54cab33 100644 --- a/sound/usb/mixer_quirks.c +++ b/sound/usb/mixer_quirks.c @@ -3360,9 +3360,10 @@ void snd_usb_mixer_fu_apply_quirk(struct usb_mixer_interface *mixer, if (unitid == 7 && cval->control == UAC_FU_VOLUME) snd_dragonfly_quirk_db_scale(mixer, cval, kctl); break; - /* lowest playback value is muted on C-Media devices */ - case USB_ID(0x0d8c, 0x000c): - case USB_ID(0x0d8c, 0x0014): + /* lowest playback value is muted on some devices */ + case USB_ID(0x0d8c, 0x000c): /* C-Media */ + case USB_ID(0x0d8c, 0x0014): /* C-Media */ + case USB_ID(0x19f7, 0x0003): /* RODE NT-USB */ if (strstr(kctl->id.name, "Playback")) cval->min_mute = 1; break; -- 2.30.2

2 1

[PATCH] ASoC: mchp-spdifrx: fix typo
by Claudiu Beznea 11 Mar '22

11 Mar '22

Fix typo in log describing failure of devm_snd_dmaengine_pcm_register(). Signed-off-by: Claudiu Beznea <claudiu.beznea(a)microchip.com> --- sound/soc/atmel/mchp-spdifrx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/atmel/mchp-spdifrx.c b/sound/soc/atmel/mchp-spdifrx.c index bcd4f3e4fb0f..5fc968483f2c 100644 --- a/sound/soc/atmel/mchp-spdifrx.c +++ b/sound/soc/atmel/mchp-spdifrx.c @@ -920,7 +920,7 @@ static int mchp_spdifrx_probe(struct platform_device *pdev) err = devm_snd_dmaengine_pcm_register(&pdev->dev, NULL, 0); if (err) { - dev_err(&pdev->dev, "failed to register PMC: %d\n", err); + dev_err(&pdev->dev, "failed to register PCM: %d\n", err); return err; } -- 2.32.0

3 2

[PATCH] ASoC: simple-card-utils: Don't reset clock of active DAI
by Sameer Pujar 11 Mar '22

11 Mar '22

Playback or capture errors are seen when clock is reset during an active stage of DAI. Presently this scenario happens when DAI has both playback and capture sessions running and one of these finishes first which will be followed by clock rate reset. The remaining active session will be affected in such case. Address this problem by allowing clock rate reset to happen only when the DAI is no more active. Signed-off-by: Sameer Pujar <spujar(a)nvidia.com> --- sound/soc/generic/simple-card-utils.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/sound/soc/generic/simple-card-utils.c b/sound/soc/generic/simple-card-utils.c index a4babfb..6a5faaf 100644 --- a/sound/soc/generic/simple-card-utils.c +++ b/sound/soc/generic/simple-card-utils.c @@ -272,15 +272,19 @@ void asoc_simple_shutdown(struct snd_pcm_substream *substream) int i; for_each_prop_dai_cpu(props, i, dai) { - if (props->mclk_fs && !dai->clk_fixed) - snd_soc_dai_set_sysclk(asoc_rtd_to_cpu(rtd, i), + struct snd_soc_dai *cpu_dai = asoc_rtd_to_cpu(rtd, i); + + if (props->mclk_fs && !dai->clk_fixed && !snd_soc_dai_active(cpu_dai)) + snd_soc_dai_set_sysclk(cpu_dai, 0, 0, SND_SOC_CLOCK_IN); asoc_simple_clk_disable(dai); } for_each_prop_dai_codec(props, i, dai) { - if (props->mclk_fs && !dai->clk_fixed) - snd_soc_dai_set_sysclk(asoc_rtd_to_codec(rtd, i), + struct snd_soc_dai *codec_dai = asoc_rtd_to_codec(rtd, i); + + if (props->mclk_fs && !dai->clk_fixed && !snd_soc_dai_active(codec_dai)) + snd_soc_dai_set_sysclk(codec_dai, 0, 0, SND_SOC_CLOCK_IN); asoc_simple_clk_disable(dai); -- 2.7.4

3 2

[PATCH 0/5] ASoC: SOF/Intel: small fixes and updates for 5.18
by Pierre-Louis Bossart 11 Mar '22

11 Mar '22

One important fix from Kai to restore DM1 L1 functionality, one important update from Peter to use DMA trace buffers as capture-only and sync them and a couple of minor updates for Intel/SOF platforms. Brent Lu (1): ASoC: SOF: Intel: add topology overwrite for Taniks Kai Vehmanen (1): ASoC: SOF: Intel: enable DMI L1 for playback streams Muralidhar Reddy (1): ASoC: Intel: soc-acpi: Add entry for rt711-sdca-sdw in ADL match table Peter Ujfalusi (1): ASoC: SOF: trace: Use proper DMA direction for the trace data buffer Weiguo Li (1): ASoC: SOF: compress: fix null check after dereference sound/soc/intel/common/soc-acpi-intel-adl-match.c | 15 +++++++++++++++ sound/soc/sof/compress.c | 6 ++++-- sound/soc/sof/intel/hda-pcm.c | 1 + sound/soc/sof/sof-pci-dev.c | 8 ++++++++ sound/soc/sof/trace.c | 12 ++++++++++-- 5 files changed, 38 insertions(+), 4 deletions(-) base-commit: 52eaf2bbcf022a61872c812ce41855a90b814ebc -- 2.30.2

2 6

[PATCH 00/10] Make the SOF pcm_hw_params DSP op IPC agnostic
by Ranjani Sridharan 11 Mar '22

11 Mar '22

In preparation for supporting the newly introduced IPC version in the SOF firmware, this patchset adds the changes required to make the pcm_hw_params DSP op IPC agnostic. Peter Ujfalusi (10): ASoC: SOF: sof-priv: Remove stale snd_sof_ipc_stream_pcm_params() declaration ASoC: SOF: Make pcm_hw_params snd_sof_dsp_ops callback IPC neutral ASoC: SOF: pcm: Remove sof_pcm_dsp_params() wrapper ASoC: SOF: Introduce optional callback to configure stream data offset ASoC: SOF: Mark snd_sof_dsp_ops.ipc_pcm_params() callback optional ASoC: SOF: stream-ipc: Add sof_set_stream_data_offset() ASoC: SOF: Intel: hda-ipc: Add hda_set_stream_data_offset() ASoC: SOF: Intel: Convert to use the generic set_stream_data_offset ops ASoC: SOF: imx: Convert to use the generic set_stream_data_offset ops ASoC: SOF: Remove ipc_pcm_params() ops sound/soc/sof/amd/acp-pcm.c | 8 ++++--- sound/soc/sof/amd/acp.h | 3 ++- sound/soc/sof/core.c | 3 +-- sound/soc/sof/imx/imx8.c | 4 ++-- sound/soc/sof/imx/imx8m.c | 2 +- sound/soc/sof/intel/apl.c | 2 +- sound/soc/sof/intel/bdw.c | 2 +- sound/soc/sof/intel/byt.c | 4 ++-- sound/soc/sof/intel/cnl.c | 2 +- sound/soc/sof/intel/hda-ipc.c | 8 +++---- sound/soc/sof/intel/hda-pcm.c | 18 ++++---------- sound/soc/sof/intel/hda.h | 8 +++---- sound/soc/sof/intel/icl.c | 2 +- sound/soc/sof/intel/pci-tng.c | 2 +- sound/soc/sof/intel/tgl.c | 2 +- sound/soc/sof/ops.h | 21 +++++++++------- sound/soc/sof/pcm.c | 45 ++++++++++++++++++++--------------- sound/soc/sof/sof-priv.h | 33 +++++++++++++++++-------- sound/soc/sof/stream-ipc.c | 9 ++++--- 19 files changed, 95 insertions(+), 83 deletions(-) -- 2.25.1

2 11

[PATCH v3 00/11] Fix broken usage of driver_override (and kfree of static memory)
by Krzysztof Kozlowski 11 Mar '22

11 Mar '22

Hi, This is a continuation of my old patchset from 2019. [1] Back then, few drivers set driver_override wrong. I fixed Exynos in a different way after discussions. QCOM NGD was not fixed and a new user appeared - IMX SCU. It seems "char *" in driver_override looks too consty, so we tend to make a mistake of storing there string literals. Changes since latest v2 ======================= 1. Make all driver_override fields as "const char *", just like SPI and VDPA. (Mark) 2. Move "count" check to the new helper and add "count" argument. (Michael) 3. Fix typos in docs, patch subject. Extend doc. (Michael, Bjorn) 4. Compare pointers to reduce number of string readings in the helper. 5. Fix clk-imx return value. Changes since latest v1 (not the old 2019 solution): ==================================================== https://lore.kernel.org/all/708eabb1-7b35-d525-d4c3-451d4a3de84f@rasmusvill… 1. Add helper for setting driver_override. 2. Use the helper. Dependencies (and stable): ========================== 1. All patches, including last three fixes, depend on the first patch introducing the helper. 2. The last three commits - fixes - are probably not backportable directly, because of this dependency. I don't know how to express this dependency here, since stable-kernel-rules.rst mentions only commits as possible dependencies. [1] https://lore.kernel.org/all/1550484960-2392-3-git-send-email-krzk@kernel.or… Best regards, Krzysztof Krzysztof Kozlowski (11): driver: platform: Add helper for safer setting of driver_override amba: Use driver_set_override() instead of open-coding fsl-mc: Use driver_set_override() instead of open-coding hv: Use driver_set_override() instead of open-coding PCI: Use driver_set_override() instead of open-coding s390: cio: Use driver_set_override() instead of open-coding spi: Use helper for safer setting of driver_override vdpa: Use helper for safer setting of driver_override clk: imx: scu: Fix kfree() of static memory on setting driver_override slimbus: qcom-ngd: Fix kfree() of static memory on setting driver_override rpmsg: Fix kfree() of static memory on setting driver_override drivers/amba/bus.c | 28 +++--------------- drivers/base/driver.c | 51 +++++++++++++++++++++++++++++++++ drivers/base/platform.c | 28 +++--------------- drivers/bus/fsl-mc/fsl-mc-bus.c | 25 +++------------- drivers/clk/imx/clk-scu.c | 7 ++++- drivers/hv/vmbus_drv.c | 28 +++--------------- drivers/pci/pci-sysfs.c | 28 +++--------------- drivers/rpmsg/rpmsg_core.c | 3 +- drivers/rpmsg/rpmsg_internal.h | 13 +++++++-- drivers/rpmsg/rpmsg_ns.c | 14 +++++++-- drivers/s390/cio/cio.h | 7 ++++- drivers/s390/cio/css.c | 28 +++--------------- drivers/slimbus/qcom-ngd-ctrl.c | 13 ++++++++- drivers/spi/spi.c | 26 +++-------------- drivers/vdpa/vdpa.c | 29 +++---------------- include/linux/amba/bus.h | 7 ++++- include/linux/device/driver.h | 2 ++ include/linux/fsl/mc.h | 6 ++-- include/linux/hyperv.h | 7 ++++- include/linux/pci.h | 7 ++++- include/linux/platform_device.h | 7 ++++- include/linux/rpmsg.h | 6 ++-- include/linux/spi/spi.h | 2 ++ include/linux/vdpa.h | 4 ++- 24 files changed, 171 insertions(+), 205 deletions(-) -- 2.32.0

8 25