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

[alsa-devel] Is period_size bytes per period or frames per period?
by GitHub issues - edited 12 Aug '19

12 Aug '19

alsa-project/alsa-lib issue #8 was edited from crocket: According to https://www.alsa-project.org/alsa-doc/alsa-lib/pcm_plugins.html period_size in dmix is bytes per period. According to aplay --dump-hw-params -D dmix ... period_size is frames per period. Which one is right? Issue URL : https://github.com/alsa-project/alsa-lib/issues/8 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[alsa-devel] Is period_size bytes per period or frames per period?
by GitHub issues - opened 12 Aug '19

12 Aug '19

alsa-project/alsa-lib issue #8 was opened from crocket: According to https://www.alsa-project.org/alsa-doc/alsa-lib/pcm_plugins.html period_size in dmix is bytes per period. According to aplay --dump-hw-params period_size is frames per period. Which one is right? Issue URL : https://github.com/alsa-project/alsa-lib/issues/8 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[alsa-devel] [PATCH v4 0/9]ASoC: sun4i-i2s: Updates to the driver
by codekipper＠gmail.com 12 Aug '19

12 Aug '19

From: Marcus Cooper <codekipper(a)gmail.com> Hi All, here is a patch series to add some improvements to the sun4i-i2s driver found whilst getting slave clocking and hdmi audio working on the newer SoCs. As the LibreELEC project is progressing extremely well then there has been some activity getting surround sound working and this is included. The functionality included with the new patch set has been extended to cover more sample resolutions, multi-lane data output for HDMI audio and some bug fixes that have been discovered along the way. I can see more usage of the tdm property since I last attempted to push these patches and the examples currently in mainline sort of the opposite to what I'm trying to achieve. When we first started looking at the i2s driver, the codecs that we were using allowed for the frame width to be determined based on the sampling resolution but in most use cases it seems that a fixed width is required(my highest priority should be to get HDMI audio support in). We're using the tdm property to override the old way to calculate the frame width. What I've seen in what has already been mainlined is that the i2s driver has a frame width that is fixed to 32 bits and this can be overridden using the tdm property. I still need to investigate the FIFO syncing issues which i've not had a chance to change or address the concerns that broonie and wens brought up. This change has been moved to the top of the patch stack. BR, CK --- v4 changes compared to v3 are: - Moved patches around so that the more controversial of patches are at the top of the stack. - Added more details to commit messages. - Fixed 20bit audio PCM format to use 4 bytes. - Reduced number of flags used to indicate a new SoC. v3 changes compared to v2 are: - added back slave mode changes - added back the use of tdm properties - changes to regmap and caching - removed loopback functionality - fixes to the channel offset mask v2 changes compared to v1 are: - removed slave mode changes which didn't set mclk and bclk div. - removed use of tdm and now use a dedicated property. - fix commit message to better explain reason for sign extending - add divider calculations for newer SoCs. - add support for multi-lane i2s data output. - add support for 20, 24 and 32 bit samples. - add loopback property so blocks can be tested without a codec. Marcus Cooper (9): ASoC: sun4i-i2s: Fix sun8i tx channel offset mask ASoC: sun4i-i2s: Add offset to RX channel select ASoC: sun4i-i2s: Add regmap field to sign extend sample ASoC: sun4i-i2s: Reduce quirks for sun8i-h3 ASoC: sun4i-i2s: Add set_tdm_slot functionality ASoC: sun4i-i2s: Add multi-lane functionality ASoC: sun4i-i2s: Add multichannel functionality ASoc: sun4i-i2s: Add 20, 24 and 32 bit support ASoC: sun4i-i2s: Adjust regmap settings sound/soc/sunxi/sun4i-i2s.c | 242 ++++++++++++++++++++++++------------ 1 file changed, 164 insertions(+), 78 deletions(-) -- 2.21.0

7 33

[alsa-devel] [asoc:for-5.4 175/205] include/linux/dynamic_debug.h:82:16: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t'
by kbuild test robot 11 Aug '19

11 Aug '19

tree: https://kernel.googlesource.com/pub/scm/linux/kernel/git/broonie/sound.git for-5.4 head: 83ee7ab1627b7597bbb9acc26b37e9f96fff40c9 commit: abf31feea26c0f412a191c83f408311a0de7435c [175/205] ASoC: Intel: Update request-reply IPC model config: i386-randconfig-a004-201932 (attached as .config) compiler: gcc-4.9 (Debian 4.9.2-10+deb8u1) 4.9.2 reproduce: git checkout abf31feea26c0f412a191c83f408311a0de7435c # save the attached .config to linux build tree make ARCH=i386 If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp(a)intel.com> All warnings (new ones prefixed by >>): In file included from include/linux/printk.h:332:0, from include/linux/kernel.h:15, from sound/soc/intel/haswell/sst-haswell-ipc.c:9: sound/soc/intel/haswell/sst-haswell-ipc.c: In function 'sst_hsw_module_set_param': >> include/linux/dynamic_debug.h:82:16: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' [-Wformat=] static struct _ddebug __aligned(8) \ ^ include/linux/dynamic_debug.h:123:2: note: in expansion of macro 'DEFINE_DYNAMIC_DEBUG_METADATA' DEFINE_DYNAMIC_DEBUG_METADATA(id, fmt); \ ^ include/linux/dynamic_debug.h:143:2: note: in expansion of macro '__dynamic_func_call' __dynamic_func_call(__UNIQUE_ID(ddebug), fmt, func, ##__VA_ARGS__) ^ include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call' _dynamic_func_call(fmt,__dynamic_dev_dbg, \ ^ include/linux/device.h:1503:2: note: in expansion of macro 'dynamic_dev_dbg' dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__) ^ sound/soc/intel/haswell/sst-haswell-ipc.c:2018:3: note: in expansion of macro 'dev_dbg' dev_dbg(dev, "transfer parameter size : %lu\n", ^ >> include/linux/dynamic_debug.h:82:16: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' [-Wformat=] static struct _ddebug __aligned(8) \ ^ include/linux/dynamic_debug.h:123:2: note: in expansion of macro 'DEFINE_DYNAMIC_DEBUG_METADATA' DEFINE_DYNAMIC_DEBUG_METADATA(id, fmt); \ ^ include/linux/dynamic_debug.h:143:2: note: in expansion of macro '__dynamic_func_call' __dynamic_func_call(__UNIQUE_ID(ddebug), fmt, func, ##__VA_ARGS__) ^ include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call' _dynamic_func_call(fmt,__dynamic_dev_dbg, \ ^ include/linux/device.h:1503:2: note: in expansion of macro 'dynamic_dev_dbg' dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__) ^ sound/soc/intel/haswell/sst-haswell-ipc.c:2022:3: note: in expansion of macro 'dev_dbg' dev_dbg(dev, "transfer parameter aligned size : %lu\n", ^ -- In file included from include/linux/printk.h:332:0, from include/linux/kernel.h:15, from sound/soc//intel/haswell/sst-haswell-ipc.c:9: sound/soc//intel/haswell/sst-haswell-ipc.c: In function 'sst_hsw_module_set_param': >> include/linux/dynamic_debug.h:82:16: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' [-Wformat=] static struct _ddebug __aligned(8) \ ^ include/linux/dynamic_debug.h:123:2: note: in expansion of macro 'DEFINE_DYNAMIC_DEBUG_METADATA' DEFINE_DYNAMIC_DEBUG_METADATA(id, fmt); \ ^ include/linux/dynamic_debug.h:143:2: note: in expansion of macro '__dynamic_func_call' __dynamic_func_call(__UNIQUE_ID(ddebug), fmt, func, ##__VA_ARGS__) ^ include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call' _dynamic_func_call(fmt,__dynamic_dev_dbg, \ ^ include/linux/device.h:1503:2: note: in expansion of macro 'dynamic_dev_dbg' dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__) ^ sound/soc//intel/haswell/sst-haswell-ipc.c:2018:3: note: in expansion of macro 'dev_dbg' dev_dbg(dev, "transfer parameter size : %lu\n", ^ >> include/linux/dynamic_debug.h:82:16: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' [-Wformat=] static struct _ddebug __aligned(8) \ ^ include/linux/dynamic_debug.h:123:2: note: in expansion of macro 'DEFINE_DYNAMIC_DEBUG_METADATA' DEFINE_DYNAMIC_DEBUG_METADATA(id, fmt); \ ^ include/linux/dynamic_debug.h:143:2: note: in expansion of macro '__dynamic_func_call' __dynamic_func_call(__UNIQUE_ID(ddebug), fmt, func, ##__VA_ARGS__) ^ include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call' _dynamic_func_call(fmt,__dynamic_dev_dbg, \ ^ include/linux/device.h:1503:2: note: in expansion of macro 'dynamic_dev_dbg' dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__) ^ sound/soc//intel/haswell/sst-haswell-ipc.c:2022:3: note: in expansion of macro 'dev_dbg' dev_dbg(dev, "transfer parameter aligned size : %lu\n", ^ vim +82 include/linux/dynamic_debug.h 923abb9d797ba0 Gal Pressman 2019-05-01 75 923abb9d797ba0 Gal Pressman 2019-05-01 76 extern __printf(3, 4) 923abb9d797ba0 Gal Pressman 2019-05-01 77 void __dynamic_ibdev_dbg(struct _ddebug *descriptor, 923abb9d797ba0 Gal Pressman 2019-05-01 78 const struct ib_device *ibdev, 923abb9d797ba0 Gal Pressman 2019-05-01 79 const char *fmt, ...); 923abb9d797ba0 Gal Pressman 2019-05-01 80 2bdde670beedf7 Rasmus Villemoes 2019-03-07 81 #define DEFINE_DYNAMIC_DEBUG_METADATA(name, fmt) \ c0d2af63786394 Joe Perches 2012-10-18 @82 static struct _ddebug __aligned(8) \ 07613b0b5ef857 Jason Baron 2011-10-04 83 __attribute__((section("__verbose"))) name = { \ 07613b0b5ef857 Jason Baron 2011-10-04 84 .modname = KBUILD_MODNAME, \ 07613b0b5ef857 Jason Baron 2011-10-04 85 .function = __func__, \ 07613b0b5ef857 Jason Baron 2011-10-04 86 .filename = __FILE__, \ 07613b0b5ef857 Jason Baron 2011-10-04 87 .format = (fmt), \ 07613b0b5ef857 Jason Baron 2011-10-04 88 .lineno = __LINE__, \ 07613b0b5ef857 Jason Baron 2011-10-04 89 .flags = _DPRINTK_FLAGS_DEFAULT, \ 2bdde670beedf7 Rasmus Villemoes 2019-03-07 90 _DPRINTK_KEY_INIT \ 07613b0b5ef857 Jason Baron 2011-10-04 91 } 07613b0b5ef857 Jason Baron 2011-10-04 92 :::::: The code at line 82 was first introduced by commit :::::: c0d2af637863940b1a4fb208224ca7acb905c39f dynamic_debug: Remove unnecessary __used :::::: TO: Joe Perches <joe(a)perches.com> :::::: CC: Greg Kroah-Hartman <gregkh(a)linuxfoundation.org> --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

1 0

[alsa-devel] snd_pcm_start question
by Eric Desjardins 11 Aug '19

11 Aug '19

Hi, This is a simple question. I call: snd_pcm_prepare snd_pcm_writei and it works fine but I would like to delay playback start. There is snd_pcm_start for that. There is a method that is deprecated to specify a start mode but I was wondering what was the correct new way to do that? Thanks, Eric

1 0

[alsa-devel] bcm2835.conf for RPi v. 4 incl. HDMI
by Pavel Hofman 10 Aug '19

10 Aug '19

Hi, may I ask for help with cards/bcm2835.conf for the new RPi? No such config exists yet: pi@raspberrypi:~ $ strace -f aplay -L 2>&1 | grep -i bcm2835 access("/usr/share/alsa/cards/bcm2835_alsa.conf", R_OK) = -1 ENOENT (No such file or directory) I understand the _alsa suffix is handled in aliases (e.g. https://www.raspberrypi.org/forums/viewtopic.php?t=221734#p1361104 ) The reason I am looking for this file are Pulseaudio profiles. PA requires front:X, surroundX, hdmi:X, etc. PCM devices, yet non of these are created due to the missing card config: pi@raspberrypi:~ $ aplay -L default Playback/recording through the PulseAudio sound server null Discard all samples (playback) or generate zero samples (capture) jack JACK Audio Connection Kit pulse PulseAudio Sound Server sysdefault:CARD=ALSA bcm2835 ALSA, bcm2835 ALSA Default Audio Device dmix:CARD=ALSA,DEV=0 bcm2835 ALSA, bcm2835 ALSA Direct sample mixing device dmix:CARD=ALSA,DEV=1 bcm2835 ALSA, bcm2835 IEC958/HDMI Direct sample mixing device dsnoop:CARD=ALSA,DEV=0 bcm2835 ALSA, bcm2835 ALSA Direct sample snooping device dsnoop:CARD=ALSA,DEV=1 bcm2835 ALSA, bcm2835 IEC958/HDMI Direct sample snooping device hw:CARD=ALSA,DEV=0 bcm2835 ALSA, bcm2835 ALSA Direct hardware device without any conversions hw:CARD=ALSA,DEV=1 bcm2835 ALSA, bcm2835 IEC958/HDMI Direct hardware device without any conversions plughw:CARD=ALSA,DEV=0 bcm2835 ALSA, bcm2835 ALSA Hardware device with all software conversions plughw:CARD=ALSA,DEV=1 bcm2835 ALSA, bcm2835 IEC958/HDMI Hardware device with all software conversions usbstream:CARD=ALSA bcm2835 ALSA USB Stream Output As a result, only the Analog Mono PA profile (based on the only existing hw:0) is available in PA: D: [pulseaudio] alsa-mixer.c: Checking for playback on Analog Mono (analog-mono) D: [pulseaudio] alsa-util.c: Trying hw:0 with SND_PCM_NO_AUTO_FORMAT ... D: [pulseaudio] alsa-util.c: Managed to open hw:0 ... D: [pulseaudio] alsa-mixer.c: Checking for playback on Analog Stereo (analog-stereo) D: [pulseaudio] alsa-util.c: Trying front:0 with SND_PCM_NO_AUTO_FORMAT ... I: [pulseaudio] (alsa-lib)conf.c: Unknown parameters 0 I: [pulseaudio] (alsa-lib)pcm.c: Unknown PCM front:0 --- D: [pulseaudio] alsa-mixer.c: Checking for playback on Digital Stereo (HDMI) (hdmi-stereo) D: [pulseaudio] alsa-util.c: Trying hdmi:0 with SND_PCM_NO_AUTO_FORMAT ... I: [pulseaudio] (alsa-lib)conf.c: Unknown parameters 0 I: [pulseaudio] (alsa-lib)pcm.c: Unknown PCM hdmi:0 ... D: [pulseaudio] alsa-mixer.c: Checking for playback on Digital Surround 7.1 (HDMI) (hdmi-surround71) D: [pulseaudio] alsa-util.c: Trying hdmi:0 with SND_PCM_NO_AUTO_FORMAT ... I: [pulseaudio] (alsa-lib)conf.c: Unknown parameters 0 I: [pulseaudio] (alsa-lib)pcm.c: Unknown PCM hdmi:0 RPi v.4 has 2 HDMI outputs: pi@raspberrypi:~ $ aplay -l **** List of PLAYBACK Hardware Devices **** card 0: ALSA [bcm2835 ALSA], device 0: bcm2835 ALSA [bcm2835 ALSA] Subdevices: 6/7 Subdevice #0: subdevice #0 Subdevice #1: subdevice #1 Subdevice #2: subdevice #2 Subdevice #3: subdevice #3 Subdevice #4: subdevice #4 Subdevice #5: subdevice #5 Subdevice #6: subdevice #6 card 0: ALSA [bcm2835 ALSA], device 1: bcm2835 IEC958/HDMI [bcm2835 IEC958/HDMI] Subdevices: 1/1 Subdevice #0: subdevice #0 card 0: ALSA [bcm2835 ALSA], device 2: bcm2835 IEC958/HDMI1 [bcm2835 IEC958/HDMI1] Subdevices: 1/1 Subdevice #0: subdevice #0 Each is 2 - 8 channel: pi@raspberrypi:~ $ aplay -v --dump-hw-params -D hw:0,1 /dev/zero Playing raw data '/dev/zero' : Unsigned 8 bit, Rate 8000 Hz, Mono HW Params of device "hw:0,1": -------------------- ACCESS: MMAP_INTERLEAVED RW_INTERLEAVED FORMAT: S16_LE SUBFORMAT: STD SAMPLE_BITS: 16 FRAME_BITS: [32 128] CHANNELS: [2 8] RATE: [44100 192000] PERIOD_TIME: [10000 743039) PERIOD_SIZE: [441 32768] PERIOD_BYTES: [1776 524288] PERIODS: [1 75) BUFFER_TIME: (2296 743039) BUFFER_SIZE: [441 32768] BUFFER_BYTES: [1764 131072] TICK_TIME: ALL I looked at hdmi section of cards/Intel-HDA.conf but that one is very complicated. May I ask for help how the config should look like to work both for multichannel PCM and digital formats (i.e. support for the AES vars)? I very much appreciate any help. Best regards, Pavel.

1 0

[alsa-devel] Unable to get sound out of Laptop
by Nilesh Kumar 10 Aug '19

10 Aug '19

Hello all, I am unable to get sound out of my "HP Notebook - 15-bs180tx" laptop, the output of alsa-info,sh is been uploaded at http://www.alsa-project.org/db/?f=5b015c625cef0cc14b0c0551cb57bc1e35d4b40a so, I would really appreciate if someone could see the log and help me out in configuring my sound card -- Nilesh Kumar

1 0

[alsa-devel] Writing /usr/share/alsa/cards/ files
by Paul Pawlowski 10 Aug '19

10 Aug '19

Hello, Is there any documentation on writing /usr/share/alsa/cards/ files? I painfully tried to create one in order to enable 2 separate audio outputs, but I ultimately failed to due to the lack of any sort of error reporting. Is there a way to enable any error reporting when parsing these files? It seems that the error output when parsing any hooks goes to nowhere. Also, is this the correct mailing list to get help regarding alsa driver development and configuration? Thank you, Paul Pawlowski

2 2

[alsa-devel] [PATCH] ALSA: hda - Fix a memory leak bug
by Wenwen Wang 10 Aug '19

10 Aug '19

In snd_hda_parse_generic_codec(), 'spec' is allocated through kzalloc(). Then, the pin widgets in 'codec' are parsed. However, if the parsing process fails, 'spec' is not deallocated, leading to a memory leak. To fix the above issue, free 'spec' before returning the error. Signed-off-by: Wenwen Wang <wenwen(a)cs.uga.edu> --- sound/pci/hda/hda_generic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/pci/hda/hda_generic.c b/sound/pci/hda/hda_generic.c index 485edab..8f2beb1 100644 --- a/sound/pci/hda/hda_generic.c +++ b/sound/pci/hda/hda_generic.c @@ -6100,7 +6100,7 @@ static int snd_hda_parse_generic_codec(struct hda_codec *codec) err = snd_hda_parse_pin_defcfg(codec, &spec->autocfg, NULL, 0); if (err < 0) - return err; + goto error; err = snd_hda_gen_parse_auto_config(codec, &spec->autocfg); if (err < 0) -- 2.7.4

2 1

[alsa-devel] [PATCH] ASoC: SOF: Intel: Add missing include file hdac_hda.h
by YueHaibing 09 Aug '19

09 Aug '19

Building with SND_SOC_SOF_HDA_AUDIO_CODEC fails: sound/soc/sof/intel/hda-bus.c: In function sof_hda_bus_init: sound/soc/sof/intel/hda-bus.c:16:25: error: implicit declaration of function snd_soc_hdac_hda_get_ops; did you mean snd_soc_jack_add_gpiods? [-Werror=implicit-function-declaration] #define sof_hda_ext_ops snd_soc_hdac_hda_get_ops() Reported-by: Hulk Robot <hulkci(a)huawei.com> Fixes: d4ff1b3917a5 ('ASoC: SOF: Intel: Initialize hdaudio bus properly") Signed-off-by: YueHaibing <yuehaibing(a)huawei.com> --- sound/soc/sof/intel/hda-bus.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/sof/intel/hda-bus.c b/sound/soc/sof/intel/hda-bus.c index 0caec3a..e28a9c4 100644 --- a/sound/soc/sof/intel/hda-bus.c +++ b/sound/soc/sof/intel/hda-bus.c @@ -13,6 +13,7 @@ #include "hda.h" #if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA_AUDIO_CODEC) +#include "../../../soc/codecs/hdac_hda.h" #define sof_hda_ext_ops snd_soc_hdac_hda_get_ops() #else #define sof_hda_ext_ops NULL -- 2.7.4

3 8