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

[PATCH v2 0/2] ASoC: mediatek: add missing initialization
by Trevor Wu 01 Mar '23

01 Mar '23

Fix dts parsing problem in mt8188 and mt8195 etdm dai drivers. Changes since v1: - split the patch into two commits and add Fixes tag. Trevor Wu (2): ASoC: mediatek: mt8188: add missing initialization ASoC: mediatek: mt8195: add missing initialization sound/soc/mediatek/mt8188/mt8188-dai-etdm.c | 3 +++ sound/soc/mediatek/mt8195/mt8195-dai-etdm.c | 3 +++ 2 files changed, 6 insertions(+) -- 2.18.0

3 5

[PATCH] ASoC: mediatek: add missing initialization
by Trevor Wu 01 Mar '23

01 Mar '23

In etdm dai driver, dai_etdm_parse_of() function is used to parse dts properties to get parameters. There are two for-loops which are sepearately for all etdm and etdm input only cases. In etdm in only loop, dai_id is not initialized, so it keeps the value intiliazed in another loop. In the patch, add the missing initialization to fix the unexpected parsing problem. Signed-off-by: Trevor Wu <trevor.wu(a)mediatek.com> --- sound/soc/mediatek/mt8188/mt8188-dai-etdm.c | 3 +++ sound/soc/mediatek/mt8195/mt8195-dai-etdm.c | 3 +++ 2 files changed, 6 insertions(+) diff --git a/sound/soc/mediatek/mt8188/mt8188-dai-etdm.c b/sound/soc/mediatek/mt8188/mt8188-dai-etdm.c index 1c53d4cb19bb..7a37752d4244 100644 --- a/sound/soc/mediatek/mt8188/mt8188-dai-etdm.c +++ b/sound/soc/mediatek/mt8188/mt8188-dai-etdm.c @@ -2498,6 +2498,9 @@ static void mt8188_dai_etdm_parse_of(struct mtk_base_afe *afe) /* etdm in only */ for (i = 0; i < 2; i++) { + dai_id = ETDM_TO_DAI_ID(i); + etdm_data = afe_priv->dai_priv[dai_id]; + snprintf(prop, sizeof(prop), "mediatek,%s-chn-disabled", of_afe_etdms[i].name); diff --git a/sound/soc/mediatek/mt8195/mt8195-dai-etdm.c b/sound/soc/mediatek/mt8195/mt8195-dai-etdm.c index c2e268054773..f2c9a1fdbe0d 100644 --- a/sound/soc/mediatek/mt8195/mt8195-dai-etdm.c +++ b/sound/soc/mediatek/mt8195/mt8195-dai-etdm.c @@ -2567,6 +2567,9 @@ static void mt8195_dai_etdm_parse_of(struct mtk_base_afe *afe) /* etdm in only */ for (i = 0; i < 2; i++) { + dai_id = ETDM_TO_DAI_ID(i); + etdm_data = afe_priv->dai_priv[dai_id]; + ret = snprintf(prop, sizeof(prop), "mediatek,%s-chn-disabled", of_afe_etdms[i].name); -- 2.18.0

3 2

[Patch] sound/aoa: Add support for iBook G4 (powerbook6,5)
by Thaison Nguyen 01 Mar '23

01 Mar '23

The audio on my iBook G4 (powerbook65) has not been working. This is because the ids of the device were missing in the aoa audio driver. With this patch, the IDs of Powerbook6,4 and Powerbook6,5 are added to the driver. Signed-off-by: Thaison Nguyen <mail(a)thaison-nguyen.de> --- sound/aoa/fabrics/layout.c | 15 +++++++++++++++ sound/aoa/soundbus/i2sbus/core.c | 2 +- 2 files changed, 16 insertions(+), 1 deletions(-) diff a/sound/aoa/fabrics/layout.c b/sound/aoa/fabrics/layout.c --- a/sound/aoa/fabrics/layout.c +++ b/sound/aoa/fabrics/layout.c @@ -111,7 +111,9 @@ MODULE_ALIAS("aoa-device-id-14"); MODULE_ALIAS("aoa-device-id-22"); MODULE_ALIAS("aoa-device-id-31"); MODULE_ALIAS("aoa-device-id-35"); +MODULE_ALIAS("aoa-device-id-38"); +MODULE_ALIAS("aoa-device-id-40"); MODULE_ALIAS("aoa-device-id-44"); /* onyx with all but microphone connected */ @@ -368,7 +368,20 @@ .connections = tas_connections_nolineout, }, }, + /* PowerBook6,4 */ + { .device_id = 40, + .codecs[0] = { + .name = "tas", + .connections = tas_connections_all, + }, + }, /* PowerBook6,5 */ + { .device_id = 38, + .codecs[0] = { + .name = "tas", + .connections = tas_connections_all, + }, + }, { .device_id = 44, .codecs[0] = { .name = "tas", diff a/sound/aoa/soundbus/i2sbus/core.c b/sound/aoa/soundbus/i2sbus/core.c --- a/sound/aoa/soundbus/i2sbus/core.c +++ b/sound/aoa/soundbus/i2sbus/core.c @@ -197,7 +197,7 @@ * so restrict to those we do handle for now. */ if (id && (*id == 22 || *id == 14 || *id == 35 || - *id == 31 || *id == 44)) { + *id == 31 || *id == 44 || *id == 40 || *id == 38)) { snprintf(dev->sound.modalias, 32, "aoa-device-id-%d", *id); ok = 1;

1 0

Re: Patch for audio on iBook G4 (powerbook6,5)
by Christophe Leroy 01 Mar '23

01 Mar '23

Le 28/02/2023 à 22:17, mail(a)thaison-nguyen.de a écrit : > Hi > > I have added a bug report and a patch regarding audio for iBook G4 > (powerbook6,5) on kernel.org bugzilla, but nobody has picked it up. > https://bugzilla.kernel.org/show_bug.cgi?id=216504 > > Now I am trying to submit the patch here. > Please read and follow the process to submit a patch : https://docs.kernel.org/process/5.Posting.html Patches must be in the core part of the mail and not as an attached file. Patches must have a description of the change. Patches must be signed Patches must be sent to the relevant maintainer(s) based on the modified files. As far as I can see your patch is for the sound subsystem. Christophe

1 0

[PATCH v3 0/3] ASoC: mchp-pdmc: fix poc noises when starting capture
by Claudiu Beznea 28 Feb '23

28 Feb '23

To start capture on Microchip PDMC the enable bits for each supported microphone need to be set. After this bit is set the PDMC starts to receive data from microphones and it considers this data as valid data. Thus if microphones are not ready the PDMC captures anyway data from its lines. This data is interpreted by the human ear as poc noises. To avoid this the following software workaround need to be applied when starting capture: 1/ enable PDMC channel 2/ wait 150ms 3/ execute 16 dummy reads from RHR 4/ clear interrupts 5/ enable interrupts 6/ enable DMA channel For this workaround to work step 6 need to be executed at the end. For step 6 was added patch 1/3 from this series. With this, component DAI driver sets its struct snd_soc_component_driver::start_dma_last = 1 and proper action is taken based on this flag when starting DAI trigger vs DMA. Thank you, Claudiu Beznea Changes in v3: - update the commit message of patch 2/3 to be more descriptive - in patch 3/3 initialize dd->startup_delay_us before calling of_property_read_u32() and don't check its return value; property is optional and the the default value is 150ms Changes in v2: - patch 1/3 from v1 is now "ASoC: soc-pcm: add option to start DMA after DAI" - pass start_dma_last from component DAI driver object (struct snd_soc_component_driver::start_dma_last); adapt patch 3/3 after this; - in patch 1/3 s/Do we need to start dma first/Do we need to start dma last in comment from soc_pcm_trigger() - collect review tag from Krzysztof Claudiu Beznea (3): ASoC: soc-pcm: add option to start DMA after DAI ASoC: dt-bindings: sama7g5-pdmc: add microchip,startup-delay-us binding ASoC: mchp-pdmc: fix poc noise at capture startup Claudiu Beznea (3): ASoC: soc-pcm: add option to start DMA after DAI ASoC: dt-bindings: sama7g5-pdmc: add microchip,startup-delay-us binding ASoC: mchp-pdmc: fix poc noise at capture startup .../sound/microchip,sama7g5-pdmc.yaml | 6 +++ include/sound/soc-component.h | 2 + sound/soc/atmel/mchp-pdmc.c | 53 +++++++++++++++++-- sound/soc/soc-pcm.c | 27 ++++++++-- 4 files changed, 78 insertions(+), 10 deletions(-) -- 2.34.1

2 4

[PATCH] ASoC: zl38060 add gpiolib dependency
by Arnd Bergmann 28 Feb '23

28 Feb '23

From: Arnd Bergmann <arnd(a)arndb.de> Without gpiolib, this driver fails to link: arm-linux-gnueabi-ld: sound/soc/codecs/zl38060.o: in function `chip_gpio_get': zl38060.c:(.text+0x30): undefined reference to `gpiochip_get_data' arm-linux-gnueabi-ld: sound/soc/codecs/zl38060.o: in function `zl38_spi_probe': zl38060.c:(.text+0xa18): undefined reference to `devm_gpiochip_add_data_with_key' This appears to have been in the driver since the start, but is hard to hit in randconfig testing since gpiolib is almost always selected by something else. Fixes: 52e8a94baf90 ("ASoC: Add initial ZL38060 driver") Signed-off-by: Arnd Bergmann <arnd(a)arndb.de> --- sound/soc/codecs/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig index bd72c426a93d..07747565c3b5 100644 --- a/sound/soc/codecs/Kconfig +++ b/sound/soc/codecs/Kconfig @@ -2103,6 +2103,7 @@ config SND_SOC_WSA883X config SND_SOC_ZL38060 tristate "Microsemi ZL38060 Connected Home Audio Processor" depends on SPI_MASTER + depends on GPIOLIB select REGMAP help Support for ZL38060 Connected Home Audio Processor from Microsemi, -- 2.39.2

2 1

[PATCH] ASoC: sam9g20ek: Disable capture unless building with microphone input
by Mark Brown 28 Feb '23

28 Feb '23

Without modification the AT91SAM9G20-EK has no capture support, none of the inputs of the CODEC are wired to anything to useful and there are no paths supporting loopback. Since the audio is clocked from the CODEC and the DAPM inputs are marked as unusable this means that capture will fail to transfer any data as the ADC path can't be powered up. Flag this in the device description so apps don't see unusable capture support, guarded with the existing optional define for mic input. Signed-off-by: Mark Brown <broonie(a)kernel.org> --- sound/soc/atmel/sam9g20_wm8731.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/soc/atmel/sam9g20_wm8731.c b/sound/soc/atmel/sam9g20_wm8731.c index 1430642c8433..785b9d01d8af 100644 --- a/sound/soc/atmel/sam9g20_wm8731.c +++ b/sound/soc/atmel/sam9g20_wm8731.c @@ -98,6 +98,9 @@ static struct snd_soc_dai_link at91sam9g20ek_dai = { .init = at91sam9g20ek_wm8731_init, .dai_fmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF | SND_SOC_DAIFMT_CBP_CFP, +#ifndef ENABLE_MIC_INPUT + .playback_only = true, +#endif SND_SOC_DAILINK_REG(pcm), }; --- base-commit: cac85e4616b1cf4a90844b952b49b9cbc4562530 change-id: 20230225-asoc-sam9g20ek-f4e775205d8c Best regards, -- Mark Brown <broonie(a)kernel.org>

2 2

[PATCH 0/2] ASoC: mt8183: Fixes from an initial glance at a kselftest run
by Mark Brown 28 Feb '23

28 Feb '23

This is a collection of fixes I came up after glancing through an initial test run with the snappily named Kukui Jacuzzi SKU16 Chromebook on KernelCI. There are more issues flagged, this is just what I fixed thus far. Signed-off-by: Mark Brown <broonie(a)kernel.org> --- Mark Brown (2): ASoC: mt8183: Remove spammy logging from I2S DAI driver ASoC: mt8183: Fix event generation for I2S DAI operations sound/soc/mediatek/mt8183/mt8183-dai-i2s.c | 21 +++------------------ 1 file changed, 3 insertions(+), 18 deletions(-) --- base-commit: d2980d8d826554fa6981d621e569a453787472f8 change-id: 20230224-asoc-mt8183-quick-fixes-ccb7c567c755 Best regards, -- Mark Brown <broonie(a)kernel.org>

2 5

[PATCH 0/3] ASoC: mt6358: Fixes from an initial glance at a kselftest run
by Mark Brown 28 Feb '23

28 Feb '23

This is a collection of fixes I came up after glancing through an initial test run with the snappily named Kukui Jacuzzi SKU16 Chromebook on KernelCI. There are more issues flagged, this is just what I fixed thus far. Signed-off-by: Mark Brown <broonie(a)kernel.org> --- Mark Brown (3): ASoC: mt6358: Fix event generation for wake on voice stage 2 switch ASoC: mt6358: Validate Wake on Voice 2 writes ASoC: mt6358: Remove undefined HPx Mux enumeration values sound/soc/codecs/mt6358.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) --- base-commit: d2980d8d826554fa6981d621e569a453787472f8 change-id: 20230224-asoc-mt6358-quick-fixes-71b4ea6b8f13 Best regards, -- Mark Brown <broonie(a)kernel.org>

2 7

[PATCH 0/4] ASoC: mt8192: Fixes from initial glance at kselftest run
by Mark Brown 28 Feb '23

28 Feb '23

This is a collection of fixes I came up after glancing through an initial test run with the Spherion Chromebook on KernelCI. There are more issues flagged, this is just what I fixed thus far - the volume controls on the MT6359 have issues for example, and a lot of controls aren't marked as Switches like they should be. Signed-off-by: Mark Brown <broonie(a)kernel.org> --- Mark Brown (4): ASoC: mt8192: Remove spammy log messages ASoC: mt8192: Fix event generation for controls ASoC: mt8192: Report an error if when an invalid sidetone gain is written ASoC: mt8192: Fix range for sidetone positive gain sound/soc/mediatek/mt8192/mt8192-dai-adda.c | 58 ++++++++++------------------- 1 file changed, 19 insertions(+), 39 deletions(-) --- base-commit: b361d5d2464a88184f6e17a6462719ba79180b1a change-id: 20230223-asoc-mt8192-quick-fixes-8f3e0a187226 Best regards, -- Mark Brown <broonie(a)kernel.org>

3 15