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

[PATCH v2 0/5] ASoC: SOF: Add PM support for i.MX8/i.MX8X/i.MX8M
by Daniel Baluta 23 Nov '21

23 Nov '21

From: Daniel Baluta <daniel.baluta(a)nxp.com> This patch series adds support for System PM and Runtime PM with SOF for i.MX8 platforms. First patch adds common code to manage clocks, patch 2/3 adds the actual PM support, patch 4/5 implement the DSP start / reset callbacks for i.mx8m. Changes since v1: * rebased on top of Mark's for-next branch Daniel Baluta (5): ASoC: SOF: imx: Add code to manage DSP related clocks ASoC: SOF: imx8: Add runtime PM / System PM support ASoC: SOF: imx8m: Add runtime PM / System PM support ASoC: SOF: imx8m: Implement DSP start ASoC: SOF: imx8m: Implement reset callback sound/soc/sof/imx/imx-common.c | 24 +++++ sound/soc/sof/imx/imx-common.h | 11 ++ sound/soc/sof/imx/imx8.c | 137 +++++++++++++++++++++++ sound/soc/sof/imx/imx8m.c | 191 ++++++++++++++++++++++++++++++++- 4 files changed, 362 insertions(+), 1 deletion(-) -- 2.27.0

2 6

[PATCH 0/5] ASoC: SOF: Add PM support for i.MX8/i.MX8X/i.MX8M
by Daniel Baluta 23 Nov '21

23 Nov '21

From: Daniel Baluta <daniel.baluta(a)nxp.com> This patch series adds support for System PM and Runtime PM with SOF for i.MX8 platforms. First patch adds common code to manage clocks, patch 2/3 adds the actual PM support, patch 4/5 implement the DSP start / reset callbacks for i.mx8m. Daniel Baluta (5): ASoC: SOF: imx: Add code to manage DSP related clocks ASoC: SOF: imx8: Add runtime PM / System PM support ASoC: SOF: imx8m: Add runtime PM / System PM support ASoC: SOF: imx8m: Implement DSP start ASoC: SOF: imx8m: Implement reset callback sound/soc/sof/imx/imx-common.c | 24 +++++ sound/soc/sof/imx/imx-common.h | 11 ++ sound/soc/sof/imx/imx8.c | 137 +++++++++++++++++++++++ sound/soc/sof/imx/imx8m.c | 191 ++++++++++++++++++++++++++++++++- 4 files changed, 362 insertions(+), 1 deletion(-) -- 2.27.0

2 7

[PATCH 5.16 regression fix] ASoC: soc-acpi: Set mach->id field on comp_ids matches
by Hans de Goede 23 Nov '21

23 Nov '21

Commit dac7cbd55dca ("ASoC: Intel: soc-acpi-byt: shrink tables using compatible IDs") and commit 959ae8215a9e ("ASoC: Intel: soc-acpi-cht: shrink tables using compatible IDs") simplified the match tables in soc-acpi-intel-byt-match.c and soc-acpi-intel-cht-match.c by merging identical entries using the new .comp_ids snd_soc_acpi_mach field to point a single entry to multiple ACPI HIDs and clearing the previously unique per entry .id field. But various machine drivers from sound/soc/intel/boards rely on mach->id in one or more ways, e.g. some drivers contain the following snippets: adev = acpi_dev_get_first_match_dev(mach->id, NULL, -1); pkg_found = snd_soc_acpi_find_package_from_hid(mach->id, ... if (!strncmp(snd_soc_cards[i].codec_id, mach->id, 8)) { ... All of which are broken by the match table shrinking. Make the snd_soc_acpi_mach.id field non const (the storage for the tables already is non const) and on a comps_ids match copy the matching HID to the id field to fix this. Fixes: dac7cbd55dca ("ASoC: Intel: soc-acpi-byt: shrink tables using compatible IDs") Fixes: 959ae8215a9e ("ASoC: Intel: soc-acpi-cht: shrink tables using compatible IDs") Suggested-by: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> Cc: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> Cc: Brent Lu <brent.lu(a)intel.com> Signed-off-by: Hans de Goede <hdegoede(a)redhat.com> --- include/sound/soc-acpi.h | 2 +- sound/soc/soc-acpi.c | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/include/sound/soc-acpi.h b/include/sound/soc-acpi.h index 31f4c4f9aeea..ac0893df9c76 100644 --- a/include/sound/soc-acpi.h +++ b/include/sound/soc-acpi.h @@ -147,7 +147,7 @@ struct snd_soc_acpi_link_adr { */ /* Descriptor for SST ASoC machine driver */ struct snd_soc_acpi_mach { - const u8 id[ACPI_ID_LEN]; + u8 id[ACPI_ID_LEN]; const struct snd_soc_acpi_codecs *comp_ids; const u32 link_mask; const struct snd_soc_acpi_link_adr *links; diff --git a/sound/soc/soc-acpi.c b/sound/soc/soc-acpi.c index 2ae99b49d3f5..cbd7ea48837b 100644 --- a/sound/soc/soc-acpi.c +++ b/sound/soc/soc-acpi.c @@ -20,8 +20,10 @@ static bool snd_soc_acpi_id_present(struct snd_soc_acpi_mach *machine) if (comp_ids) { for (i = 0; i < comp_ids->num_codecs; i++) { - if (acpi_dev_present(comp_ids->codecs[i], NULL, -1)) + if (acpi_dev_present(comp_ids->codecs[i], NULL, -1)) { + strscpy(machine->id, comp_ids->codecs[i], ACPI_ID_LEN); return true; + } } } -- 2.31.1

3 2

Xonar U7 MKII Analog Multichannel and SPDIF
by GitHub issues - opened 21 Nov '21

21 Nov '21

alsa-project/alsa-lib issue #194 was opened from Gebbx: Hi, Usb sound card Xonar U7 MKII plays multichannel content without issue, but only through aplay `aplay -D hw:2,0 test.wav #6 channels file` in other cases only two channels work (downmixed) I have tried different combinations of settings. Editing asound.conf gives no results too. The digital output also does not work in any combination of settings, but works (only two channels) in the following case: `pcm_slave.u7spdif { pcm "hw:2,1" } pcm.u7spplug { type plug slave u7spdif hint { show on } }` If I understand correctly, in this case it output the decoded signal and it will always be two-channel. I found a topic: https://forum.kodi.tv/showthread.php?tid=253179 So, i have to edit USB-Audio.conf, ``` USB-Audio.pcm.iec958_device { # "NoiseBlaster 3000" 42 "USB Sound Blaster HD" 1 "Xonar U7" 1 "MKII" 1 # my card } ``` but this is not recommended. How can this be done in another way? Example given in the USB-Audio.conf file forces system to reboot. `cards.USB-Audio.pcm.iec958_device."MKII" 1` Thanks! My system: RPI4, LibreELEC 10 Issue URL : https://github.com/alsa-project/alsa-lib/issues/194 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

Where can I find alsa forum?
by GitHub issues - edited 21 Nov '21

21 Nov '21

alsa-project/alsa-lib issue #193 was edited from yangyunfei16: Dear alsa developers: Hello. I am a graduate student in the Institute of information engineering, Chinese Academy of Sciences. We are doing a project to investigate the use of this library by some programmers who use alsa for programming, so we want to ask if there are any websites or forums on which more programmers who use this library communicate and discuss, We want them to do a questionnaire later. Thank you very much for your help! Issue URL : https://github.com/alsa-project/alsa-lib/issues/193 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

project
by GitHub issues - opened 21 Nov '21

21 Nov '21

alsa-project/alsa-lib issue #193 was opened from yangyunfei16: Dear alsa developers: Hello. I am a graduate student in the Institute of information engineering, Chinese Academy of Sciences. We are doing a project to investigate the use of this library by some programmers who use alsa for programming, so we want to ask if there are any websites or forums on which more programmers who use this library communicate and discuss, We want them to do a questionnaire later. Thank you very much for your help! Issue URL : https://github.com/alsa-project/alsa-lib/issues/193 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[PATCH 0/2] ASoC: wm8962: Conversion to json-schema and fix
by Geert Uytterhoeven 21 Nov '21

21 Nov '21

Hi all, This patch series converts the Wolfson WM8962 Device Tree binding documentation to json-schema, after fixing an issue in the imx8mn-beacon DTS file. Thanks for your comments! Geert Uytterhoeven (2): arm64: dts: imx: imx8mn-beacon: Drop undocumented clock-names reference ASoC: dt-bindings: wlf,wm8962: Convert to json-schema .../devicetree/bindings/sound/wlf,wm8962.yaml | 118 ++++++++++++++++++ .../devicetree/bindings/sound/wm8962.txt | 43 ------- .../freescale/imx8mn-beacon-baseboard.dtsi | 1 - 3 files changed, 118 insertions(+), 44 deletions(-) create mode 100644 Documentation/devicetree/bindings/sound/wlf,wm8962.yaml delete mode 100644 Documentation/devicetree/bindings/sound/wm8962.txt -- 2.25.1 Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert(a)linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds

5 6

[PATCH] ALSA: intel-dsp-config: add quirk for JSL devices based on ES8336 codec
by Bard Liao 20 Nov '21

20 Nov '21

From: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> These devices are based on an I2C/I2S device, we need to force the use of the SOF driver otherwise the legacy HDaudio driver will be loaded - only HDMI will be supported. We previously added support for other Intel platforms but missed JasperLake. BugLink: https://github.com/thesofproject/linux/issues/3210 Fixes: 9d36ceab9415 ('ALSA: intel-dsp-config: add quirk for APL/GLK/TGL devices based on ES8336 codec') Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> Reviewed-by: Kai Vehmanen <kai.vehmanen(a)intel.com> Signed-off-by: Bard Liao <yung-chuan.liao(a)linux.intel.com> --- sound/hda/intel-dsp-config.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/sound/hda/intel-dsp-config.c b/sound/hda/intel-dsp-config.c index b9ac9e9e45a4..10a0bffc3cf6 100644 --- a/sound/hda/intel-dsp-config.c +++ b/sound/hda/intel-dsp-config.c @@ -299,6 +299,15 @@ static const struct config_entry config_table[] = { }, #endif +/* JasperLake */ +#if IS_ENABLED(CONFIG_SND_SOC_SOF_JASPERLAKE) + { + .flags = FLAG_SOF, + .device = 0x4dc8, + .codec_hid = "ESSX8336", + }, +#endif + /* Tigerlake */ #if IS_ENABLED(CONFIG_SND_SOC_SOF_TIGERLAKE) { -- 2.17.1

3 3

Warning due to "ALSA: hda: intel: More comprehensive PM runtime setup for controller driver"
by Heiner Kallweit 19 Nov '21

19 Nov '21

I get the following warning caused by 4f66a9ef37d3 ("ALSA: hda: intel: More comprehensive PM runtime setup for controller driver"): snd_hda_intel 0000:00:1f.3: Unbalanced pm_runtime_enable! Not sure how this patch was tested because the warning is obvious. The patch doesn't consider what the PCI sub-system does with regard to RPM. Have a look at pci_pm_init(). I'd understand to add the call to pm_runtime_dont_use_autosuspend(), but for all other added calls I see no justification. If being unsure about when to use which RPM call best involve linux-pm(a)vger.kernel.org.

3 5

[PATCH] ALSA: hda: Remove redundant runtime PM calls
by Takashi Iwai 19 Nov '21

19 Nov '21

The previous fix for more comprehensive runtime PM calls turned out to be not good as hoped; a few calls including pm_runtime_enable() and pm_runtime_disable() are rather utterly superfluous for PCI devices, even triggering a kernel error message. Better to drop those calls. Note that the problem we wanted to solve with that commit seems irrelevant with the fix itself; the original bug (a GPF at azx_remove()) was likely a regression by the recent PCI core cleanup, and the buggy PCI change has been already reverted. So basically we were scratching a wrong surface. OTOH, making the runtime PM calls symmetric for both probe and remove is more consistent, and maybe that's a sensible outcome. Fixes: 4f66a9ef37d3 ("ALSA: hda: intel: More comprehensive PM runtime setup for controller driver") Reported-by: Heiner Kallweit <hkallweit1(a)gmail.com> Link: https://lore.kernel.org/r/d9d76980-966a-e031-70d1-3254ba5be5eb@gmail.com Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/pci/hda/hda_intel.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index 45e85180048c..221afacbc7fd 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -1349,8 +1349,6 @@ static void azx_free(struct azx *chip) if (azx_has_pm_runtime(chip) && chip->running) { pm_runtime_get_noresume(&pci->dev); - pm_runtime_disable(&pci->dev); - pm_runtime_set_suspended(&pci->dev); pm_runtime_forbid(&pci->dev); pm_runtime_dont_use_autosuspend(&pci->dev); } @@ -2328,8 +2326,6 @@ static int azx_probe_continue(struct azx *chip) if (azx_has_pm_runtime(chip)) { pm_runtime_use_autosuspend(&pci->dev); pm_runtime_allow(&pci->dev); - pm_runtime_set_active(&pci->dev); - pm_runtime_enable(&pci->dev); pm_runtime_put_autosuspend(&pci->dev); } -- 2.31.1

1 0