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

[alsa-devel] [PATCH v1 0/2] conf/ucm: Remove intel UCM conf files from alsa-lib repo
by fuweix.tang＠intel.com 21 Mar '17

21 Mar '17

From: Fuwei Tang <fuweix.tang(a)intel.com> Remove the intel UCM configs from alsa-lib repo, and move them to a dedicated UCM and topology file repo. Another series will add them to a dedicated UCM conf repo. License can be changed to BSD for intel files, and left as "unknown" for other conf files when they are moved from alsa-lib to the new repo. Fuwei Tang (2): conf/ucm: Remove conf files of broxton-rt298 platform conf/ucm: Remove conf files of skylake-rt286 platform configure.ac | 2 - src/conf/ucm/Makefile.am | 2 +- src/conf/ucm/broxton-rt298/Hdmi1 | 22 ----- src/conf/ucm/broxton-rt298/Hdmi2 | 14 --- src/conf/ucm/broxton-rt298/HiFi | 129 -------------------------- src/conf/ucm/broxton-rt298/Makefile.am | 4 - src/conf/ucm/broxton-rt298/broxton-rt298.conf | 58 ------------ src/conf/ucm/skylake-rt286/Hdmi1 | 20 ---- src/conf/ucm/skylake-rt286/Hdmi2 | 20 ---- src/conf/ucm/skylake-rt286/HiFi | 129 -------------------------- src/conf/ucm/skylake-rt286/Makefile.am | 4 - src/conf/ucm/skylake-rt286/skylake-rt286.conf | 60 ------------ 12 files changed, 1 insertion(+), 463 deletions(-) delete mode 100644 src/conf/ucm/broxton-rt298/Hdmi1 delete mode 100644 src/conf/ucm/broxton-rt298/Hdmi2 delete mode 100644 src/conf/ucm/broxton-rt298/HiFi delete mode 100644 src/conf/ucm/broxton-rt298/Makefile.am delete mode 100644 src/conf/ucm/broxton-rt298/broxton-rt298.conf delete mode 100644 src/conf/ucm/skylake-rt286/Hdmi1 delete mode 100644 src/conf/ucm/skylake-rt286/Hdmi2 delete mode 100644 src/conf/ucm/skylake-rt286/HiFi delete mode 100644 src/conf/ucm/skylake-rt286/Makefile.am delete mode 100644 src/conf/ucm/skylake-rt286/skylake-rt286.conf -- 2.7.4

1 2

[alsa-devel] [PATCH v1 0/3] ucm-conf: Add intel UCM conf files to alsa-ucm-conf repo
by fuweix.tang＠intel.com 21 Mar '17

21 Mar '17

From: Fuwei Tang <fuweix.tang(a)intel.com> Add the intel UCM configs to a dedicated UCM conf repo.License can be changed to BSD for intel files,and left as "unknown" for other conf files when they are moved from alsa-lib to the new repo.The alsa-lib LGPL license does not cover runtime configuration files. The configs are moved from alsa-lib repo.The original authorship and commit message of all config files will be reserved. I write the setup code from scratch instead of copying it from Takashi in this version. Fuwei Tang (1): ucm-conf: Set up for installation Vinod Koul (2): ucm-conf: broxton: add broxton-rt298 conf files ucm-conf: skylake: add skylake-rt286 conf files .gitignore | 13 ++++ ChangeLog | 1 + INSTALL | 1 + Makefile.am | 1 + broxton-rt298/Hdmi1 | 22 +++++++ broxton-rt298/Hdmi2 | 14 +++++ broxton-rt298/HiFi | 129 +++++++++++++++++++++++++++++++++++++++ broxton-rt298/Makefile.am | 4 ++ broxton-rt298/broxton-rt298.conf | 58 ++++++++++++++++++ configure.ac | 24 ++++++++ gitcompile | 5 ++ skylake-rt286/Hdmi1 | 20 ++++++ skylake-rt286/Hdmi2 | 20 ++++++ skylake-rt286/HiFi | 129 +++++++++++++++++++++++++++++++++++++++ skylake-rt286/Makefile.am | 4 ++ skylake-rt286/skylake-rt286.conf | 60 ++++++++++++++++++ 16 files changed, 505 insertions(+) create mode 100644 .gitignore create mode 100644 ChangeLog create mode 120000 INSTALL create mode 100644 Makefile.am create mode 100644 broxton-rt298/Hdmi1 create mode 100644 broxton-rt298/Hdmi2 create mode 100644 broxton-rt298/HiFi create mode 100644 broxton-rt298/Makefile.am create mode 100644 broxton-rt298/broxton-rt298.conf create mode 100644 configure.ac create mode 100755 gitcompile create mode 100644 skylake-rt286/Hdmi1 create mode 100644 skylake-rt286/Hdmi2 create mode 100644 skylake-rt286/HiFi create mode 100644 skylake-rt286/Makefile.am create mode 100644 skylake-rt286/skylake-rt286.conf -- 2.7.4

3 5

[alsa-devel] snd_hda_intel: probe of 0000:00:01.1 failed with error -2
by Jorge Fernandez Monteagudo 20 Mar '17

20 Mar '17

Hi all! I've a system with a custom distribution and a 3.16.7 linux kernel. Sometimes when the sound system is initialized I get this error and the sound is not working. I see in dmesg [ 36.104413] snd_hda_intel 0000:00:01.1: irq 80 for MSI/MSI-X [ 36.111812] sound hdaudioC1D0: autoconfig: line_outs=1 (0xa/0x0/0x0/0x0/0x0) type:line [ 36.111815] sound hdaudioC1D0: speaker_outs=1 (0xb/0x0/0x0/0x0/0x0) [ 36.111817] sound hdaudioC1D0: hp_outs=1 (0x9/0x0/0x0/0x0/0x0) [ 36.111819] sound hdaudioC1D0: mono: mono_out=0x0 [ 36.111820] sound hdaudioC1D0: dig-out=0x10/0x15 [ 36.111821] sound hdaudioC1D0: inputs: [ 36.111823] sound hdaudioC1D0: Mic=0xd [ 36.111824] sound hdaudioC1D0: Line=0xc [ 36.111826] sound hdaudioC1D0: dig-in=0x12 [ 36.115940] sound hdaudioC0D0: ignore pin 0x7, too many assigned pins [ 36.115942] sound hdaudioC0D0: autoconfig: line_outs=0 (0x0/0x0/0x0/0x0/0x0) type:line [ 36.115944] sound hdaudioC0D0: speaker_outs=0 (0x0/0x0/0x0/0x0/0x0) [ 36.115945] sound hdaudioC0D0: hp_outs=0 (0x0/0x0/0x0/0x0/0x0) [ 36.115946] sound hdaudioC0D0: mono: mono_out=0x0 [ 36.115947] sound hdaudioC0D0: dig-out=0x3/0x5 [ 36.115948] sound hdaudioC0D0: inputs: [ 37.119900] snd_hda_intel: probe of 0000:00:01.1 failed with error -2 [ 37.127608] sound hdaudioC0D0: autoconfig: line_outs=1 (0xa/0x0/0x0/0x0/0x0) type:line [ 37.127613] sound hdaudioC0D0: speaker_outs=1 (0xb/0x0/0x0/0x0/0x0) [ 37.127615] sound hdaudioC0D0: hp_outs=1 (0x9/0x0/0x0/0x0/0x0) [ 37.127617] sound hdaudioC0D0: mono: mono_out=0x0 [ 37.127618] sound hdaudioC0D0: dig-out=0x10/0x15 [ 37.127620] sound hdaudioC0D0: inputs: [ 37.127622] sound hdaudioC0D0: Mic=0xd [ 37.127624] sound hdaudioC0D0: Line=0xc [ 37.127625] sound hdaudioC0D0: dig-in=0x12 and # cat /proc/asound/cards 0 [Generic ]: HDA-Intel - HD-Audio Generic HD-Audio Generic at 0xff760000 irq 39 What's the meaning of this error? Regards, Jorge

1 0

[alsa-devel] [RFC PATCH 0/2] ASoC: wm8994: master clock issues
by olivier moysan 20 Mar '17

20 Mar '17

Hello, I face two issues using wm8994 codec driver. Below is a description of the use cases rising the problem. I added the patches which allow to fix these issues. These minimalist patches can be seen as a starting point for a discussion on the best way to handle the use cases exposed. - ASoC: wm8994: default master clock selection Use case: CPU DAI and codec are managed through simple card. Wolson codec wm8994 is set as slave of CPU DAI and CPU DAI feeds codec with master clock. Master clock is derived from mclk-fs property and provided to CPU DAI and codec through snd_soc_dai_set_sysclk API. Analysis: The simple card assumes id 0 for master clock. However wm8994 codec exposes master clock ids from 1 to 4. (MCLK1, MCLK2 ..) The proposal in this patch is to define a default selection regarding master clock when id 0 is provided. - ASoC: wm8994: delay aifxclk activation Use case: Wolson codec wm8994 is set as slave of CPU DAI and CPU DAI feeds codec with master clock. CPU DAI generates master clock when enabled in trigger callback. So master clock is available only at runtime. Master clock is not available when configuring codec through alsa controls. Analysis: When we configure aifx clock while master clock is not active, related registers are not updated in wm8894 codec. A way to go around this problem, is to delay aifxclk activation until SND_SOC_DAPM_POST_PMU DAPM stage. The purpose of patch here is to illustrate this workaround. What I cannot clearly figure out, is how this may impact other uses of wm8894 codec. I try to know if it's worth investigating further in this direction or if it will not be applicable. If not applicable, what maybe the other options ? regards olivier olivier moysan (2): ASoC: wm8994: default master clock selection ASoC: wm8994: delay aifxclk activation sound/soc/codecs/wm8994.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) -- 1.9.1

3 5

[alsa-devel] [PATCH 0/2] conf/ucm: Remove intel UCM conf files from alsa-lib repo
by fuweix.tang＠intel.com 20 Mar '17

20 Mar '17

From: Fuwei Tang <fuweix.tang(a)intel.com> Remove the intel UCM configs from alsa-lib repo. Another series will add them to a dedicated UCM conf repo.License can be changed to BSD for intel files,and left as "unknown" for other conf files when they are moved from alsa-lib to the new repo.The alsa-lib LGPL license does not cover runtime configuration files. Fuwei Tang (2): conf/ucm: Remove conf files of broxton-rt298 platform conf/ucm: Remove conf files of skylake-rt286 platfrom configure.ac | 2 - src/conf/ucm/Makefile.am | 2 +- src/conf/ucm/broxton-rt298/Hdmi1 | 22 ----- src/conf/ucm/broxton-rt298/Hdmi2 | 14 --- src/conf/ucm/broxton-rt298/HiFi | 129 -------------------------- src/conf/ucm/broxton-rt298/Makefile.am | 4 - src/conf/ucm/broxton-rt298/broxton-rt298.conf | 58 ------------ src/conf/ucm/skylake-rt286/Hdmi1 | 20 ---- src/conf/ucm/skylake-rt286/Hdmi2 | 20 ---- src/conf/ucm/skylake-rt286/HiFi | 129 -------------------------- src/conf/ucm/skylake-rt286/Makefile.am | 4 - src/conf/ucm/skylake-rt286/skylake-rt286.conf | 60 ------------ 12 files changed, 1 insertion(+), 463 deletions(-) delete mode 100644 src/conf/ucm/broxton-rt298/Hdmi1 delete mode 100644 src/conf/ucm/broxton-rt298/Hdmi2 delete mode 100644 src/conf/ucm/broxton-rt298/HiFi delete mode 100644 src/conf/ucm/broxton-rt298/Makefile.am delete mode 100644 src/conf/ucm/broxton-rt298/broxton-rt298.conf delete mode 100644 src/conf/ucm/skylake-rt286/Hdmi1 delete mode 100644 src/conf/ucm/skylake-rt286/Hdmi2 delete mode 100644 src/conf/ucm/skylake-rt286/HiFi delete mode 100644 src/conf/ucm/skylake-rt286/Makefile.am delete mode 100644 src/conf/ucm/skylake-rt286/skylake-rt286.conf -- 2.7.4

3 4

[alsa-devel] [BUG] src/conf/alsa.conf: Hard-coded path `/etc/asound.conf`, needs `@func sysconfdir`.
by Alain Kalker 20 Mar '17

20 Mar '17

The file `src/conf/alsa.conf` contains the hard-coded path `/etc/asound.conf`, which in some use cases may not be correct. One particular use case: Termux[1], a "Linux subsystem for Android", plans to use ALSA plugins to communicate with the OpenSL ES implementation in Android. In doing so, it needs to not conflict with any vendor provided ALSA implentation, which could conceivably use an `/etc/asound.conf` of its own. At any rate, the system `/etc` directory is normally not writable by non-root users of Android. I would suggest adding a `snd_func_sysconfdir` in `src/confmisc.c (analogous to `snd_func_datadir`) so packagers and users can place a global `asound.conf` file in a directory to which they have write access and which doesn't conflict with the systems ALSA. The desired path can already be specified at build time by using `./configure --sysconfdir=<dir>`, which conveniently defaults to PREFIX/etc in any case. It is available as configure variable: `sysconfdir='${prefix}/etc'`, and only needs to be substituted to be available to `src/confmisc.c`. Kind regards, Alain Kalker [1]: https://termux.com/

2 1

Re: [alsa-devel] [PATCH] ALSA: hda/ca0132: Remove double parentheses
by Takashi Iwai 20 Mar '17

20 Mar '17

On Wed, 15 Mar 2017 23:41:23 +0100, Matthias Kaehlcke wrote: > > The extra pairs of parantheses are not needed and causes clang to > generate warnings like this: > > sound/pci/hda/patch_ca0132.c:1171:14: error: equality comparison with extraneous parentheses [-Werror,-Wparentheses-equality] > if ((buffer == NULL)) > ~~~~~~~^~~~~~~ > sound/pci/hda/patch_ca0132.c:1171:14: note: remove extraneous parentheses around the comparison to silence this warning > if ((buffer == NULL)) > ~ ^ ~ > sound/pci/hda/patch_ca0132.c:1171:14: note: use '=' to turn this equality comparison into an assignment > if ((buffer == NULL)) > > Signed-off-by: Matthias Kaehlcke <mka(a)chromium.org> Applied, thanks. Takashi

1 0

[alsa-devel] [PATCH 1/1] ALSA: es1688: Use strcpy() instead of sprintf()
by Nicolas Iooss 20 Mar '17

20 Mar '17

There is no point in using sprintf() without a format string when strcpy() can perform the same operation. Signed-off-by: Nicolas Iooss <nicolas.iooss_linux(a)m4x.org> --- sound/isa/es1688/es1688_lib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/isa/es1688/es1688_lib.c b/sound/isa/es1688/es1688_lib.c index e2cf508841b1..81cf26fa28d6 100644 --- a/sound/isa/es1688/es1688_lib.c +++ b/sound/isa/es1688/es1688_lib.c @@ -742,7 +742,7 @@ int snd_es1688_pcm(struct snd_card *card, struct snd_es1688 *chip, int device) pcm->private_data = chip; pcm->info_flags = SNDRV_PCM_INFO_HALF_DUPLEX; - sprintf(pcm->name, snd_es1688_chip_id(chip)); + strcpy(pcm->name, snd_es1688_chip_id(chip)); chip->pcm = pcm; snd_pcm_lib_preallocate_pages_for_all(pcm, SNDRV_DMA_TYPE_DEV, -- 2.12.0

2 1

[alsa-devel] [PATCH] ALSA: hda - add support for docking station for HP 840 G3
by Jaroslav Kysela 20 Mar '17

20 Mar '17

This tested patch adds missing initialization for Line-In/Out PINs for the docking station for HP 840 G3. Signed-off-by: Jaroslav Kysela <perex(a)perex.cz> --- sound/pci/hda/patch_conexant.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/sound/pci/hda/patch_conexant.c b/sound/pci/hda/patch_conexant.c index c15c51b..69266b8 100644 --- a/sound/pci/hda/patch_conexant.c +++ b/sound/pci/hda/patch_conexant.c @@ -261,6 +261,7 @@ enum { CXT_FIXUP_HP_530, CXT_FIXUP_CAP_MIX_AMP_5047, CXT_FIXUP_MUTE_LED_EAPD, + CXT_FIXUP_HP_DOCK, CXT_FIXUP_HP_SPECTRE, CXT_FIXUP_HP_GATE_MIC, }; @@ -778,6 +779,14 @@ static const struct hda_fixup cxt_fixups[] = { .type = HDA_FIXUP_FUNC, .v.func = cxt_fixup_mute_led_eapd, }, + [CXT_FIXUP_HP_DOCK] = { + .type = HDA_FIXUP_PINS, + .v.pins = (const struct hda_pintbl[]) { + { 0x16, 0x21011020 }, /* line-out */ + { 0x18, 0x2181103f }, /* line-in */ + { } + } + }, [CXT_FIXUP_HP_SPECTRE] = { .type = HDA_FIXUP_PINS, .v.pins = (const struct hda_pintbl[]) { @@ -839,6 +848,7 @@ static const struct snd_pci_quirk cxt5066_fixups[] = { SND_PCI_QUIRK(0x1025, 0x0543, "Acer Aspire One 522", CXT_FIXUP_STEREO_DMIC), SND_PCI_QUIRK(0x1025, 0x054c, "Acer Aspire 3830TG", CXT_FIXUP_ASPIRE_DMIC), SND_PCI_QUIRK(0x1025, 0x054f, "Acer Aspire 4830T", CXT_FIXUP_ASPIRE_DMIC), + SND_PCI_QUIRK(0x103c, 0x8079, "HP EliteBook 840 G3", CXT_FIXUP_HP_DOCK), SND_PCI_QUIRK(0x103c, 0x8174, "HP Spectre x360", CXT_FIXUP_HP_SPECTRE), SND_PCI_QUIRK(0x103c, 0x8115, "HP Z1 Gen3", CXT_FIXUP_HP_GATE_MIC), SND_PCI_QUIRK(0x1043, 0x138d, "Asus", CXT_FIXUP_HEADPHONE_MIC_PIN), @@ -871,6 +881,7 @@ static const struct hda_model_fixup cxt5066_fixup_models[] = { { .id = CXT_PINCFG_LEMOTE_A1205, .name = "lemote-a1205" }, { .id = CXT_FIXUP_OLPC_XO, .name = "olpc-xo" }, { .id = CXT_FIXUP_MUTE_LED_EAPD, .name = "mute-led-eapd" }, + { .id = CXT_FIXUP_HP_DOCK, .name = "hp-dock" }, {} }; -- 2.9.3

2 1

[alsa-devel] [PATCH] ALSA: hda - add support for docking station for HP 820 G2
by Jaroslav Kysela 20 Mar '17

20 Mar '17

This tested patch adds missing initialization for Line-In/Out PINs for the docking station for HP 820 G2. Signed-off-by: Jaroslav Kysela <perex(a)perex.cz> --- sound/pci/hda/patch_realtek.c | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 73a0046..147bddb 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -4847,6 +4847,7 @@ enum { ALC286_FIXUP_HP_GPIO_LED, ALC280_FIXUP_HP_GPIO2_MIC_HOTKEY, ALC280_FIXUP_HP_DOCK_PINS, + ALC269_FIXUP_HP_DOCK_GPIO_MIC1_LED, ALC280_FIXUP_HP_9480M, ALC288_FIXUP_DELL_HEADSET_MODE, ALC288_FIXUP_DELL1_MIC_NO_PRESENCE, @@ -5388,6 +5389,16 @@ static const struct hda_fixup alc269_fixups[] = { .chained = true, .chain_id = ALC280_FIXUP_HP_GPIO4 }, + [ALC269_FIXUP_HP_DOCK_GPIO_MIC1_LED] = { + .type = HDA_FIXUP_PINS, + .v.pins = (const struct hda_pintbl[]) { + { 0x1b, 0x21011020 }, /* line-out */ + { 0x18, 0x2181103f }, /* line-in */ + { }, + }, + .chained = true, + .chain_id = ALC269_FIXUP_HP_GPIO_MIC1_LED + }, [ALC280_FIXUP_HP_9480M] = { .type = HDA_FIXUP_FUNC, .v.func = alc280_fixup_hp_9480m, @@ -5646,7 +5657,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x103c, 0x2256, "HP", ALC269_FIXUP_HP_GPIO_MIC1_LED), SND_PCI_QUIRK(0x103c, 0x2257, "HP", ALC269_FIXUP_HP_GPIO_MIC1_LED), SND_PCI_QUIRK(0x103c, 0x2259, "HP", ALC269_FIXUP_HP_GPIO_MIC1_LED), - SND_PCI_QUIRK(0x103c, 0x225a, "HP", ALC269_FIXUP_HP_GPIO_MIC1_LED), + SND_PCI_QUIRK(0x103c, 0x225a, "HP", ALC269_FIXUP_HP_DOCK_GPIO_MIC1_LED), SND_PCI_QUIRK(0x103c, 0x2260, "HP", ALC269_FIXUP_HP_MUTE_LED_MIC1), SND_PCI_QUIRK(0x103c, 0x2263, "HP", ALC269_FIXUP_HP_MUTE_LED_MIC1), SND_PCI_QUIRK(0x103c, 0x2264, "HP", ALC269_FIXUP_HP_MUTE_LED_MIC1), @@ -5814,6 +5825,7 @@ static const struct hda_model_fixup alc269_fixup_models[] = { {.id = ALC269_FIXUP_HEADSET_MODE_NO_HP_MIC, .name = "headset-mode-no-hp-mic"}, {.id = ALC269_FIXUP_LENOVO_DOCK, .name = "lenovo-dock"}, {.id = ALC269_FIXUP_HP_GPIO_LED, .name = "hp-gpio-led"}, + {.id = ALC269_FIXUP_HP_DOCK_GPIO_MIC1_LED, .name = "hp-dock-gpio-mic1-led"}, {.id = ALC269_FIXUP_DELL1_MIC_NO_PRESENCE, .name = "dell-headset-multi"}, {.id = ALC269_FIXUP_DELL2_MIC_NO_PRESENCE, .name = "dell-headset-dock"}, {.id = ALC283_FIXUP_CHROME_BOOK, .name = "alc283-dac-wcaps"}, -- 2.9.3

2 1