March 2012 - Alsa-devel - mailman.alsa-project.org

Re: [alsa-devel] [PATCH] ASoC: ep93xx-ac97: use devm_* helpers to cleanup probe
by Mark Brown 28 Mar '12

28 Mar '12

On Mon, Mar 26, 2012 at 04:00:08PM -0700, H Hartley Sweeten wrote: > Use the devm_* helpers to cleanup the probe routine. This also eliminates > having to carry the mem and irq values in the private data for the remove. > Signed-off-by: H Hartley Sweeten <hsweeten(a)visionengravers.com> > Cc: Mika Westerberg <mika.westerberg(a)iki.fi> > Cc: Liam Girdwood <lrg(a)ti.com> > Cc: Mark Brown <broonie(a)opensource.wolfsonmicro.com> > Cc: Jaroslav Kysela <perex(a)perex.cz> > CC: Takashi Iwai <tiwai(a)suse.de> Applied, but please: - Think more about your CC list. - Try to avoid putting it in the body of the e-mail.

1 0

[alsa-devel] [PATCH] ASoC: fsl: assign dma peripheral type according to bus topology
by Shawn Guo 28 Mar '12

28 Mar '12

The dma peripheral_type for SSI should be IMX_DMATYPE_SSI_SP if the SSI is on SPBA bus. Signed-off-by: Shawn Guo <shawn.guo(a)linaro.org> --- sound/soc/fsl/fsl_ssi.c | 6 ++++++ sound/soc/fsl/imx-pcm-dma.c | 3 ++- sound/soc/fsl/imx-pcm.h | 1 + 3 files changed, 9 insertions(+), 1 deletions(-) diff --git a/sound/soc/fsl/fsl_ssi.c b/sound/soc/fsl/fsl_ssi.c index 30ff605..4f76b5d 100644 --- a/sound/soc/fsl/fsl_ssi.c +++ b/sound/soc/fsl/fsl_ssi.c @@ -746,6 +746,12 @@ static int __devinit fsl_ssi_probe(struct platform_device *pdev) } ssi_private->dma_params_tx.dma = dma_events[0]; ssi_private->dma_params_rx.dma = dma_events[1]; + + ssi_private->dma_params_tx.shared_peripheral = + of_device_is_compatible(of_get_parent(np), + "fsl,spba-bus"); + ssi_private->dma_params_rx.shared_peripheral = + ssi_private->dma_params_tx.shared_peripheral; } /* Initialize the the device_attribute structure */ diff --git a/sound/soc/fsl/imx-pcm-dma.c b/sound/soc/fsl/imx-pcm-dma.c index e43c8fa..8c43262 100644 --- a/sound/soc/fsl/imx-pcm-dma.c +++ b/sound/soc/fsl/imx-pcm-dma.c @@ -106,7 +106,8 @@ static int snd_imx_open(struct snd_pcm_substream *substream) dma_params = snd_soc_dai_get_dma_data(rtd->cpu_dai, substream); dma_data = kzalloc(sizeof(*dma_data), GFP_KERNEL); - dma_data->peripheral_type = IMX_DMATYPE_SSI; + dma_data->peripheral_type = dma_params->shared_peripheral ? + IMX_DMATYPE_SSI_SP : IMX_DMATYPE_SSI; dma_data->priority = DMA_PRIO_HIGH; dma_data->dma_request = dma_params->dma; diff --git a/sound/soc/fsl/imx-pcm.h b/sound/soc/fsl/imx-pcm.h index b5f5c3a..83c0ed7 100644 --- a/sound/soc/fsl/imx-pcm.h +++ b/sound/soc/fsl/imx-pcm.h @@ -22,6 +22,7 @@ struct imx_pcm_dma_params { int dma; unsigned long dma_addr; int burstsize; + bool shared_peripheral; /* The peripheral is on SPBA bus */ }; int snd_imx_pcm_mmap(struct snd_pcm_substream *substream, -- 1.7.4.1

2 1

[alsa-devel] DMA problem with The GTA02 Machine.
by Denis 'GNUtoo' Carikli 28 Mar '12

28 Mar '12

Hi, There is a problem with DMA with the GTA02 machine: instead of playing a wav file saying "incomming call..." it plays "in-in-in-co-co-co-mi-mi-mi...". I don't know how that problem is called. In any case I've bisected it during hours(the commit that broke it was in an untestable area, so I had to bisect what made the area untestable too(it's already fixed in linux-next)). Then after identifying the commit I've tried to narrow it more. But I don't know how to narrow it even more than what I've in the patch attached. Denis.

3 3

[alsa-devel] [PATCH v5] ASoC: add imx-sgtl5000 machine driver working with fsl_ssi
by Shawn Guo 28 Mar '12

28 Mar '12

Changes since v4: * Add new_binding flag for fsl_ssi driver to support both "old binding" and "new binding" --- Shawn Guo (8): ASoC: fsl: separate SSI and DMA Kconfig options ASoC: imx: merge sound/soc/imx into sound/soc/fsl ASoC: fsl: rename imx-pcm Kconfig options and filename ASoC: fsl: create fsl_utils to accommodate the common functions ASoC: fsl: remove helper fsl_asoc_get_codec_dev_name ASoC: fsl: make fsl_ssi driver compilable on ARM/IMX ASoC: fsl: let fsl_ssi work with imx pcm and machine drivers ASoC: fsl: add imx-sgtl5000 machine driver .../bindings/sound/imx-audio-sgtl5000.txt | 24 +++ arch/powerpc/configs/86xx/mpc8610_hpcd_defconfig | 1 + arch/powerpc/configs/mpc85xx_defconfig | 1 + arch/powerpc/configs/mpc85xx_smp_defconfig | 1 + sound/soc/Kconfig | 1 - sound/soc/Makefile | 1 - sound/soc/fsl/Kconfig | 124 +++++++++++++- sound/soc/fsl/Makefile | 29 +++- sound/soc/{imx => fsl}/eukrea-tlv320.c | 2 +- sound/soc/fsl/fsl_ssi.c | 141 ++++++++++++---- sound/soc/fsl/fsl_utils.c | 91 ++++++++++ sound/soc/fsl/fsl_utils.h | 26 +++ sound/soc/{imx => fsl}/imx-audmux.c | 0 sound/soc/{imx => fsl}/imx-audmux.h | 0 .../{imx/imx-pcm-dma-mx2.c => fsl/imx-pcm-dma.c} | 0 sound/soc/{imx => fsl}/imx-pcm-fiq.c | 0 sound/soc/{imx => fsl}/imx-pcm.c | 0 sound/soc/{imx => fsl}/imx-pcm.h | 0 sound/soc/fsl/imx-sgtl5000.c | 177 ++++++++++++++++++++ sound/soc/{imx => fsl}/imx-ssi.c | 2 +- sound/soc/{imx => fsl}/imx-ssi.h | 0 sound/soc/fsl/mpc8610_hpcd.c | 166 ++----------------- sound/soc/{imx => fsl}/mx27vis-aic32x4.c | 0 sound/soc/fsl/p1022_ds.c | 158 +---------------- sound/soc/{imx => fsl}/phycore-ac97.c | 0 sound/soc/{imx => fsl}/wm1133-ev1.c | 0 sound/soc/imx/Kconfig | 79 --------- sound/soc/imx/Makefile | 22 --- 28 files changed, 601 insertions(+), 445 deletions(-)

5 21

Re: [alsa-devel] [PATCH] ASoC: ep93xx-i2s: use devm_* helpers to cleanup probe
by Mika Westerberg 27 Mar '12

27 Mar '12

On Mon, Mar 26, 2012 at 04:00:17PM -0700, H Hartley Sweeten wrote: > Use the devm_* helpers to cleanup the probe routine. This also eliminates > having to carry the mem value in the private data for the remove. > > Signed-off-by: H Hartley Sweeten <hsweeten(a)visionengravers.com> > Cc: Mika Westerberg <mika.westerberg(a)iki.fi> Acked-by: Mika Westerberg <mika.westerberg(a)iki.fi> > Cc: Liam Girdwood <lrg(a)ti.com> > Cc: Mark Brown <broonie(a)opensource.wolfsonmicro.com> > Cc: Jaroslav Kysela <perex(a)perex.cz> > CC: Takashi Iwai <tiwai(a)suse.de> > Cc: Vinod Koul <vinod.koul(a)intel.com>

1 0

Re: [alsa-devel] [PATCH] ASoC: ep93xx-ac97: use devm_* helpers to cleanup probe
by Mika Westerberg 27 Mar '12

27 Mar '12

On Mon, Mar 26, 2012 at 04:00:08PM -0700, H Hartley Sweeten wrote: > Use the devm_* helpers to cleanup the probe routine. This also eliminates > having to carry the mem and irq values in the private data for the remove. > > Signed-off-by: H Hartley Sweeten <hsweeten(a)visionengravers.com> > Cc: Mika Westerberg <mika.westerberg(a)iki.fi> Tested-by: Mika Westerberg <mika.westerberg(a)iki.fi> Acked-by: Mika Westerberg <mika.westerberg(a)iki.fi> > Cc: Liam Girdwood <lrg(a)ti.com> > Cc: Mark Brown <broonie(a)opensource.wolfsonmicro.com> > Cc: Jaroslav Kysela <perex(a)perex.cz> > CC: Takashi Iwai <tiwai(a)suse.de>

1 0

[alsa-devel] dsp-soc: question regarding dsp_add_new_paths
by Bensaid, Selma 27 Mar '12

27 Mar '12

Hi, I'm reversing the dsp-soc code and I have a question regarding the function dsp_add_new_paths: - Why do you check if (be->dai_link->dynamic) when looking for the FE AIF_IN/OUT, in my comprehension the for FE the dai_link->dynamic = 1? - For both loops used to find out the FE and BE why do you check the be->dai_link->dynamicsince in the snd_soc_dapm_get_aif loops over all card widget? Thanks in advance for your answer. Selma. --------------------------------------------------------------------- Intel Corporation SAS (French simplified joint stock company) Registered headquarters: "Les Montalets"- 2, rue de Paris, 92196 Meudon Cedex, France Registration Number: 302 456 199 R.C.S. NANTERRE Capital: 4,572,000 Euros This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies.

1 1

[alsa-devel] M-audio fasttrack pro, input device index
by David Henningsson 27 Mar '12

27 Mar '12

Hi, M-Audio fasttrack pro has two stereo outputs and one stereo input. It looks like the input sometimes shows up at device 0 and sometimes at device 1. I e, a device string would sometimes be hw:1,0,0 and sometimes hw:1,1,0. (Assuming the card index is 1) Do we know if there is a reason behind this behaviour, or should we just try both device strings? I don't have this device myself, so I don't know if it's consistent given a specific physical device, if it differs between kernel versions, or anything else. I've just seen both. Example alsa-info: https://launchpadlibrarian.net/45386782/alsa-info.txt.5TIcWGnRjL -- David Henningsson, Canonical Ltd. http://launchpad.net/~diwic

2 1

Re: [alsa-devel] ALSA async callback re-enter and DEADLOCK.
by Clemens Ladisch 27 Mar '12

27 Mar '12

Павел Колодин wrote: > So, in multi-threaded appliction, the better solution would be to start separate thread that would talk with ALSA only? Yes. If you already have a poll()-based event loop in some thread, it might make sense to put the ALSA code there, but this depends on how these threads affect each other. Regards, Clemens

1 0

[alsa-devel] [PATCH 0/8] Add ASoC drivers for SPEAr platform
by Rajeev Kumar 27 Mar '12

27 Mar '12

This patch set adds support for ASoC sound drivers on ST's SPEAr platforms. Details of the SPEAr platforms can be seen here: http://www.st.com/internet/mcu/product/250658.jsp This patch set supports the following modules: 1. Audio codec sta529 (connected to I2S controller). 2. SPDIF IN dummy Codec. 3. Synopsys Designware I2S controller. 4. ST SPDIF IN controller. 5. ST SPDIF OUT controller. This patch set has been tested for linux-kernel-version 3.3-rc3 on SPEAr platform. In past, I had circulated the patch for the same and got some review comments. I have incorporate those commnets also. Please find the link below. http://mailman.alsa-project.org/pipermail/alsa-devel/2011-April/038738.html Rajeev Kumar (6): sound:asoc: Add support for STA529 Audio Codec sound:asoc: Add support for synopsys i2s controller as per ASoC framework. sound:asoc: Add support for SPEAr ASoC pcm layer. sound:asoc: Add support for SPEAr ASoC machine driver. sound:asoc: Add Kconfig and Makefile to support SPEAr audio driver sound:asoc: Update Kconfig and Makefile(sound/soc/) to support SPEAr audio Vipin Kumar (2): sound:asoc:spdif_in: Add spdif IN support sound:asoc:spdif_out: Add spdif out support include/linux/designware_i2s.h | 63 ++++ sound/soc/Kconfig | 1 + sound/soc/Makefile | 1 + sound/soc/codecs/Kconfig | 5 + sound/soc/codecs/Makefile | 2 + sound/soc/codecs/sta529.c | 414 ++++++++++++++++++++++++++ sound/soc/spear/Kconfig | 46 +++ sound/soc/spear/Makefile | 16 + sound/soc/spear/designware_i2s.c | 610 ++++++++++++++++++++++++++++++++++++++ sound/soc/spear/spdif_in.c | 304 +++++++++++++++++++ sound/soc/spear/spdif_in_regs.h | 60 ++++ sound/soc/spear/spdif_out.c | 397 +++++++++++++++++++++++++ sound/soc/spear/spdif_out_regs.h | 79 +++++ sound/soc/spear/spear_evb.c | 278 +++++++++++++++++ sound/soc/spear/spear_pcm.c | 460 ++++++++++++++++++++++++++++ sound/soc/spear/spear_pcm.h | 33 ++ 16 files changed, 2769 insertions(+), 0 deletions(-) create mode 100644 include/linux/designware_i2s.h create mode 100644 sound/soc/codecs/sta529.c create mode 100644 sound/soc/spear/Kconfig create mode 100644 sound/soc/spear/Makefile create mode 100644 sound/soc/spear/designware_i2s.c create mode 100644 sound/soc/spear/spdif_in.c create mode 100644 sound/soc/spear/spdif_in_regs.h create mode 100644 sound/soc/spear/spdif_out.c create mode 100644 sound/soc/spear/spdif_out_regs.h create mode 100644 sound/soc/spear/spear_evb.c create mode 100644 sound/soc/spear/spear_pcm.c create mode 100644 sound/soc/spear/spear_pcm.h -- 1.7.2.2

3 24