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

UCM fails as CardDriver may not be defined
by GitHub issues - opened 18 Dec '20

18 Dec '20

alsa-project/alsa-ucm-conf issue #69 was opened from coolstar: UCM doesn't load as CardDriver isn't set on some platforms Changing https://github.com/alsa-project/alsa-ucm-conf/blob/master/ucm2/ucm.conf#L69 to `Directory "${CardLongName}"` resolves the issue ``` I: [pulseaudio] source.c: Created source 1 "alsa_input.platform-sound.analog-stereo" with sample spec s16le 2ch 44100Hz and channel map front-left,front-right I: [pulseaudio] source.c: alsa.resolution_bits = "16" I: [pulseaudio] source.c: device.api = "alsa" I: [pulseaudio] source.c: device.class = "sound" I: [pulseaudio] source.c: alsa.class = "generic" I: [pulseaudio] source.c: alsa.subclass = "generic-mix" I: [pulseaudio] source.c: alsa.name = "" I: [pulseaudio] source.c: alsa.id = "ADMAIF1 CIF ADMAIF1-0" I: [pulseaudio] source.c: alsa.subdevice = "0" I: [pulseaudio] source.c: alsa.subdevice_name = "subdevice #0" I: [pulseaudio] source.c: alsa.device = "0" I: [pulseaudio] source.c: alsa.card = "1" I: [pulseaudio] source.c: alsa.card_name = "tegra-snd-t210ref-mobile-rt565x" I: [pulseaudio] source.c: alsa.long_card_name = "tegra-snd-t210ref-mobile-rt565x" I: [pulseaudio] source.c: device.bus_path = "platform-sound" I: [pulseaudio] source.c: sysfs.path = "/devices/sound/sound/card1" I: [pulseaudio] source.c: device.form_factor = "internal" I: [pulseaudio] source.c: device.string = "front:1" I: [pulseaudio] source.c: device.buffering.buffer_size = "32768" I: [pulseaudio] source.c: device.buffering.fragment_size = "16384" I: [pulseaudio] source.c: device.access_mode = "mmap+timer" I: [pulseaudio] source.c: device.profile.name = "analog-stereo" I: [pulseaudio] source.c: device.profile.description = "Analog Stereo" I: [pulseaudio] source.c: device.description = "Built-in Audio Analog Stereo" I: [pulseaudio] source.c: module-udev-detect.discovered = "1" I: [pulseaudio] source.c: device.icon_name = "audio-card" ``` Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/69 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[PATCH v1 ] ALSA: core: memalloc: add page alignment for iram
by Robin Gong 17 Dec '20

17 Dec '20

Since mmap for userspace is based on page alignment, add page alignment for iram alloc from pool, otherwise, some good data located in the same page of dmab->area maybe touched wrongly by userspace like pulseaudio. Signed-off-by: Robin Gong <yibin.gong(a)nxp.com> --- sound/core/memalloc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/core/memalloc.c b/sound/core/memalloc.c index 0aeeb62..0f33516 100644 --- a/sound/core/memalloc.c +++ b/sound/core/memalloc.c @@ -77,7 +77,8 @@ static void snd_malloc_dev_iram(struct snd_dma_buffer *dmab, size_t size) /* Assign the pool into private_data field */ dmab->private_data = pool; - dmab->area = gen_pool_dma_alloc(pool, size, &dmab->addr); + dmab->area = gen_pool_dma_alloc_align(pool, size, &dmab->addr, + PAGE_SIZE); } /** -- 2.7.4

3 13

wallclock time for midi events without audio port
by Michael Kirchner 17 Dec '20

17 Dec '20

Hi, i try to get an absolute wallclock time for midi events using event->time.time.tv_sec. As I understand I have to add trigger time to get wall clock time. Therefor I use snd_pcm_status_get_trigger_htstamp(). If I use alsa audio via snd_pcm_open there is no problem. But: how can I get trigger time if I use only a midi port without audio? (usecase is: soft synth with alsa midi in -> jack audio out) Sincerely, Michael

1 0

utils/alsa.m4: update for newer autoconf.
by GitHub pull_request - opened 17 Dec '20

17 Dec '20

alsa-project/alsa-lib pull request #106 was opened from sezero: fixes autoconf-2.70 warnings. should be compatible down to autoconf-2.59. Request URL : https://github.com/alsa-project/alsa-lib/pull/106 Patch URL : https://github.com/alsa-project/alsa-lib/pull/106.patch Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[PATCH alsa-lib 1/2] pcm: Add snd_pcm_audio_tstamp_type_t constants
by Takashi Iwai 17 Dec '20

17 Dec '20

From: David Henningsson <diwic(a)ubuntu.com> These are mostly a copy-paste from the kernel headers. But since functions snd_pcm_audio_tstamp_config make use of these they should be added to the public API as well. Reported-by: Alex Moon <alex.r.moon(a)gmail.com> Signed-off-by: David Henningsson <diwic(a)ubuntu.com> Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- include/pcm.h | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/include/pcm.h b/include/pcm.h index 5b0782315585..cf1eea8bbc9b 100644 --- a/include/pcm.h +++ b/include/pcm.h @@ -350,6 +350,20 @@ typedef enum _snd_pcm_tstamp_type { SND_PCM_TSTAMP_TYPE_LAST = SND_PCM_TSTAMP_TYPE_MONOTONIC_RAW, } snd_pcm_tstamp_type_t; +typedef enum _snd_pcm_audio_tstamp_type { + /** + * first definition for backwards compatibility only, + * maps to wallclock/link time for HDAudio playback and DEFAULT/DMA time for everything else + */ + SND_PCM_AUDIO_TSTAMP_TYPE_COMPAT = 0, + SND_PCM_AUDIO_TSTAMP_TYPE_DEFAULT = 1, /**< DMA time, reported as per hw_ptr */ + SND_PCM_AUDIO_TSTAMP_TYPE_LINK = 2, /**< link time reported by sample or wallclock counter, reset on startup */ + SND_PCM_AUDIO_TSTAMP_TYPE_LINK_ABSOLUTE = 3, /**< link time reported by sample or wallclock counter, not reset on startup */ + SND_PCM_AUDIO_TSTAMP_TYPE_LINK_ESTIMATED = 4, /**< link time estimated indirectly */ + SND_PCM_AUDIO_TSTAMP_TYPE_LINK_SYNCHRONIZED = 5, /**< link time synchronized with system time */ + SND_PCM_AUDIO_TSTAMP_TYPE_LAST = SNDRV_PCM_AUDIO_TSTAMP_TYPE_LINK_SYNCHRONIZED +} snd_pcm_audio_tstamp_type_t; + typedef struct _snd_pcm_audio_tstamp_config { /* 5 of max 16 bits used */ unsigned int type_requested:4; -- 2.26.2

1 1

Re: [PATCH 01/01] Add VID to support native DSD reproduction on FiiO devices.
by Amadej Kastelic 17 Dec '20

17 Dec '20

Message-Id: <c759bc50134ea434bfed0a183e0ce72984f5b609.camel(a)gmail.com> Add VID to support native DSD reproduction on FiiO devices. Tested-by: Amadej Kastelic <amadejkastelic7(a)gmail.com> Signed-off-by: Emilio Moretti <emilio.moretti(a)gmail.com> --- sound/usb/quirks.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c index 349e1e52996d..f5fc65aef628 100644 --- a/sound/usb/quirks.c +++ b/sound/usb/quirks.c @@ -1658,6 +1658,7 @@ u64 snd_usb_interface_dsd_format_quirks(struct snd_usb_audio *chip, case 0x25ce: /* Mytek devices */ case 0x278b: /* Rotel? */ case 0x292b: /* Gustard/Ess based devices */ + case 0x2972: /* FiiO devices */ case 0x2ab6: /* T+A devices */ case 0x3842: /* EVGA */ case 0xc502: /* HiBy devices */ -- 2.20.1

2 3

Dell fixed type headset platform
by Kailang 17 Dec '20

17 Dec '20

Hi Takashi, Dell has fixed type headset platform for desktop. Attach patch was add pin verb. BIOS verb table was setting for fixed type coef. Many Thanks. BR, Kailang

2 1

[PATCH v4 0/2] Platform driver update to support playback recover after resume
by Srinivasa Rao Mandadapu 17 Dec '20

17 Dec '20

This patch set is to add support for playback recover after hard suspend and resume. It includes: 1. Reverting part of previous commit, which is for handling registers invalid state after hard suspend. 2. Adding pm ops in component driver and do regcache sync. Srinivasa Rao Mandadapu (2): Partially revert ASoC: qcom: Fix enabling BCLK and LRCLK in LPAIF invalid state ASoC: qcom: Add support for playback recover after resume sound/soc/qcom/lpass-cpu.c | 20 ++---------------- sound/soc/qcom/lpass-platform.c | 46 +++++++++++++++++++++++++++++++---------- 2 files changed, 37 insertions(+), 29 deletions(-) -- Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc., is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.

3 11

rsnd: clk-multiplier already disabled warning on Ebisu
by Geert Uytterhoeven 17 Dec '20

17 Dec '20

Hi Morimoto-san, While booting Ebisu-4D, I saw the below warnings (once). rcar_sound ec500000.sound: can't use clk 1 rsnd_adg_clk_control() failed to enable CLKB, but continues. ------------[ cut here ]------------ clk-multiplier already disabled WARNING: CPU: 1 PID: 52 at drivers/clk/clk.c:952 clk_core_disable+0x30/0x9c Modules linked in: CPU: 1 PID: 52 Comm: kworker/1:2 Not tainted 5.10.0-rc7-rcar3-initrd-00582-gadecf297acf4-dirty #183 Hardware name: Renesas Ebisu-4D board based on r8a77990 (DT) Workqueue: events deferred_probe_work_func pstate: 60000085 (nZCv daIf -PAN -UAO -TCO BTYPE=--) pc : clk_core_disable+0x30/0x9c lr : clk_core_disable+0x30/0x9c sp : ffffffc010f939f0 x29: ffffffc010f939f0 x28: 0000000000000000 x27: ffffff800949eb00 x26: ffffff80093c40f8 x25: 0000000000000001 x24: ffffffc010805fea x23: 0000000000000000 x22: ffffff80082b8810 x21: ffffff80093c4080 x20: ffffff8009368b00 x19: ffffff8009368b00 x18: 0000000000000000 x17: 000000004ef774c4 x16: 0000000000000014 x15: ffffffffffffffff x14: ffffffc010b8e8b0 x13: fffffffffff9fd47 x12: 0720072007200720 x11: fffffffffffc0000 x10: ffffffc010b8e8d8 x9 : 0720072007200720 x8 : 2079646165726c61 x7 : 0000000000000004 x6 : 00000000ffffe6ad x5 : ffffffc010f93718 x4 : 0000000000000000 x3 : 0000000000000000 x2 : 00000000ffffffff x1 : 0000000000000000 x0 : 0000000000000000 Call trace: clk_core_disable+0x30/0x9c clk_core_disable_lock+0x24/0x3c clk_disable+0x20/0x30 rsnd_adg_clk_control+0xa0/0xd4 rsnd_adg_remove+0x58/0x6c rsnd_probe+0x134/0x3cc Something else fails (-EPROBE_DEFER?), and thus rsnd_adg_remove() is called. The latter disables all clocks, including the one that failed to enable before, leading to the warning. platform_drv_probe+0x54/0xa4 really_probe+0x234/0x340 driver_probe_device+0xa0/0xb0 __device_attach_driver+0x9c/0xa8 bus_for_each_drv+0x9c/0xc4 __device_attach+0xd4/0x140 device_initial_probe+0x14/0x20 bus_probe_device+0x30/0x94 deferred_probe_work_func+0x9c/0xb0 process_one_work+0x180/0x1f0 process_scheduled_works+0x44/0x48 worker_thread+0x1ec/0x270 kthread+0xdc/0xec ret_from_fork+0x10/0x30 ---[ end trace 8d8c2a1b7537ca36 ]--- ------------[ cut here ]------------ clk-multiplier already unprepared WARNING: CPU: 1 PID: 52 at drivers/clk/clk.c:810 clk_core_unprepare+0x30/0xd0 [...] ---[ end trace 8d8c2a1b7537ca37 ]--- Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert(a)linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds

3 5

[PATCH v2] ASoC: AMD Renoir - add DMI table to avoid the ACP mic probe (broken BIOS)
by Jaroslav Kysela 17 Dec '20

17 Dec '20

Users reported that some Lenovo AMD platforms do not have ACP microphone, but the BIOS advertises it via ACPI. This patch create a simple DMI table, where those machines with the broken BIOS can be added. The DMI description for Lenovo IdeaPad 5 and IdeaPad Flex 5 devices are added there. Also describe the dmic_acpi_check kernel module parameter in a more understandable way. BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1892115 Cc: <stable(a)kernel.org> Cc: Vijendar Mukunda <Vijendar.Mukunda(a)amd.com> Cc: Mark Brown <broonie(a)kernel.org> Signed-off-by: Jaroslav Kysela <perex(a)perex.cz> --- sound/soc/amd/renoir/rn-pci-acp3x.c | 28 +++++++++++++++++++++++----- 1 file changed, 23 insertions(+), 5 deletions(-) diff --git a/sound/soc/amd/renoir/rn-pci-acp3x.c b/sound/soc/amd/renoir/rn-pci-acp3x.c index b943e59fc302..60a3e2f442db 100644 --- a/sound/soc/amd/renoir/rn-pci-acp3x.c +++ b/sound/soc/amd/renoir/rn-pci-acp3x.c @@ -6,6 +6,7 @@ #include <linux/pci.h> #include <linux/acpi.h> +#include <linux/dmi.h> #include <linux/module.h> #include <linux/io.h> #include <linux/delay.h> @@ -20,14 +21,13 @@ module_param(acp_power_gating, int, 0644); MODULE_PARM_DESC(acp_power_gating, "Enable acp power gating"); /** - * dmic_acpi_check = -1 - Checks ACPI method to know DMIC hardware status runtime - * = 0 - Skips the DMIC device creation and returns probe failure - * = 1 - Assumes that platform has DMIC support and skips ACPI - * method check + * dmic_acpi_check = -1 - Use ACPI/DMI method to detect the DMIC hardware presence at runtime + * = 0 - Skip the DMIC device creation and return probe failure + * = 1 - Force DMIC support */ static int dmic_acpi_check = ACP_DMIC_AUTO; module_param(dmic_acpi_check, bint, 0644); -MODULE_PARM_DESC(dmic_acpi_check, "checks Dmic hardware runtime"); +MODULE_PARM_DESC(dmic_acpi_check, "Digital microphone presence (-1=auto, 0=none, 1=force)"); struct acp_dev_data { void __iomem *acp_base; @@ -163,6 +163,17 @@ static int rn_acp_deinit(void __iomem *acp_base) return 0; } +static const struct dmi_system_id rn_acp_quirk_table[] = { + { + /* Lenovo IdeaPad Flex 5 14ARE05, IdeaPad 5 15ARE05 */ + .matches = { + DMI_EXACT_MATCH(DMI_BOARD_VENDOR, "LENOVO"), + DMI_EXACT_MATCH(DMI_BOARD_NAME, "LNVNB161216"), + } + }, + {} +}; + static int snd_rn_acp_probe(struct pci_dev *pci, const struct pci_device_id *pci_id) { @@ -172,6 +183,7 @@ static int snd_rn_acp_probe(struct pci_dev *pci, acpi_handle handle; acpi_integer dmic_status; #endif + const struct dmi_system_id *dmi_id; unsigned int irqflags; int ret, index; u32 addr; @@ -232,6 +244,12 @@ static int snd_rn_acp_probe(struct pci_dev *pci, goto de_init; } #endif + dmi_id = dmi_first_match(rn_acp_quirk_table); + if (dmi_id && !dmi_id->driver_data) { + dev_info(&pci->dev, "ACPI settings override using DMI (ACP mic is not present)"); + ret = -ENODEV; + goto de_init; + } } adata->res = devm_kzalloc(&pci->dev, -- 2.26.2

4 3