February 2010 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH v3 1/3] ASoC: OMAP: data_type and sync_mode configurable in audio dma
by Candelaria Villareal, Jorge 19 Feb '10

19 Feb '10

From: Misael Lopez Cruz <x0052729(a)ti.com> Allow client drivers to set the data_type (16, 32) and the sync_mode (element, packet, etc) of the audio dma transferences. McBSP dai driver configures it for a data type of 16 bits and element sync mode. Signed-off-by: Misael Lopez Cruz <x0052729(a)ti.com> Signed-off-by: Jorge Eduardo Candelaria <jorge.candelaria(a)ti.com> --- sound/soc/omap/omap-mcbsp.c | 2 ++ sound/soc/omap/omap-pcm.c | 15 ++++++++------- sound/soc/omap/omap-pcm.h | 4 +++- 3 files changed, 13 insertions(+), 8 deletions(-) diff --git a/sound/soc/omap/omap-mcbsp.c b/sound/soc/omap/omap-mcbsp.c index 6bbbd2a..d297256 100644 --- a/sound/soc/omap/omap-mcbsp.c +++ b/sound/soc/omap/omap-mcbsp.c @@ -287,6 +287,8 @@ static int omap_mcbsp_dai_hw_params(struct snd_pcm_substream *substream, omap_mcbsp_dai_dma_params[id][substream->stream].dma_req = dma; omap_mcbsp_dai_dma_params[id][substream->stream].port_addr = port; omap_mcbsp_dai_dma_params[id][substream->stream].sync_mode = sync_mode; + omap_mcbsp_dai_dma_params[id][substream->stream].data_type = + OMAP_DMA_DATA_TYPE_S16; cpu_dai->dma_data = &omap_mcbsp_dai_dma_params[id][substream->stream]; if (mcbsp_data->configured) { diff --git a/sound/soc/omap/omap-pcm.c b/sound/soc/omap/omap-pcm.c index 9db2770..825db38 100644 --- a/sound/soc/omap/omap-pcm.c +++ b/sound/soc/omap/omap-pcm.c @@ -37,7 +37,8 @@ static const struct snd_pcm_hardware omap_pcm_hardware = { SNDRV_PCM_INFO_INTERLEAVED | SNDRV_PCM_INFO_PAUSE | SNDRV_PCM_INFO_RESUME, - .formats = SNDRV_PCM_FMTBIT_S16_LE, + .formats = SNDRV_PCM_FMTBIT_S16_LE | + SNDRV_PCM_FMTBIT_S32_LE, .period_bytes_min = 32, .period_bytes_max = 64 * 1024, .periods_min = 2, @@ -149,6 +150,7 @@ static int omap_pcm_prepare(struct snd_pcm_substream *substream) struct omap_runtime_data *prtd = runtime->private_data; struct omap_pcm_dma_data *dma_data = prtd->dma_data; struct omap_dma_channel_params dma_params; + int bytes; /* return if this is a bufferless transfer e.g. * codec <--> BT codec or GSM modem -- lg FIXME */ @@ -156,11 +158,7 @@ static int omap_pcm_prepare(struct snd_pcm_substream *substream) return 0; memset(&dma_params, 0, sizeof(dma_params)); - /* - * Note: Regardless of interface data formats supported by OMAP McBSP - * or EAC blocks, internal representation is always fixed 16-bit/sample - */ - dma_params.data_type = OMAP_DMA_DATA_TYPE_S16; + dma_params.data_type = dma_data->data_type; dma_params.trigger = dma_data->dma_req; dma_params.sync_mode = dma_data->sync_mode; if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) { @@ -170,6 +168,7 @@ static int omap_pcm_prepare(struct snd_pcm_substream *substream) dma_params.src_start = runtime->dma_addr; dma_params.dst_start = dma_data->port_addr; dma_params.dst_port = OMAP_DMA_PORT_MPUI; + dma_params.dst_fi = dma_data->packet_size; } else { dma_params.src_amode = OMAP_DMA_AMODE_CONSTANT; dma_params.dst_amode = OMAP_DMA_AMODE_POST_INC; @@ -177,6 +176,7 @@ static int omap_pcm_prepare(struct snd_pcm_substream *substream) dma_params.src_start = dma_data->port_addr; dma_params.dst_start = runtime->dma_addr; dma_params.src_port = OMAP_DMA_PORT_MPUI; + dma_params.src_fi = dma_data->packet_size; } /* * Set DMA transfer frame size equal to ALSA period size and frame @@ -184,7 +184,8 @@ static int omap_pcm_prepare(struct snd_pcm_substream *substream) * we can transfer the whole ALSA buffer with single DMA transfer but * still can get an interrupt at each period bounary */ - dma_params.elem_count = snd_pcm_lib_period_bytes(substream) / 2; + bytes = snd_pcm_lib_period_bytes(substream); + dma_params.elem_count = bytes >> dma_data->data_type; dma_params.frame_count = runtime->periods; omap_set_dma_params(prtd->dma_ch, &dma_params); diff --git a/sound/soc/omap/omap-pcm.h b/sound/soc/omap/omap-pcm.h index 38a821d..b19975d 100644 --- a/sound/soc/omap/omap-pcm.h +++ b/sound/soc/omap/omap-pcm.h @@ -29,8 +29,10 @@ struct omap_pcm_dma_data { char *name; /* stream identifier */ int dma_req; /* DMA request line */ unsigned long port_addr; /* transmit/receive register */ - int sync_mode; /* DMA sync mode */ void (*set_threshold)(struct snd_pcm_substream *substream); + int data_type; /* data type 8,16,32 */ + int sync_mode; /* DMA sync mode */ + int packet_size; /* packet size only in PACKET mode */ }; extern struct snd_soc_platform omap_soc_platform; -- 1.6.0.4

3 3

[alsa-devel] Since kernel-2.6.32, alsactl fails to restore defaults
by Pacho Ramos 19 Feb '10

19 Feb '10

I originally reported this issue downstream to: http://bugs.gentoo.org/show_bug.cgi?id=299362 Later to kernel: http://bugzilla.kernel.org/show_bug.cgi?id=15107 And they told me that this is an alsactl issue, since it's not exiting with 0 when it notices that the device doesn't need special quirks: # alsactl -f /var/lib/alsa/asound.state restore Unknown hardware: "VIA8237" "Realtek ALC658D" "AC97a:414c4781" "0x147b" "0x1415" Hardware is initialized using a guess method # echo $? 157 What could I do? Thanks for helping me, if you need more information, simply ask me for it :-)

2 5

[alsa-devel] [PATCH v3 3/3] ASoC: OMAP4: Add McPDM platform driver
by Candelaria Villareal, Jorge 19 Feb '10

19 Feb '10

From: Misael Lopez Cruz <x0052729(a)ti.com> McPDM platform driver is configured to use sDMA in order to transfer to/from memory. Support for interfacing with ABE will be added later. McPDM dai currently supports up to 4 downlink channels and 2 uplink channels simultaneously, as well as 88.2 and 96 KHz, and a sample size of 32 bits. Signed-off-by: Misael Lopez Cruz <x0052729(a)ti.com> Signed-off-by: Margarita Olaya <x0080101(a)ti.com> Signed-off-by: Jorge Eduardo Candelaria <jorge.candelaria(a)ti.com> --- sound/soc/omap/Makefile | 2 +- sound/soc/omap/omap-mcpdm.c | 251 +++++++++++++++++++++++++++++++++++++++++++ sound/soc/omap/omap-mcpdm.h | 29 +++++ 3 files changed, 281 insertions(+), 1 deletions(-) create mode 100644 sound/soc/omap/omap-mcpdm.c create mode 100644 sound/soc/omap/omap-mcpdm.h diff --git a/sound/soc/omap/Makefile b/sound/soc/omap/Makefile index bf8b388..d6382fa 100644 --- a/sound/soc/omap/Makefile +++ b/sound/soc/omap/Makefile @@ -1,7 +1,7 @@ # OMAP Platform Support snd-soc-omap-objs := omap-pcm.o snd-soc-omap-mcbsp-objs := omap-mcbsp.o -snd-soc-omap-mcpdm-objs := mcpdm.o +snd-soc-omap-mcpdm-objs := mcpdm.o omap-mcpdm.o obj-$(CONFIG_SND_OMAP_SOC) += snd-soc-omap.o obj-$(CONFIG_SND_OMAP_SOC_MCBSP) += snd-soc-omap-mcbsp.o diff --git a/sound/soc/omap/omap-mcpdm.c b/sound/soc/omap/omap-mcpdm.c new file mode 100644 index 0000000..25f19e4 --- /dev/null +++ b/sound/soc/omap/omap-mcpdm.c @@ -0,0 +1,251 @@ +/* + * omap-mcpdm.c -- OMAP ALSA SoC DAI driver using McPDM port + * + * Copyright (C) 2009 Texas Instruments + * + * Author: Misael Lopez Cruz <x0052729(a)ti.com> + * Contact: Jorge Eduardo Candelaria <x0107209(a)ti.com> + * Margarita Olaya <magi.olaya(a)ti.com> + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * version 2 as published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA + * 02110-1301 USA + * + */ + +#include <linux/init.h> +#include <linux/module.h> +#include <linux/device.h> +#include <sound/core.h> +#include <sound/pcm.h> +#include <sound/pcm_params.h> +#include <sound/initval.h> +#include <sound/soc.h> + +#include <plat/control.h> +#include <plat/dma.h> +#include <plat/mcbsp.h> +#include "mcpdm.h" +#include "omap-mcpdm.h" +#include "omap-pcm.h" + +struct omap_mcpdm_data { + struct omap_mcpdm_link *links; + int active; +}; + +static struct omap_mcpdm_link omap_mcpdm_links[] = { + /* downlink */ + { + .irq_mask = MCPDM_DN_IRQ_EMPTY | MCPDM_DN_IRQ_FULL, + .threshold = 1, + .format = PDMOUTFORMAT_LJUST, + }, + /* uplink */ + { + .irq_mask = MCPDM_UP_IRQ_EMPTY | MCPDM_UP_IRQ_FULL, + .threshold = 1, + .format = PDMOUTFORMAT_LJUST, + }, +}; + +static struct omap_mcpdm_data mcpdm_data = { + .links = omap_mcpdm_links, + .active = 0, +}; + +/* + * Stream DMA parameters + */ +static struct omap_pcm_dma_data omap_mcpdm_dai_dma_params[] = { + { + .name = "Audio playback", + .dma_req = OMAP44XX_DMA_MCPDM_DL, + .data_type = OMAP_DMA_DATA_TYPE_S32, + .sync_mode = OMAP_DMA_SYNC_PACKET, + .packet_size = 16, + .port_addr = OMAP44XX_MCPDM_L3_BASE + MCPDM_DN_DATA, + }, + { + .name = "Audio capture", + .dma_req = OMAP44XX_DMA_MCPDM_UP, + .data_type = OMAP_DMA_DATA_TYPE_S32, + .sync_mode = OMAP_DMA_SYNC_PACKET, + .packet_size = 16, + .port_addr = OMAP44XX_MCPDM_L3_BASE + MCPDM_UP_DATA, + }, +}; + +static int omap_mcpdm_dai_startup(struct snd_pcm_substream *substream, + struct snd_soc_dai *dai) +{ + struct snd_soc_pcm_runtime *rtd = substream->private_data; + struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai; + int err = 0; + + if (!cpu_dai->active) + err = omap_mcpdm_request(); + + return err; +} + +static void omap_mcpdm_dai_shutdown(struct snd_pcm_substream *substream, + struct snd_soc_dai *dai) +{ + struct snd_soc_pcm_runtime *rtd = substream->private_data; + struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai; + + if (!cpu_dai->active) + omap_mcpdm_free(); +} + +static int omap_mcpdm_dai_trigger(struct snd_pcm_substream *substream, int cmd, + struct snd_soc_dai *dai) +{ + struct snd_soc_pcm_runtime *rtd = substream->private_data; + struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai; + struct omap_mcpdm_data *mcpdm_priv = cpu_dai->private_data; + int stream = substream->stream; + int err = 0; + + switch (cmd) { + case SNDRV_PCM_TRIGGER_START: + case SNDRV_PCM_TRIGGER_RESUME: + case SNDRV_PCM_TRIGGER_PAUSE_RELEASE: + if (!mcpdm_priv->active++) + omap_mcpdm_start(stream); + break; + + case SNDRV_PCM_TRIGGER_STOP: + case SNDRV_PCM_TRIGGER_SUSPEND: + case SNDRV_PCM_TRIGGER_PAUSE_PUSH: + if (!--mcpdm_priv->active) + omap_mcpdm_stop(stream); + break; + default: + err = -EINVAL; + } + + return err; +} + +static int omap_mcpdm_dai_hw_params(struct snd_pcm_substream *substream, + struct snd_pcm_hw_params *params, + struct snd_soc_dai *dai) +{ + struct snd_soc_pcm_runtime *rtd = substream->private_data; + struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai; + struct omap_mcpdm_data *mcpdm_priv = cpu_dai->private_data; + struct omap_mcpdm_link *mcpdm_links = mcpdm_priv->links; + int stream = substream->stream; + int channels, err, link_mask = 0; + + cpu_dai->dma_data = &omap_mcpdm_dai_dma_params[stream]; + + channels = params_channels(params); + switch (channels) { + case 4: + if (stream == SNDRV_PCM_STREAM_CAPTURE) + /* up to 2 channels for capture */ + return -EINVAL; + link_mask |= 1 << 3; + case 3: + if (stream == SNDRV_PCM_STREAM_CAPTURE) + /* up to 2 channels for capture */ + return -EINVAL; + link_mask |= 1 << 2; + case 2: + link_mask |= 1 << 1; + case 1: + link_mask |= 1 << 0; + break; + default: + /* unsupported number of channels */ + return -EINVAL; + } + + if (stream == SNDRV_PCM_STREAM_PLAYBACK) { + mcpdm_links[stream].channels = link_mask << 3; + err = omap_mcpdm_playback_open(&mcpdm_links[stream]); + } else { + mcpdm_links[stream].channels = link_mask << 0; + err = omap_mcpdm_capture_open(&mcpdm_links[stream]); + } + + return err; +} + +static int omap_mcpdm_dai_hw_free(struct snd_pcm_substream *substream, + struct snd_soc_dai *dai) +{ + struct snd_soc_pcm_runtime *rtd = substream->private_data; + struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai; + struct omap_mcpdm_data *mcpdm_priv = cpu_dai->private_data; + struct omap_mcpdm_link *mcpdm_links = mcpdm_priv->links; + int stream = substream->stream; + int err; + + if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) + err = omap_mcpdm_playback_close(&mcpdm_links[stream]); + else + err = omap_mcpdm_capture_close(&mcpdm_links[stream]); + + return err; +} + +static struct snd_soc_dai_ops omap_mcpdm_dai_ops = { + .startup = omap_mcpdm_dai_startup, + .shutdown = omap_mcpdm_dai_shutdown, + .trigger = omap_mcpdm_dai_trigger, + .hw_params = omap_mcpdm_dai_hw_params, + .hw_free = omap_mcpdm_dai_hw_free, +}; + +#define OMAP_MCPDM_RATES (SNDRV_PCM_RATE_88200 | SNDRV_PCM_RATE_96000) +#define OMAP_MCPDM_FORMATS (SNDRV_PCM_FMTBIT_S32_LE) + +struct snd_soc_dai omap_mcpdm_dai = { + .name = "omap-mcpdm", + .id = -1, + .playback = { + .channels_min = 1, + .channels_max = 4, + .rates = OMAP_MCPDM_RATES, + .formats = OMAP_MCPDM_FORMATS, + }, + .capture = { + .channels_min = 1, + .channels_max = 2, + .rates = OMAP_MCPDM_RATES, + .formats = OMAP_MCPDM_FORMATS, + }, + .ops = &omap_mcpdm_dai_ops, + .private_data = &mcpdm_data, +}; +EXPORT_SYMBOL_GPL(omap_mcpdm_dai); + +static int __init snd_omap_mcpdm_init(void) +{ + return snd_soc_register_dai(&omap_mcpdm_dai); +} +module_init(snd_omap_mcpdm_init); + +static void __exit snd_omap_mcpdm_exit(void) +{ + snd_soc_unregister_dai(&omap_mcpdm_dai); +} +module_exit(snd_omap_mcpdm_exit); + +MODULE_AUTHOR("Misael Lopez Cruz <x0052729(a)ti.com>"); +MODULE_DESCRIPTION("OMAP PDM SoC Interface"); +MODULE_LICENSE("GPL"); diff --git a/sound/soc/omap/omap-mcpdm.h b/sound/soc/omap/omap-mcpdm.h new file mode 100644 index 0000000..73b80d5 --- /dev/null +++ b/sound/soc/omap/omap-mcpdm.h @@ -0,0 +1,29 @@ +/* + * omap-mcpdm.h + * + * Copyright (C) 2009 Texas Instruments + * + * Contact: Misael Lopez Cruz <x0052729(a)ti.com> + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * version 2 as published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA + * 02110-1301 USA + * + */ + +#ifndef __OMAP_MCPDM_H__ +#define __OMAP_MCPDM_H__ + +extern struct snd_soc_dai omap_mcpdm_dai; + +#endif /* End of __OMAP_MCPDM_H__ */ -- 1.6.0.4

2 1

Re: [alsa-devel] [patch] oxygen: clean up. make precedence explicit
by Clemens Ladisch 19 Feb '10

19 Feb '10

> This doesn't change anything, but I think it makes the code clearer. > It silences a smatch warning: > sound/pci/oxygen/oxygen_mixer.c +91 dac_mute_put(7) warn: add some parenthesis here? That message doesn't say why some parentheses should be added. And it's a question; how do I give it the answer "no"? :-) > - changed = !value->value.integer.value[0] != chip->dac_mute; > + changed = (!value->value.integer.value[0]) != chip->dac_mute; This doesn't look any clearer to me; I don't think that the unary negation operator could be thought to have lower precedence than "!=". Why does smatch warn about this combination? Do such errors actually happen? Regards, Clemens

2 1

[alsa-devel] [patch] oxygen: clean up. make precedence explicit
by Dan Carpenter 19 Feb '10

19 Feb '10

This doesn't change anything, but I think it makes the code clearer. It silences a smatch warning: sound/pci/oxygen/oxygen_mixer.c +91 dac_mute_put(7) warn: add some parenthesis here? Signed-off-by: Dan Carpenter <error27(a)gmail.com> --- Compile tested. diff --git a/sound/pci/oxygen/oxygen_mixer.c b/sound/pci/oxygen/oxygen_mixer.c index f375b8a..28590b9 100644 --- a/sound/pci/oxygen/oxygen_mixer.c +++ b/sound/pci/oxygen/oxygen_mixer.c @@ -88,7 +88,7 @@ static int dac_mute_put(struct snd_kcontrol *ctl, int changed; mutex_lock(&chip->mutex); - changed = !value->value.integer.value[0] != chip->dac_mute; + changed = (!value->value.integer.value[0]) != chip->dac_mute; if (changed) { chip->dac_mute = !value->value.integer.value[0]; chip->model.update_dac_mute(chip);

1 0

[alsa-devel] [PATCH v3 0/3] McPDM driver patches series
by Candelaria Villareal, Jorge 19 Feb '10

19 Feb '10

McPDM module is the interface between the OMAP and TWL6030 audio codec. The interface supports the following: * Up to 5 downlink channels and 3 uplink channels * 88.2 and 96 KHz * 32 bit sample size Currently, driver is set to support up to 4 downlink channels. If 2 channels are selected, output will go to the headset, and if 4 channels are selected, output will go to the headset and speakers. Functionality to select the output at runtime will be added afterwards. Command and status mode is not supported in current patches. Changes from v2: - Functions have been renamed using playback/capture convention -- Jorge Eduardo Candelaria (1): ASoC: OMAP4: Add support for McPDM Misael Lopez Cruz (2): ASoC: OMAP: data_type and sync_mode configurable in audio dma ASoC: OMAP4: Add McPDM platform driver sound/soc/omap/Kconfig | 3 + sound/soc/omap/Makefile | 2 + sound/soc/omap/mcpdm.c | 490 +++++++++++++++++++++++++++++++++++++++++++ sound/soc/omap/mcpdm.h | 151 +++++++++++++ sound/soc/omap/omap-mcbsp.c | 2 + sound/soc/omap/omap-mcpdm.c | 251 ++++++++++++++++++++++ sound/soc/omap/omap-mcpdm.h | 29 +++ sound/soc/omap/omap-pcm.c | 15 +- sound/soc/omap/omap-pcm.h | 4 +- 9 files changed, 939 insertions(+), 8 deletions(-) create mode 100644 sound/soc/omap/mcpdm.c create mode 100644 sound/soc/omap/mcpdm.h create mode 100644 sound/soc/omap/omap-mcpdm.c create mode 100644 sound/soc/omap/omap-mcpdm.h

1 0

[alsa-devel] Troubles using the external SDK
by Hans-Kristian Arntzen 19 Feb '10

19 Feb '10

I am trying to make an alsa output plugin that streams the audio over network (something like esd, just even more simplistic) using the external SDK, however, I am having trouble implementing some things correctly it seems, and the docs weren't descriptive enough. So far, it works quite well with music playback that doesn't really care about audio blocking, etc. The main problem is audio sync and blocking with more complex applications like MPlayer. As far as I can see in the MPlayer source, to retain audio sync, it uses snd_pcm_delay() for delay, and snd_pcm_status_get_avail() to figure out how much it can write without audio blocking. This is fine, but how can snd_pcm_status_get_avail() get the data it needs from the plugins? It doesn't seem to call on any callbacks to do this. It seems so far that snd_pcm_status_get_avail() always returns buffer_size, since I seem to get working audio, but the video starts to stutter. The stuttering goes away when I force a really small maximum buffer size (we're talking 512-ish bytes), but this is really, really bad, and doesn't work too well on my netbook (performance/timing issues I guess). Is it possible to define a snd_pcm_status_get_avail() callback in the SDK in some way? Anyways. I want to know how you'd do this. The OSS driver for example doesn't even seem to define a delay() callback, which strikes me as odd. How can it know about the delay? Also, I want to know the reasoning behind the pointer() function. It seems to be called all the time. I don't know what it really does, and what good it does. Is it supposed to represent where in the buffer we are? As in, it's supposed to stay somewhere between 0 and buffer_size? If so, my experience with it doesn't make sense at all. If I return that actual value, the programs simply stop up and loop endlessly over the pointer() function. The temporary "hack" (as in probably very wrong) was to return the total amount of data that's been sent since the start. Can anyone help a frustrated programmer? :P -- Themaister

1 0

[alsa-devel] [PATCHv2 0/2] McBSP: OMAP3: Add sidetone feature
by Ilkka Koskinen 18 Feb '10

18 Feb '10

The first patch implements McBSP sidetone feature on OMAP3 while the second one provides ALSA interface for using it. The patch set is based on the patch Eduardo Valentin sent to alsa-devel mailing list in October 2009. Channel gain control and enabling the sidetone have been moved to ALSA interface as suggested in comments. Since previous review round, I have corrected both the pointed issues in ASoC patch. In addition, I rebased this patch on top of Janusz Krzysztofik's McBSP register cache patches in linux-omap-2.6.git Eero Nurkkala (1): McBSP: OMAP3: Add sidetone feature Ilkka Koskinen (1): ASoC: OMAP-McBSP: ASoC interface for McBSP sidetone arch/arm/mach-omap2/mcbsp.c | 2 + arch/arm/plat-omap/include/plat/mcbsp.h | 63 +++++ arch/arm/plat-omap/mcbsp.c | 394 ++++++++++++++++++++++++++++++- sound/soc/omap/omap-mcbsp.c | 176 ++++++++++++++ sound/soc/omap/omap-mcbsp.h | 2 + 5 files changed, 636 insertions(+), 1 deletions(-)

7 11

[alsa-devel] [PATCHv2 0/2] McBSP: OMAP3: Add sidetone feature
by Ilkka Koskinen 18 Feb '10

18 Feb '10

The first patch implements McBSP sidetone feature on OMAP3 while the second one provides ALSA interface for using it. The patch set is based on the patch Eduardo Valentin sent to alsa-devel mailing list in October 2009. Channel gain control and enabling the sidetone have been moved to ALSA interface as suggested in comments. These patches apply on top of Janusz Krzysztofik's McBSP register cache patches in linux-omap-2.6.git Since previous review round, I have combined McBSP port specific omap_mcbsp_st_set_mode() and omap_mcbsp_st_put_mode() functions to corresponding ones as suggested by Peter Ujfalusi. Eero Nurkkala (1): McBSP: OMAP3: Add sidetone feature Ilkka Koskinen (1): ASoC: OMAP-McBSP: ASoC interface for McBSP sidetone arch/arm/mach-omap2/mcbsp.c | 2 + arch/arm/plat-omap/include/plat/mcbsp.h | 63 +++++ arch/arm/plat-omap/mcbsp.c | 394 ++++++++++++++++++++++++++++++- sound/soc/omap/omap-mcbsp.c | 176 ++++++++++++++ sound/soc/omap/omap-mcbsp.h | 2 + 5 files changed, 636 insertions(+), 1 deletions(-)

4 7

[alsa-devel] [LOCKDEP] 33-rc8 Running aplay with pulse as the default
by Ed Tomlinson 18 Feb '10

18 Feb '10

Hi, Inorder to get skype working with linux I have the following in .asoundrc for my normal userid. --- pcm.pulse { type pulse } ctl.pulse { type pulse } pcm.!default { type pulse } ctl.!default { type pulse --- Try #1 Starting as root if I do: su - <id> aplay sound.wav <traceback inlined below> aplay sound.wav (silence) killall pulseaudio aplay sound.wav (works as expected now going thru pulseaudio) Try #2 Starting as root if I do: aplay sound.wav (works as expected using alsa alone) su - <id> aplay sound.wav <another lockdep traceback> aplay sound.wav (works as expected now going thru pulseaudio) With .32 there were no tracebacks (lockdep was enabled) and try #1 would work where I now get silence. Ideas? Ed Tomlinson aplay -l **** List of PLAYBACK Hardware Devices **** card 0: SB [HDA ATI SB], device 0: ALC1200 Analog [ALC1200 Analog] Subdevices: 0/1 Subdevice #0: subdevice #0 card 0: SB [HDA ATI SB], device 1: ALC1200 Digital [ALC1200 Digital] Subdevices: 1/1 Subdevice #0: subdevice #0 card 1: HDMI [HDA ATI HDMI], device 3: ATI HDMI [ATI HDMI] Subdevices: 1/1 Subdevice #0: subdevice #0 "ALC1200 Analog" is what has speakers connected The em28xx in the traceback is an input source from a usb hdtv dongle. pavucontrol setup has been verified. [ 91.070620] [ 91.070621] ======================================================= [ 91.071378] [ INFO: possible circular locking dependency detected ] [ 91.071378] 2.6.33-rc8-crc #106 [ 91.071378] ------------------------------------------------------- [ 91.071378] pulseaudio/2717 is trying to acquire lock: [ 91.071378] (&dev->lock){+.+.+.}, at: [<ffffffffa02797f1>] snd_em28xx_capture_open+0x61/0x190 [em28xx_alsa] [ 91.071378] [ 91.071378] but task is already holding lock: [ 91.071378] (&pcm->open_mutex){+.+.+.}, at: [<ffffffffa010ecd2>] snd_pcm_open+0x182/0x450 [snd_pcm] [ 91.071378] [ 91.071378] which lock already depends on the new lock. [ 91.071378] [ 91.071378] [ 91.071378] the existing dependency chain (in reverse order) is: [ 91.071378] [ 91.071378] -> #3 (&pcm->open_mutex){+.+.+.}: [ 91.071378] [<ffffffff81083215>] __lock_acquire+0xfc5/0x1550 [ 91.071378] [<ffffffff8108383c>] lock_acquire+0x9c/0x140 [ 91.071378] [<ffffffff814b96c1>] __mutex_lock_common+0x61/0x610 [ 91.071378] [<ffffffff814b9d53>] mutex_lock_nested+0x43/0x50 [ 91.071378] [<ffffffffa010ad32>] snd_pcm_release+0x42/0xb0 [snd_pcm] [ 91.071378] [<ffffffff8112115d>] __fput+0x15d/0x290 [ 91.071378] [<ffffffff811212ad>] fput+0x1d/0x30 [ 91.071378] [<ffffffff810fe001>] remove_vma+0x51/0x80 [ 91.071378] [<ffffffff810ffa1d>] do_munmap+0x2fd/0x390 [ 91.071378] [<ffffffff810ffb06>] sys_munmap+0x56/0x80 [ 91.071378] [<ffffffff81002adb>] system_call_fastpath+0x16/0x1b [ 91.071378] [ 91.071378] -> #2 (&mm->mmap_sem){++++++}: [ 91.071378] [<ffffffff81083215>] __lock_acquire+0xfc5/0x1550 [ 91.071378] [<ffffffff8108383c>] lock_acquire+0x9c/0x140 [ 91.071378] [<ffffffff810f62c7>] might_fault+0xa7/0xd0 [ 91.071378] [<ffffffff8113102b>] filldir+0x7b/0xe0 [ 91.071378] [<ffffffff8118aa21>] sysfs_readdir+0xf1/0x1b0 [ 91.071378] [<ffffffff81131268>] vfs_readdir+0xb8/0xe0 [ 91.071378] [<ffffffff8113144d>] sys_getdents+0xad/0x110 [ 91.071378] [<ffffffff81002adb>] system_call_fastpath+0x16/0x1b [ 91.071378] [ 91.071378] -> #1 (sysfs_mutex){+.+.+.}: [ 91.071378] [<ffffffff81083215>] __lock_acquire+0xfc5/0x1550 [ 91.071378] [<ffffffff8108383c>] lock_acquire+0x9c/0x140 [ 91.071378] [<ffffffff814b96c1>] __mutex_lock_common+0x61/0x610 [ 91.071378] [<ffffffff814b9d53>] mutex_lock_nested+0x43/0x50 [ 91.071378] [<ffffffff8118b082>] sysfs_addrm_start+0x22/0x30 [ 91.071378] [<ffffffff8118bc58>] create_dir+0x58/0xb0 [ 91.514885] [<ffffffff8118bd0a>] sysfs_create_dir+0x5a/0x70 [ 91.514885] [<ffffffff8123ebbd>] kobject_add_internal+0xbd/0x1f0 [ 91.514885] [<ffffffff8123ee08>] kobject_add_varg+0x38/0x60 [ 91.514885] [<ffffffff8123eee4>] kobject_add+0x44/0x70 [ 91.514885] [<ffffffff81392510>] device_add+0xb0/0x5e0 [ 91.514885] [<ffffffff81392a5e>] device_register+0x1e/0x30 [ 91.514885] [<ffffffff8140a50d>] i2c_register_adapter+0x12d/0x250 [ 91.514885] [<ffffffff8140a7f1>] i2c_add_adapter+0xb1/0xd0 [ 91.514885] [<ffffffffa0231ba4>] em28xx_i2c_register+0x104/0x520 [em28xx] [ 91.514885] [<ffffffffa0233daa>] em28xx_usb_probe+0x68a/0xb90 [em28xx] [ 91.514885] [<ffffffff813eb4ab>] usb_probe_interface+0xeb/0x1b0 [ 91.514885] [<ffffffff81395546>] driver_probe_device+0xc6/0x1d0 [ 91.514885] [<ffffffff813956eb>] __driver_attach+0x9b/0xa0 [ 91.514885] [<ffffffff8139492c>] bus_for_each_dev+0x6c/0xa0 [ 91.514885] [<ffffffff8139535e>] driver_attach+0x1e/0x20 [ 91.514885] [<ffffffff81394c11>] bus_add_driver+0xe1/0x280 [ 91.514885] [<ffffffff81395a58>] driver_register+0x98/0x140 [ 91.514885] [<ffffffff813ea06c>] usb_register_driver+0xdc/0x1a0 [ 91.514885] [<ffffffffa0248023>] 0xffffffffa0248023 [ 91.514885] [<ffffffff810001dc>] do_one_initcall+0x3c/0x1d0 [ 91.514885] [<ffffffff81092d75>] sys_init_module+0xe5/0x250 [ 91.514885] [<ffffffff81002adb>] system_call_fastpath+0x16/0x1b [ 91.514885] [ 91.514885] -> #0 (&dev->lock){+.+.+.}: [ 91.514885] [<ffffffff81083668>] __lock_acquire+0x1418/0x1550 [ 91.514885] [<ffffffff8108383c>] lock_acquire+0x9c/0x140 [ 91.514885] [<ffffffff814b96c1>] __mutex_lock_common+0x61/0x610 [ 91.514885] [<ffffffff814b9d53>] mutex_lock_nested+0x43/0x50 [ 91.514885] [<ffffffffa02797f1>] snd_em28xx_capture_open+0x61/0x190 [em28xx_alsa] [ 91.514885] [<ffffffffa010adee>] snd_pcm_open_substream+0x4e/0x90 [snd_pcm] [ 91.514885] [<ffffffffa010ece9>] snd_pcm_open+0x199/0x450 [snd_pcm] [ 91.514885] [<ffffffffa010efd4>] snd_pcm_capture_open+0x34/0x40 [snd_pcm] [ 91.514885] [<ffffffffa00c5778>] snd_open+0x198/0x4e0 [snd] [ 91.514885] [<ffffffff8112384d>] chrdev_open+0x17d/0x320 [ 91.514885] [<ffffffff8111d358>] __dentry_open+0x1a8/0x400 [ 91.514885] [<ffffffff8111d6b4>] nameidata_to_filp+0x54/0x70 [ 91.514885] [<ffffffff8112e7f1>] do_filp_open+0x841/0xc00 [ 91.514885] [<ffffffff8111e954>] do_sys_open+0xa4/0x180 [ 91.514885] [<ffffffff8111ea70>] sys_open+0x20/0x30 [ 91.514885] [<ffffffff81002adb>] system_call_fastpath+0x16/0x1b [ 91.514885] [ 91.514885] other info that might help us debug this: [ 91.514885] [ 91.514885] 1 lock held by pulseaudio/2717: [ 91.514885] #0: (&pcm->open_mutex){+.+.+.}, at: [<ffffffffa010ecd2>] snd_pcm_open+0x182/0x450 [snd_pcm] [ 91.514885] [ 91.514885] stack backtrace: [ 91.514885] Pid: 2717, comm: pulseaudio Not tainted 2.6.33-rc8-crc #106 [ 91.514885] Call Trace: [ 91.514885] [<ffffffff81080fd9>] print_circular_bug+0xe9/0xf0 [ 91.514885] [<ffffffff81083668>] __lock_acquire+0x1418/0x1550 [ 91.514885] [<ffffffff81081dc8>] ? debug_check_no_locks_freed+0xc8/0x150 [ 91.514885] [<ffffffff8108383c>] lock_acquire+0x9c/0x140 [ 91.514885] [<ffffffffa02797f1>] ? snd_em28xx_capture_open+0x61/0x190 [em28xx_alsa] [ 91.514885] [<ffffffff814b96c1>] __mutex_lock_common+0x61/0x610 [ 91.514885] [<ffffffffa02797f1>] ? snd_em28xx_capture_open+0x61/0x190 [em28xx_alsa] [ 91.514885] [<ffffffffa02797f1>] ? snd_em28xx_capture_open+0x61/0x190 [em28xx_alsa] [ 91.514885] [<ffffffffa0109c70>] ? snd_pcm_hw_rule_muldivk+0x0/0xa0 [snd_pcm] [ 91.514885] [<ffffffff8107fd54>] ? lockdep_init_map+0x44/0x130 [ 91.514885] [<ffffffff814b9d53>] mutex_lock_nested+0x43/0x50 [ 91.514885] [<ffffffffa02797f1>] snd_em28xx_capture_open+0x61/0x190 [em28xx_alsa] [ 91.514885] [<ffffffffa010adee>] snd_pcm_open_substream+0x4e/0x90 [snd_pcm] [ 91.514885] [<ffffffffa010ece9>] snd_pcm_open+0x199/0x450 [snd_pcm] [ 91.514885] [<ffffffff81081cfd>] ? trace_hardirqs_on+0xd/0x10 [ 91.514885] [<ffffffff810434f0>] ? default_wake_function+0x0/0x20 [ 91.514885] [<ffffffff814b965e>] ? mutex_unlock+0xe/0x10 [ 91.514885] [<ffffffffa010efd4>] snd_pcm_capture_open+0x34/0x40 [snd_pcm] [ 91.514885] [<ffffffffa00c5778>] snd_open+0x198/0x4e0 [snd] [ 91.514885] [<ffffffff8112384d>] chrdev_open+0x17d/0x320 [ 91.514885] [<ffffffff8111d358>] __dentry_open+0x1a8/0x400 [ 91.514885] [<ffffffff811236d0>] ? chrdev_open+0x0/0x320 [ 91.514885] [<ffffffff8111d6b4>] nameidata_to_filp+0x54/0x70 [ 91.514885] [<ffffffff8112e7f1>] do_filp_open+0x841/0xc00 [ 91.514885] [<ffffffff814bf981>] ? sub_preempt_count+0x51/0x60 [ 91.514885] [<ffffffff814bbb7c>] ? _raw_spin_unlock+0x5c/0x70 [ 91.514885] [<ffffffff8111e954>] do_sys_open+0xa4/0x180 [ 91.514885] [<ffffffff8111ea70>] sys_open+0x20/0x30 [ 91.514885] [<ffffffff81002adb>] system_call_fastpath+0x16/0x1b [ 92.643423] hda-intel: IRQ timing workaround is activated for card #0. Suggest a bigger bdl_pos_adj.

2 12