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

Missed review / mailing list submission?
by Andy Shevchenko 23 Apr '24

23 Apr '24

https://git.kernel.org/pub/scm/linux/kernel/git/broonie/ci.git/commit/?h=as… I can't find this in the mailing list. Moreover, it has an obstacle to be applied, i.e. as per https://elixir.bootlin.com/linux/v6.9-rc5/source/include/linux/gpio.h#L5 the gpio.h must not be included in a new code and brief looking into that driver shows that legacy APIs are being used without _any_ good reason. This has to be fixed (by the author). -- With Best Regards, Andy Shevchenko

2 4

[PATCH v4 0/7] Patches to update for rockchip pdm
by Sugar Zhang 23 Apr '24

23 Apr '24

These patches fixup or update for rockchip pdm. Changes in v4: - Acked by Rob Herring Changes in v3: - Fix property 'path-map' suggested by Rob Herring. Changes in v2: - Fix yamllint errors. Sugar Zhang (7): ASoC: rockchip: Add support for rv1126 pdm ASoC: dt-bindings: rockchip: Add binding for rv1126 pdm ASoC: rockchip: pdm: Add support for rk3568 pdm ASoC: dt-bindings: rockchip: Add binding for rk3568 pdm ASoC: rockchip: pdm: Add support for path map ASoC: dt-bindings: rockchip: pdm: Document property 'rockchip,path-map' ASoC: dt-bindings: rockchip: Convert pdm bindings to yaml .../devicetree/bindings/sound/rockchip,pdm.txt | 46 -------- .../devicetree/bindings/sound/rockchip,pdm.yaml | 120 +++++++++++++++++++++ sound/soc/rockchip/rockchip_pdm.c | 112 +++++++++++++++++-- sound/soc/rockchip/rockchip_pdm.h | 6 ++ 4 files changed, 232 insertions(+), 52 deletions(-) delete mode 100644 Documentation/devicetree/bindings/sound/rockchip,pdm.txt create mode 100644 Documentation/devicetree/bindings/sound/rockchip,pdm.yaml -- 2.7.4

3 9

[PATCH] ASoC: cs35l41: Delete unnecessary condition in cs35l41_pcm_hw_params()
by Dan Carpenter 23 Apr '24

23 Apr '24

This code returns -EINVAL if "i" is out of bounds a few lines earlier. Delete this unnecessary check and pull the code in a tab. Signed-off-by: Dan Carpenter <dan.carpenter(a)linaro.org> --- sound/soc/codecs/cs35l41.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/sound/soc/codecs/cs35l41.c b/sound/soc/codecs/cs35l41.c index 2799ccd6b5c7..d3da6a9f7f2b 100644 --- a/sound/soc/codecs/cs35l41.c +++ b/sound/soc/codecs/cs35l41.c @@ -772,10 +772,9 @@ static int cs35l41_pcm_hw_params(struct snd_pcm_substream *substream, asp_wl = params_width(params); - if (i < ARRAY_SIZE(cs35l41_fs_rates)) - regmap_update_bits(cs35l41->regmap, CS35L41_GLOBAL_CLK_CTRL, - CS35L41_GLOBAL_FS_MASK, - cs35l41_fs_rates[i].fs_cfg << CS35L41_GLOBAL_FS_SHIFT); + regmap_update_bits(cs35l41->regmap, CS35L41_GLOBAL_CLK_CTRL, + CS35L41_GLOBAL_FS_MASK, + cs35l41_fs_rates[i].fs_cfg << CS35L41_GLOBAL_FS_SHIFT); if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) { regmap_update_bits(cs35l41->regmap, CS35L41_SP_FORMAT, -- 2.43.0

2 1

conf: aliases: hdmi: Include unconditionally the pcm/hdmi.conf
by GitHub pull_request - opened 23 Apr '24

23 Apr '24

alsa-project/alsa-lib pull request #393 was opened from ujfalusi: The hdmi.conf contains the high level macro to be used by cards to create the hdmi: device. Instead of including it in different config files, include it in the main aliases.conf and remove it's inclusion by other config files. This change is needed to add support for the hdmi: device mapping via UCM. Suggested-by: Jaroslav Kysela <perex(a)perex.cz> Request URL : https://github.com/alsa-project/alsa-lib/pull/393 Patch URL : https://github.com/alsa-project/alsa-lib/pull/393.patch Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[PATCH] ASoC: cs35l56: Avoid static analysis warning of uninitialised variable
by Richard Fitzgerald 23 Apr '24

23 Apr '24

From: Simon Trimmer <simont(a)opensource.cirrus.com> Static checkers complain that the silicon_uid variable passed by pointer to cs35l56_read_silicon_uid() could later be used uninitialised when calling cs_amp_get_efi_calibration_data(). cs35l56_read_silicon_uid() must have succeeded to call cs_amp_get_efi_calibration_data() and that would have populated the variable. However, initialise the value so we are not haunted by it forevermore. Signed-off-by: Simon Trimmer <simont(a)opensource.cirrus.com> Fixes: e1830f66f6c6 ("ASoC: cs35l56: Add helper functions for amp calibration") Signed-off-by: Richard Fitzgerald <rf(a)opensource.cirrus.com> --- sound/soc/codecs/cs35l56-shared.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/cs35l56-shared.c b/sound/soc/codecs/cs35l56-shared.c index 3aa39f5923eb..8af89a263594 100644 --- a/sound/soc/codecs/cs35l56-shared.c +++ b/sound/soc/codecs/cs35l56-shared.c @@ -707,7 +707,7 @@ EXPORT_SYMBOL_NS_GPL(cs35l56_calibration_controls, SND_SOC_CS35L56_SHARED); int cs35l56_get_calibration(struct cs35l56_base *cs35l56_base) { - u64 silicon_uid; + u64 silicon_uid = 0; int ret; /* Driver can't apply calibration to a secured part, so skip */ -- 2.39.2

2 1

CM106L, CM6206, CM6206-LX
by James Lehman 22 Apr '24

22 Apr '24

Hello everyone. I have been using CMedia USB sound devices to make laser control signals for many years. I have sound devices with CM106L, CM6206 and CM6206-LX. Some are populated for 6 channels and some for 8. The CM106L chip works perfectly. I can send it a stream of 16-bit data that uses the full range of -32767 to +32767 and at 100% volume in either alsamixer or Pulseaudio Volume Control, I get a perfect image. https://laserboy.org/piDAC/perfect_circle.jpg If I use a device with a CM6206, I get a very near perfect image (only clipped a tiny bit). But If I use a CM6206-LX chip, it clips hard. https://laserboy.org/piDAC/CM6206-LX_100.jpg Even if I reduce the volume to 50%, it just seems to reduce the clipped signals. https://laserboy.org/piDAC/CM6206-LX_50.jpg I'm not sure what is going on here, but it seems like there is some kind of internal digital gain of more than 1.0. I also notice that lsusb shows exactly the same device ID for both the CM6206 and the CM6206-LX. .... 0d8c:0102 C-Media Electronics, Inc. CM106 Like Sound Device And what's really weird is that a genuine CM106L shows up as something else! .... 0d8c:0006 C-Media Electronics, Inc. Storm HP-USB500 5.1 Headset Is there something I can do in alsa config to fix this issue? The CM106L is long since out of production and it seems like any device that uses the CM6206 might have the LX version (or not). There's no way to know until I get them and open them up. I'm using this on an Orange Pi Zero 3 4gb running dietpi with the 6.6.16-current-sunxi64 kernel, but that doesn't seem to matter. I get the same results any other way I try it. The LX version of this chip clips the same way in Windows too! Thanks for your time. James.

1 0

[tiwai-sound:for-next 34/51] sound/usb/mixer_scarlett2.c:3690 scarlett2_ag_target_ctl_get() error: uninitialized symbol 'err'.
by Dan Carpenter 22 Apr '24

22 Apr '24

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next head: 32cb23a0f911317cdb5030035e49a204aa86fef5 commit: e30ea5340c25d41484d551f61d8c93106989019b [34/51] ALSA: scarlett2: Add autogain target controls config: csky-randconfig-r081-20240420 (https://download.01.org/0day-ci/archive/20240420/202404201509.mWdaL3xF-lkp@…) compiler: csky-linux-gcc (GCC) 13.2.0 If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp(a)intel.com> | Reported-by: Dan Carpenter <dan.carpenter(a)linaro.org> | Closes: https://lore.kernel.org/r/202404201509.mWdaL3xF-lkp@intel.com/ New smatch warnings: sound/usb/mixer_scarlett2.c:3690 scarlett2_ag_target_ctl_get() error: uninitialized symbol 'err'. vim +/err +3690 sound/usb/mixer_scarlett2.c e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3665 static int scarlett2_ag_target_ctl_get( e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3666 struct snd_kcontrol *kctl, struct snd_ctl_elem_value *ucontrol) e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3667 { e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3668 struct usb_mixer_elem_info *elem = kctl->private_data; e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3669 struct usb_mixer_interface *mixer = elem->head.mixer; e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3670 struct scarlett2_data *private = mixer->private_data; e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3671 int err; e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3672 e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3673 mutex_lock(&private->data_mutex); e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3674 e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3675 if (private->hwdep_in_use) { e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3676 err = -EBUSY; e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3677 goto unlock; e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3678 } e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3679 e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3680 if (private->autogain_updated) { e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3681 err = scarlett2_update_autogain(mixer); e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3682 if (err < 0) e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3683 goto unlock; e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3684 } err uninitialized on else path. e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3685 e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3686 ucontrol->value.integer.value[0] = private->ag_targets[elem->control]; e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3687 e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3688 unlock: e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3689 mutex_unlock(&private->data_mutex); e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 @3690 return err; e30ea5340c25d41 Geoffrey D. Bennett 2024-03-13 3691 } -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki

1 0

snd_hda_intel with buffer_size set to 32768 sounds horrible. Kernel driver bug?
by Jaime 22 Apr '24

22 Apr '24

Fresh Debian Bookworm (v12) installation with alsa but no sound server. aplay with no .asoundrc sounds great (default buffer_size is 16384). If I force the buffer_size to 32768 ([1]), then aplay sounds horrible ([2]). Is this a kernel driver bug? (If yes, I'll log it at https://bugzilla.kernel.org) I discovered this bug because pipewire sets the buffer_size to 32768 hence it triggers this bug - I've logged a bug against pipewire at [3]. Output from alsa-info at [4]. TIA. [1] To set aplay's buffer_size to 32768, I use the following ~/.asoundrc: pcm.!default { type plug slave { pcm { type dmix ipc_key 1234 slave { period_size 1024 buffer_size 32768 pcm "hw:0" format S32_LE } } } } [2] Upload of horrible sound at https://audio.com/jt-1796478503182974/audio/alsaonlybuf-32768nopipewire [3] https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/3966 [4] http://alsa-project.org/db/?f=5dbfb02c81bb1523505c725a189b6f0133f2b3d7

1 0

[PATCH v2 0/8] ASoC: codecs: wcd937x: add wcd937x audio codec support
by Mohammad Rafi Shaik 22 Apr '24

22 Apr '24

This patchset adds support for Qualcomm WCD937X codec. Qualcomm WCD9370/WCD9375 Codec is a standalone Hi-Fi audio codec IC connected over SoundWire. This device has two SoundWire devices, RX and TX respectively supporting 3 x ADCs, ClassH, Ear, Aux PA, 2xHPH, 6 DMICs and MBHC. For codec driver to be functional it would need both tx and rx Soundwire devices to be up and this is taken care by using device component framework and device-links are used to ensure proper pm dependencies. Ex tx does not enter suspend before rx or codec is suspended. This patchset along with other SoundWire patches on the list have been tested on QCM6490 IDP device. Changes since v8: - Split the patch per driver for easier review as suggested by Krzysztof - Used devm_gpiod_get api to get reset gpio as suggested by Krzysztof Prasad Kumpatla (8): ASoC: dt-bindings: wcd937x: add bindings for wcd937x ASoC: codecs: wcd937x: add wcd937x codec driver ASoC: dt-bindings: wcd937x-sdw: add bindings for wcd937x-sdw ASoC: codecs: wcd937x-sdw: add SoundWire driver ASoC: codecs: wcd937x: add basic controls ASoC: codecs: wcd937x: add playback dapm widgets ASoC: codecs: wcd937x: add capture dapm widgets ASoC: codecs: wcd937x: add audio routing and Kconfig .../bindings/sound/qcom,wcd937x-sdw.yaml | 71 + .../bindings/sound/qcom,wcd937x.yaml | 119 + sound/soc/codecs/Kconfig | 20 + sound/soc/codecs/Makefile | 7 + sound/soc/codecs/wcd937x-sdw.c | 1148 +++++++ sound/soc/codecs/wcd937x.c | 3036 +++++++++++++++++ sound/soc/codecs/wcd937x.h | 655 ++++ 7 files changed, 5056 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/qcom,wcd937x-sdw.yaml create mode 100644 Documentation/devicetree/bindings/sound/qcom,wcd937x.yaml create mode 100644 sound/soc/codecs/wcd937x-sdw.c create mode 100644 sound/soc/codecs/wcd937x.c create mode 100644 sound/soc/codecs/wcd937x.h base-commit: 6bd343537461b57f3efe5dfc5fc193a232dfef1e -- 2.25.1

5 18

sof-hda-dsp/sof-soundwire: Create "hdmi:" mapping PCMs to allow passthrough if supported
by GitHub pull_request - opened 22 Apr '24

22 Apr '24

alsa-project/alsa-ucm-conf pull request #411 was opened from ujfalusi: Hi, SOF with IPC4 can use 'ChainDMA' on selected PCMs (HDMI/DP currently) which allows them to be used for bytestream passthrough since the data is passed through unmodified. The kernel will list the PCMs with ChainDMA to the card's components list: https://github.com/thesofproject/linux/pull/4921 For example for sof-dsp-hda cards `iec61937-pcm:5,4,3` will be added. For user space to use HDMI passthrough, the `hdmi:` PCM device should be present correctly mapping to the machine, for example for sof-dsp-hda: ``` aplay -L | grep hdmi hdmi:CARD=sofhdadsp,DEV=0 hdmi:CARD=sofhdadsp,DEV=1 hdmi:CARD=sofhdadsp,DEV=2 ``` This PR does this by 1. Checking the `iec61937-pcm` indexes against the expected HDMI devices (sof-dsp-hda: 3-5, sof-soundwire: 5-7) 2. If there is a match (passthrough can be supported) then we will create three conf files: The generated files are: [1] /var/lib/alsa/conf.d/42-sof-hdmi.conf [2] /var/lib/alsa/card[card_number].conf.d/30-sof-hdmi-common.conf [3] /var/lib/alsa/card[card_number].conf.d/31-sof-hdmi.conf [1] includes the pcm/iec958.conf and pcm/hdmi.conf to global space of alsaconf to be used by the card macros [2] Card specific macros for hdmi PCM definition, mapping [3] Card specific definitions of the three HDMI port Note for [1]: I needed to use `shell "echo '...` since the `cfg-save` would expand the includes into the saved config file and that just does not result a working alsa configuration. @perexg, I'm sure all this can be done in a cleaner way... We cannot do this unconditionally as if the HDMI is not using ChainDMA (and SOF is not IPC4) then the passthrough is not possible since the firmware might touch the data. Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/411 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/411.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0