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

[PATCHv4 8/9] ASoC: fsl-asoc-card: add DT property "cpu-system-clock-direction-out"
by Elinor Montmasson 31 May '24

31 May '24

Add new optional DT property "cpu-system-clock-direction-out" to set sysclk direction as "out" for the CPU DAI when using the generic codec. It is set for both Tx and Rx. If not set, the direction is "in". The way the direction value is used is up to the CPU DAI driver implementation. Signed-off-by: Elinor Montmasson <elinor.montmasson(a)savoirfairelinux.com> --- sound/soc/fsl/fsl-asoc-card.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/sound/soc/fsl/fsl-asoc-card.c b/sound/soc/fsl/fsl-asoc-card.c index c7fc9c16f761..f3fc2b29c92f 100644 --- a/sound/soc/fsl/fsl-asoc-card.c +++ b/sound/soc/fsl/fsl-asoc-card.c @@ -760,6 +760,10 @@ static int fsl_asoc_card_probe(struct platform_device *pdev) priv->cpu_priv.sysclk_freq[RX] = priv->cpu_priv.sysclk_freq[TX]; clk_put(cpu_sysclk); } + priv->cpu_priv.sysclk_dir[TX] = + of_property_read_bool(np, "cpu-system-clock-direction-out") ? + SND_SOC_CLOCK_OUT : SND_SOC_CLOCK_IN; + priv->cpu_priv.sysclk_dir[RX] = priv->cpu_priv.sysclk_dir[TX]; } else { dev_err(&pdev->dev, "unknown Device Tree compatible\n"); ret = -EINVAL; -- 2.34.1

2 5

[PATCH] ALSA: hda: cs35l41: Possible null pointer dereference in cs35l41_hda_unbind()
by Simon Trimmer 31 May '24

31 May '24

The cs35l41_hda_unbind() function clears the hda_component entry matching it's index and then dereferences the codec pointer held in the first element of the hda_component array, this is an issue when the device index was 0. Instead use the codec pointer stashed in the cs35l41_hda structure as it will still be valid. Fixes: 7cf5ce66dfda ("ALSA: hda: cs35l41: Add device_link between HDA and cs35l41_hda") Signed-off-by: Simon Trimmer <simont(a)opensource.cirrus.com> --- sound/pci/hda/cs35l41_hda.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/pci/hda/cs35l41_hda.c b/sound/pci/hda/cs35l41_hda.c index 6c49e5c6cd20..d54d4d60b03e 100644 --- a/sound/pci/hda/cs35l41_hda.c +++ b/sound/pci/hda/cs35l41_hda.c @@ -1495,7 +1495,7 @@ static void cs35l41_hda_unbind(struct device *dev, struct device *master, void * if (comps[cs35l41->index].dev == dev) { memset(&comps[cs35l41->index], 0, sizeof(*comps)); sleep_flags = lock_system_sleep(); - device_link_remove(&comps->codec->core.dev, cs35l41->dev); + device_link_remove(&cs35l41->codec->core.dev, cs35l41->dev); unlock_system_sleep(sleep_flags); } } -- 2.34.1

2 1

[PATCH] ALSA: hda: cs35l56: Fix lifecycle of codec pointer
by Simon Trimmer 31 May '24

31 May '24

The codec should be cleared when the amp driver is unbound and when resuming it should be tested to prevent loading firmware into the device and ALSA in a partially configured system state. Signed-off-by: Simon Trimmer <simont(a)opensource.cirrus.com> --- sound/pci/hda/cs35l56_hda.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/sound/pci/hda/cs35l56_hda.c b/sound/pci/hda/cs35l56_hda.c index 11b0570ff56d..0923e2589f5f 100644 --- a/sound/pci/hda/cs35l56_hda.c +++ b/sound/pci/hda/cs35l56_hda.c @@ -735,6 +735,8 @@ static void cs35l56_hda_unbind(struct device *dev, struct device *master, void * if (comps[cs35l56->index].dev == dev) memset(&comps[cs35l56->index], 0, sizeof(*comps)); + cs35l56->codec = NULL; + dev_dbg(cs35l56->base.dev, "Unbound\n"); } @@ -840,6 +842,9 @@ static int cs35l56_hda_system_resume(struct device *dev) cs35l56->suspended = false; + if (!cs35l56->codec) + return 0; + ret = cs35l56_is_fw_reload_needed(&cs35l56->base); dev_dbg(cs35l56->base.dev, "fw_reload_needed: %d\n", ret); if (ret > 0) { -- 2.34.1

2 1

[PATCH v6 0/1] Add tas2781 driver for SPI.
by Baojun Xu 31 May '24

31 May '24

This patch was used to add TAS2781 devices on SPI support in sound/pci/hda. It use ACPI node descript about parameters of TAS2781 on SPI, it like: Scope (_SB.PC00.SPI0) { Device (GSPK) { Name (_HID, "TXNW2781") // _HID: Hardware ID Method (_CRS, 0, NotSerialized) { Name (RBUF, ResourceTemplate () { SpiSerialBusV2 (...) SpiSerialBusV2 (...) } } } } And in platform/x86/serial-multi-instantiate.c, those spi devices will be added into system as a single SPI device, so TAS2781 SPI driver will probe twice for every single SPI device. And driver will also parser mono DSP firmware binary and RCA binary for itself. Signed-off-by: Baojun Xu <baojun.xu(a)ti.com> Baojun Xu (1): ALSA: hda/tas2781: Add tas2781 hda driver based on SPI drivers/acpi/scan.c | 1 + .../platform/x86/serial-multi-instantiate.c | 12 + sound/pci/hda/Kconfig | 14 + sound/pci/hda/Makefile | 2 + sound/pci/hda/patch_realtek.c | 13 + sound/pci/hda/tas2781-spi.h | 148 ++ sound/pci/hda/tas2781_hda_spi.c | 1266 ++++++++++ sound/pci/hda/tas2781_spi_fwlib.c | 2101 +++++++++++++++++ 8 files changed, 3557 insertions(+) create mode 100644 sound/pci/hda/tas2781-spi.h create mode 100644 sound/pci/hda/tas2781_hda_spi.c create mode 100644 sound/pci/hda/tas2781_spi_fwlib.c -- 2.40.1

2 2

[GIT PULL] ASoC fixes for v6.10-rc1
by Mark Brown 30 May '24

30 May '24

The following changes since commit 1613e604df0cd359cf2a7fbd9be7a0bcfacfabd0: Linux 6.10-rc1 (2024-05-26 15:20:12 -0700) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-fix-v6.10-rc1 for you to fetch changes up to c85578e730135d47271ca874e4436dd87e87f252: ASoC: SOF: ipc4-topology: Fix nhlt configuration blob (2024-05-30 14:33:14 +0100) ---------------------------------------------------------------- ASoC: Fixes for v6.10 Several serieses of fixes that have come in since the merge window, mostly for Intel systems. The biggest thing is some updates from Peter which fix support for a series of Intel laptops which have been found to use 16 bit rather than 32 bit DMIC configuration blobs in their firmware descriptions. We also have a bunch of fixes for module annotations, and some smaller single patch fixes. ---------------------------------------------------------------- Charles Keepax (3): ASoC: cs42l43: Only restrict 44.1kHz for the ASP ASoC: wm_adsp: Add missing MODULE_DESCRIPTION() MAINTAINERS: Remove James Schulman from Cirrus audio maintainers Mark Brown (2): ASoC: SOF: add missing MODULE_DESCRIPTION ASoC: SOF: ipc4-topology: Fix nhlt configuration blob Peter Ujfalusi (6): ASoC: SOF: ipc4-topology: Fix input format query of process modules without base extension ASoC: SOF: ipc4-topology: Add support for NHLT with 16-bit only DMIC blob ASoC: SOF: ipc4-topology: Print out the channel count in sof_ipc4_dbg_audio_format ASoC: SOF: ipc4-topology/pcm: Rename sof_ipc4_copier_is_single_format() ASoC: SOF: ipc4-topology: Improve readability of sof_ipc4_prepare_dai_copier() ASoC: SOF: ipc4-topology: Adjust the params based on DAI formats Pierre-Louis Bossart (5): ASoC: SOF: stream-ipc: remove unnecessary MODULE_LICENSE ASoC: SOF: AMD: group all module related information ASoC: SOF: reorder MODULE_ definitions ASoC: SOF: add missing MODULE_DESCRIPTION() ASoC: Intel: sof-sdw: fix missing SPI_MASTER dependency MAINTAINERS | 1 - sound/soc/codecs/cs42l43.c | 5 +- sound/soc/codecs/wm_adsp.c | 1 + sound/soc/intel/boards/Kconfig | 2 +- sound/soc/sof/amd/acp-common.c | 4 +- sound/soc/sof/amd/acp.c | 2 +- sound/soc/sof/amd/acp63.c | 4 - sound/soc/sof/amd/pci-acp63.c | 1 + sound/soc/sof/amd/pci-rmb.c | 1 + sound/soc/sof/amd/pci-rn.c | 1 + sound/soc/sof/amd/pci-vangogh.c | 1 + sound/soc/sof/amd/rembrandt.c | 4 - sound/soc/sof/amd/renoir.c | 4 - sound/soc/sof/amd/vangogh.c | 4 - sound/soc/sof/core.c | 2 +- sound/soc/sof/imx/imx-common.c | 1 + sound/soc/sof/imx/imx8.c | 3 +- sound/soc/sof/imx/imx8m.c | 3 +- sound/soc/sof/imx/imx8ulp.c | 3 +- sound/soc/sof/intel/atom.c | 1 + sound/soc/sof/intel/bdw.c | 1 + sound/soc/sof/intel/byt.c | 1 + sound/soc/sof/intel/hda-codec.c | 1 + sound/soc/sof/intel/hda-ctrl.c | 1 + sound/soc/sof/intel/hda-mlink.c | 1 + sound/soc/sof/intel/hda.c | 1 + sound/soc/sof/intel/pci-apl.c | 1 + sound/soc/sof/intel/pci-cnl.c | 1 + sound/soc/sof/intel/pci-icl.c | 1 + sound/soc/sof/intel/pci-lnl.c | 1 + sound/soc/sof/intel/pci-mtl.c | 1 + sound/soc/sof/intel/pci-skl.c | 1 + sound/soc/sof/intel/pci-tgl.c | 1 + sound/soc/sof/intel/pci-tng.c | 1 + sound/soc/sof/ipc4-pcm.c | 12 +- sound/soc/sof/ipc4-topology.c | 163 +++++++++++++++++-------- sound/soc/sof/ipc4-topology.h | 6 +- sound/soc/sof/mediatek/mt8186/mt8186.c | 3 +- sound/soc/sof/mediatek/mt8195/mt8195.c | 3 +- sound/soc/sof/mediatek/mtk-adsp-common.c | 1 + sound/soc/sof/nocodec.c | 2 +- sound/soc/sof/sof-acpi-dev.c | 1 + sound/soc/sof/sof-client-ipc-flood-test.c | 2 +- sound/soc/sof/sof-client-ipc-kernel-injector.c | 2 +- sound/soc/sof/sof-client-ipc-msg-injector.c | 2 +- sound/soc/sof/sof-client-probes.c | 2 +- sound/soc/sof/sof-of-dev.c | 1 + sound/soc/sof/sof-pci-dev.c | 1 + sound/soc/sof/sof-utils.c | 1 + sound/soc/sof/stream-ipc.c | 2 - sound/soc/sof/xtensa/core.c | 2 +- 51 files changed, 173 insertions(+), 95 deletions(-)

2 1

Re:[PATCH 6.8 215/493] cpufreq: exit() callback is optional
by lizhe 30 May '24

30 May '24

Hi, Viresh Kumar Do you take to heart the efforts and contributions of others to the community ? 1. I submitted this patch. You included it in your patch completely. > > static int cpufreq_offline(unsigned int cpu) > >@@ -1731,7 +1734,7 @@ static void cpufreq_remove_dev(struct device *dev, struct subsys_interface *sif) > > } > > > > /* We did light-weight exit earlier, do full tear down now */ > >- if (cpufreq_driver->offline) > >+ if (cpufreq_driver->offline && cpufreq_driver->exit) > > cpufreq_driver->exit(policy); > > > > up_write(&policy->rwsem); > >-- 2. You have seen the three patches I submitted to you and the main line before submitting this patch. > >@@ -1670,10 +1670,13 @@ static void __cpufreq_offline(unsigned int cpu, struct cpufreq_policy *policy) > > */ > > if (cpufreq_driver->offline) { > > cpufreq_driver->offline(policy); > >- } else if (cpufreq_driver->exit) { > >- cpufreq_driver->exit(policy); > >- policy->freq_table = NULL; > >+ return; > > } > >+ > >+ if (cpufreq_driver->exit) > >+ cpufreq_driver->exit(policy); > >+ > >+ policy->freq_table = NULL; > > } As a senior member of the kernel community, you should uphold the fairness and justice of the community. Why do you have to rush in and take over others' patches? Isn't that selfish of you? Your behavior disrespects others' efforts and enthusiasm, and it undermines the collaborative, harmonious, and fair atmosphere of the community. I have tried to communicate with you several times, but you have refused. Is it because you are the maintainer of the community ? Where does your authority come from ? thanks. Lizhe At 2024-05-28 02:53:37, "Greg Kroah-Hartman" <gregkh(a)linuxfoundation.org> wrote: >6.8-stable review patch. If anyone has any objections, please let me know. > >------------------ > >From: Viresh Kumar <viresh.kumar(a)linaro.org> > >[ Upstream commit b8f85833c05730d631576008daaa34096bc7f3ce ] > >The exit() callback is optional and shouldn't be called without checking >a valid pointer first. > >Also, we must clear freq_table pointer even if the exit() callback isn't >present. > >Signed-off-by: Viresh Kumar <viresh.kumar(a)linaro.org> >Fixes: 91a12e91dc39 ("cpufreq: Allow light-weight tear down and bring up of CPUs") >Fixes: f339f3541701 ("cpufreq: Rearrange locking in cpufreq_remove_dev()") >Reported-by: Lizhe <sensor1010(a)163.com> >Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki(a)intel.com> >Signed-off-by: Sasha Levin <sashal(a)kernel.org> >--- > drivers/cpufreq/cpufreq.c | 11 +++++++---- > 1 file changed, 7 insertions(+), 4 deletions(-) > >diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c >index 3c2c955fbbbd6..86f1bc7754ea6 100644 >--- a/drivers/cpufreq/cpufreq.c >+++ b/drivers/cpufreq/cpufreq.c >@@ -1670,10 +1670,13 @@ static void __cpufreq_offline(unsigned int cpu, struct cpufreq_policy *policy) > */ > if (cpufreq_driver->offline) { > cpufreq_driver->offline(policy); >- } else if (cpufreq_driver->exit) { >- cpufreq_driver->exit(policy); >- policy->freq_table = NULL; >+ return; > } >+ >+ if (cpufreq_driver->exit) >+ cpufreq_driver->exit(policy); >+ >+ policy->freq_table = NULL; > } > > static int cpufreq_offline(unsigned int cpu) >@@ -1731,7 +1734,7 @@ static void cpufreq_remove_dev(struct device *dev, struct subsys_interface *sif) > } > > /* We did light-weight exit earlier, do full tear down now */ >- if (cpufreq_driver->offline) >+ if (cpufreq_driver->offline && cpufreq_driver->exit) > cpufreq_driver->exit(policy); > > up_write(&policy->rwsem); >-- >2.43.0 > >

1 0

[PATCH v2 0/2] Support Tegra I2S client format conversion
by Sameer Pujar 30 May '24

30 May '24

From: Mohan Kumar <mkumard(a)nvidia.com> The AHUB HW modules are interconnected with CIF which are capable of supporting Channel and Sample bit format conversion. Due to this, the I2S Client can have different Channel and Sample bit from the hw_params() and this config is passed from CIF port of I2S DT node which can help to perform this conversion. - First change to split simple_fixup_sample_fmt to support returning sample format value - Second patch to support Tegra I2S client channel and sample format programming based on CIF port from DT node. Changelog ========= v2: - Fix sparse warning related to incorrect type assignment for variable 'sample_format' in hw_params() call. The defect was reported by "kernel test robot <lkp(a)intel.com>" and details can be found below: https://lore.kernel.org/oe-kbuild-all/202405211805.UMAdiH0d-lkp@intel.com/ Mohan Kumar (2): ASoC: simple-card-utils: Split simple_fixup_sample_fmt func ASoC: tegra: I2S client convert formats handling include/sound/simple_card_utils.h | 2 + sound/soc/generic/simple-card-utils.c | 26 +++++++--- sound/soc/tegra/tegra210_i2s.c | 71 +++++++++++++++++++++++++-- sound/soc/tegra/tegra210_i2s.h | 2 + 4 files changed, 91 insertions(+), 10 deletions(-) -- 2.45.1

2 4

[RFC 00/12] ASoC: remove/replace dpcm_xxx flag
by Kuninori Morimoto 30 May '24

30 May '24

Hi each ASoC driver maintainer As I posted in "ASoC: grace time for DPCM cleanup", dpcm_xxx flag will be remove. Then, some driver might got damage. This patch-set try to remove/replace dpcm_xxx flag, but is not tesed. It is just "RFC" patch, but I'm happy if you can reuse it on your driver. Thank you for your help !! Best regards --- Kuninori Morimoto

1 12

[PATCH v6 06/49] ASoC: Intel: avs: es8336: Switch to new Intel CPU model defines
by Tony Luck 29 May '24

29 May '24

New CPU #defines encode vendor and family as well as model. Signed-off-by: Tony Luck <tony.luck(a)intel.com> Reviewed-by: Amadeusz Sławiński <amadeuszx.slawinski(a)linux.intel.com> Acked-by: Mark Brown <broonie(a)kernel.org> --- sound/soc/intel/avs/boards/es8336.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sound/soc/intel/avs/boards/es8336.c b/sound/soc/intel/avs/boards/es8336.c index 3bf37a8fd6e6..c8522e2430f8 100644 --- a/sound/soc/intel/avs/boards/es8336.c +++ b/sound/soc/intel/avs/boards/es8336.c @@ -18,7 +18,7 @@ #include <sound/pcm_params.h> #include <sound/soc.h> #include <sound/soc-acpi.h> -#include <asm/intel-family.h> +#include <asm/cpu_device_id.h> #include "../utils.h" #define ES8336_CODEC_DAI "ES8316 HiFi" @@ -153,9 +153,9 @@ static int avs_es8336_hw_params(struct snd_pcm_substream *substream, int clk_freq; int ret; - switch (boot_cpu_data.x86_model) { - case INTEL_FAM6_KABYLAKE_L: - case INTEL_FAM6_KABYLAKE: + switch (boot_cpu_data.x86_vfm) { + case INTEL_KABYLAKE_L: + case INTEL_KABYLAKE: clk_freq = 24000000; break; default: -- 2.45.0

1 0

[PATCH 0/4] ASoC: SOF: add missing MODULE_DESCRIPTION
by Pierre-Louis Bossart 29 May '24

29 May '24

'make W=1' now reports missing MODULE_DESCRIPTION lines. This patchset cleans-up all the module definitions and adds MODULE_DESCRIPTION lines as needed. Pierre-Louis Bossart (4): ASoC: SOF: stream-ipc: remove unnecessary MODULE_LICENSE ASoC: SOF: AMD: group all module related information ASoC: SOF: reorder MODULE_ definitions ASoC: SOF: add missing MODULE_DESCRIPTION() sound/soc/sof/amd/acp-common.c | 4 ++-- sound/soc/sof/amd/acp.c | 2 +- sound/soc/sof/amd/acp63.c | 4 ---- sound/soc/sof/amd/pci-acp63.c | 1 + sound/soc/sof/amd/pci-rmb.c | 1 + sound/soc/sof/amd/pci-rn.c | 1 + sound/soc/sof/amd/pci-vangogh.c | 1 + sound/soc/sof/amd/rembrandt.c | 4 ---- sound/soc/sof/amd/renoir.c | 4 ---- sound/soc/sof/amd/vangogh.c | 4 ---- sound/soc/sof/core.c | 2 +- sound/soc/sof/imx/imx-common.c | 1 + sound/soc/sof/imx/imx8.c | 3 ++- sound/soc/sof/imx/imx8m.c | 3 ++- sound/soc/sof/imx/imx8ulp.c | 3 ++- sound/soc/sof/intel/atom.c | 1 + sound/soc/sof/intel/bdw.c | 1 + sound/soc/sof/intel/byt.c | 1 + sound/soc/sof/intel/hda-codec.c | 1 + sound/soc/sof/intel/hda-ctrl.c | 1 + sound/soc/sof/intel/hda-mlink.c | 1 + sound/soc/sof/intel/hda.c | 1 + sound/soc/sof/intel/pci-apl.c | 1 + sound/soc/sof/intel/pci-cnl.c | 1 + sound/soc/sof/intel/pci-icl.c | 1 + sound/soc/sof/intel/pci-lnl.c | 1 + sound/soc/sof/intel/pci-mtl.c | 1 + sound/soc/sof/intel/pci-skl.c | 1 + sound/soc/sof/intel/pci-tgl.c | 1 + sound/soc/sof/intel/pci-tng.c | 1 + sound/soc/sof/mediatek/mt8186/mt8186.c | 3 ++- sound/soc/sof/mediatek/mt8195/mt8195.c | 3 ++- sound/soc/sof/mediatek/mtk-adsp-common.c | 1 + sound/soc/sof/nocodec.c | 2 +- sound/soc/sof/sof-acpi-dev.c | 1 + sound/soc/sof/sof-client-ipc-flood-test.c | 2 +- sound/soc/sof/sof-client-ipc-kernel-injector.c | 2 +- sound/soc/sof/sof-client-ipc-msg-injector.c | 2 +- sound/soc/sof/sof-client-probes.c | 2 +- sound/soc/sof/sof-of-dev.c | 1 + sound/soc/sof/sof-pci-dev.c | 1 + sound/soc/sof/sof-utils.c | 1 + sound/soc/sof/stream-ipc.c | 2 -- sound/soc/sof/xtensa/core.c | 2 +- 44 files changed, 45 insertions(+), 33 deletions(-) -- 2.43.0

2 5