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

[PATCH 0/3] hwdep: define interface for hwdep device
by GitHub pull_request - edited 02 Jun '22

02 Jun '22

alsa-project/alsa-gobject pull request #73 was edited from takaswie: The protocol in ALSA HwDep character device has common feature to each device. It's suitable for userspace implementation to have GObject interface for such features. This patchset adds the interface. The implementation of interface should have four methods; open, get_protocol_version, get_device_info, and create_source. Additionally, it should emit handle-disconnection signal when detecting ENODEV error in the interation of source. It requires to notify userspace application since the application should release character device immediately. ``` Takashi Sakamoto (3): hwdep: use explicit value for iface type enumeration hwdep: device-common: add interface for common feature of ALSA HwDep device hwdep: add error domain for implementators of DeviceCommon interface src/hwdep/alsahwdep-enum-types.h | 89 +++++++++------- src/hwdep/alsahwdep.h | 2 + src/hwdep/alsahwdep.map | 10 ++ src/hwdep/device-common.c | 178 +++++++++++++++++++++++++++++++ src/hwdep/device-common.h | 106 ++++++++++++++++++ src/hwdep/meson.build | 2 + tests/alsahwdep-enums | 10 ++ 7 files changed, 356 insertions(+), 41 deletions(-) create mode 100644 src/hwdep/device-common.c create mode 100644 src/hwdep/device-common.h ``` Request URL : https://github.com/alsa-project/alsa-gobject/pull/73 Patch URL : https://github.com/alsa-project/alsa-gobject/pull/73.patch Repository URL: https://github.com/alsa-project/alsa-gobject

1 0

[PATCH 0/2] hwdep: define interface for hwdep device
by GitHub pull_request - edited 02 Jun '22

02 Jun '22

alsa-project/alsa-gobject pull request #73 was edited from takaswie: The protocol in ALSA HwDep character device has common feature to each device. It's suitable for userspace implementation to have GObject interface for such features. This patchset adds the interface. The implementation of interface should have four methods; open, get_protocol_version, get_device_info, and create_source. Additionally, it should emit handle-disconnection signal when detecting ENODEV error in the interation of source. It requires to notify userspace application since the application should release character device immediately. ``` Takashi Sakamoto (3): hwdep: use explicit value for iface type enumeration hwdep: device-common: add interface for common feature of ALSA HwDep device hwdep: add error domain for implementators of DeviceCommon interface src/hwdep/alsahwdep-enum-types.h | 89 +++++++++------- src/hwdep/alsahwdep.h | 2 + src/hwdep/alsahwdep.map | 10 ++ src/hwdep/device-common.c | 178 +++++++++++++++++++++++++++++++ src/hwdep/device-common.h | 106 ++++++++++++++++++ src/hwdep/meson.build | 2 + tests/alsahwdep-enums | 10 ++ 7 files changed, 356 insertions(+), 41 deletions(-) create mode 100644 src/hwdep/device-common.c create mode 100644 src/hwdep/device-common.h ``` Request URL : https://github.com/alsa-project/alsa-gobject/pull/73 Patch URL : https://github.com/alsa-project/alsa-gobject/pull/73.patch Repository URL: https://github.com/alsa-project/alsa-gobject

1 0

[PATCH 0/2] hwdep: define interface for hwdep device
by GitHub pull_request - opened 02 Jun '22

02 Jun '22

alsa-project/alsa-gobject pull request #73 was opened from takaswie: The protocol in ALSA HwDep character device has common feature to each device. It's suitable for userspace implementation to have GObject interface for such features. This patchset adds the interface. The implementation of interface should have four methods; open, get_protocol_version, get_device_info, and create_source. Additionally, it should emit handle-disconnection signal when detecting ENODEV error in the interation of source. It requires to notify userspace application since the application should release character device immediately. ``` Takashi Sakamoto (2): hwdep: use explicit value for iface type enumeration hwdep: device-common: add interface for common feature of ALSA HwDep device src/hwdep/alsahwdep-enum-types.h | 67 +++++++--------- src/hwdep/alsahwdep.h | 2 + src/hwdep/alsahwdep.map | 6 ++ src/hwdep/device-common.c | 128 +++++++++++++++++++++++++++++++ src/hwdep/device-common.h | 102 ++++++++++++++++++++++++ src/hwdep/meson.build | 2 + 6 files changed, 267 insertions(+), 40 deletions(-) create mode 100644 src/hwdep/device-common.c create mode 100644 src/hwdep/device-common.h ``` Request URL : https://github.com/alsa-project/alsa-gobject/pull/73 Patch URL : https://github.com/alsa-project/alsa-gobject/pull/73.patch Repository URL: https://github.com/alsa-project/alsa-gobject

1 0

Semantics for _SX controls
by Mark Brown 02 Jun '22

02 Jun '22

Hi, The fixes for input validation have highlighed a bunch of problems with the _SX controls. These aren't widely used, it's just some Cirrus Logic and Qualcomm CODECs plus the obsolte ab8500 driver, but it has become apparent that at least some of the drivers were relying on the lack of validation we used to have to work. For the Cirrus case 34198710f55b5f ("ASoC: Add info callback for SX_TLV controls") says that the intended semantic is Currently every CODEC implementing these controls specifies the minimum as the non-zero value for the minimum and the maximum as the number of gain settings available. which was from Charles at Cirrus so hopefully that's the semantic used by Cirrus drivers. However Tan Nayir pointed at some out of tree patches from Qualcomm which change the info callback such that max is instead the maximum register value that can be written and reports that since we started validating written values properly they can no longer set some valid values in some controls on at least some Qualcomm devices (this was a wcd9340) which clearly indicates that there's some problems that need fixing. This isn't helped by the fact that snd_soc_info_volsw_sx() is implemented by calling snd_soc_info_volsw() and then applying an offset to the maximum value which makes things harder to follow - I think we should make snd_soc_info_volsw_sx() a standalone function, and we probably need some fixes in some combination of the shared functions and the drivers as well. Can people please check how the values in the current drivers correspond to what the actual bitfields in the registers have so we can get a handle on what's going on here? I've pasted a quick grep for _SX controls below: sound/soc/codecs/ab8500-codec.c: SOC_SINGLE_XR_SX("ANC Warp Delay Shift", sound/soc/codecs/ab8500-codec.c: SOC_SINGLE_XR_SX("ANC FIR Output Shift", sound/soc/codecs/ab8500-codec.c: SOC_SINGLE_XR_SX("ANC IIR Output Shift", sound/soc/codecs/ab8500-codec.c: SOC_SINGLE_XR_SX("ANC Warp Delay", sound/soc/codecs/cs35l33.c: SOC_SINGLE_SX_TLV("DAC Volume", CS35L33_DIG_VOL_CTL, sound/soc/codecs/cs35l34.c: SOC_SINGLE_SX_TLV("Digital Volume", CS35L34_AMP_DIG_VOL, sound/soc/codecs/cs35l35.c: SOC_SINGLE_SX_TLV("Digital Audio Volume", CS35L35_AMP_DIG_VOL, sound/soc/codecs/cs35l35.c: SOC_SINGLE_SX_TLV("Digital Advisory Volume", CS35L35_ADV_DIG_VOL, sound/soc/codecs/cs35l36.c: SOC_SINGLE_SX_TLV("Digital PCM Volume", CS35L36_AMP_DIG_VOL_CTRL, sound/soc/codecs/cs35l41.c: SOC_SINGLE_SX_TLV("Digital PCM Volume", CS35L41_AMP_DIG_VOL_CTRL, sound/soc/codecs/cs4265.c: SOC_DOUBLE_R_SX_TLV("PGA Volume", CS4265_CHA_PGA_CTL, sound/soc/codecs/cs42l51.c: SOC_DOUBLE_R_SX_TLV("PCM Playback Volume", sound/soc/codecs/cs42l51.c: SOC_DOUBLE_R_SX_TLV("Analog Playback Volume", sound/soc/codecs/cs42l51.c: SOC_DOUBLE_R_SX_TLV("ADC Mixer Volume", sound/soc/codecs/cs42l51.c: SOC_DOUBLE_R_SX_TLV("ADC Attenuator Volume", sound/soc/codecs/cs42l51.c: SOC_DOUBLE_R_SX_TLV("PGA Volume", sound/soc/codecs/cs42l52.c: SOC_DOUBLE_R_SX_TLV("Master Volume", CS42L52_MASTERA_VOL, sound/soc/codecs/cs42l52.c: SOC_DOUBLE_R_SX_TLV("Headphone Volume", CS42L52_HPA_VOL, sound/soc/codecs/cs42l52.c: SOC_DOUBLE_R_SX_TLV("Speaker Volume", CS42L52_SPKA_VOL, sound/soc/codecs/cs42l52.c: SOC_DOUBLE_R_SX_TLV("Bypass Volume", CS42L52_PASSTHRUA_VOL, sound/soc/codecs/cs42l52.c: SOC_DOUBLE_R_SX_TLV("ADC Volume", CS42L52_ADCA_VOL, sound/soc/codecs/cs42l52.c: SOC_DOUBLE_R_SX_TLV("ADC Mixer Volume", sound/soc/codecs/cs42l52.c: SOC_DOUBLE_R_SX_TLV("PGA Volume", CS42L52_PGAA_CTL, sound/soc/codecs/cs42l52.c: SOC_DOUBLE_R_SX_TLV("PCM Mixer Volume", sound/soc/codecs/cs42l52.c: SOC_SINGLE_SX_TLV("Beep Volume", CS42L52_BEEP_VOL, sound/soc/codecs/cs42l56.c: SOC_DOUBLE_R_SX_TLV("Master Volume", CS42L56_MASTER_A_VOLUME, sound/soc/codecs/cs42l56.c: SOC_DOUBLE_R_SX_TLV("ADC Mixer Volume", CS42L56_ADCA_MIX_VOLUME, sound/soc/codecs/cs42l56.c: SOC_DOUBLE_R_SX_TLV("PCM Mixer Volume", CS42L56_PCMA_MIX_VOLUME, sound/soc/codecs/cs42l56.c: SOC_DOUBLE_R_SX_TLV("PGA Volume", CS42L56_PGAA_MUX_VOLUME, sound/soc/codecs/cs42l56.c: SOC_DOUBLE_R_SX_TLV("Headphone Volume", CS42L56_HPA_VOLUME, sound/soc/codecs/cs42l56.c: SOC_DOUBLE_R_SX_TLV("LineOut Volume", CS42L56_LOA_VOLUME, sound/soc/codecs/cs42l56.c: SOC_SINGLE_SX_TLV("Beep Volume", CS42L56_BEEP_FREQ_OFFTIME, sound/soc/codecs/cs42l73.c: SOC_DOUBLE_R_SX_TLV("Headphone Analog Playback Volume", sound/soc/codecs/cs42l73.c: SOC_DOUBLE_R_SX_TLV("LineOut Analog Playback Volume", CS42L73_LOAAVOL, sound/soc/codecs/cs42l73.c: SOC_DOUBLE_R_SX_TLV("Input PGA Analog Volume", CS42L73_MICAPREPGAAVOL, sound/soc/codecs/cs42l73.c: SOC_DOUBLE_R_SX_TLV("Input Path Digital Volume", CS42L73_IPADVOL, sound/soc/codecs/cs42l73.c: SOC_DOUBLE_R_SX_TLV("HL Digital Playback Volume", sound/soc/codecs/cs42l73.c: SOC_SINGLE_SX_TLV("Speakerphone Digital Volume", sound/soc/codecs/cs42l73.c: SOC_SINGLE_SX_TLV("Ear Speaker Digital Volume", sound/soc/codecs/cs53l30.c: SOC_SINGLE_SX_TLV("ADC1A PGA Volume", sound/soc/codecs/cs53l30.c: SOC_SINGLE_SX_TLV("ADC1B PGA Volume", sound/soc/codecs/cs53l30.c: SOC_SINGLE_SX_TLV("ADC2A PGA Volume", sound/soc/codecs/cs53l30.c: SOC_SINGLE_SX_TLV("ADC2B PGA Volume", sound/soc/codecs/cs53l30.c: SOC_SINGLE_SX_TLV("ADC1A Digital Volume", sound/soc/codecs/cs53l30.c: SOC_SINGLE_SX_TLV("ADC1B Digital Volume", sound/soc/codecs/cs53l30.c: SOC_SINGLE_SX_TLV("ADC2A Digital Volume", sound/soc/codecs/cs53l30.c: SOC_SINGLE_SX_TLV("ADC2B Digital Volume", sound/soc/codecs/msm8916-wcd-digital.c: SOC_SINGLE_SX_TLV("IIR1 INP1 Volume", LPASS_CDC_IIR1_GAIN_B1_CTL, sound/soc/codecs/msm8916-wcd-digital.c: SOC_SINGLE_SX_TLV("IIR1 INP2 Volume", LPASS_CDC_IIR1_GAIN_B2_CTL, sound/soc/codecs/msm8916-wcd-digital.c: SOC_SINGLE_SX_TLV("IIR1 INP3 Volume", LPASS_CDC_IIR1_GAIN_B3_CTL, sound/soc/codecs/msm8916-wcd-digital.c: SOC_SINGLE_SX_TLV("IIR1 INP4 Volume", LPASS_CDC_IIR1_GAIN_B4_CTL, sound/soc/codecs/msm8916-wcd-digital.c: SOC_SINGLE_SX_TLV("IIR2 INP1 Volume", LPASS_CDC_IIR2_GAIN_B1_CTL, sound/soc/codecs/msm8916-wcd-digital.c: SOC_SINGLE_SX_TLV("IIR2 INP2 Volume", LPASS_CDC_IIR2_GAIN_B2_CTL, sound/soc/codecs/msm8916-wcd-digital.c: SOC_SINGLE_SX_TLV("IIR2 INP3 Volume", LPASS_CDC_IIR2_GAIN_B3_CTL, sound/soc/codecs/msm8916-wcd-digital.c: SOC_SINGLE_SX_TLV("IIR2 INP4 Volume", LPASS_CDC_IIR2_GAIN_B4_CTL, sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX0 Digital Volume", WCD9335_CDC_RX0_RX_VOL_CTL, sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX1 Digital Volume", WCD9335_CDC_RX1_RX_VOL_CTL, sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX2 Digital Volume", WCD9335_CDC_RX2_RX_VOL_CTL, sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX3 Digital Volume", WCD9335_CDC_RX3_RX_VOL_CTL, sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX4 Digital Volume", WCD9335_CDC_RX4_RX_VOL_CTL, sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX5 Digital Volume", WCD9335_CDC_RX5_RX_VOL_CTL, sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX6 Digital Volume", WCD9335_CDC_RX6_RX_VOL_CTL, sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX7 Digital Volume", WCD9335_CDC_RX7_RX_VOL_CTL, sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX8 Digital Volume", WCD9335_CDC_RX8_RX_VOL_CTL, sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX0 Mix Digital Volume", sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX1 Mix Digital Volume", sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX2 Mix Digital Volume", sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX3 Mix Digital Volume", sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX4 Mix Digital Volume", sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX5 Mix Digital Volume", sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX6 Mix Digital Volume", sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX7 Mix Digital Volume", sound/soc/codecs/wcd9335.c: SOC_SINGLE_SX_TLV("RX8 Mix Digital Volume", Thanks, Mark

3 4

"topology: nhlt plugin: second attempt to fix the .so linking" cause build static library failed
by GitHub issues - opened 02 Jun '22

02 Jun '22

alsa-project/alsa-utils issue #152 was opened from youling257: ./configure --prefix=/system/usr --with-alsa-inc-prefix=/system/usr/include --with-alsa-prefix=/system/usr/lib --enable-static=yes --enable-shared=no make LDFLAGS="--static" -j8 Revert "topology: nhlt plugin: second attempt to fix the .so linking" can build static library. Issue URL : https://github.com/alsa-project/alsa-utils/issues/152 Repository URL: https://github.com/alsa-project/alsa-utils

1 0

[PATCH v3] ASoC: fsl_xcvr:Fix unbalanced pm_runtime_enable in fsl_xcvr_probe
by zhangqilong 02 Jun '22

02 Jun '22

a) Add missing pm_runtime_disable() when probe error out. It could avoid pm_runtime implementation complains when removing and probing again the driver. b) Add remove for missing pm_runtime_disable(). v2: --- Add remove to put PM usage counter. v3: --- Modify commit message. Fix:c590fa80b3928 ("ASoC: fsl_xcvr: register platform component before registering cpu dai") Signed-off-by: Zhang Qilong <zhangqilong3(a)huawei.com> --- sound/soc/fsl/fsl_xcvr.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/sound/soc/fsl/fsl_xcvr.c b/sound/soc/fsl/fsl_xcvr.c index d0556c79fdb1..55e640cba87d 100644 --- a/sound/soc/fsl/fsl_xcvr.c +++ b/sound/soc/fsl/fsl_xcvr.c @@ -1228,6 +1228,7 @@ static int fsl_xcvr_probe(struct platform_device *pdev) */ ret = devm_snd_dmaengine_pcm_register(dev, NULL, 0); if (ret) { + pm_runtime_disable(dev); dev_err(dev, "failed to pcm register\n"); return ret; } @@ -1235,6 +1236,7 @@ static int fsl_xcvr_probe(struct platform_device *pdev) ret = devm_snd_soc_register_component(dev, &fsl_xcvr_comp, &fsl_xcvr_dai, 1); if (ret) { + pm_runtime_disable(dev); dev_err(dev, "failed to register component %s\n", fsl_xcvr_comp.name); } @@ -1242,6 +1244,12 @@ static int fsl_xcvr_probe(struct platform_device *pdev) return ret; } +static int fsl_xcvr_remove(struct platform_device *pdev) +{ + pm_runtime_disable(&pdev->dev); + return 0; +} + static __maybe_unused int fsl_xcvr_runtime_suspend(struct device *dev) { struct fsl_xcvr *xcvr = dev_get_drvdata(dev); @@ -1370,6 +1378,7 @@ static struct platform_driver fsl_xcvr_driver = { .pm = &fsl_xcvr_pm_ops, .of_match_table = fsl_xcvr_dt_ids, }, + .remove = fsl_xcvr_remove, }; module_platform_driver(fsl_xcvr_driver); -- 2.31.1

1 0

[PATCH V2] ASoC: fsl_xcvr:Fix unbalanced pm_runtime_enable in fsl_xcvr_probe
by zhangqilong 02 Jun '22

02 Jun '22

a) Add missing pm_runtime_disable() when probe error out. It could avoid pm_runtime implementation complains when removing and probing again the driver. b) Add remove for missing pm_runtime_disable(). v2: add remove to put PM usage counter. Fix:c590fa80b3928 ("ASoC: fsl_xcvr: register platform component before registering cpu dai") Signed-off-by: Zhang Qilong <zhangqilong3(a)huawei.com> --- sound/soc/fsl/fsl_xcvr.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/sound/soc/fsl/fsl_xcvr.c b/sound/soc/fsl/fsl_xcvr.c index d0556c79fdb1..55e640cba87d 100644 --- a/sound/soc/fsl/fsl_xcvr.c +++ b/sound/soc/fsl/fsl_xcvr.c @@ -1228,6 +1228,7 @@ static int fsl_xcvr_probe(struct platform_device *pdev) */ ret = devm_snd_dmaengine_pcm_register(dev, NULL, 0); if (ret) { + pm_runtime_disable(dev); dev_err(dev, "failed to pcm register\n"); return ret; } @@ -1235,6 +1236,7 @@ static int fsl_xcvr_probe(struct platform_device *pdev) ret = devm_snd_soc_register_component(dev, &fsl_xcvr_comp, &fsl_xcvr_dai, 1); if (ret) { + pm_runtime_disable(dev); dev_err(dev, "failed to register component %s\n", fsl_xcvr_comp.name); } @@ -1242,6 +1244,12 @@ static int fsl_xcvr_probe(struct platform_device *pdev) return ret; } +static int fsl_xcvr_remove(struct platform_device *pdev) +{ + pm_runtime_disable(&pdev->dev); + return 0; +} + static __maybe_unused int fsl_xcvr_runtime_suspend(struct device *dev) { struct fsl_xcvr *xcvr = dev_get_drvdata(dev); @@ -1370,6 +1378,7 @@ static struct platform_driver fsl_xcvr_driver = { .pm = &fsl_xcvr_pm_ops, .of_match_table = fsl_xcvr_dt_ids, }, + .remove = fsl_xcvr_remove, }; module_platform_driver(fsl_xcvr_driver); -- 2.31.1

2 1

1.2.7: missing git tag
by GitHub issues - opened 01 Jun '22

01 Jun '22

alsa-project/alsa-utils issue #151 was opened from kloczek: https://www.alsa-project.org/wiki/Main_Page says that last release is 1.2.7 but there is no git tag for that version here. Issue URL : https://github.com/alsa-project/alsa-utils/issues/151 Repository URL: https://github.com/alsa-project/alsa-utils

1 0

1.2.7: missing git tag
by GitHub issues - opened 01 Jun '22

01 Jun '22

alsa-project/alsa-lib issue #232 was opened from kloczek: https://www.alsa-project.org/wiki/Main_Page says that last release is 1.2.7 but there is no git tag for that version here. Issue URL : https://github.com/alsa-project/alsa-lib/issues/232 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

soundwire: device drivers for PC based hardware
by Muni Sekhar 01 Jun '22

01 Jun '22

Hi all, I am looking for the soundwire device drivers which run on x86 CPU for PC based platforms. linux/drivers/soundwire at master · torvalds/linux · GitHub Can the above mentioned mainline tree drivers can be used for this purpose? config SOUNDWIRE_INTEL depends on ACPI && SND_SOC config SOUNDWIRE_QCOM depends on SND_SOC As per the Kconfig file, both the Intel and Qualcomm SoundWire Master driver depend on SND_SOC, does it mean it is only possible to run on SoC CPU(Arm chipsets)? -- Thanks, Sekhar

2 1