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

Re: [PATCH v2] ALSA: hda: cs35l41: Stop creating ALSA Controls for firmware coefficients
by Takashi Iwai 05 Aug '24

05 Aug '24

On Thu, 01 Aug 2024 17:50:44 +0200, Stefan Binding wrote: > > When the CS35L41 loads its firmware, it has a number of controls to > affect its behaviour. Currently, these controls are exposed as ALSA > Controls. > > These controls were never intended to be exposed to users but the > firmware doesn't mark them hidden, so make the driver ignore them. > > Any changes in the coefficients handled by these controls needs to > be matched to the individual system by SSID, which is already handled > using the tuning file, when firmware is loaded, so UCM should not be > setting these controls anyway. > > Signed-off-by: Stefan Binding <sbinding(a)opensource.cirrus.com> Applied to for-linus branch now. Thanks. Takashi

1 0

Re: [PATCH v3] ALSA: hda: cs35l56: Stop creating ALSA controls for firmware coefficients
by Takashi Iwai 05 Aug '24

05 Aug '24

On Thu, 01 Aug 2024 16:31:39 +0200, Simon Trimmer wrote: > > A number of laptops have gone to market with old firmware versions that > export controls that have since been hidden, but we can't just install a > newer firmware because the firmware for each product is customized and > qualified by the OEM. The issue is that alsactl save and restore has no > idea what controls are good to persist which can lead to > misconfiguration. > > There is no reason that the UCM or user should need to interact with any > of the ALSA controls for the firmware coefficients so they can be > removed entirely, this also simplifies the driver. > > Signed-off-by: Simon Trimmer <simont(a)opensource.cirrus.com> > --- > Changes in v3: > - Remove ability to add firmware coefficients as ALSA controls > entirely > > Changes in v2: > - v1 was accidentally the backport version for older kernels instead of > the one for v6.11 Applied now to for-linus branch. Thanks. Takashi

1 0

ucm2: sof-soundwire: add rt712 VA device
by GitHub pull_request - opened 05 Aug '24

05 Aug '24

alsa-project/alsa-ucm-conf pull request #435 was opened from shumingfan: Add support headphone/headset mic/speaker/dmic for rt712 VA Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/435 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/435.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

a52 plugin Unknown field card error
by GitHub issues - edited 04 Aug '24

04 Aug '24

alsa-project/alsa-lib issue #405 was edited from DocMAX: ``` pcm.a52 { @args [CARD] @args.CARD { type string } type rate slave { pcm { type a52 bitrate 448 channels 6 card $CARD } rate 48000 } } a52:CARD=Adapter USB SPDIF Adapter Plugin to convert multichannel stream to A52 (AC3) bitstream [root@desktop ~]# speaker-test -c6 -Da52 speaker-test 1.2.12 Playback device is a52 Stream parameters are 48000Hz, S16_LE, 6 channels Using 16 octaves of pink noise ALSA lib pcm_rate.c:1701:(_snd_pcm_rate_open) Unknown field card Playback open error: -22,Invalid argument ``` how to fix this error? running arch linux all updated Issue URL : https://github.com/alsa-project/alsa-lib/issues/405 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

a52 plugin Unknown field card error
by GitHub issues - opened 04 Aug '24

04 Aug '24

alsa-project/alsa-lib issue #405 was opened from DocMAX: ``` a52:CARD=Adapter USB SPDIF Adapter Plugin to convert multichannel stream to A52 (AC3) bitstream [root@desktop ~]# speaker-test -c6 -Da52 speaker-test 1.2.12 Playback device is a52 Stream parameters are 48000Hz, S16_LE, 6 channels Using 16 octaves of pink noise ALSA lib pcm_rate.c:1701:(_snd_pcm_rate_open) Unknown field card Playback open error: -22,Invalid argument ``` how to fix this error? running arch linux all updated Issue URL : https://github.com/alsa-project/alsa-lib/issues/405 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[PATCH v1] MAINTAINERS: update entries in TEXAS INSTRUMENTS AUDIO (ASoC/HDA) DRIVERS and add entries for haptic driver
by Shenghao Ding 04 Aug '24

04 Aug '24

Due to internal re-org, Kevin is no longer mnaintaining audio driver. Due to job change to Shenghao Ding and Baojun Xu, drop entries for the audio converter and add entries for both haptics drivers and middle- power audio amplifier drivers. Signed-off-by: Shenghao Ding <shenghao-ding(a)ti.com> --- MAINTAINERS | 29 +++++++++++------------------ 1 file changed, 11 insertions(+), 18 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 8766f3e5e87e..34639103e463 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -22567,37 +22567,30 @@ S: Maintained F: Documentation/devicetree/bindings/sound/davinci-mcasp-audio.yaml F: sound/soc/ti/ -TEXAS INSTRUMENTS AUDIO (ASoC/HDA) DRIVERS +TEXAS INSTRUMENTS AUDIO (ASoC/HDA) & HAPTICS DRIVERS M: Shenghao Ding <shenghao-ding(a)ti.com> -M: Kevin Lu <kevin-lu(a)ti.com> M: Baojun Xu <baojun.xu(a)ti.com> L: alsa-devel(a)alsa-project.org (moderated for non-subscribers) S: Maintained +F: Documentation/devicetree/bindings/input/ti,drv260x.yaml +F: Documentation/devicetree/bindings/input/ti,drv266x.yaml F: Documentation/devicetree/bindings/sound/tas2552.txt +F: Documentation/devicetree/bindings/sound/tas5720.txt F: Documentation/devicetree/bindings/sound/ti,tas2562.yaml F: Documentation/devicetree/bindings/sound/ti,tas2770.yaml +F: Documentation/devicetree/bindings/sound/ti,tas2781.yaml F: Documentation/devicetree/bindings/sound/ti,tas27xx.yaml -F: Documentation/devicetree/bindings/sound/ti,pcm1681.yaml -F: Documentation/devicetree/bindings/sound/ti,pcm3168a.yaml -F: Documentation/devicetree/bindings/sound/ti,tlv320*.yaml -F: Documentation/devicetree/bindings/sound/ti,tlv320adcx140.yaml -F: Documentation/devicetree/bindings/sound/tlv320aic31xx.txt +F: Documentation/devicetree/bindings/sound/ti,tas5086.txt +F: Documentation/devicetree/bindings/sound/ti,tas57xx.yaml +F: Documentation/devicetree/bindings/sound/ti,tas5805m.yaml F: Documentation/devicetree/bindings/sound/tpa6130a2.txt +F: drivers/input/misc/drv2*.c F: include/sound/tas2*.h -F: include/sound/tlv320*.h +F: include/sound/tas5086.h F: include/sound/tpa6130a2-plat.h F: sound/pci/hda/tas2781_hda_i2c.c -F: sound/soc/codecs/pcm1681.c -F: sound/soc/codecs/pcm1789*.* -F: sound/soc/codecs/pcm179x*.* -F: sound/soc/codecs/pcm186x*.* -F: sound/soc/codecs/pcm3008.* -F: sound/soc/codecs/pcm3060*.* -F: sound/soc/codecs/pcm3168a*.* -F: sound/soc/codecs/pcm5102a.c -F: sound/soc/codecs/pcm512x*.* F: sound/soc/codecs/tas2*.* -F: sound/soc/codecs/tlv320*.* +F: sound/soc/codecs/tas5*.* F: sound/soc/codecs/tpa6130a2.* TEXAS INSTRUMENTS DMA DRIVERS -- 2.34.1

1 0

[PATCH 0/4] ASoC: SOF: reshuffle and optimize structures
by Pierre-Louis Bossart 03 Aug '24

03 Aug '24

pahole reports a number of sub-optimal structure definitions with holes and cache-line alignment problems, this patchset optimizes a couple of structure frequently used. No functionality change, only re-ordering of structure members. Pierre-Louis Bossart (4): ASoC: SOF: sof-priv.h: optimize snd_sof_platform_stream_params ASoC: SOF: sof-priv.h: optimize snd_sof_mailbox ASoC: SOF: sof-priv.h: optimize snd_sof_ipc_msg ASoC: SOF: sof-audio.h: optimize snd_sof_pcm_stream_pipeline_list sound/soc/sof/sof-audio.h | 4 ++-- sound/soc/sof/sof-priv.h | 16 +++++++++------- 2 files changed, 11 insertions(+), 9 deletions(-) -- 2.43.0

2 5

[PATCH v4 0/5] ASoC/SOF/PCI/Intel: add PantherLake support
by Pierre-Louis Bossart 03 Aug '24

03 Aug '24

Add initial support for the PantherLake platform, and initial ACPI configurations. All the dependencies required in the initial versions are now available in ASoc for-next branch. v4: topology name simplification for rt722 v3: add configuration with rt722 on Link3 v2: changed order of patches to fix git bisect error reported by Intel kbuild bot https://lore.kernel.org/oe-kbuild-all/202406131144.L6gW0I47-lkp@intel.com/ Bard Liao (2): ASoC: Intel: soc-acpi-intel-ptl-match: add rt711-sdca table ASoC: Intel: soc-acpi-intel-ptl-match: Add rt722 support Fred Oh (1): ASoC: SOF: Intel: add PTL specific power control register Pierre-Louis Bossart (2): ASoC: Intel: soc-acpi: add PTL match tables ASoC: SOF: Intel: add initial support for PTL include/sound/soc-acpi-intel-match.h | 2 + sound/soc/intel/common/Makefile | 1 + .../intel/common/soc-acpi-intel-ptl-match.c | 145 ++++++++++++++++++ sound/soc/sof/intel/Kconfig | 17 ++ sound/soc/sof/intel/Makefile | 2 + sound/soc/sof/intel/hda-dsp.c | 1 + sound/soc/sof/intel/hda.h | 1 + sound/soc/sof/intel/lnl.c | 27 ++++ sound/soc/sof/intel/mtl.c | 16 +- sound/soc/sof/intel/mtl.h | 2 + sound/soc/sof/intel/pci-ptl.c | 77 ++++++++++ sound/soc/sof/intel/shim.h | 1 + 12 files changed, 290 insertions(+), 2 deletions(-) create mode 100644 sound/soc/intel/common/soc-acpi-intel-ptl-match.c create mode 100644 sound/soc/sof/intel/pci-ptl.c -- 2.43.0

2 6

[PATCH] ASoC: cs43130: Constify snd_soc_component_driver struct
by Christophe JAILLET 02 Aug '24

02 Aug '24

In order to constify `snd_soc_component_driver` struct, duplicate `soc_component_dev_cs43130` into a `soc_component_dev_cs43130_digital` and `soc_component_dev_cs43130_analog`. These 2 new structures share the same .dapm_widgets and .dapm_routes arrays but differ for .num_dapm_widgets and .num_dapm_routes. In the digital case, the last entries are not taken into account. Doing so has several advantages: - `snd_soc_component_driver` can be declared as const to move their declarations to read-only sections. - code in the probe is simpler. There is no need to concatenate some arrays to handle the "analog" case - this saves some memory because all_hp_widgets and analog_hp_routes can be removed. Before : ====== text data bss dec hex filename 53965 8265 4512 66742 104b6 sound/soc/codecs/cs43130.o After : ===== text data bss dec hex filename 54409 7881 64 62354 f392 sound/soc/codecs/cs43130.o Signed-off-by: Christophe JAILLET <christophe.jaillet(a)wanadoo.fr> --- Compile tested-only. See discussion at [1]. [1]: https://lore.kernel.org/all/ZqNawRmAqBRLIoQq@opensource.cirrus.com/ --- sound/soc/codecs/cs43130.c | 73 +++++++++++++++++--------------------- 1 file changed, 32 insertions(+), 41 deletions(-) diff --git a/sound/soc/codecs/cs43130.c b/sound/soc/codecs/cs43130.c index be4037890fdb..cb4ca80f36d2 100644 --- a/sound/soc/codecs/cs43130.c +++ b/sound/soc/codecs/cs43130.c @@ -1415,7 +1415,7 @@ static const char * const bypass_mux_text[] = { static SOC_ENUM_SINGLE_DECL(bypass_enum, SND_SOC_NOPM, 0, bypass_mux_text); static const struct snd_kcontrol_new bypass_ctrl = SOC_DAPM_ENUM("Switch", bypass_enum); -static const struct snd_soc_dapm_widget digital_hp_widgets[] = { +static const struct snd_soc_dapm_widget hp_widgets[] = { SND_SOC_DAPM_MUX("Bypass Switch", SND_SOC_NOPM, 0, 0, &bypass_ctrl), SND_SOC_DAPM_OUTPUT("HPOUTA"), SND_SOC_DAPM_OUTPUT("HPOUTB"), @@ -1447,19 +1447,16 @@ static const struct snd_soc_dapm_widget digital_hp_widgets[] = { CS43130_PDN_HP_SHIFT, 1, cs43130_dac_event, (SND_SOC_DAPM_PRE_PMU | SND_SOC_DAPM_POST_PMU | SND_SOC_DAPM_POST_PMD)), -}; -static const struct snd_soc_dapm_widget analog_hp_widgets[] = { +/* Some devices have some extra analog widgets */ +#define NUM_ANALOG_WIDGETS 1 + SND_SOC_DAPM_DAC_E("Analog Playback", NULL, CS43130_HP_OUT_CTL_1, CS43130_HP_IN_EN_SHIFT, 0, cs43130_hpin_event, (SND_SOC_DAPM_PRE_PMU | SND_SOC_DAPM_POST_PMD)), }; -static struct snd_soc_dapm_widget all_hp_widgets[ - ARRAY_SIZE(digital_hp_widgets) + - ARRAY_SIZE(analog_hp_widgets)]; - -static const struct snd_soc_dapm_route digital_hp_routes[] = { +static const struct snd_soc_dapm_route hp_routes[] = { {"ASPIN PCM", NULL, "ASP PCM Playback"}, {"ASPIN DoP", NULL, "ASP DoP Playback"}, {"XSPIN DoP", NULL, "XSP DoP Playback"}, @@ -1472,15 +1469,12 @@ static const struct snd_soc_dapm_route digital_hp_routes[] = { {"Bypass Switch", "Internal", "HiFi DAC"}, {"HPOUTA", NULL, "Bypass Switch"}, {"HPOUTB", NULL, "Bypass Switch"}, -}; -static const struct snd_soc_dapm_route analog_hp_routes[] = { +/* Some devices have some extra analog routes */ +#define NUM_ANALOG_ROUTES 1 {"Bypass Switch", "Alternative", "Analog Playback"}, }; -static struct snd_soc_dapm_route all_hp_routes[ - ARRAY_SIZE(digital_hp_routes) + - ARRAY_SIZE(analog_hp_routes)]; static const unsigned int cs43130_asp_src_rates[] = { 32000, 44100, 48000, 88200, 96000, 176400, 192000, 352800, 384000 @@ -2398,7 +2392,23 @@ static int cs43130_probe(struct snd_soc_component *component) return 0; } -static struct snd_soc_component_driver soc_component_dev_cs43130 = { +static const struct snd_soc_component_driver soc_component_dev_cs43130_digital = { + .probe = cs43130_probe, + .controls = cs43130_snd_controls, + .num_controls = ARRAY_SIZE(cs43130_snd_controls), + .set_sysclk = cs43130_component_set_sysclk, + .set_pll = cs43130_set_pll, + .idle_bias_on = 1, + .use_pmdown_time = 1, + .endianness = 1, + /* Don't take into account the ending analog widgets and routes */ + .dapm_widgets = hp_widgets, + .num_dapm_widgets = ARRAY_SIZE(hp_widgets) - NUM_ANALOG_WIDGETS, + .dapm_routes = hp_routes, + .num_dapm_routes = ARRAY_SIZE(hp_routes) - NUM_ANALOG_ROUTES, +}; + +static const struct snd_soc_component_driver soc_component_dev_cs43130_analog = { .probe = cs43130_probe, .controls = cs43130_snd_controls, .num_controls = ARRAY_SIZE(cs43130_snd_controls), @@ -2407,6 +2417,10 @@ static struct snd_soc_component_driver soc_component_dev_cs43130 = { .idle_bias_on = 1, .use_pmdown_time = 1, .endianness = 1, + .dapm_widgets = hp_widgets, + .num_dapm_widgets = ARRAY_SIZE(hp_widgets), + .dapm_routes = hp_routes, + .num_dapm_routes = ARRAY_SIZE(hp_routes), }; static const struct regmap_config cs43130_regmap = { @@ -2479,6 +2493,7 @@ static int cs43130_handle_device_data(struct cs43130_private *cs43130) static int cs43130_i2c_probe(struct i2c_client *client) { + const struct snd_soc_component_driver *component_driver; struct cs43130_private *cs43130; int ret; unsigned int reg; @@ -2596,39 +2611,15 @@ static int cs43130_i2c_probe(struct i2c_client *client) switch (cs43130->dev_id) { case CS43130_CHIP_ID: case CS43131_CHIP_ID: - memcpy(all_hp_widgets, digital_hp_widgets, - sizeof(digital_hp_widgets)); - memcpy(all_hp_widgets + ARRAY_SIZE(digital_hp_widgets), - analog_hp_widgets, sizeof(analog_hp_widgets)); - memcpy(all_hp_routes, digital_hp_routes, - sizeof(digital_hp_routes)); - memcpy(all_hp_routes + ARRAY_SIZE(digital_hp_routes), - analog_hp_routes, sizeof(analog_hp_routes)); - - soc_component_dev_cs43130.dapm_widgets = - all_hp_widgets; - soc_component_dev_cs43130.num_dapm_widgets = - ARRAY_SIZE(all_hp_widgets); - soc_component_dev_cs43130.dapm_routes = - all_hp_routes; - soc_component_dev_cs43130.num_dapm_routes = - ARRAY_SIZE(all_hp_routes); + component_driver = &soc_component_dev_cs43130_analog; break; case CS43198_CHIP_ID: case CS4399_CHIP_ID: - soc_component_dev_cs43130.dapm_widgets = - digital_hp_widgets; - soc_component_dev_cs43130.num_dapm_widgets = - ARRAY_SIZE(digital_hp_widgets); - soc_component_dev_cs43130.dapm_routes = - digital_hp_routes; - soc_component_dev_cs43130.num_dapm_routes = - ARRAY_SIZE(digital_hp_routes); + component_driver = &soc_component_dev_cs43130_digital; break; } - ret = devm_snd_soc_register_component(cs43130->dev, - &soc_component_dev_cs43130, + ret = devm_snd_soc_register_component(cs43130->dev, component_driver, cs43130_dai, ARRAY_SIZE(cs43130_dai)); if (ret < 0) { dev_err(cs43130->dev, -- 2.45.2

2 1

Soundblaster AE-5 100% Muted on alsamixer every boot!!
by GitHub issues - opened 02 Aug '24

02 Aug '24

alsa-project/alsa-lib issue #404 was opened from DavidRLTG: I am on Linux Mint Wilma (22), 64-bit. Obviously, I have a Soundblaster AE-5 Sound card The issue I am encountering is, well, every boot I have to enter Alsamixer and unmute every channel on my Soundblaster AE-5. alsactl store does nothing, and yes I ran it as superuser. Also, I'm on pipewire! I think Wilma comes with it preinstalled. sudo alsactl store is also temporary, the config is lost at every reboot, and doing sudo alsactl restore does nothing. Any ideas for a fix? Issue URL : https://github.com/alsa-project/alsa-lib/issues/404 Repository URL: https://github.com/alsa-project/alsa-lib

1 0