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

Re: [alsa-devel] Internal Speaker problem
by Laszlo Papp 15 Apr '17

15 Apr '17

Guys, seriously, no one willing to help with a completely broken setup?! It works fine on Windows, so I would really like to get it working on Linux, too. On Mon, Mar 27, 2017 at 8:29 AM, Laszlo Papp <lpapp(a)kde.org> wrote: > I am sorry about flooding with my emails. I just thought that I would > amend some information that I forgot to mention in my original email. > > The same laptop and internal speaker work ok on Windows 10. Also, the > setup used to work about 1-2 months ago. I cannot remember what exactly > broke, perhaps a system upgrade. If it matters, I am using Archlinux. > > On Mon, Mar 27, 2017 at 8:27 AM, Laszlo Papp <lpapp(a)kde.org> wrote: > >> Dear Alsa Developers, >> >> My internal speaker in the Lenovo Thinkpad T510 laptop stopped working. >> Normally, it would provide either no sound or just for a couple of seconds >> and then it would go off. >> >> I do not have auto-mute enabled. I checked it with alsamixer that it is >> disabled. >> >> My ALSA information is located at http://www.alsa-project.org/db >> /?f=87547de5ff55e44e360aa2382d4e5d3b7bbed091 >> >> Do you know how I could fix this? >> >> Ys, L. >> >> >

4 12

[alsa-devel] [PATCH 1/3] ASoC: rt5514: Mark rt5514_i2c_driver as static
by Douglas Anderson 14 Apr '17

14 Apr '17

There's no reason for rt5514_i2c_driver to be non-static. Signed-off-by: Douglas Anderson <dianders(a)chromium.org> --- sound/soc/codecs/rt5514.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/rt5514.c b/sound/soc/codecs/rt5514.c index b281a46d769d..481e77763fe4 100644 --- a/sound/soc/codecs/rt5514.c +++ b/sound/soc/codecs/rt5514.c @@ -1149,7 +1149,7 @@ static int rt5514_i2c_remove(struct i2c_client *i2c) return 0; } -struct i2c_driver rt5514_i2c_driver = { +static struct i2c_driver rt5514_i2c_driver = { .driver = { .name = "rt5514", .of_match_table = of_match_ptr(rt5514_of_match), -- 2.12.2.762.g0e3151a226-goog

2 5

[alsa-devel] [PATCH] ALSA: fireface: obsolete usage of ktime_set() for zero assignment
by Takashi Sakamoto 14 Apr '17

14 Apr '17

In development period for Linux v4.10, ktime_t became an alias of s64, instead of union. I forgot it. We can just assign zero, instead of usage of ktime_set(0, 0). Fixes: 19174295788 ("ALSA: fireface: add transaction support") Signed-off-by: Takashi Sakamoto <o-takashi(a)sakamocchi.jp> --- sound/firewire/fireface/ff-transaction.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/firewire/fireface/ff-transaction.c b/sound/firewire/fireface/ff-transaction.c index d1b098f..dd6c8e8 100644 --- a/sound/firewire/fireface/ff-transaction.c +++ b/sound/firewire/fireface/ff-transaction.c @@ -21,7 +21,7 @@ static void finish_transmit_midi_msg(struct snd_ff *ff, unsigned int port, if (rcode != RCODE_COMPLETE) { /* Transfer the message again, immediately. */ - ff->next_ktime[port] = ktime_set(0, 0); + ff->next_ktime[port] = 0; schedule_work(&ff->rx_midi_work[port]); return; } -- 2.9.3

1 0

[alsa-devel] [PATCH v2 0/8] ALSA: firewire-lib/firewire-tascam: localize async midi port
by Takashi Sakamoto 14 Apr '17

14 Apr '17

Hi, This patchset is a revised version of my previous one: [alsa-devel] [PATCH 0/8] ALSA: firewire-lib/firewire-tascam: localize async midi port http://mailman.alsa-project.org/pipermail/alsa-devel/2017-April/119748.html A bug of this feature was reported: http://mailman.alsa-project.org/pipermail/alsa-devel/2017-April/119775.html And I posted a fix: http://mailman.alsa-project.org/pipermail/alsa-devel/2017-April/119782.html The fix brought conflict to the previous patchset and this is a revised version. Takashi Sakamoto (8): ALSA: firewire-lib/firewire-tascam: localize async midi port ALSA: firewire-tascam: remove callback function from async midi port ALSA: firewire-tascam: send fixed-length transaction for async midi port ALSA: firewire-tascam: use the same address for asynchronous transaction for MIDI message ALSA: firewire-tascam: use fixed-length array for message cache to async midi port ALSA: firewire-tascam: initialize parameters at open of rawmidi character devices ALSA: firewire-tascam: move message parameters for async midi port ALSA: firewire-tascam: support drain callback for MIDI playback substream sound/firewire/lib.c | 141 ---------------------------- sound/firewire/lib.h | 54 ----------- sound/firewire/tascam/tascam-midi.c | 13 ++- sound/firewire/tascam/tascam-transaction.c | 142 ++++++++++++++++++++++++----- sound/firewire/tascam/tascam.h | 39 +++++++- 5 files changed, 162 insertions(+), 227 deletions(-) -- 2.9.3

2 9

[alsa-devel] usx2y: usx2yhwdeppcm.c gives "should not be here with counts=40" (linux 4.9.0)
by Jaime T 14 Apr '17

14 Apr '17

Hi all. I'm running a fresh and cleanly-installed debian testing (stretch) with my tascam us-122. After about 6 hours of perfect, joyous sound, the kernel (4.9.0-2-686) emits: should not be here with counts=40 and my jack server stops working. I'm aware that I should be running a vanilla kernel before logging a bug against the driver - is that my next step? (it might take a very long time!) (FWIW, I've run alsa-info.sh and the results are available at http://www.alsa-project.org/db/?f=f5e0f8f654ef467cedaaed7ae129b02267c46a95) Many thanks, Jaime

1 0

[alsa-devel] [PATCH 0/8] ALSA: firewire-lib/firewire-tascam: localize async midi port
by Takashi Sakamoto 14 Apr '17

14 Apr '17

Hi, At a development period for Linux v4.4, firewire-lib module got an abstraction of asynchronous transaction for transmission of MIDI message. I did this with an assumption that this is required for Digi00x series, as well as TASCAM FireWire series. However, my recent work reveal it's actually not[0]. This patchset localizes the abstraction just for TASCAM FireWire series. [0] [alsa-devel] [PATCH 0/4] ALSA: firewire-digi00x: integration to support console models http://mailman.alsa-project.org/pipermail/alsa-devel/2017-April/119397.html Takashi Sakamoto (8): firewire-lib/firewire-tascam: localize async midi port firewire-tascam: remove callback function from async midi port firewire-tascam: send fixed-length transaction for async midi port firewire-tascam: use the same address for asynchronous transaction for MIDI message firewire-tascam: use fixed-length array for message cache to async midi port firewire-tascam: initialize parameters at open of rawmidi character devices firewire-tascam: move message parameters for async midi port firewire-tascam: support drain callback for MIDI playback substream sound/firewire/lib.c | 141 ---------------------------- sound/firewire/lib.h | 54 ----------- sound/firewire/tascam/tascam-midi.c | 13 ++- sound/firewire/tascam/tascam-transaction.c | 142 ++++++++++++++++++++++++----- sound/firewire/tascam/tascam.h | 39 +++++++- 5 files changed, 162 insertions(+), 227 deletions(-) -- 2.9.3

2 9

[alsa-devel] [PATCH] ALSA: usb-line6: constify snd_kcontrol_new strucutre array
by Takashi Sakamoto 14 Apr '17

14 Apr '17

In kernel APIs of ALSA control interface, drivers can create a control element set by a call of snd_ctl_new1() with a template. This template is known to have const qualifier in general cases. This commit adds the qualifier to template array, for safer program and runtime. Application of this change moves the symbol from .data section to .rodata section. Cc: Bhumika Goyal <bhumirks(a)gmail.com> Signed-off-by: Takashi Sakamoto <o-takashi(a)sakamocchi.jp> --- sound/usb/line6/pcm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/usb/line6/pcm.c b/sound/usb/line6/pcm.c index fab53f5..b3854f8 100644 --- a/sound/usb/line6/pcm.c +++ b/sound/usb/line6/pcm.c @@ -430,7 +430,7 @@ static int snd_line6_control_playback_put(struct snd_kcontrol *kcontrol, } /* control definition */ -static struct snd_kcontrol_new line6_controls[] = { +static const struct snd_kcontrol_new line6_controls[] = { { .iface = SNDRV_CTL_ELEM_IFACE_MIXER, .name = "PCM Playback Volume", -- 2.9.3

2 1

[alsa-devel] [PATCH] ALSA: firewire-lib: fix inappropriate assignment between signed/unsigned type
by Takashi Sakamoto 14 Apr '17

14 Apr '17

An abstraction of asynchronous transaction for transmission of MIDI messages was introduced in Linux v4.4. Each driver can utilize this abstraction to transfer MIDI messages via fixed-length payload of transaction to a certain unit address. Filling payload of the transaction is done by callback. In this callback, each driver can return negative error code, however current implementation assigns the return value to unsigned variable. This commit changes type of the variable to fix the bug. Reported-by: Julia Lawall <Julia.Lawall(a)lip6.fr> Cc: <stable(a)vger.kernel.org> # 4.4+ Fixes: 585d7cba5e1f ("ALSA: firewire-lib: add helper functions for asynchronous transactions to transfer MIDI messages") Signed-off-by: Takashi Sakamoto <o-takashi(a)sakamocchi.jp> --- sound/firewire/lib.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/firewire/lib.h b/sound/firewire/lib.h index f676931..c3768cd 100644 --- a/sound/firewire/lib.h +++ b/sound/firewire/lib.h @@ -45,7 +45,7 @@ struct snd_fw_async_midi_port { struct snd_rawmidi_substream *substream; snd_fw_async_midi_port_fill fill; - unsigned int consume_bytes; + int consume_bytes; }; int snd_fw_async_midi_port_init(struct snd_fw_async_midi_port *port, -- 2.9.3

2 1

Re: [alsa-devel] [PATCH 2/8] firewire-tascam: remove callback function from async midi port (fwd)
by Julia Lawall 14 Apr '17

14 Apr '17

Hello, I haven't checked on the type, but it may be worth looking into (line 201). julia ---------- Forwarded message ---------- Date: Thu, 13 Apr 2017 18:20:17 +0800 From: kbuild test robot <fengguang.wu(a)intel.com> To: kbuild(a)01.org Cc: Julia Lawall <julia.lawall(a)lip6.fr> Subject: Re: [alsa-devel] [PATCH 2/8] firewire-tascam: remove callback function from async midi port In-Reply-To: <20170413051527.21396-3-o-takashi(a)sakamocchi.jp> Hi Takashi, [auto build test WARNING on sound/for-next] [also build test WARNING on v4.11-rc6 next-20170413] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Takashi-Sakamoto/ALSA-firewire-lib… base: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next :::::: branch date: 3 hours ago :::::: commit date: 3 hours ago >> sound/firewire/tascam/tascam-transaction.c:201:5-24: WARNING: Unsigned expression compared with zero: port -> consume_bytes <= 0 git remote add linux-review https://github.com/0day-ci/linux git remote update linux-review git checkout b866eaa9a02893e6383eb5177c11078a16af4e80 vim +201 sound/firewire/tascam/tascam-transaction.c 741f7c77a Takashi Sakamoto 2017-04-13 185 /* Nothing to do. */ 741f7c77a Takashi Sakamoto 2017-04-13 186 if (substream == NULL || snd_rawmidi_transmit_empty(substream)) 741f7c77a Takashi Sakamoto 2017-04-13 187 return; 741f7c77a Takashi Sakamoto 2017-04-13 188 741f7c77a Takashi Sakamoto 2017-04-13 189 /* Do it in next chance. */ 741f7c77a Takashi Sakamoto 2017-04-13 190 if (ktime_after(port->next_ktime, ktime_get())) { 741f7c77a Takashi Sakamoto 2017-04-13 191 schedule_work(&port->work); 741f7c77a Takashi Sakamoto 2017-04-13 192 return; 741f7c77a Takashi Sakamoto 2017-04-13 193 } 741f7c77a Takashi Sakamoto 2017-04-13 194 741f7c77a Takashi Sakamoto 2017-04-13 195 /* 741f7c77a Takashi Sakamoto 2017-04-13 196 * Fill the buffer. The callee must use snd_rawmidi_transmit_peek(). 741f7c77a Takashi Sakamoto 2017-04-13 197 * Later, snd_rawmidi_transmit_ack() is called. 741f7c77a Takashi Sakamoto 2017-04-13 198 */ 741f7c77a Takashi Sakamoto 2017-04-13 199 memset(port->buf, 0, port->len); b866eaa9a Takashi Sakamoto 2017-04-13 200 port->consume_bytes = fill_message(substream, port->buf); 741f7c77a Takashi Sakamoto 2017-04-13 @201 if (port->consume_bytes <= 0) { 741f7c77a Takashi Sakamoto 2017-04-13 202 /* Do it in next chance, immediately. */ 741f7c77a Takashi Sakamoto 2017-04-13 203 if (port->consume_bytes == 0) { 741f7c77a Takashi Sakamoto 2017-04-13 204 port->next_ktime = 0; 741f7c77a Takashi Sakamoto 2017-04-13 205 schedule_work(&port->work); 741f7c77a Takashi Sakamoto 2017-04-13 206 } else { 741f7c77a Takashi Sakamoto 2017-04-13 207 /* Fatal error. */ 741f7c77a Takashi Sakamoto 2017-04-13 208 port->error = true; 741f7c77a Takashi Sakamoto 2017-04-13 209 } :::::: The code at line 201 was first introduced by commit :::::: 741f7c77a1a15d04ebafcce4e079cfe1cfb612ff firewire-lib/firewire-tascam: localize async midi port :::::: TO: Takashi Sakamoto <o-takashi(a)sakamocchi.jp> :::::: CC: 0day robot <fengguang.wu(a)intel.com> --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

2 1

[alsa-devel] [PATCH v4 0/2] Add STM32 SAI support
by olivier moysan 13 Apr '17

13 Apr '17

This patch-set handles the Serial Audio Interface (SAI) IP on STM32 platforms. The SAI IP is composed of two Sub-block A and B. (see the figure below) Each sub-block makes use of individual and shared resources. Shared resources: - Reset line. Reset common and sub-block registers. - Bus interface clocks. This clock is not exposed in device as it it managed by clock driver, according to kernel clocks gating. - Common register. SAI IP exhibits a common configuration register to manage synchronization modes. NB: These modes are not yet implemented in this version of the SAI driver. - Interrupt. Sub-blocks have their own interrupt status registers but they share the same interrupt line. Individual resources: - Register set - DMA request line. - Communication interface. Each sub-block has its own GPIOs and associated bus lines. - Kernel clock. Each sub-block has its own dedicated clock for its communication interface. To reflect this architecture Sub-block A and B are handled by 2 child devices. Sub-block A and B devices can be configured independently either as transmitter or receiver. A PCM device is associated to each sub-block. A sub-block has to select the appropriated parent clock at runtime, depending on the audio stream sampling rate to be played or captured. Two parent clocks must be provided to support sampling rates multiples of 8 kHz or 11.025kHz. Interface clock +--------------------------------+ +---------------> | SAI IP | Reset | +-----------------+ | +---------------> | |common registers | | | +-----------------+ | | | | +----------------------+ | | | Sub-block A | | Kernel clock A | | | | Bus A +--------------------> | +----------------+ +----------------> | | |A registers | | | | | +----------------+ | | | +----------------------+ | | | | +----------------------+ | | | Sub-block B | | Kernel clock B | | | | Bus B +--------------------> | +----------------+ +-----------------> | | |B registers | | | | | +----------------+ | | | +----------------------+ | | | +--------------------------------+ v3 -> v4 Apply minor changes from Peter review olivier moysan (2): ASoC: stm32: add bindings for SAI ASoC: stm32: add SAI driver .../devicetree/bindings/sound/st,stm32-sai.txt | 89 +++ sound/soc/Kconfig | 1 + sound/soc/Makefile | 1 + sound/soc/stm/Kconfig | 8 + sound/soc/stm/Makefile | 6 + sound/soc/stm/stm32_sai.c | 115 +++ sound/soc/stm/stm32_sai.h | 199 +++++ sound/soc/stm/stm32_sai_sub.c | 881 +++++++++++++++++++++ 8 files changed, 1300 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/st,stm32-sai.txt create mode 100644 sound/soc/stm/Kconfig create mode 100644 sound/soc/stm/Makefile create mode 100644 sound/soc/stm/stm32_sai.c create mode 100644 sound/soc/stm/stm32_sai.h create mode 100644 sound/soc/stm/stm32_sai_sub.c -- 1.9.1

2 3