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

[PATCH 0/3] ASoC: SOF: minor updates for 5.20
by Pierre-Louis Bossart 26 Jul '22

26 Jul '22

One sanity check for SSP index reported by NHLT/BIOS and two updates for Mediatek and Intel Chromebooks related to already-merged firmware changes. Ideally this should go in 5.20, time-permitting. Brent Lu (1): ASoC: SOF: dai-intel: add SOF_DAI_INTEL_SSP_CLKCTRL_MCLK_AON bit Pierre-Louis Bossart (1): ASoC: SOF: Intel: hda: add sanity check on SSP index reported by NHLT YC Hung (1): ASoC: SOF: Add cont_update_posn to platform parameters include/sound/sof/dai-intel.h | 2 ++ include/uapi/sound/sof/abi.h | 2 +- sound/soc/sof/intel/hda.c | 7 +++++++ sound/soc/sof/ipc3-pcm.c | 3 +++ sound/soc/sof/sof-priv.h | 1 + 5 files changed, 14 insertions(+), 1 deletion(-) -- 2.34.1

2 4

[PATCH 00/10] ASoC: Intel: machine driver updates for 5.20
by Pierre-Louis Bossart 26 Jul '22

26 Jul '22

A couple of minor updates contributed and tested by Intel teams or end-users. Ideally these should be applied for 5.20, time-permitting. Andrey Turkin (2): ASoC: Intel: sof_es8336: Fix GPIO quirks set via module option ASoC: Intel: sof_es8336: ignore GpioInt when looking for speaker/headset GPIO lines Muralidhar Reddy (3): ASoC: Intel: soc-acpi: Add entry for sof_es8336 in ADL match table ASoC: Intel: sof_es8336: add support for HDMI_In capture ASoC: Intel: sof_es8336: reset the num_links during probe Pierre-Louis Bossart (2): ASoC: Intel: sof_sdw: add support for Dell SKU 0AF0 ASoC: Intel: sof_es8336: remove hard-coded SSP selection Yong Zhi (3): ASoC: Intel: sof_rt5682: Add support for mtl_mx98357_rt5682 ASoC: Intel: sof_rt5682: Perform quirk check first in card late probe ASoC: Intel: sof_nau8825: Move quirk check to the front in late probe sound/hda/intel-dsp-config.c | 5 + sound/soc/intel/boards/sof_es8336.c | 160 +++++++++++++----- sound/soc/intel/boards/sof_nau8825.c | 10 +- sound/soc/intel/boards/sof_rt5682.c | 26 ++- sound/soc/intel/boards/sof_sdw.c | 10 ++ .../intel/common/soc-acpi-intel-adl-match.c | 25 +++ .../intel/common/soc-acpi-intel-mtl-match.c | 17 ++ 7 files changed, 201 insertions(+), 52 deletions(-) -- 2.34.1

2 11

[PATCH -next] ASoC: mediatek: mt8186: fix Kconfig dependency
by Ren Zhijie 26 Jul '22

26 Jul '22

If CONFIG_SND_SOC_MT8186=y and CONFIG_SND_SOC_MT6358 is not set, make ARCH=x86_64 CROSS_COMPILE=x86_64-linux-gnu-, will be failed, like this: sound/soc/mediatek/mt8186/mt8186-mt6366-common.o: In function `mt8186_mt6366_init': mt8186-mt6366-common.c:(.text+0x4d): undefined reference to `mt6358_set_mtkaif_protocol' make: *** [vmlinux] Error 1 To fix this build error, add select SND_SOC_MT6358 to config SND_SOC_MT8186 dependency. Fixes: 097e874ad3fc ("ASoC: mediatek: mt8186: add platform driver") Signed-off-by: Ren Zhijie <renzhijie2(a)huawei.com> --- sound/soc/mediatek/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/mediatek/Kconfig b/sound/soc/mediatek/Kconfig index 96917743b0db..363fa4d47680 100644 --- a/sound/soc/mediatek/Kconfig +++ b/sound/soc/mediatek/Kconfig @@ -157,6 +157,7 @@ config SND_SOC_MT8186 depends on ARCH_MEDIATEK || COMPILE_TEST depends on COMMON_CLK select SND_SOC_MEDIATEK + select SND_SOC_MT6358 select MFD_SYSCON if SND_SOC_MT6358 help This adds ASoC driver for Mediatek MT8186 boards -- 2.17.1

2 1

[PATCH] ASoC: max98373: Removing 0x203E from the volatile reg
by Ryan Lee 26 Jul '22

26 Jul '22

The 0x203E speaker gain register should be non-volatile. This register value was not able to be synced because it was marked as volatile. Signed-off-by: Ryan Lee <ryans.lee(a)analog.com> --- sound/soc/codecs/max98373-i2c.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/codecs/max98373-i2c.c b/sound/soc/codecs/max98373-i2c.c index 4fe065ece17c..3e04c7f0cce4 100644 --- a/sound/soc/codecs/max98373-i2c.c +++ b/sound/soc/codecs/max98373-i2c.c @@ -442,7 +442,6 @@ static bool max98373_volatile_reg(struct device *dev, unsigned int reg) { switch (reg) { case MAX98373_R2000_SW_RESET ... MAX98373_R2009_INT_FLAG3: - case MAX98373_R203E_AMP_PATH_GAIN: case MAX98373_R2054_MEAS_ADC_PVDD_CH_READBACK: case MAX98373_R2055_MEAS_ADC_THERM_CH_READBACK: case MAX98373_R20B6_BDE_CUR_STATE_READBACK: -- 2.37.0

3 2

[PATCH] ASoC: ti: remove sound support for DaVinci DM644x and DM646x
by Lukas Bulwahn 26 Jul '22

26 Jul '22

Commit 7dd33764486d ("ARM: davinci: Delete DM644x board files") and commit b4aed01de486 ("ARM: davinci: Delete DM646x board files") removes the support for DaVinci DM644x and DM646x boards. Hence, remove the sound support for those boards as well. Signed-off-by: Lukas Bulwahn <lukas.bulwahn(a)gmail.com> --- sound/soc/ti/Kconfig | 5 ----- 1 file changed, 5 deletions(-) diff --git a/sound/soc/ti/Kconfig b/sound/soc/ti/Kconfig index 40110e9a9e8a..35fddd7c501a 100644 --- a/sound/soc/ti/Kconfig +++ b/sound/soc/ti/Kconfig @@ -180,9 +180,6 @@ config SND_SOC_DAVINCI_EVM select SND_SOC_DAVINCI_ASP if MACH_DAVINCI_DM355_EVM select SND_SOC_DAVINCI_ASP if SND_SOC_DM365_AIC3X_CODEC select SND_SOC_DAVINCI_VCIF if SND_SOC_DM365_VOICE_CODEC - select SND_SOC_DAVINCI_ASP if MACH_DAVINCI_EVM # DM6446 - select SND_SOC_DAVINCI_MCASP if MACH_DAVINCI_DM6467_EVM - select SND_SOC_SPDIF if MACH_DAVINCI_DM6467_EVM select SND_SOC_DAVINCI_MCASP if MACH_DAVINCI_DA830_EVM select SND_SOC_DAVINCI_MCASP if MACH_DAVINCI_DA850_EVM select SND_SOC_TLV320AIC3X @@ -191,8 +188,6 @@ config SND_SOC_DAVINCI_EVM DaVinci EVM platforms: - DM355 - DM365 - - DM6446 - - DM6447 - DM830 - DM850 -- 2.17.1

3 2

Question about DPCM locking
by Kuninori Morimoto 26 Jul '22

26 Jul '22

Hi Takashi May I ask about this patch ? ASoC: soc-pcm: Fix and cleanup DPCM locking b7898396f4bbe160f546d0c5e9fa17cca9a7d153 Q1. It exchanges many function parameters - func(substream, xxx) + func(rtd, substream, xxx) ^^^^ I guess the purpose is it want to call snd_soc_dpcm_mutex_assert_held() inside the func ? But it looks very verbose. we can do like this, but am I misunderstanding ? func(substream, xxx) { struct snd_soc_pcm_runtime *rtd = asoc_substream_to_rtd(substream); snd_soc_dpcm_mutex_assert_held(rtd); ... } Q2. It added new __soc_pcm_close(). But soc_pcm_close() is using soc_pcm_clean() instead of __soc_pcm_close(). Is this just a mistake ? static int soc_pcm_close(struct snd_pcm_substream *substream) { struct snd_soc_pcm_runtime *rtd = asoc_substream_to_rtd(substream); snd_soc_dpcm_mutex_lock(rtd); - soc_pcm_clean(substream, 0); + __soc_pcm_close(substream); snd_soc_dpcm_mutex_unlock(rtd); return 0; } Thank you for your help !! Best regards --- Kuninori Morimoto

2 2

Re: [PATCH v8] ASoc: dt-bindings: tas27xx: revise tas27xx format
by Rob Herring 26 Jul '22

26 Jul '22

On Fri, Jul 22, 2022 at 06:55:12PM +0800, Raphael-Xu wrote: > revise the format Complete sentences. And this is not v8. Each new change is v1. > > Signed-off-by: Raphael-Xu <13691752556(a)139.com> > --- > .../devicetree/bindings/sound/tas27xx.yaml | 16 +++++++++++----- > 1 file changed, 11 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/sound/tas27xx.yaml b/Documentation/devicetree/bindings/sound/tas27xx.yaml > index 66a0df8850ea..573652577462 100644 > --- a/Documentation/devicetree/bindings/sound/tas27xx.yaml > +++ b/Documentation/devicetree/bindings/sound/tas27xx.yaml > @@ -8,7 +8,7 @@ $schema: "http://devicetree.org/meta-schemas/core.yaml#" > title: Texas Instruments TAS2764/TAS2780 Smart PA > > maintainers: > - - Shenghao Ding <shenghao-ding(a)ti.com> > + - Raphael Xu <raphael-xu(a)ti.com> > > description: | > The TAS2764/TAS2780 is a mono, digital input Class-D audio amplifier > @@ -23,17 +23,19 @@ properties: > - ti,tas2780 > > reg: > - maxItems: 1 > description: | > - I2C address of the device can be between 0x38 to 0x45. 0x40-0x45 is no longer valid? > + I2C address of the device can be between 0x38 to 0x3f. > + minimum: 0x38 > + maximum: 0x3f > > reset-gpios: > - maxItems: 1 > description: GPIO used to reset the device. > + maxItems: 1 Unnecessary change. > > shutdown-gpios: > + description: | Don't need '|' if no formatting. > + GPIO used to control the state of the device.Only for TAS2764. Space needed after period. > maxItems: 1 > - description: GPIO used to control the state of the device. > > interrupts: > maxItems: 1 > @@ -41,10 +43,14 @@ properties: > ti,imon-slot-no: > $ref: /schemas/types.yaml#/definitions/uint32 > description: TDM TX current sense time slot. > + minimum: 0 > + maximum: 3 > > ti,vmon-slot-no: > $ref: /schemas/types.yaml#/definitions/uint32 > description: TDM TX voltage sense time slot. > + minimum: 0 > + maximum: 3 > > '#sound-dai-cells': > const: 1 > -- > 2.35.1 > > >

1 0

[PATCH v2 0/5] ALSA: Drop async signal support
by Takashi Iwai 25 Jul '22

25 Jul '22

Hi, this is a revised patch set for dropping fasync support from ALSA core. The async signal itself is very difficult to use properly due to various restrictions (e.g. you cannot perform any I/O in the context), hence it's a feature that has been never used by real applications. OTOH, the real problem is that there have been quite a few syzcaller reports indicating that fasync code path may lead to some potential deadlocks for long time. Dropping the feature is the easiest solution, obviously. The corresponding update for alsa-lib will follow once when we agree with this approach. thanks, Takashi === v1: https://lore.kernel.org/r/20220715102935.4695-1-tiwai@suse.de v1->v2: Fix unused variable warning in patch 2 === Takashi Iwai (5): ALSA: timer: Drop async signal support ALSA: pcm: Drop async signal support ALSA: control: Drop async signal support ALSA: core: Drop async signal support ALSA: doc: Drop stale fasync entry .../kernel-api/writing-an-alsa-driver.rst | 1 - include/sound/control.h | 1 - include/sound/pcm.h | 1 - sound/core/control.c | 11 ----------- sound/core/init.c | 11 +---------- sound/core/pcm_lib.c | 8 +------- sound/core/pcm_native.c | 18 ------------------ sound/core/timer.c | 13 ------------- 8 files changed, 2 insertions(+), 62 deletions(-) -- 2.35.3

2 7

[bug report] ASoC: mediatek: mt8186: support tdm in platform driver
by Dan Carpenter 25 Jul '22

25 Jul '22

Hello Jiaxin Yu, This is a semi-automatic email about new static checker warnings. The patch ae92dcbee8b6: "ASoC: mediatek: mt8186: support tdm in platform driver" from May 23, 2022, leads to the following Smatch complaint: sound/soc/mediatek/mt8186/mt8186-dai-tdm.c:424 mtk_dai_tdm_hw_params() warn: variable dereferenced before check 'tdm_priv' (see line 406) sound/soc/mediatek/mt8186/mt8186-dai-tdm.c 405 struct mtk_afe_tdm_priv *tdm_priv = afe_priv->dai_priv[tdm_id]; 406 unsigned int tdm_mode = tdm_priv->tdm_mode; ^^^^^^^^^^^^^^^^^^^ Lot's of dereferences 407 unsigned int data_mode = tdm_priv->data_mode; 408 unsigned int rate = params_rate(params); 409 unsigned int channels = params_channels(params); 410 snd_pcm_format_t format = params_format(params); 411 unsigned int bit_width = 412 snd_pcm_format_physical_width(format); 413 unsigned int tdm_channels = (data_mode == TDM_DATA_ONE_PIN) ? 414 get_tdm_ch_per_sdata(tdm_mode, channels) : 2; 415 unsigned int lrck_width = 416 get_tdm_lrck_width(format, tdm_mode); 417 unsigned int tdm_con = 0; 418 bool slave_mode = tdm_priv->slave_mode; 419 bool lrck_inv = tdm_priv->lck_invert; 420 bool bck_inv = tdm_priv->bck_invert; 421 unsigned int tran_rate; 422 unsigned int tran_relatch_rate; 423 424 if (!tdm_priv) { ^^^^^^^^^ Checked too late 425 dev_err(afe->dev, "%s(), tdm_priv == NULL", __func__); 426 return -EINVAL; regards, dan carpenter

2 1

[bug report] ASoC: mediatek: mt8186: support adda in platform driver
by Dan Carpenter 25 Jul '22

25 Jul '22

Hello Jiaxin Yu, The patch b65c466220b3: "ASoC: mediatek: mt8186: support adda in platform driver" from May 23, 2022, leads to the following Smatch static checker warning: sound/soc/mediatek/mt8186/mt8186-dai-adda.c:78 get_adda_priv_by_name() warn: strncmp() with weird length: 17 vs 7 sound/soc/mediatek/mt8186/mt8186-dai-adda.c 72 static struct mtk_afe_adda_priv *get_adda_priv_by_name(struct mtk_base_afe *afe, 73 const char *name) 74 { 75 struct mt8186_afe_private *afe_priv = afe->platform_priv; 76 int dai_id; 77 --> 78 if (strncmp(name, "aud_dac_hires_clk", 7) == 0 || 79 strncmp(name, "aud_adc_hires_clk", 7) == 0) 7 is weird. It would be more readable to say it as: if (strncmp(name, "aud_dac", 7) == 0 || strncmp(name, "aud_adc", 7) == 0) 80 dai_id = MT8186_DAI_ADDA; 81 else 82 return NULL; 83 84 return afe_priv->dai_priv[dai_id]; 85 } regards, dan carpenter

2 1