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

Re: [alsa-devel] [GIT PULL] ASoC: Samsung: Updates for v3.8
by Mark Brown 24 Nov '12

24 Nov '12

On Sat, Nov 24, 2012 at 05:20:52AM +0000, PADMAVATHI VENNA wrote: > <HTML><HEAD><TITLE>Samsung Enterprise Portal mySingle</TITLE> > <META content="text/html; charset=windows-1252" http-equiv=Content-Type> > <STYLE id=mysingle_style type=text/css>P { > MARGIN-TOP: 5px; FONT-FAMILY: Arial, arial; MARGIN-BOTTOM: 5px; FONT-SIZE: 9pt > } Please fix your mailer, the above is exactly how your mail comes out in my mail client (modulo me deleting the rest of it) - there's no text version.

1 0

[alsa-devel] [GIT PULL] ASoC: Samsung: Updates for v3.8
by Sangbeom Kim 24 Nov '12

24 Nov '12

Hi Mark, It's Samsung audio updates which are mainly device tree supports. We had fully tested on SMDK5250. Thanks, Sangbeom. --------- The following changes since commit 208229ecb95b58a627d2f01fc4e2a1652c27f4d9: ASoC: bells: Up to 512fs (2012-11-16 17:42:40 -0800) (sound.git topic/samsung) are available in the git repository at: git://github.com/cornus/linux-samsung.git for-broonie ---------------------------------------------------------------- Padmavathi Venna (2): ASoC: SAMSUNG: Add DT support for i2s ASoC: Samsung: Register the audio dma platform device Sachin Kamat (1): ASoC: Samsung-dma: Fix potential NULL pointer dereference Sangsu Park (1): ASoC: Samsung: Update Kconfig for I2S,SPDIF and PCM audio .../devicetree/bindings/sound/samsung-i2s.txt | 69 ++++++ sound/soc/samsung/Kconfig | 6 +- sound/soc/samsung/dma.c | 44 ++++- sound/soc/samsung/dma.h | 1 + sound/soc/samsung/i2s.c | 234 ++++++++++++++++---- 5 files changed, 298 insertions(+), 56 deletions(-)

2 5

Re: [alsa-devel] [PATCH V3 0/2] ARM: Exynos: Enable I2S platform support for Exynos5
by Sangbeom Kim 24 Nov '12

24 Nov '12

On Tue, Nov 6, 2012 at 1:45 PM, Padmavathi Venna wrote: > This patch set enables device tree based platform support for the three I2S > controllers on Samsung's Exynos5 SoC's. > This patches add Device Tree Support for Exynos5250. I ack 2 patches. Acked-by: Sangbeom Kim <sbkim73(a)samsung.com> Thanks, Sangbeom.

2 1

[alsa-devel] [PATCH v2 0/2] ARM: OMAP2+: HDMI: Update platform devices for audio
by Ricardo Neri 23 Nov '12

23 Nov '12

Hi Mark, Tomi, Liam, Tony, This set aims to be the version 2 of my previous submission[1] and aims to address the potential issues that Mark and Tomi described on such submission. The creation of the platform device for HDMI audio from the OMAPDSS HDMI driver that was previously submitted[2] is resubmitted to be complemented with code to remove the platform device created in /arch/arm/mach-omap2/devices.c in order to have a complete series that does not break functionality in any patch. Also, a second patch proposes new names and locations for the ASoC HDMI CPU DAI and machine drivers. This intends to give to the platform devices more descriptive names and create them from more relevant locations. This set of patches requires that commit 14840b9a83c6a56629db2ba0ec247503e975f143 Author: Ricardo Neri <ricardo.neri(a)ti.com> Date: Tue Nov 6 00:19:17 2012 -0600 OMAPDSS: HDMI: Create platform device for audio support is reverted in Tomi's git://gitorious.org/linux-omap-dss2/linux.git master Changes from v1. *Put in a single series all the patches related to platform device updates. *Now HDMI audio works correctly in every patch. *Remove reference to the TPD12S015 HDMI companion chip as the ASoC drivers are not aware of this and other chips could be used in the future. BR, Ricardo [1]. http://www.mail-archive.com/linux-omap@vger.kernel.org/msg80678.html [2]. http://www.mail-archive.com/linux-omap@vger.kernel.org/msg79795.html Ricardo Neri (2): ARM: OMAP2+: HDMI: Relocate audio platform device creation ARM: OMAP4+: HDMI: Rearrange platform devices for ASoC drivers arch/arm/mach-omap2/board-4430sdp.c | 6 +++ arch/arm/mach-omap2/board-omap4panda.c | 6 +++ arch/arm/mach-omap2/devices.c | 31 ---------------- drivers/video/omap2/dss/hdmi.c | 62 ++++++++++++++++++++++++++++++++ sound/soc/omap/omap-hdmi-card.c | 4 +- sound/soc/omap/omap-hdmi.c | 5 +-- sound/soc/omap/omap-hdmi.h | 2 - 7 files changed, 78 insertions(+), 38 deletions(-) -- 1.7.5.4

3 16

[alsa-devel] SNDRV_PCM_TRIGGER_STOP not called
by irfanks 23 Nov '12

23 Nov '12

Hello All, We are facing issue when forceful disconnection of isochronous USB ALSA device (IPOD). Problems as below : 1) SNDRV_PCM_TRIGGER_STOP in snd_usb_substream_capture_trigger() is not getting called. 2) We have tried calling snd_pcm_drain(), snd_pcm_drop() and snd_pcm_close() from application, which should STOP the PCM device as from API description. But still SNDRV_PCM_TRIGGER_STOP is not getting called. Please can you suggest the possibilities of SNDRV_PCM_TRIGGER_STOP not called in usb alsa capture driver. Also please suggest how to achieve the above ? Thanks in advance !! Response wil be appreciated !!! Best Regards,

2 1

[alsa-devel] [asoc:topic/wm8994 8/8] sound/soc/codecs/wm8994.c:3454:29: error: invalid type argument of '->' (have 'struct wm8994_pdata')
by kbuild test robot 23 Nov '12

23 Nov '12

tree: git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/wm8994 head: 0b1a5f157fd0248cb96cbd2dda3b9c5e8ab21b82 commit: 0b1a5f157fd0248cb96cbd2dda3b9c5e8ab21b82 [8/8] ASoC: wm8994: Use pdata cached in MFD driver config: make ARCH=i386 allyesconfig All error/warnings: sound/soc/codecs/wm8994.c: In function 'wm8958_default_micdet': sound/soc/codecs/wm8994.c:3454:29: error: invalid type argument of '->' (have 'struct wm8994_pdata') vim +3454 sound/soc/codecs/wm8994.c af6b6fe4 Mark Brown 2011-11-30 3448 c986564b Mark Brown 2012-03-12 3449 wm1811_jackdet_set_mode(codec, c986564b Mark Brown 2012-03-12 3450 WM1811_JACKDET_MODE_JACK); c986564b Mark Brown 2012-03-12 3451 c986564b Mark Brown 2012-03-12 3452 mutex_unlock(&wm8994->accdet_lock); c986564b Mark Brown 2012-03-12 3453 0b1a5f15 Mark Brown 2012-10-08 @3454 if (wm8994->wm8994->pdata->jd_ext_cap) 07fb9d9e Mark Brown 2012-02-21 3455 snd_soc_dapm_disable_pin(&codec->dapm, 07fb9d9e Mark Brown 2012-02-21 3456 "MICBIAS2"); af6b6fe4 Mark Brown 2011-11-30 3457 } --- 0-DAY kernel build testing backend Open Source Technology Center Fengguang Wu, Yuanhan Liu Intel Corporation

1 0

Re: [alsa-devel] [BUGFIX PATCH] ASoC: imx-sgtl5000: prevent module from being unloaded when in use
by Mark Brown 23 Nov '12

23 Nov '12

On Thu, Nov 22, 2012 at 01:31:08PM +0100, Lothar Waßmann wrote: > set the 'onwer' field of the registered snd_soc_card object to prevent > removal of the module when its resources are in use. Applied, thanks.

1 0

Re: [alsa-devel] [PATCH] ASoC: imx: add MODULE_LICENSE("GPL")
by Mark Brown 23 Nov '12

23 Nov '12

On Thu, Nov 22, 2012 at 01:31:07PM +0100, Lothar Waßmann wrote: > also set MODULE_AUTHOR and MODULE_DESCRIPTION Applied, thanks. But as ever please do write better changelogs, for example "Add MODULE_ information" or something as the subject.

1 0

Re: [alsa-devel] [PATCH 00/10] ASoC: codec: devm_ related modifications
by Mark Brown 23 Nov '12

23 Nov '12

On Thu, Nov 22, 2012 at 09:38:32AM +0530, Tushar Behera wrote: > This patchset includes devm_ related modifications for sound codecs. > > First 2 patches fix warning for freeing of devm_ allocated data. > The next patch introduces usage of devm_kzalloc() in wm8995 codec. > Following 7 patches introduce usage of devm_regmap_init_xxx() in > remaining codecs. Applied all, thanks. If you're doing stuff like this it's helpful to sort by part.

1 0

[alsa-devel] [PATCH 1/1] ASoC: Ux500: Initialise PCM from MSP probe rather than as a device
by Lee Jones 23 Nov '12

23 Nov '12

The PCM is a pseudo-device. It doesn't have any of it's own registers or hardware. It rather acts as a layer of abstraction for DMA transfers. Hence, instead of classifying it as a device in its own right, we call the initialisation from the MSP driver. Cc: Mark Brown <broonie(a)opensource.wolfsonmicro.com> Cc: Ola LILJA2 <ola.o.lilja(a)stericsson.com> Cc: alsa-devel(a)alsa-project.org Signed-off-by: Lee Jones <lee.jones(a)linaro.org> --- sound/soc/ux500/mop500.c | 4 ++-- sound/soc/ux500/ux500_msp_dai.c | 14 +++++++++++++- sound/soc/ux500/ux500_pcm.c | 19 ++++--------------- sound/soc/ux500/ux500_pcm.h | 3 +++ 4 files changed, 22 insertions(+), 18 deletions(-) diff --git a/sound/soc/ux500/mop500.c b/sound/soc/ux500/mop500.c index 54f7e25..651a52a 100644 --- a/sound/soc/ux500/mop500.c +++ b/sound/soc/ux500/mop500.c @@ -33,7 +33,7 @@ struct snd_soc_dai_link mop500_dai_links[] = { .stream_name = "ab8500_0", .cpu_dai_name = "ux500-msp-i2s.1", .codec_dai_name = "ab8500-codec-dai.0", - .platform_name = "ux500-pcm.0", + .platform_name = "ux500-msp-i2s.1", .codec_name = "ab8500-codec.0", .init = mop500_ab8500_machine_init, .ops = mop500_ab8500_ops, @@ -43,7 +43,7 @@ struct snd_soc_dai_link mop500_dai_links[] = { .stream_name = "ab8500_1", .cpu_dai_name = "ux500-msp-i2s.3", .codec_dai_name = "ab8500-codec-dai.1", - .platform_name = "ux500-pcm.0", + .platform_name = "ux500-msp-i2s.3", .codec_name = "ab8500-codec.0", .init = NULL, .ops = mop500_ab8500_ops, diff --git a/sound/soc/ux500/ux500_msp_dai.c b/sound/soc/ux500/ux500_msp_dai.c index be94bf9..66ca889 100644 --- a/sound/soc/ux500/ux500_msp_dai.c +++ b/sound/soc/ux500/ux500_msp_dai.c @@ -28,6 +28,7 @@ #include "ux500_msp_i2s.h" #include "ux500_msp_dai.h" +#include "ux500_pcm.h" static int setup_pcm_multichan(struct snd_soc_dai *dai, struct ux500_msp_config *msp_config) @@ -806,11 +807,20 @@ static int __devinit ux500_msp_drv_probe(struct platform_device *pdev) goto err_init_msp; } + ret = ux500_pcm_register_platform(pdev); + if (ret < 0) { + dev_err(&pdev->dev, + "Error: %s: Failed to register PCM platform device!\n", + __func__); + goto err_reg_plat; + } + return 0; +err_reg_plat: + snd_soc_unregister_dais(&pdev->dev, ARRAY_SIZE(ux500_msp_dai_drv)); err_init_msp: clk_put(drvdata->clk); - err_clk: devm_regulator_put(drvdata->reg_vape); @@ -821,6 +831,8 @@ static int __devexit ux500_msp_drv_remove(struct platform_device *pdev) { struct ux500_msp_i2s_drvdata *drvdata = dev_get_drvdata(&pdev->dev); + ux500_pcm_unregister_platform(pdev); + snd_soc_unregister_dais(&pdev->dev, ARRAY_SIZE(ux500_msp_dai_drv)); devm_regulator_put(drvdata->reg_vape); diff --git a/sound/soc/ux500/ux500_pcm.c b/sound/soc/ux500/ux500_pcm.c index 1a04e24..894c9f4 100644 --- a/sound/soc/ux500/ux500_pcm.c +++ b/sound/soc/ux500/ux500_pcm.c @@ -282,7 +282,7 @@ static struct snd_soc_platform_driver ux500_pcm_soc_drv = { .pcm_new = ux500_pcm_new, }; -static int __devexit ux500_pcm_drv_probe(struct platform_device *pdev) +int __devinit ux500_pcm_register_platform(struct platform_device *pdev) { int ret; @@ -296,23 +296,12 @@ static int __devexit ux500_pcm_drv_probe(struct platform_device *pdev) return 0; } +EXPORT_SYMBOL_GPL(ux500_pcm_register_platform); -static int __devinit ux500_pcm_drv_remove(struct platform_device *pdev) +int __devexit ux500_pcm_unregister_platform(struct platform_device *pdev) { snd_soc_unregister_platform(&pdev->dev); return 0; } - -static struct platform_driver ux500_pcm_driver = { - .driver = { - .name = "ux500-pcm", - .owner = THIS_MODULE, - }, - - .probe = ux500_pcm_drv_probe, - .remove = __devexit_p(ux500_pcm_drv_remove), -}; -module_platform_driver(ux500_pcm_driver); - -MODULE_LICENSE("GPL v2"); +EXPORT_SYMBOL_GPL(ux500_pcm_unregister_platform); diff --git a/sound/soc/ux500/ux500_pcm.h b/sound/soc/ux500/ux500_pcm.h index 77ed44d..76d3444 100644 --- a/sound/soc/ux500/ux500_pcm.h +++ b/sound/soc/ux500/ux500_pcm.h @@ -32,4 +32,7 @@ #define UX500_PLATFORM_PERIODS_MAX 48 #define UX500_PLATFORM_BUFFER_BYTES_MAX (2048 * PAGE_SIZE) +int ux500_pcm_register_platform(struct platform_device *pdev); +int ux500_pcm_unregister_platform(struct platform_device *pdev); + #endif -- 1.7.9.5

2 1