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

[PATCH] ALSA: hda/realtek - Fix microphone noise on ASUS TUF B550M-PLUS
by Marios Levogiannis 30 May '22

30 May '22

Set microphone pins 0x18 (rear) and 0x19 (front) to VREF_50 to fix the microphone noise on ASUS TUF B550M-PLUS which uses the ALCS1200A codec. The initial value was VREF_80. The same issue is also present on Windows using both the default Windows driver and all tested Realtek drivers before version 6.0.9049.1. Comparing Realtek driver 6.0.9049.1 (the first one without the microphone noise) to Realtek driver 6.0.9047.1 (the last one with the microphone noise) revealed that the fix is the result of setting pins 0x18 and 0x19 to VREF_50. This fix may also work for other boards that have been reported to have the same microphone issue and use the ALC1150 and ALCS1200A codecs, since these codecs are similar and the fix in the Realtek driver on Windows is common for both. However, it is currently enabled only for ASUS TUF B550M-PLUS as this is the only board that could be tested. --- sound/pci/hda/patch_realtek.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 89d610afa..f3ad454b3 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -1981,6 +1981,7 @@ enum { ALC1220_FIXUP_CLEVO_PB51ED_PINS, ALC887_FIXUP_ASUS_AUDIO, ALC887_FIXUP_ASUS_HMIC, + ALCS1200A_FIXUP_MIC_VREF, }; static void alc889_fixup_coef(struct hda_codec *codec, @@ -2526,6 +2527,14 @@ static const struct hda_fixup alc882_fixups[] = { .chained = true, .chain_id = ALC887_FIXUP_ASUS_AUDIO, }, + [ALCS1200A_FIXUP_MIC_VREF] = { + .type = HDA_FIXUP_PINCTLS, + .v.pins = (const struct hda_pintbl[]) { + { 0x18, PIN_VREF50 }, /* rear mic */ + { 0x19, PIN_VREF50 }, /* front mic */ + {} + } + }, }; static const struct snd_pci_quirk alc882_fixup_tbl[] = { @@ -2563,6 +2572,7 @@ static const struct snd_pci_quirk alc882_fixup_tbl[] = { SND_PCI_QUIRK(0x1043, 0x835f, "Asus Eee 1601", ALC888_FIXUP_EEE1601), SND_PCI_QUIRK(0x1043, 0x84bc, "ASUS ET2700", ALC887_FIXUP_ASUS_BASS), SND_PCI_QUIRK(0x1043, 0x8691, "ASUS ROG Ranger VIII", ALC882_FIXUP_GPIO3), + SND_PCI_QUIRK(0x1043, 0x8797, "ASUS TUF B550M-PLUS", ALCS1200A_FIXUP_MIC_VREF), SND_PCI_QUIRK(0x104d, 0x9043, "Sony Vaio VGC-LN51JGB", ALC882_FIXUP_NO_PRIMARY_HP), SND_PCI_QUIRK(0x104d, 0x9044, "Sony VAIO AiO", ALC882_FIXUP_NO_PRIMARY_HP), SND_PCI_QUIRK(0x104d, 0x9047, "Sony Vaio TT", ALC889_FIXUP_VAIO_TT), -- 2.36.1

2 3

[PATCH 00/15] rewrite public API according to convention of GNOME project
by GitHub pull_request - opened 30 May '22

30 May '22

alsa-project/alsa-gobject pull request #70 was opened from takaswie: In GNOME convention for throw function which has an argument of GError to report failure, the function is programmed to return value of gboolean type to report whether the argument is filled with instance of GError. On the other hand, I wrote the throw functions to return nothing (void). This patchset rewrite public API according to it. Unfortunately, it brings loss of backward compatibility. ``` Takashi Sakamoto (15): ctl: remove useless exposed symbols timer: user-instance: fix closure marshaller for event signal ctl: query: rewrite public API to return gboolean according to GNOME convention ctl: card: rewrite public API to return gboolean according to GNOME convention ctl: elem_info: rewrite public API to return gboolean according to GNOME convention rawmidi: query: rewrite public API to return gboolean according to GNOME convention rawmidi: stream_pair: rewrite public API to return gboolean according to GNOME convention hwdep: query: rewrite public API to return gboolean according to GNOME convention timer: query: rewrite public API to return gboolean according to GNOME convention timer: user_instance: rewrite public API to return gboolean according to GNOME convention timer: instance-params: rewrite public API to return gboolean according to GNOME convention seq: query: rewrite public API to return gboolean according to GNOME convention seq: user_client: rewrite public API to return gboolean according to GNOME convention seq: client_info: rewrite public API to return gboolean according to GNOME convention seq: event_cntr: rewrite public API to return gboolean according to GNOME convention samples/ctl | 21 +- samples/hwdep | 8 +- samples/rawmidi | 14 +- samples/seq | 37 +- samples/timer | 9 +- src/ctl/alsactl.map | 73 ++-- src/ctl/card.c | 322 +++++++++------- src/ctl/card.h | 82 ++--- src/ctl/elem-info.c | 84 +++-- src/ctl/elem-info.h | 12 +- src/ctl/query.c | 59 ++- src/ctl/query.h | 8 +- src/hwdep/alsahwdep.map | 13 +- src/hwdep/query.c | 64 ++-- src/hwdep/query.h | 17 +- src/rawmidi/alsarawmidi.map | 37 +- src/rawmidi/privates.h | 2 +- src/rawmidi/query.c | 96 +++-- src/rawmidi/query.h | 37 +- src/rawmidi/stream-pair.c | 163 +++++---- src/rawmidi/stream-pair.h | 31 +- src/seq/alsaseq.map | 98 ++--- src/seq/client-info.c | 38 +- src/seq/client-info.h | 16 +- src/seq/event-cntr.c | 711 +++++++++++++++++++++--------------- src/seq/event-cntr.h | 203 +++++----- src/seq/query.c | 298 +++++++++------ src/seq/query.h | 46 +-- src/seq/user-client.c | 413 +++++++++++++-------- src/seq/user-client.h | 140 +++---- src/timer/alsatimer.map | 44 ++- src/timer/instance-params.c | 30 +- src/timer/instance-params.h | 4 +- src/timer/query.c | 163 ++++++--- src/timer/query.h | 23 +- src/timer/user-instance.c | 192 ++++++---- src/timer/user-instance.h | 43 +-- 37 files changed, 2117 insertions(+), 1534 deletions(-) ``` Request URL : https://github.com/alsa-project/alsa-gobject/pull/70 Patch URL : https://github.com/alsa-project/alsa-gobject/pull/70.patch Repository URL: https://github.com/alsa-project/alsa-gobject

1 0

[PATCH v2 0/6] OPE support on Tegra210 and later
by Sameer Pujar 30 May '22

30 May '22

This series adds support for Output Prcoessing Module (OPE) module on Tegra210 and later generations of SoCs. OPE is a client of AHUB and it has sub blocks of PEQ (Parametric Equalizer) and MBDRC (Multi Band Dynamic Range Compressor) for data processing. An ASoC component is registered for OPE, which includes PEQ and MBDRC functions as well. This can be plugged in audio path using ALSA mixer controls. The series adds necessary binding documentaion, driver and DT binding patches to enable OPE module on Jetson platforms. Changelog ========= v1 -> v2: --------- * Use generic node names for OPE, PEQ and MBDRC devices. Update binding doc and DT patches for this. * Remove redundant nodename rule enforcement from OPE, PEQ and MBDRC nodes. Update binding doc patch for this. * Fix spaces before binding doc examples and remove '|' from binding doc descriptions. Sameer Pujar (6): ASoC: tegra: Add binding doc for OPE module ASoC: tegra: Add Tegra210 based OPE driver ASoC: tegra: AHUB routes for OPE module arm64: defconfig: Build Tegra OPE module arm64: tegra: Add OPE device on Tegra210 and later arm64: tegra: Enable OPE on various platforms .../bindings/sound/nvidia,tegra210-ahub.yaml | 4 + .../bindings/sound/nvidia,tegra210-mbdrc.yaml | 47 + .../bindings/sound/nvidia,tegra210-ope.yaml | 87 ++ .../bindings/sound/nvidia,tegra210-peq.yaml | 48 + arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts | 43 + arch/arm64/boot/dts/nvidia/tegra186.dtsi | 23 + arch/arm64/boot/dts/nvidia/tegra194-p2972-0000.dts | 43 + .../arm64/boot/dts/nvidia/tegra194-p3509-0000.dtsi | 43 + arch/arm64/boot/dts/nvidia/tegra194.dtsi | 23 + arch/arm64/boot/dts/nvidia/tegra210-p2371-2180.dts | 84 ++ arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts | 84 ++ arch/arm64/boot/dts/nvidia/tegra210.dtsi | 40 + .../dts/nvidia/tegra234-p3737-0000+p3701-0000.dts | 43 + arch/arm64/boot/dts/nvidia/tegra234.dtsi | 23 + arch/arm64/configs/defconfig | 1 + sound/soc/tegra/Kconfig | 9 + sound/soc/tegra/Makefile | 2 + sound/soc/tegra/tegra210_ahub.c | 39 +- sound/soc/tegra/tegra210_mbdrc.c | 1012 ++++++++++++++++++++ sound/soc/tegra/tegra210_mbdrc.h | 215 +++++ sound/soc/tegra/tegra210_ope.c | 419 ++++++++ sound/soc/tegra/tegra210_ope.h | 90 ++ sound/soc/tegra/tegra210_peq.c | 434 +++++++++ sound/soc/tegra/tegra210_peq.h | 56 ++ 24 files changed, 2908 insertions(+), 4 deletions(-) create mode 100644 Documentation/devicetree/bindings/sound/nvidia,tegra210-mbdrc.yaml create mode 100644 Documentation/devicetree/bindings/sound/nvidia,tegra210-ope.yaml create mode 100644 Documentation/devicetree/bindings/sound/nvidia,tegra210-peq.yaml create mode 100644 sound/soc/tegra/tegra210_mbdrc.c create mode 100644 sound/soc/tegra/tegra210_mbdrc.h create mode 100644 sound/soc/tegra/tegra210_ope.c create mode 100644 sound/soc/tegra/tegra210_ope.h create mode 100644 sound/soc/tegra/tegra210_peq.c create mode 100644 sound/soc/tegra/tegra210_peq.h -- 2.7.4

2 9

ucm2 - splitting 10 channel interface into mix of mono and stereo channels
by GitHub issues - opened 29 May '22

29 May '22

alsa-project/alsa-ucm-conf issue #161 was opened from smangels: Hi, I've pushed a PR for the Behringer Flow8 digital mixer configured to be in "streaming" mode. Now that I've managed to wrap my head around the configuration, I would like to continue with the so called "recording" mode (different ID in `lsusb`). Thus I wonder whether UCM2 syntax allows me to split a 10 channel interface into a mix of mono and stereo channels (mic - mono, master, stereo)? Regards, Sebastian Mangelsen Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/161 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[PATCH v2] ASoC: dt-bindings: mediatek: mt8192: Add i2s-share properties
by Nícolas F. R. A. Prado 29 May '22

29 May '22

The Mediatek AFE PCM controller for MT8192 allows two I2S interfaces to share the same clock and act as a single interface with both input and output. Add patterns for these properties in the dt-binding. The property is split into two patterns in order to allow all valid interface pairings. Signed-off-by: Nícolas F. R. A. Prado <nfraprado(a)collabora.com> --- The series from v1 of this patch was merged although some changes were still needed in this patch, so the v1 of this patch was reverted [1] and this standalone commit addresses the feedback from v1 and readds the property. [1] https://lore.kernel.org/all/20220509185625.580811-1-nfraprado@collabora.com v1: https://lore.kernel.org/all/20220429203039.2207848-2-nfraprado@collabora.co… Changes in v2: - Added "mediatek," prefix to property - Rewrote and added more information to property description - Split into two patterns to validate that output-input pairings are done .../bindings/sound/mt8192-afe-pcm.yaml | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml b/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml index 7a25bc9b8060..2abf43c6c2c3 100644 --- a/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml +++ b/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml @@ -54,6 +54,22 @@ properties: - const: aud_infra_clk - const: aud_infra_26m_clk +patternProperties: + "^mediatek,i2s[13579]-share$": + description: + Each I2S interface has a single data line, input if its index is even or + output if the index is odd. An input and an output I2S interface can be + used together as if they were a single I2S interface with both input and + output data lines by sharing the same clock. This property represents this + pairing. The value should be the name of the interface whose clock is + used, and the property name the other interface that depends on this + clock. + pattern: "^I2S[0268]$" + + "^mediatek,i2s[0268]-share$": + description: Same as above. + pattern: "^I2S[13579]$" + required: - compatible - interrupts -- 2.36.0

3 5

ucm2 profile for Behringer Flow8
by GitHub pull_request - edited 29 May '22

29 May '22

alsa-project/alsa-ucm-conf pull request #160 was edited from smangels: - kudos to Pekka Oinas for supporting me finding the right settings - provides an UCM2 profile for Behringer's digital mixer Flow8 configured into "streaming" mode - 2 stereo outputs, 1 stereo input, when vendor ID product ID are `1397:050d` - prepare filenames for Flow8's recording mode `1397:050c` (10 chan input, 4 chan output) Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/160 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/160.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

ucm2 profile for Behringer Flow8
by GitHub pull_request - edited 29 May '22

29 May '22

alsa-project/alsa-ucm-conf pull request #160 was edited from smangels: - kudos to Pekka Oinas for supporting me finding the right settings - provides an UCM2 profile for Behringer's digital mixer Flow8 configured into "streaming" mode - 2 stereo outputs, 1 stereo input - prepare filenames for Flow8's recording mode (10 chan input, 4 chan output) Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/160 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/160.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

ucm2 profile for Behringer Flow8
by GitHub pull_request - opened 29 May '22

29 May '22

alsa-project/alsa-ucm-conf pull request #160 was opened from smangels: - kudos to Pekka Oinas for supporting me finding the right settings - provides an UCM2 profile for Behringer's digital mixer Flow configured into "streaming" mode - 2 stereo outputs, 1 stereo input - prepare filenames for Flow8's recording mode (10 chan input, 4 chan output) Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/160 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/160.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

ucm2 profile for Behringer Flow8
by GitHub pull_request - opened 28 May '22

28 May '22

alsa-project/alsa-ucm-conf pull request #159 was opened from smangels: - kudos to Pekka Oinas for supporting me finding the right settings - provides an UCM2 profile for Behringer's digital mixer Flow configured into "streaming" mode - 2 stereo outputs, 1 stereo input Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/159 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/159.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[PATCH] ALSA: hda/via: Delete does not require return
by Xiang wangx 27 May '22

27 May '22

Void function return statements are not generally useful. Signed-off-by: Xiang wangx <wangxiang(a)cdjrlc.com> --- sound/pci/hda/patch_via.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/sound/pci/hda/patch_via.c b/sound/pci/hda/patch_via.c index 773a136161f1..a05304f340df 100644 --- a/sound/pci/hda/patch_via.c +++ b/sound/pci/hda/patch_via.c @@ -449,8 +449,6 @@ static void vt1708_set_pinconfig_connect(struct hda_codec *codec, hda_nid_t nid) def_conf = def_conf & (~(AC_JACK_PORT_BOTH << 30)); snd_hda_codec_set_pincfg(codec, nid, def_conf); } - - return; } static int vt1708_jack_detect_get(struct snd_kcontrol *kcontrol, -- 2.36.1

2 1