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

[GIT PULL] ASoC fixes for v6.2-rc4
by Mark Brown 12 Nov '22

12 Nov '22

The following changes since commit e9441675edc1bb8dbfadacf68aafacca60d65a25: ASoC: SOF: Intel: hda-codec: fix possible memory leak in hda_codec_device_init() (2022-10-21 13:04:14 +0100) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-fix-v6.2-rc4 for you to fetch changes up to 7d945b046be3d2605dbb1806e73095aadd7ae129: ASoC: stm32: dfsdm: manage cb buffers cleanup (2022-11-09 17:56:37 +0000) ---------------------------------------------------------------- ASoC: Fixes for v6.1 A relatively large collection of fixes and new platform quirks here, they're all fairly minor though - the widest possible impact is the fix to the use of prefixes on regulator names which would have broken any device that integrates regulators with DAPM and was used in a system where it had a name prefix assigning to it. ---------------------------------------------------------------- Brent Mendelsohn (1): ASoC: amd: yc: Add Alienware m17 R5 AMD into DMI table Chen Zhongjin (2): ASoC: core: Fix use-after-free in snd_soc_exit() ASoC: soc-utils: Remove __exit for snd_soc_util_exit() Hans de Goede (1): ASoC: Intel: bytcht_es8316: Add quirk for the Nanote UMPC-01 Jason Montleon (2): ASoC: rt5514: fix legacy dai naming ASoC: rt5677: fix legacy dai naming Kai Vehmanen (1): ASoC: SOF: ipc3-topology: use old pipeline teardown flow with SOF2.1 and older Maarten Zanders (1): ASoC: fsl_asrc fsl_esai fsl_sai: allow CONFIG_PM=N Martin Povišer (3): ASoC: tas2770: Fix set_tdm_slot in case of single slot ASoC: tas2764: Fix set_tdm_slot in case of single slot ASoC: tas2780: Fix set_tdm_slot in case of single slot Olivier Moysan (1): ASoC: stm32: dfsdm: manage cb buffers cleanup Paul Cercueil (1): ASoC: dapm: Don't use prefix for regulator name Peter Ujfalusi (1): ASoC: SOF: topology: No need to assign core ID if token parsing failed Pierre-Louis Bossart (2): ASoC: Intel: soc-acpi: add ES83x6 support to IceLake ASoC: hda: intel-dsp-config: add ES83x6 quirk for IceLake Tzung-Bi Shih (1): MAINTAINERS: update Tzung-Bi's email address Zhu Ning (1): ASoC: sof_es8336: reduce pop noise on speaker .mailmap | 1 + .../bindings/sound/google,cros-ec-codec.yaml | 2 +- .../devicetree/bindings/sound/realtek,rt1015p.yaml | 2 +- MAINTAINERS | 2 +- include/sound/sof/info.h | 4 ++ sound/hda/intel-dsp-config.c | 5 ++ sound/soc/amd/yc/acp6x-mach.c | 7 +++ sound/soc/codecs/rt5514-spi.c | 15 +++--- sound/soc/codecs/rt5677-spi.c | 19 +++---- sound/soc/codecs/tas2764.c | 19 +++---- sound/soc/codecs/tas2770.c | 20 +++----- sound/soc/codecs/tas2780.c | 19 +++---- sound/soc/fsl/fsl_asrc.c | 2 +- sound/soc/fsl/fsl_esai.c | 2 +- sound/soc/fsl/fsl_sai.c | 2 +- sound/soc/intel/boards/bytcht_es8316.c | 7 +++ sound/soc/intel/boards/sof_es8336.c | 60 +++++++++++++++++----- sound/soc/intel/common/soc-acpi-intel-icl-match.c | 13 +++++ sound/soc/soc-core.c | 17 +++++- sound/soc/soc-dapm.c | 2 +- sound/soc/soc-utils.c | 2 +- sound/soc/sof/ipc3-topology.c | 15 ++++-- sound/soc/sof/topology.c | 20 ++++---- sound/soc/stm/stm32_adfsdm.c | 11 ++++ 24 files changed, 173 insertions(+), 95 deletions(-)

2 2

[PATCH] ALSA: memalloc: Try dma_alloc_noncontiguous() at first
by Takashi Iwai 12 Nov '22

12 Nov '22

The latest fix for the non-contiguous memalloc helper changed the allocation method for a non-IOMMU system to use only the fallback allocator. This should have worked, but it caused a problem sometimes when too many non-contiguous pages are allocated that can't be treated by HD-audio controller. As a quirk workaround, go back to the original strategy: use dma_alloc_noncontiguous() at first, and apply the fallback only when it fails, but only for non-IOMMU case. We'll need a better fix in the fallback code as well, but this workaround should paper over most cases. Fixes: 9736a325137b ("ALSA: memalloc: Don't fall back for SG-buffer with IOMMU") Reported-by: Linus Torvalds <torvalds(a)linux-foundation.org> Link: https://lore.kernel.org/r/CAHk-=wgSH5ubdvt76gNwa004ooZAEJL_1Q-Fyw5M2FDdqL==… Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/core/memalloc.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/sound/core/memalloc.c b/sound/core/memalloc.c index 6a81aaab25ab..ba095558b6d1 100644 --- a/sound/core/memalloc.c +++ b/sound/core/memalloc.c @@ -542,8 +542,10 @@ static void *snd_dma_noncontig_alloc(struct snd_dma_buffer *dmab, size_t size) struct sg_table *sgt; void *p; + sgt = dma_alloc_noncontiguous(dmab->dev.dev, size, dmab->dev.dir, + DEFAULT_GFP, 0); #ifdef CONFIG_SND_DMA_SGBUF - if (!get_dma_ops(dmab->dev.dev)) { + if (!sgt && !get_dma_ops(dmab->dev.dev)) { if (dmab->dev.type == SNDRV_DMA_TYPE_DEV_WC_SG) dmab->dev.type = SNDRV_DMA_TYPE_DEV_WC_SG_FALLBACK; else @@ -551,9 +553,6 @@ static void *snd_dma_noncontig_alloc(struct snd_dma_buffer *dmab, size_t size) return snd_dma_sg_fallback_alloc(dmab, size); } #endif - - sgt = dma_alloc_noncontiguous(dmab->dev.dev, size, dmab->dev.dir, - DEFAULT_GFP, 0); if (!sgt) return NULL; -- 2.35.3

1 0

Build failure with upcoming Clang 16 (-Wincompatible-function-pointer-types)
by GitHub issues - opened 12 Nov '22

12 Nov '22

alsa-project/alsa-tools issue #12 was opened from thesamesam: Clang 16 will make -Wincompatible-function-pointer-types an error by default. You can emulate this error using: * GCC via -Werror=incompatible-pointer-types (GCC lacks a more specific warning for just func. ptrs) * Clang <16 via -Werror=incompatible-function-pointer-types Originally reported downstream in Gentoo at https://bugs.gentoo.org/880997. ``` echomixer.c:2108:7: error: incompatible function pointer types passing 'void (GtkWidget *, gpointer)' (aka 'void (struct _GtkWidget *, void *)') to parameter of type 'GCallback' (aka 'void (*)(void)') [-Werror,-Wincompatible-function-pointer-types] ...gtk_signal_connect(GTK_OBJECT(menuitem), "activate", Digital_mode_activate, (gpointer)(long)i); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /usr/include/gtk-2.0/gtk/gtksignal.h:51:47: note: expanded from macro 'gtk_signal_connect' gtk_signal_connect_full ((object), (name), (func), NULL, (func_data)... ^~~~~~ /usr/include/gtk-2.0/gtk/gtksignal.h:123:23: note: passing argument to parameter 'func' here GCallback func, ^ [...] ``` Full build log: https://bugs.gentoo.org/attachment.cgi?id=831477 Issue URL : https://github.com/alsa-project/alsa-tools/issues/12 Repository URL: https://github.com/alsa-project/alsa-tools

1 0

[PATCH v2] ASoC: codecs: wsa883x: use correct header file
by Randy Dunlap 11 Nov '22

11 Nov '22

Fix build errors when GPIOLIB is not set/enabled: ../sound/soc/codecs/wsa883x.c: In function 'wsa883x_probe': ../sound/soc/codecs/wsa883x.c:1394:25: error: implicit declaration of function 'devm_gpiod_get_optional'; did you mean 'devm_regulator_get_optional'? [-Werror=implicit-function-declaration] wsa883x->sd_n = devm_gpiod_get_optional(&pdev->dev, "powerdown", ../sound/soc/codecs/wsa883x.c:1395:49: error: 'GPIOD_FLAGS_BIT_NONEXCLUSIVE' undeclared (first use in this function) GPIOD_FLAGS_BIT_NONEXCLUSIVE); ../sound/soc/codecs/wsa883x.c:1414:9: error: implicit declaration of function 'gpiod_direction_output'; did you mean 'gpio_direction_output'? [-Werror=implicit-function-declaration] gpiod_direction_output(wsa883x->sd_n, 1); Fixes: 43b8c7dc85a1 ("ASoC: codecs: add wsa883x amplifier support") Signed-off-by: Randy Dunlap <rdunlap(a)infradead.org> Reported-by: kernel test robot <lkp(a)intel.com> Cc: Srinivas Kandagatla <srinivas.kandagatla(a)linaro.org> Cc: Banajit Goswami <bgoswami(a)quicinc.com> Cc: Mark Brown <broonie(a)kernel.org> Cc: Liam Girdwood <lgirdwood(a)gmail.com> Cc: alsa-devel(a)alsa-project.org Cc: Jaroslav Kysela <perex(a)perex.cz> Cc: Takashi Iwai <tiwai(a)suse.com> --- v2: add the correct header file instead of messing with GPIOLIB in Kconfig sound/soc/codecs/wsa883x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -- a/sound/soc/codecs/wsa883x.c b/sound/soc/codecs/wsa883x.c --- a/sound/soc/codecs/wsa883x.c +++ b/sound/soc/codecs/wsa883x.c @@ -7,7 +7,7 @@ #include <linux/debugfs.h> #include <linux/delay.h> #include <linux/device.h> -#include <linux/gpio.h> +#include <linux/gpio/consumer.h> #include <linux/init.h> #include <linux/kernel.h> #include <linux/module.h>

3 2

[PATCH v2 1/2] ASoC: dt-bindings: qcom, wsa883x: Use correct SD_N polarity
by Krzysztof Kozlowski 11 Nov '22

11 Nov '22

Use correct polarity in example and powerdown-gpios description. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski(a)linaro.org> --- Changes since v1: 1. New patch. --- Documentation/devicetree/bindings/sound/qcom,wsa883x.yaml | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/sound/qcom,wsa883x.yaml b/Documentation/devicetree/bindings/sound/qcom,wsa883x.yaml index 6113f65f2990..99f9c10bbc83 100644 --- a/Documentation/devicetree/bindings/sound/qcom,wsa883x.yaml +++ b/Documentation/devicetree/bindings/sound/qcom,wsa883x.yaml @@ -23,7 +23,7 @@ properties: maxItems: 1 powerdown-gpios: - description: GPIO spec for Powerdown/Shutdown line to use + description: GPIO spec for Powerdown/Shutdown line to use (pin SD_N) maxItems: 1 vdd-supply: @@ -47,6 +47,8 @@ additionalProperties: false examples: - | + #include <dt-bindings/gpio/gpio.h> + soundwire-controller@3250000 { #address-cells = <2>; #size-cells = <0>; @@ -55,7 +57,7 @@ examples: speaker@0,1 { compatible = "sdw10217020200"; reg = <0 1>; - powerdown-gpios = <&tlmm 1 0>; + powerdown-gpios = <&tlmm 1 GPIO_ACTIVE_LOW>; vdd-supply = <&vreg_s10b_1p8>; #thermal-sensor-cells = <0>; #sound-dai-cells = <0>; @@ -64,7 +66,7 @@ examples: speaker@0,2 { compatible = "sdw10217020200"; reg = <0 2>; - powerdown-gpios = <&tlmm 89 0>; + powerdown-gpios = <&tlmm 89 GPIO_ACTIVE_LOW>; vdd-supply = <&vreg_s10b_1p8>; #thermal-sensor-cells = <0>; #sound-dai-cells = <0>; -- 2.34.1

3 4

[PATCH 0/4] ASoC: Intel: add new Dell devices support
by Bard Liao 11 Nov '22

11 Nov '22

Add 2 Dell SKU support. Gongjun Song (4): ASoC: Intel: sof_sdw: Add support for SKU 0C10 product ASoC: Intel: soc-acpi: add SKU 0C10 SoundWire configuration ASoC: Intel: sof_sdw: Add support for SKU 0C40 product ASoC: Intel: soc-acpi: add SKU 0C40 SoundWire configuration sound/soc/intel/boards/sof_sdw.c | 21 ++++ .../intel/common/soc-acpi-intel-rpl-match.c | 109 ++++++++++++++++++ 2 files changed, 130 insertions(+) -- 2.25.1

2 5

[PATCH] ASoC: audio-graph-card2: remove Experimental announce
by Kuninori Morimoto 11 Nov '22

11 Nov '22

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Audio Graph Card2 has been announcing "Audio Graph Card2 is still under Experimental stage", but it is time to remove it. This patch removes it. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/generic/audio-graph-card2.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/sound/soc/generic/audio-graph-card2.c b/sound/soc/generic/audio-graph-card2.c index 8ac6df645ee6..0b0fec9ada51 100644 --- a/sound/soc/generic/audio-graph-card2.c +++ b/sound/soc/generic/audio-graph-card2.c @@ -1271,9 +1271,6 @@ int audio_graph2_parse_of(struct asoc_simple_priv *priv, struct device *dev, if (ret < 0) dev_err_probe(dev, ret, "parse error\n"); - if (ret == 0) - dev_warn(dev, "Audio Graph Card2 is still under Experimental stage\n"); - return ret; } EXPORT_SYMBOL_GPL(audio_graph2_parse_of); -- 2.25.1

2 1

[PATCH] ASoC: qcom: q6prm: Correct module description
by Krzysztof Kozlowski 11 Nov '22

11 Nov '22

The description was copied from APM driver. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski(a)linaro.org> --- sound/soc/qcom/qdsp6/q6prm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/qcom/qdsp6/q6prm.c b/sound/soc/qcom/qdsp6/q6prm.c index cda33ded29be..8aa1a213bfb7 100644 --- a/sound/soc/qcom/qdsp6/q6prm.c +++ b/sound/soc/qcom/qdsp6/q6prm.c @@ -247,5 +247,5 @@ static gpr_driver_t prm_driver = { }; module_gpr_driver(prm_driver); -MODULE_DESCRIPTION("Audio Process Manager"); +MODULE_DESCRIPTION("Q6 Proxy Resource Manager"); MODULE_LICENSE("GPL"); -- 2.34.1

2 1

[PATCH 0/5] ASoC: Intel: add more ACPI tables/quirks for RPL/MTL
by Pierre-Louis Bossart 11 Nov '22

11 Nov '22

Add support for two RaptorLake SoundWire SKUs and one test board for MeteorLake. There will be additional RaptorLake SKUs shared when validation is complete. Gongjun Song (4): ASoC: Intel: sof_sdw: Add support for SKU 0C10 product ASoC: Intel: soc-acpi: add SKU 0C10 SoundWire configuration ASoC: Intel: sof_sdw: Add support for SKU 0C40 product ASoC: Intel: soc-acpi: add SKU 0C40 SoundWire configuration Yong Zhi (1): ASoC: Intel: soc-acpi: add MTL AIC SoundWire configurations sound/soc/intel/boards/sof_sdw.c | 21 ++++ .../intel/common/soc-acpi-intel-mtl-match.c | 59 ++++++++++ .../intel/common/soc-acpi-intel-rpl-match.c | 109 ++++++++++++++++++ 3 files changed, 189 insertions(+) -- 2.34.1

2 6

[PATCH] ASoC: soc-pcm: Don't zero TDM masks in __soc_pcm_open()
by Richard Fitzgerald 11 Nov '22

11 Nov '22

The DAI tx_mask and rx_mask are set by snd_soc_dai_set_tdm_slot() and used by later code that depends on the TDM settings. So __soc_pcm_open() should not be obliterating those mask values. The code in __soc_pcm_hw_params() uses these masks to calculate the active channels so that only the AIF_IN/AIF_OUT widgets for the active TDM slots are enabled. The zeroing of the masks in __soc_pcm_open() disables this functionality so all AIF widgets were enabled even for channels that are not assigned to a TDM slot. Signed-off-by: Richard Fitzgerald <rf(a)opensource.cirrus.com> Fixes: 2e5894d73789 ("ASoC: pcm: Add support for DAI multicodec") --- sound/soc/soc-pcm.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c index d8e4677f3002..493f003273d0 100644 --- a/sound/soc/soc-pcm.c +++ b/sound/soc/soc-pcm.c @@ -822,11 +822,6 @@ static int __soc_pcm_open(struct snd_soc_pcm_runtime *rtd, ret = snd_soc_dai_startup(dai, substream); if (ret < 0) goto err; - - if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) - dai->tx_mask = 0; - else - dai->rx_mask = 0; } /* Dynamic PCM DAI links compat checks use dynamic capabilities */ -- 2.30.2

2 1