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

[PATCH] slimbus: qcom-ngd: Add check for platform_driver_register
by Jiasheng Jiang 12 Dec '22

12 Dec '22

Add the check for the return value of platform_driver_register in order to catch the exception. Fixes: 917809e2280b ("slimbus: ngd: Add qcom SLIMBus NGD driver") Signed-off-by: Jiasheng Jiang <jiasheng(a)iscas.ac.cn> --- drivers/slimbus/qcom-ngd-ctrl.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/slimbus/qcom-ngd-ctrl.c b/drivers/slimbus/qcom-ngd-ctrl.c index 76c5e446d243..c16fc9e6d1c6 100644 --- a/drivers/slimbus/qcom-ngd-ctrl.c +++ b/drivers/slimbus/qcom-ngd-ctrl.c @@ -1590,7 +1590,10 @@ static int qcom_slim_ngd_ctrl_probe(struct platform_device *pdev) goto err_pdr_lookup; } - platform_driver_register(&qcom_slim_ngd_driver); + ret = platform_driver_register(&qcom_slim_ngd_driver); + if (ret < 0) + goto err_pdr_lookup; + return of_qcom_slim_ngd_register(dev, ctrl); err_pdr_alloc: -- 2.25.1

2 1

Add support for ASUS ROG Crosshair X670 Extreme
by GitHub pull_request - opened 11 Dec '22

11 Dec '22

alsa-project/alsa-ucm-conf pull request #249 was opened from ioquatix: - Confirmed working. Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/249 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/249.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

Sound is pitched and slowed down when going through the sound card of a mixer controller
by GitHub issues - opened 10 Dec '22

10 Dec '22

alsa-project/alsa-firmware issue #7 was opened from mYnDstrEAm: First of all I'm not sure if that's a bug in ALSA and if so if this is the right repo to file it. When I connect my mixer to my Debian11/KDE machine the sound is slowed and pitched down. Sometimes it takes days or weeks until this effect occurs, most of the time everything works well and I don't know what causes this to occur. **See https://unix.stackexchange.com/questions/572895/linux-sound-is-pitched-and-… (and comments below)** `aplay -l` says the driver is module-alsa-card.c. The track is still pitched down when played with `aplay`. I'm not sure if the sound really is slowed down or if it was changed in a slightly different way in the past (had this bug for years now), but right now when I use my mixer rather than internal audio, the sound is lower in pitch. Let me know if there's something I could check like something in some logs. Issue URL : https://github.com/alsa-project/alsa-firmware/issues/7 Repository URL: https://github.com/alsa-project/alsa-firmware

1 0

Simple card and PLL/FLL
by Emanuele Ghidoli 09 Dec '22

09 Dec '22

Hello, I want to understand what is the right way to implement PLL/FLL usage on audio codec when using simple-card. In wm8904 driver I found that FLL is used if set_sysclk is called (that means mclk-fs property is set, otherwise it is not called). In this case the frequency passed to set_sysclk is assumed to be the "desired" frequency, the codec read the currently clkrate coming from "cpu" ("clk_get_rate(priv->mclk)") and use these information to program FLL. In other codec drives I do not find a similar approach. In general codec register the set_pll function but it is never called by simple-card driver. I guess what is the "right/better" implementation? Have we to add the set_pll call in simple-card? Or have we to add it to e.g. fsl_sai driver? Or, in some way, the wm8904 codec driver is approaching in the right way? Is there any documentation that explain all of that (I have already looked at Documentation/sound)? Any driver that is considered well written and complete I should use as a reference? Thanks a lot for your help Best regards, Emanuele

4 4

[PATCH v1] ASoC: Intel: sof_nau8825: add support for nau8825 with amp nau8318
by Ajye Huang 09 Dec '22

09 Dec '22

This patch adds the driver data for two nau8318 speaker amplifiers on SSP1 and nau8825 on SSP0 for ADL platform. And reusing max98360's topology since DAI setting could be leveraged. Signed-off-by: Ajye Huang <ajye_huang(a)compal.corp-partner.google.com> --- sound/soc/intel/boards/Kconfig | 1 + sound/soc/intel/boards/sof_nau8825.c | 23 +++++++++++++++++++ .../intel/common/soc-acpi-intel-adl-match.c | 12 ++++++++++ 3 files changed, 36 insertions(+) diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig index a472de1909f4..3f68e9edd853 100644 --- a/sound/soc/intel/boards/Kconfig +++ b/sound/soc/intel/boards/Kconfig @@ -554,6 +554,7 @@ config SND_SOC_INTEL_SOF_NAU8825_MACH select SND_SOC_RT1015P select SND_SOC_MAX98373_I2C select SND_SOC_MAX98357A + select SND_SOC_NAU8315 select SND_SOC_DMIC select SND_SOC_HDAC_HDMI select SND_SOC_INTEL_HDA_DSP_COMMON diff --git a/sound/soc/intel/boards/sof_nau8825.c b/sound/soc/intel/boards/sof_nau8825.c index 27880224359d..0936450be153 100644 --- a/sound/soc/intel/boards/sof_nau8825.c +++ b/sound/soc/intel/boards/sof_nau8825.c @@ -48,6 +48,7 @@ #define SOF_MAX98373_SPEAKER_AMP_PRESENT BIT(15) #define SOF_MAX98360A_SPEAKER_AMP_PRESENT BIT(16) #define SOF_RT1015P_SPEAKER_AMP_PRESENT BIT(17) +#define SOF_NAU8318_SPEAKER_AMP_PRESENT BIT(18) static unsigned long sof_nau8825_quirk = SOF_NAU8825_SSP_CODEC(0); @@ -338,6 +339,13 @@ static struct snd_soc_dai_link_component rt1019p_component[] = { } }; +static struct snd_soc_dai_link_component nau8318_components[] = { + { + .name = "NVTN2012:00", + .dai_name = "nau8315-hifi", + } +}; + static struct snd_soc_dai_link_component dummy_component[] = { { .name = "snd-soc-dummy", @@ -486,6 +494,11 @@ static struct snd_soc_dai_link *sof_card_dai_links_create(struct device *dev, max_98360a_dai_link(&links[id]); } else if (sof_nau8825_quirk & SOF_RT1015P_SPEAKER_AMP_PRESENT) { sof_rt1015p_dai_link(&links[id]); + } else if (sof_nau8825_quirk & + SOF_NAU8318_SPEAKER_AMP_PRESENT) { + links[id].codecs = nau8318_components; + links[id].num_codecs = ARRAY_SIZE(nau8318_components); + links[id].init = speaker_codec_init; } else { goto devm_err; } @@ -657,6 +670,16 @@ static const struct platform_device_id board_ids[] = { SOF_BT_OFFLOAD_SSP(2) | SOF_SSP_BT_OFFLOAD_PRESENT), }, + { + .name = "adl_nau8318_nau8825", + .driver_data = (kernel_ulong_t)(SOF_NAU8825_SSP_CODEC(0) | + SOF_SPEAKER_AMP_PRESENT | + SOF_NAU8318_SPEAKER_AMP_PRESENT | + SOF_NAU8825_SSP_AMP(1) | + SOF_NAU8825_NUM_HDMIDEV(4) | + SOF_BT_OFFLOAD_SSP(2) | + SOF_SSP_BT_OFFLOAD_PRESENT), + }, { } }; MODULE_DEVICE_TABLE(platform, board_ids); diff --git a/sound/soc/intel/common/soc-acpi-intel-adl-match.c b/sound/soc/intel/common/soc-acpi-intel-adl-match.c index 60aee56f94bd..282b9c966ce6 100644 --- a/sound/soc/intel/common/soc-acpi-intel-adl-match.c +++ b/sound/soc/intel/common/soc-acpi-intel-adl-match.c @@ -450,6 +450,11 @@ static const struct snd_soc_acpi_codecs adl_lt6911_hdmi = { .codecs = {"INTC10B0"} }; +static const struct snd_soc_acpi_codecs adl_nau8318_amp = { + .num_codecs = 1, + .codecs = {"NVTN2012"} +}; + struct snd_soc_acpi_mach snd_soc_acpi_intel_adl_machines[] = { { .comp_ids = &adl_rt5682_rt5682s_hp, @@ -507,6 +512,13 @@ struct snd_soc_acpi_mach snd_soc_acpi_intel_adl_machines[] = { .quirk_data = &adl_rt1015p_amp, .sof_tplg_filename = "sof-adl-rt1015-nau8825.tplg", }, + { + .id = "10508825", + .drv_name = "adl_nau8318_nau8825", + .machine_quirk = snd_soc_acpi_codec_list, + .quirk_data = &adl_nau8318_amp, + .sof_tplg_filename = "sof-adl-max98360a-nau8825.tplg", + }, { .id = "10508825", .drv_name = "sof_nau8825", -- 2.25.1

2 4

alsa-project.org is down
by GitHub issues - opened 09 Dec '22

09 Dec '22

alsa-project/alsa-lib issue #287 was opened from afq984: ``` curl -i https://alsa-project.org HTTP/2 404 server: nginx/1.14.1 date: Fri, 09 Dec 2022 16:34:22 GMT content-type: text/html; charset=UTF-8 x-powered-by: PHP/7.4.30 No input file specified. ``` Issue URL : https://github.com/alsa-project/alsa-lib/issues/287 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

Re: ASoC: soc-pcm: Don't zero TDM masks in __soc_pcm_open() breaks SOF Audio in Lenovo laptops
by Richard Fitzgerald 09 Dec '22

09 Dec '22

On 9/12/22 14:42, Pierre-Louis Bossart wrote: > > > On 12/9/22 01:37, Péter Ujfalusi wrote: >> >> >> On 08/12/2022 18:37, Pierre-Louis Bossart wrote: >>> >>> >>> On 12/8/22 10:17, Mark Brown wrote: >>>> On Thu, Dec 08, 2022 at 02:02:02PM +0000, Joakim Tjernlund wrote: >>>>> Several of our Lenovo laptops lost PC audio output in Teams in 5.15.81(also in 5.15.82) >>>>> Revering above patch:https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/comm… >>>>> fixes it. >>>>> >>>>> Any idea what the real fix is? >>>> >>>> Adding the Intel people. I've no idea if there's issues with >>>> dependencies, missing quirks for the hardare or anything in that stable >>>> version. >>> >>> Humm, yes in the past we used the TDM masks to convey the 'stream_tag' >>> for HDaudio. >>> >>> I can still see this in v5.15.85 in sound/soc/sof/intel/hda-dai.c >>> >>> /* set the stream tag in the codec dai dma params */ >>> if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) >>> snd_soc_dai_set_tdm_slot(codec_dai, stream_tag, 0, 0, 0); >>> else >>> snd_soc_dai_set_tdm_slot(codec_dai, 0, stream_tag, 0, 0); >>> >>> >>> That was changed in >>> 636110411ca72 ASoC: Intel/SOF: use set_stream() instead of >>> set_tdm_slots() for HDAudio >>> >>> snd_soc_dai_set_stream(codec_dai, hdac_stream(hext_stream), >>> substream->stream); >>> >>> So my guess is that zeroing out TDM masks has a side effect on older >>> stable kernels, and that effect is not seen on newer kernels. >>> >>> I don't really understand what the side effect might be though. >> >> The reason is that on the the tdm mask now became persistent and the HDA >> code relied on the fact that it is volatile, it is reset for each stream >> open. >> The core would do some fixup if the tx/rx_mask is set: >> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/sound… >> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/sound… >> >> This did not happened when the tx/rx_mask was reset at open - and later >> set bny the HDA driver, I guess after these checks, so the fixup did not >> happened, but without resetting them we would use the previously set >> masks to do fixups for the number of channels (!) based on a stream_tag >> value, which has nothing to do with channels. > > Agree with the analysis, so what would be the least bad recommendation? > a) revert the "don't zero TDM masks" patch > b) backport the change to use set_stream()? Adding alsa-devel(a)alsa-project.org and patches(a)opensource.cirrus.com. Please add relevant lists when sending emails so that other people who might need to know, or have an opinion, are aware of the discussion.

1 0

Re: [PATCH 2/2] dt-bindings: sound: rt5640: Allow to describe how LOUT is wired
by Krzysztof Kozlowski 09 Dec '22

09 Dec '22

On 09/12/2022 11:56, Jarrah Gosbell wrote: > From: Ondrej Jirman <megi(a)xff.cz> > > Depending on HW design, som boards may expect mono differential output > on LOUT. Describe a property that enables it. > > Signed-off-by: Ondrej Jirman <megi(a)xff.cz> > Signed-off-by: Jarrah Gosbell <kernel(a)undef.tools> > --- > Documentation/devicetree/bindings/sound/rt5640.txt | 3 +++ > 1 file changed, 3 insertions(+) Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski(a)linaro.org> Best regards, Krzysztof

1 0

[PATCH v2 0/3] ALSA: hda/hdmi: i915 keepalive fixes
by Kai Vehmanen 09 Dec '22

09 Dec '22

A series with multiple fixes to i915 keepalive (KAE) functionality. First patch fixes issue that is hit on some A750/770 cards: https://gitlab.freedesktop.org/drm/intel/-/issues/7353 The two other improve behaviour especially with certain USB-C docks. Changes in v2: - check for errors from pin_id_to_pin_index() Kai Vehmanen (3): ALSA: hda/hdmi: fix i915 silent stream programming flow ALSA: hda/hdmi: set default audio parameters for KAE silent-stream ALSA: hda/hdmi: fix stream-id config keep-alive for rt suspend include/sound/hda_codec.h | 1 + sound/pci/hda/hda_codec.c | 3 +- sound/pci/hda/patch_hdmi.c | 119 ++++++++++++++++++++++++++++++++++++- 3 files changed, 119 insertions(+), 4 deletions(-) base-commit: 81a2da5a10a6eaa6ae16108eed4e74651cc296bf -- 2.38.1

2 4

[PATCH] ALSA: hda: Error out if invalid stream is being setup
by Cezary Rojewski 09 Dec '22

09 Dec '22

Scenario when snd_hdac_stream_setup_periods() receives an instance of struct hdac_stream with neither ->substream nor ->cstream initialized is invalid. Simultaneously addresses "uninitialized symbol 'dmab'" error reported by Smatch. Fixes: 3e9582267e3a ("ALSA: hda: Interrupt servicing and BDL setup for compress streams") Signed-off-by: Cezary Rojewski <cezary.rojewski(a)intel.com> --- sound/hda/hdac_stream.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/hda/hdac_stream.c b/sound/hda/hdac_stream.c index 8f625402505f..547adbc22590 100644 --- a/sound/hda/hdac_stream.c +++ b/sound/hda/hdac_stream.c @@ -500,6 +500,9 @@ int snd_hdac_stream_setup_periods(struct hdac_stream *azx_dev) dmab = snd_pcm_get_dma_buf(substream); } else if (cstream) { dmab = snd_pcm_get_dma_buf(cstream); + } else { + WARN(1, "No substream or cstream assigned\n"); + return -EINVAL; } /* reset BDL address */ -- 2.25.1

2 1