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

[alsa-devel] Potential Audio miniconference at LPC/KS
by Mark Brown 24 Oct '16

24 Oct '16

Hi, As in previous years we're trying to organize an audio miniconference so we can get together and talk through issues, especially design decisons, face to face. This year we're trying a slightly different tack and trying to organize this as part of the workshops day at Kernel Summit and Linux Plumbers in Santa Fe the week 31st October to 4th November (I'm writing this offline so don't have the exact date for the workshop day, sorry). There is a wiki page with an initial list of proposed topics here: http://wiki.linuxplumbersconf.org/2016:workshops In order to get this workshop approved we're going to need both a clear set of things to discuss and visible interest from attendees so if people could both follow up here and add to the wiki with suggestions of topics that'd be especially appreciated. Thanks, Mark

4 14

[alsa-devel] [PATCH] ASoC: topology: Reenable use from userspace
by Mark Brown 11 Oct '16

11 Oct '16

We had inserted a #error into the topology UAPI code to ensure that the ABI was not adopted by userspace while final review and testing was ongoing. Since we are now comfortable that the ABI is suitable for production remove the #error, future changes will include provision for backwards compatibility. Signed-off-by: Mark Brown <broonie(a)kernel.org> --- Are we OK with this? Looking for some acks and reviews... include/uapi/sound/asoc.h | 6 ------ 1 file changed, 6 deletions(-) diff --git a/include/uapi/sound/asoc.h b/include/uapi/sound/asoc.h index 247c50b..51b8066 100644 --- a/include/uapi/sound/asoc.h +++ b/include/uapi/sound/asoc.h @@ -18,12 +18,6 @@ #include <linux/types.h> #include <sound/asound.h> -#ifndef __KERNEL__ -#error This API is an early revision and not enabled in the current -#error kernel release, it will be enabled in a future kernel version -#error with incompatible changes to what is here. -#endif - /* * Maximum number of channels topology kcontrol can represent. */ -- 2.6.1

4 6

[alsa-devel] [PATCH v2] ASoC: cs4270: fix DAPM stream name mismatch
by murray foster 06 Oct '16

06 Oct '16

Mismatching stream names in DAPM route and widget definitions are causing compilation errors. Fixing these names allows the cs4270 driver to compile and function. Signed-off-by: Murray Foster <mrafoster(a)gmail.com> --- Changes in v2: - Remove the 's' typo at the beginning of the codec driver. sound/soc/codecs/cs4270.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sound/soc/codecs/cs4270.c b/sound/soc/codecs/cs4270.c index e07807d..3670086 100644 --- a/sound/soc/codecs/cs4270.c +++ b/sound/soc/codecs/cs4270.c @@ -148,11 +148,11 @@ SND_SOC_DAPM_OUTPUT("AOUTR"), }; static const struct snd_soc_dapm_route cs4270_dapm_routes[] = { - { "Capture", NULL, "AINA" }, - { "Capture", NULL, "AINB" }, + { "Capture", NULL, "AINL" }, + { "Capture", NULL, "AINR" }, - { "AOUTA", NULL, "Playback" }, - { "AOUTB", NULL, "Playback" }, + { "AOUTL", NULL, "Playback" }, + { "AOUTR", NULL, "Playback" }, }; /** -- 2.4.9

4 5

[alsa-devel] [PATCH 0/5] ALSA: soc: fix sparse warnings
by Takashi Sakamoto 28 Sep '16

28 Sep '16

Hi, This patchset is to fix some sparse warnings in ALSA SoC part. Below items are going to be fixed: * fix a condition statement with a logical operation * purge old way to initialize array * missing 'static' qualifier for file local symbols Although I got many warnings from Intel SST drivers due to confusions of address spaces (__iomem and __kernel), I still left them as they are, because they require more codes to fix. I hope Intel developers will do additional work for them... Takashi Sakamoto (5): ALSA: soc-da7219: fix inappropriate condition statement ALSA: soc-sst-bxt-rt298: fix obsoleted initializers for array ALSA: soc-sst-bxt-da7219_max98357a: fix obsoleted initializers for array ALSA: soc-rt5616: add static qualifier for file local symbols ALSA: soc-tpa6130a2: add static qualifier for file local symbols sound/soc/codecs/da7219.c | 2 +- sound/soc/codecs/rt5616.c | 4 ++-- sound/soc/codecs/tpa6130a2.c | 2 +- sound/soc/intel/boards/bxt_da7219_max98357a.c | 12 ++++++------ sound/soc/intel/boards/bxt_rt298.c | 14 +++++++------- 5 files changed, 17 insertions(+), 17 deletions(-) -- 2.7.4

4 13

[alsa-devel] [snd_ali5451] snd_ali5451 0000:00:06.0: BUG: , pos = 16384, buffer size = 16384, period size = 1024
by Enrico Mioso 21 Sep '16

21 Sep '16

Hello guys. The message on the subject comes from my dmesg and is generated by the said module when using mplayer. (MPlayer SVN-r37379 (C) 2000-2015 MPlayer Team) Including some devices from my lspci: if you wish to see the integral one, let me know. 00:06.0 Multimedia audio controller: ULi Electronics Inc. M5451 PCI AC-Link Controller Audio Device (rev 02) Subsystem: Hewlett-Packard Company Device 002a Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx- Status: Cap+ 66MHz- UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort- <TAbort- <MAbort- >SERR+ <PERR+ INTx- Latency: 64 (500ns min, 6000ns max) Interrupt: pin A routed to IRQ 5 Region 0: I/O ports at 1000 [size=256] Region 1: Memory at d0001000 (32-bit, non-prefetchable) [size=4K] Capabilities: [dc] Power Management version 2 Flags: PMEClk- DSI+ D1+ D2+ AuxCurrent=0mA PME(D0-,D1-,D2+,D3hot+,D3cold+) Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME- Kernel driver in use: snd_ali5451 00:07.0 ISA bridge: ULi Electronics Inc. M1533/M1535/M1543 PCI to ISA Bridge [Aladdin IV/V/V+] Subsystem: ULi Electronics Inc. ALi M1533 Aladdin IV/V ISA Bridge Control: I/O+ Mem+ BusMaster+ SpecCycle+ MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx- Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=medium >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx- Latency: 0 Capabilities: [a0] Power Management version 1 Flags: PMEClk- DSI- D1- D2- AuxCurrent=0mA PME(D0-,D1-,D2-,D3hot-,D3cold-) Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME- 00:08.0 Modem: ULi Electronics Inc. M5457 AC'97 Modem Controller (prog-if 00 [Generic]) Subsystem: Hewlett-Packard Company Device 002a Control: I/O+ Mem+ BusMaster- SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx- Status: Cap+ 66MHz- UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx- Interrupt: pin A routed to IRQ 10 Region 0: Memory at d0002000 (32-bit, non-prefetchable) [size=4K] Region 1: I/O ports at 1400 [size=256] Capabilities: [40] Power Management version 2 Flags: PMEClk- DSI+ D1- D2- AuxCurrent=0mA PME(D0-,D1-,D2-,D3hot+,D3cold+) Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME- Thank you. P.S.: sorry for giving no follow-up to my previous hd-audio thing; thanks.

2 4

[alsa-devel] Dell XPS One 2710 HDA Intel PCH - No S/PDIF Output
by Marve 20 Sep '16

20 Sep '16

Hi all, I've been trying stuff myself and got help from someone (debianuser) at #alsa IRC, but without success! I got the advice to explain the issue in this mailing list! I'm running Linux Mint 18 on a all-in-one Dell XPS One 2710. Most stuff works, except sound. The former Mint version also sound from the speakers worked out of the box. Now, I have to switch from the Pulse Audio volume control from "Speakers" to "Headphones (unplugged)" every time I start up to get audio from the internal speakers, which is working well then, annoying, is probably an issue that belongs in another group and not the reason I'm writing you! :) The XPS One 2710 has a S/PDIF Toslink (optical) connection. Back in the days that Microsoft wasn't trying to shove up Windows 10 to everyone, I successfully could get output from this connection to my JVC Receiver from Windows7 of the box. I find it hard to have it configured under Linux. Partly, with help from a anonymous debianuser on #alsa IRC I checked with alsamixer that the S/PDIF was not muted. Also, with the Jack Retasking for HDA tool, I enforced Pin ID: 0x1e is a SPDIF Out connection. After this, the Pulse Audio Tools shows several new interesting Built-in Audio options and I think "Digital Stereo (IEC958) Outout looks most promising. So I select this one and with Output Devices I select "Digital Output (S/PDIF). Then, when I play something on the box, the volume meters actually show that there is something playing, but I can't hear anything from the JVC receiver! The receiver supports PCM to DTS, but no sound is coming out of it! Thanks if you could help me out! I am happy to help you when you need more information or want we to test something. I had this problem on the previous Linux Mint LTS version and tried to install a newer kernel and alsa driver, which didn't work - so I decided to wait for the current LTS. I made a fresh info: http://www.alsa-project.org/db/?f=556ae2bdae66341579831231ef2d90ca0d7c8e24 If you need one with other settings, please let me know! Best wishes, M.

1 3

[alsa-devel] CIP Discontinuity with snd-firewire-digi00x
by lin-alsa＠noblejury.com 01 Sep '16

01 Sep '16

All, I've been working to set up a Digi002 Rack mixer with Jack using the Alsa backend. Unfortunately, the mixer seems to desync or something after a few minutes - hours. dmesg gives me `snd-firewire-digi00x fw1.0: Detect discontinuity of CIP: XX XX` with XX XX replaced with various hex pairs. I'm not sure how to debug this, and any help would be much appreciated. Also, cat /proc/asound/card1/firewire/clock gives `Optical mode: adat Sampling Rate: 88200 Clock Source: internal` so I don't think it's an external clock issue. Lin

2 7

[alsa-devel] [PATCH] ASoC: rockchip: implement system suspend/resume for i2s
by Sugar Zhang 01 Sep '16

01 Sep '16

restore hw registers after power loss during a suspend/resume cycle. Signed-off-by: Sugar Zhang <sugar.zhang(a)rock-chips.com> --- sound/soc/rockchip/rockchip_i2s.c | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/sound/soc/rockchip/rockchip_i2s.c b/sound/soc/rockchip/rockchip_i2s.c index 574c6af..53970ac 100644 --- a/sound/soc/rockchip/rockchip_i2s.c +++ b/sound/soc/rockchip/rockchip_i2s.c @@ -614,9 +614,35 @@ static const struct of_device_id rockchip_i2s_match[] = { {}, }; +#ifdef CONFIG_PM_SLEEP +static int rockchip_i2s_suspend(struct device *dev) +{ + struct rk_i2s_dev *i2s = dev_get_drvdata(dev); + + regcache_mark_dirty(i2s->regmap); + + return 0; +} + +static int rockchip_i2s_resume(struct device *dev) +{ + struct rk_i2s_dev *i2s = dev_get_drvdata(dev); + int ret; + + ret = pm_runtime_get_sync(dev); + if (ret < 0) + return ret; + ret = regcache_sync(i2s->regmap); + pm_runtime_put(dev); + + return ret; +} +#endif + static const struct dev_pm_ops rockchip_i2s_pm_ops = { SET_RUNTIME_PM_OPS(i2s_runtime_suspend, i2s_runtime_resume, NULL) + SET_SYSTEM_SLEEP_PM_OPS(rockchip_i2s_suspend, rockchip_i2s_resume) }; static struct platform_driver rockchip_i2s_driver = { -- 1.9.1

2 1

[alsa-devel] [RFC PATCH v2 0/7] AC97 device/driver model revamp
by Robert Jarzmik 29 Aug '16

29 Aug '16

It all started in the pxa device-tree submission here : https://lkml.org/lkml/2016/2/25/965 It will be maintained in : git fetch https://github.com/rjarzmik/linux.git work/ac97 And now it transformed into this RFC, which would bring a ground for AC'97 devices closer to the linux device/driver model. The driving ideas are still the same, and I put them in [1] for memory. This is the second opus of the RFC. I'm intending to stop the RFC cycle here if possible and have a true PATCH v1 submission after this RFC v2 unless there is a deep design flaw uncovered. I took into account Mark's and Takashi's remarks, I hope I forgot none. All the changes should be documented in the first 2 patches mainly. I also added : - the AC97 link clock For now, bus code doesn't disable it in suspend and re-enable in resume, leaving the controller decide. I'm still pondering if for S2RAM (as opposed to runtime suspend), bus code should disable the clock. - the .h guards I'm not particularly happy with my naming even for v2, feel free to propose anything better, the codec.h is particularly ugly. - statics and namespace pollution I made more functions static, limiting further the namespace pollution. - Kconfig/Makefile layout change I get the feeling that the KConfig "select" flavor would have better been a "depends on" one. And yet I didn't find a good way to enforce it, because of the way sound/soc/codecs/Kconfig is designed, and I'm afraid to break the structure by doing a "depends on AC97_BUS_NEW" in WM9713 configuration. Let's have another review cycle, that will let me test this serie more thoroughly, especially the suspend/resume and the reset in the resume, which doesn't work, and requires me to test more deeply. I'll also test by removing the wm9713 change to see how robust the implementation is. Happy review. -- Robert Robert Jarzmik (7): ALSA: ac97: split out the generic ac97 registers ALSA: ac97: add an ac97 bus ASoC: add new ac97 bus support ASoC: wm9713: add ac97 new bus support ASoC: pxa: switch to new ac97 bus support ARM: pxa: mioa701 convert to the new AC97 bus ASoC: mioa701_wm9713: convert to new ac97 bus arch/arm/mach-pxa/mioa701.c | 12 +- include/sound/ac97/codec.h | 111 ++++++++++++ include/sound/ac97/compat.h | 21 +++ include/sound/ac97/controller.h | 80 +++++++++ include/sound/ac97/regs.h | 262 +++++++++++++++++++++++++++ include/sound/ac97_codec.h | 239 +------------------------ include/sound/pxa2xx-lib.h | 15 +- sound/Kconfig | 2 + sound/Makefile | 1 + sound/ac97/Kconfig | 18 ++ sound/ac97/Makefile | 8 + sound/ac97/ac97_core.h | 10 ++ sound/ac97/bus.c | 381 ++++++++++++++++++++++++++++++++++++++++ sound/ac97/codec.c | 15 ++ sound/ac97/snd_ac97_compat.c | 104 +++++++++++ sound/arm/Kconfig | 1 - sound/arm/pxa2xx-ac97-lib.c | 39 ++-- sound/soc/Kconfig | 4 + sound/soc/codecs/Kconfig | 1 + sound/soc/codecs/wm9713.c | 62 ++++++- sound/soc/pxa/Kconfig | 4 +- sound/soc/pxa/mioa701_wm9713.c | 4 +- sound/soc/pxa/pxa2xx-ac97.c | 20 ++- 23 files changed, 1132 insertions(+), 282 deletions(-) create mode 100644 include/sound/ac97/codec.h create mode 100644 include/sound/ac97/compat.h create mode 100644 include/sound/ac97/controller.h create mode 100644 include/sound/ac97/regs.h create mode 100644 sound/ac97/Kconfig create mode 100644 sound/ac97/Makefile create mode 100644 sound/ac97/ac97_core.h create mode 100644 sound/ac97/bus.c create mode 100644 sound/ac97/codec.c create mode 100644 sound/ac97/snd_ac97_compat.c -- 2.1.4 [1] RFC PATCH reminder: Amongst the driving ideas behind this RFC : - use device/driver model for AC'97 device/driver binding is based on vendor_id{1,2} - use auto-probe of the AC'97 bus to enumerate codecs and create devices - be compatible with previous platform_data model for codecs => this enables a smooth transition, where a codec (here wm9713) can be used either through platform_device or automatic discovery - struct snd_ac97 is not used This structure is really heavy, and doesn't represent an AC97 device, but rather an AC'97 codec internals IIUC. I think it's right place would be in include/sound/ac97/codec.h. This is also a problem for regmap adherence, hence the include/sound/ac97/compat.h. - a new AC'97 bus Kconfig is created This was done for a smooth transition ... let's see if it's a good idea. - split the AC'97 into : - the codec - the digital controller - the bus Amongst the things that are not all touched yet : - ac97_codec.c The generic AC'97 codec in sound/pci/ac97/ac97_codec.c has really a lot features, which are not at all PCI related. They could be transfered to sound/ac97, without the pci specific field. => this deserves a deep ahead thinking, which I'll do if the current approach is accepted by the communauty. The most important patch is 2/7 "ALSA: ac97: add an ac97 bus". This is the main evaluation point for the serie, the others are here so that I can test it all. Well, this is a long term effort, which might need a complete rewrite according to the comments it'll get. Let's expose it for comments and see how I can progress with it.

2 10

[alsa-devel] [PATCH 0/2] ASoC: Extended SPDIF support to Allwinner A31 SoCs
by codekipper＠gmail.com 22 Aug '16

22 Aug '16

From: Marcus Cooper <codekipper(a)gmail.com> This patch set extends the current support of the Allwinner SPDIF transceiver to include the A31 SoC. For now just the SPDIF transmitter has been tested on the Mele I7. In order for this patch set to be functional then we require changes to the clocks and the device tree. For those that are interested I've pushed all the required patches here https://github.com/codekipper/linux-sunxi/commits/a31-spdif Thanks in advance, CK Marcus Cooper (2): ASoC: sunxi: Add bindings for sun6i to SPDIF ASoC: sunxi: compatibility for sun6i to SPDIF .../devicetree/bindings/sound/sunxi,sun4i-spdif.txt | 3 +++ sound/soc/sunxi/sun4i-spdif.c | 17 ++++++++++++++++- 2 files changed, 19 insertions(+), 1 deletion(-) -- 2.9.2

8 15