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

Re: Re: [PATCH] ALSA: intel_hdmi: Check for error num after setting mask
by Jiasheng Jiang 06 Jan '22

06 Jan '22

On Thu, Jan 06, 2022 at 04:18:01PM +0800, Takashi Iwai wrote: > Well, 32bit DMA mask practically never fails on x86 (and other > architectures, IIRC). It's fine to add a sanity check, but it's > better to be mentioned that it never fails. Actually, I have already found many place check the 32bit DMA mask. For example: snd_ad1889_create() in `sound/pci/ad1889.c`, snd_vortex_create() in `sound/pci/au88x0/au88x0.c` tegra_pcm_dma_allocate() in `sound/soc/tegra/tegra_pcm.c`. Therefore, I think there must be some reason that 32bit may fail. So, to make the system more robust, it should be better to add the sanity check. Sincerely thanks, Jiang

2 1

[PATCH] ALSA: intel_hdmi: Check for error num after setting mask
by Jiasheng Jiang 06 Jan '22

06 Jan '22

As the dma_supported() may fail, the dma_set_mask_and_coherent() may fail too. Therefore, it should be better to check it and return error if fails. Fixes: da8648097497 ("ALSA: x86: Flatten two abstraction layers") Signed-off-by: Jiasheng Jiang <jiasheng(a)iscas.ac.cn> --- sound/x86/intel_hdmi_audio.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sound/x86/intel_hdmi_audio.c b/sound/x86/intel_hdmi_audio.c index 33b12aa67cf5..6caea517f07f 100644 --- a/sound/x86/intel_hdmi_audio.c +++ b/sound/x86/intel_hdmi_audio.c @@ -1770,7 +1770,9 @@ static int hdmi_lpe_audio_probe(struct platform_device *pdev) card_ctx->irq = irq; /* only 32bit addressable */ - dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32)); + ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32)); + if (ret) + goto err; init_channel_allocations(); -- 2.25.1

2 1

hda-analyzer porting to GTK3
by GitHub issues - opened 06 Jan '22

06 Jan '22

alsa-project/alsa-utils issue #134 was opened from kosiu: I know that hda-analyzer is not a part of alsa-utils. I'm trying to port it into GTK3 at first but I encounter problems with scrolling of the graph. Source is here: https://github.com/kosiu/alsa/tree/gtk3 Issue URL : https://github.com/alsa-project/alsa-utils/issues/134 Repository URL: https://github.com/alsa-project/alsa-utils

1 0

UAC2: are UAC2 latency controls included in snd_pcm_status_get_delay?
by Pavel Hofman 06 Jan '22

06 Jan '22

Hi, The UAC2 ch. "5.2.4.8 Latency Control" specifies latencies optionally reported by the device for individual entities. Especially the overall terminal latency TE_LATENCY_CONTROL is interesting. Some video players include the delay reported by snd_pcm_status_get_delay into their AV sync calculation. A linux USB audio gadget can perform some advanced DSP, which including the output device latency can result in a non-negligible overall latency. The gadget userspace software could configure the actual value to be reported by the gadget to the host in the selector UAC2_TE_LATENCY (audio_v2.h). The UAC2-specced max value of over 4 secs is more than sufficient for standard use cases. But I could not find any read of the UAC2_TE_LATENCY control selector in the snd-usb-audio driver ( https://elixir.bootlin.com/linux/latest/C/ident/UAC2_TE_LATENCY ). Please does the driver include this value into the overall delay reported through alsa-lib? If not, would it make sense to add such support? Thanks a lot, Pavel.

1 0

[PATCH v3] dt-bindings: dsp: mediatek: add mt8195 dsp document
by YC Hung 06 Jan '22

06 Jan '22

From: "YC Hung" <yc.hung(a)mediatek.com> This patch adds mt8195 dsp document. The dsp is used for Sound Open Firmware driver node. It includes registers, clocks, memory regions, and mailbox for dsp. Signed-off-by: yc.hung <yc.hung(a)mediatek.com> --- Changes since v2: Remove useless watchdog interrupt. Add commit message more detail description. Changes since v1: Rename yaml file name as mediatek,mt8195-dsp.yaml Refine descriptions for mailbox, memory-region and drop unused labels in examples. --- --- .../devicetree/bindings/dsp/mediatek,mt8195-dsp.yaml | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/Documentation/devicetree/bindings/dsp/mediatek,mt8195-dsp.yaml b/Documentation/devicetree/bindings/dsp/mediatek,mt8195-dsp.yaml index bde763191d86..779daa786739 100644 --- a/Documentation/devicetree/bindings/dsp/mediatek,mt8195-dsp.yaml +++ b/Documentation/devicetree/bindings/dsp/mediatek,mt8195-dsp.yaml @@ -27,14 +27,6 @@ properties: - const: cfg - const: sram - interrupts: - items: - - description: watchdog interrupt - - interrupt-names: - items: - - const: wdt - clocks: items: - description: mux for audio dsp clock @@ -75,8 +67,6 @@ required: - compatible - reg - reg-names - - interrupts - - interrupt-names - clocks - clock-names - memory-region @@ -95,8 +85,6 @@ examples: reg = <0x10803000 0x1000>, <0x10840000 0x40000>; reg-names = "cfg", "sram"; - interrupts = <GIC_SPI 694 IRQ_TYPE_LEVEL_HIGH 0>; - interrupt-names = "wdt"; clocks = <&topckgen 10>, //CLK_TOP_ADSP <&clk26m>, <&topckgen 107>, //CLK_TOP_AUDIO_LOCAL_BUS -- 2.18.0

2 2

[PATCH] ALSA: hda: ALC287: Add Lenovo IdeaPad Slim 9i 14ITL5 speaker quirk
by Bart Kroon 05 Jan '22

05 Jan '22

Hello, The speaker fixup that is used for the Yoga 7 14ITL5 also applies to the IdeaPad Slim 9i 14ITL5. The attached patch applies the quirk to initialise the amplifier on the IdeaPad Slim 9i as well. This is validated to work on my laptop. Signed-off-by: Bart Kroon <bart(a)tarmack.eu> Regards, Bart

2 1

[PATCH] ASoC: ak4375: Fix unused function error
by Takashi Iwai 05 Jan '22

05 Jan '22

A randconfig caught a compile warning that is now treated as a fatal error: sound/soc/codecs/ak4375.c:415:13: error: ‘ak4375_power_off’ defined but not used [-Werror=unused-function] where ak4375_power_off() is used only from the PM handler. As both suspend and resumes are already marked with __maybe_unused, let's rip off the superfluous ifdef CONFIG_PM, so that the error above can be avoided. Fixes: 53778b8292b5 ("ASoC: Add AK4375 support") Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/soc/codecs/ak4375.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/sound/soc/codecs/ak4375.c b/sound/soc/codecs/ak4375.c index 22cda0699341..9a7b662016b9 100644 --- a/sound/soc/codecs/ak4375.c +++ b/sound/soc/codecs/ak4375.c @@ -438,7 +438,6 @@ static int ak4375_power_on(struct ak4375_priv *ak4375) return 0; } -#ifdef CONFIG_PM static int __maybe_unused ak4375_runtime_suspend(struct device *dev) { struct ak4375_priv *ak4375 = dev_get_drvdata(dev); @@ -463,7 +462,6 @@ static int __maybe_unused ak4375_runtime_resume(struct device *dev) return regcache_sync(ak4375->regmap); } -#endif /* CONFIG_PM */ static const struct snd_soc_component_driver soc_codec_dev_ak4375 = { .controls = ak4375_snd_controls, -- 2.31.1

2 1

Handling SIGTSTP (Ctrl+Z) /SIGCONT/fg with aplay/cplay
by Daniel Baluta 05 Jan '22

05 Jan '22

Hi all, Is there any quick document for handling Ctrl + Z with aplay/cplay? I don't see this signal handled with alsa-lib or tinycompress although linux kernel offers PAUSE/RESUME ioctls. What should be the expected behavior? thanks, Daniel.

3 2

[PATCH 0/8] Add low power hibernation support to cs35l41
by Charles Keepax 05 Jan '22

05 Jan '22

This patch series adds support for the low power hibernation feature on cs35l41. This allows the DSP memory to be retained whilst the device enters a very low power state. Patches 1-6 can happily be applied straight away and are mostly bug fixes to set things up for the series specifically around getting the cache handling corrected in the driver. Patches 7,8 specifically will cause some very minor conflicts with Lucas's currently outstanding work on the HDA version of cs35l41. Whilst things will still build, this patch adds a test key function his code will now have to call. If his patches are getting merged first I will respin this series to update his code, he is currently on holiday until the 12th of Jan, so if we want to wait for another spin of those patches I can work with him to update them at that time. Or alternatively we could just merge them all and I will do a quick fixup patch at the end, since there is no build breakage. Thanks, Charles Charles Keepax (7): ASoC: cs35l41: Remove incorrect comment ASoC: cs35l41: Correct DSP power down ASoC: cs35l41: Correct handling of some registers in the cache ASoC: cs35l41: Update handling of test key registers firmware: cs_dsp: Clear core reset for cache ASoC: wm_adsp: Add support for "toggle" preloaders ASoC: cs35l41: Add support for hibernate memory retention mode David Rhodes (1): ASoC: cs35l41: Add cs35l51/53 IDs drivers/firmware/cirrus/cs_dsp.c | 14 ++- include/sound/cs35l41.h | 7 ++ sound/soc/codecs/cs35l41-i2c.c | 3 + sound/soc/codecs/cs35l41-lib.c | 152 ++++++++++-------------- sound/soc/codecs/cs35l41-spi.c | 3 + sound/soc/codecs/cs35l41.c | 246 +++++++++++++++++++++++++++++++++++---- sound/soc/codecs/cs35l41.h | 4 + sound/soc/codecs/wm_adsp.c | 14 ++- sound/soc/codecs/wm_adsp.h | 8 ++ 9 files changed, 328 insertions(+), 123 deletions(-) -- 2.11.0 Charles Keepax (7): ASoC: cs35l41: Remove incorrect comment ASoC: cs35l41: Correct DSP power down ASoC: cs35l41: Correct handling of some registers in the cache firmware: cs_dsp: Clear core reset for cache ASoC: wm_adsp: Add support for "toggle" preloaders ASoC: cs35l41: Update handling of test key registers ASoC: cs35l41: Add support for hibernate memory retention mode David Rhodes (1): ASoC: cs35l41: Add cs35l51/53 IDs drivers/firmware/cirrus/cs_dsp.c | 14 ++- include/sound/cs35l41.h | 7 ++ sound/soc/codecs/cs35l41-i2c.c | 3 + sound/soc/codecs/cs35l41-lib.c | 152 ++++++++++-------------- sound/soc/codecs/cs35l41-spi.c | 3 + sound/soc/codecs/cs35l41.c | 246 +++++++++++++++++++++++++++++++++++---- sound/soc/codecs/cs35l41.h | 4 + sound/soc/codecs/wm_adsp.c | 14 ++- sound/soc/codecs/wm_adsp.h | 8 ++ 9 files changed, 328 insertions(+), 123 deletions(-) -- 2.11.0

3 15

[PATCH v3] ASoC: cs4265: Add a remove() function
by Fabio Estevam 05 Jan '22

05 Jan '22

From: Fabio Estevam <festevam(a)denx.de> When the reset_gpio GPIO is used, it is better to put the codec back into reset state when the driver unbinds. Add a remove() function to accomplish that. Suggested-by: Charles Keepax <ckeepax(a)opensource.cirrus.com> Signed-off-by: Fabio Estevam <festevam(a)denx.de> --- Changes since v2: - Keep using the current polarity logic. sound/soc/codecs/cs4265.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/sound/soc/codecs/cs4265.c b/sound/soc/codecs/cs4265.c index 294fa7ac16cb..8fa166e4b2a9 100644 --- a/sound/soc/codecs/cs4265.c +++ b/sound/soc/codecs/cs4265.c @@ -626,6 +626,16 @@ static int cs4265_i2c_probe(struct i2c_client *i2c_client, ARRAY_SIZE(cs4265_dai)); } +static int cs4265_i2c_remove(struct i2c_client *i2c) +{ + struct cs4265_private *cs4265 = i2c_get_clientdata(i2c); + + if (cs4265->reset_gpio) + gpiod_set_value_cansleep(cs4265->reset_gpio, 0); + + return 0; +} + static const struct of_device_id cs4265_of_match[] = { { .compatible = "cirrus,cs4265", }, { } @@ -645,6 +655,7 @@ static struct i2c_driver cs4265_i2c_driver = { }, .id_table = cs4265_id, .probe = cs4265_i2c_probe, + .remove = cs4265_i2c_remove, }; module_i2c_driver(cs4265_i2c_driver); -- 2.25.1

3 2