August 2013 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH] ASoC: compress: Use power efficient workqueue
by Mark Brown 12 Aug '13

12 Aug '13

From: Mark Brown <broonie(a)linaro.org> There is no need for the power down work to be done on a per CPU workqueue especially considering the fairly long delay before powerdown. Signed-off-by: Mark Brown <broonie(a)linaro.org> --- sound/soc/soc-compress.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/sound/soc/soc-compress.c b/sound/soc/soc-compress.c index d220150..53c9ecd 100644 --- a/sound/soc/soc-compress.c +++ b/sound/soc/soc-compress.c @@ -149,8 +149,9 @@ static int soc_compr_free(struct snd_compr_stream *cstream) SND_SOC_DAPM_STREAM_STOP); } else { rtd->pop_wait = 1; - schedule_delayed_work(&rtd->delayed_work, - msecs_to_jiffies(rtd->pmdown_time)); + queue_delayed_work(system_power_efficient_wq, + &rtd->delayed_work, + msecs_to_jiffies(rtd->pmdown_time)); } } else { /* capture streams can be powered down now */ -- 1.8.4.rc1

2 1

[alsa-devel] BeagleBoard driver development (PCM3168)
by Michael Jordan 12 Aug '13

12 Aug '13

Okay, before I went too far I thought to download archives and found that Klimann has been working on this, but last post I see is April 2013. I picked through and got what I think are the last discussed of the 3 files omap-pcm3168.c pcm3168.c and pcm3168.h. I did not mean to cross paths with anybody. I will compare these to what I have worked up and see what I can contribute. I am a bit confused about one thing though. When I began this (before I knew I might be stepping on somebody else's work) I downloaded 3.10 sources from kernel.org. I have previously and again grepped through the sources and I find no mention of pcm3168. Has this project just not made it that far? And do I have the latest versions (plucked from the text archives, or are they in a git somewhere that I don't know about? Michael

1 0

[alsa-devel] Hello
by Michael Jordan 11 Aug '13

11 Aug '13

Hello all, I am involved in a university project using a TI OMAPL138 EXPKIT (Logic PD Zoom Board) to create a sound processing box. I am tasked with developing the code to replace the TLV320AIC3106 with a PCM3168A due to the greater ADC/DAC count. We have already accomplished the hardware end, and I have been exercising my linux skills at trying to put this together. I found this list and subscribed, and I am glad I sat and listened for a bit before I said anything. 457 patches in the last 10 days! My fingers would fall off! I will get together what I have been working on (2 files pcm3168a.c and pcm3168a.h) and post them. I have some questions, obviously, but I will follow up with them in a bit. If someone could help me along it would be greatly appreciated. Or if I can just provide some relevant information, I would be happy to do that as well. Thanks, Michael

1 0

[alsa-devel] [PATCH] ASoC: doc: Add documentation to MAINTAINERS patterns
by Mark Brown 11 Aug '13

11 Aug '13

From: Mark Brown <broonie(a)linaro.org> Signed-off-by: Mark Brown <broonie(a)linaro.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 427a930..07e2bc8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -7710,6 +7710,7 @@ T: git git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git L: alsa-devel(a)alsa-project.org (moderated for non-subscribers) W: http://alsa-project.org/main/index.php/ASoC S: Supported +F: Documentation/sound/alsa/soc/ F: sound/soc/ F: include/sound/soc* -- 1.8.4.rc1

1 0

[alsa-devel] [PATCH 1/16] sound/soc/pxa/mioa701_wm9713.c: Avoid using ARRAY_AND_SIZE(e) as a function argument
by Julia Lawall 11 Aug '13

11 Aug '13

From: Julia Lawall <Julia.Lawall(a)lip6.fr> Replace ARRAY_AND_SIZE(e) in function argument position to avoid hiding the arity of the called function. The semantic match that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression e,f; @@ f(..., - ARRAY_AND_SIZE(e) + e,ARRAY_SIZE(e) ,...) // </smpl> Signed-off-by: Julia Lawall <Julia.Lawall(a)lip6.fr> --- Not compiled. sound/soc/pxa/mioa701_wm9713.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/sound/soc/pxa/mioa701_wm9713.c b/sound/soc/pxa/mioa701_wm9713.c index 97b711e..20fdce6 100644 --- a/sound/soc/pxa/mioa701_wm9713.c +++ b/sound/soc/pxa/mioa701_wm9713.c @@ -133,10 +133,11 @@ static int mioa701_wm9713_init(struct snd_soc_pcm_runtime *rtd) unsigned short reg; /* Add mioa701 specific widgets */ - snd_soc_dapm_new_controls(dapm, ARRAY_AND_SIZE(mioa701_dapm_widgets)); + snd_soc_dapm_new_controls(dapm, mioa701_dapm_widgets, + ARRAY_SIZE(mioa701_dapm_widgets)); /* Set up mioa701 specific audio path audio_mapnects */ - snd_soc_dapm_add_routes(dapm, ARRAY_AND_SIZE(audio_map)); + snd_soc_dapm_add_routes(dapm, audio_map, ARRAY_SIZE(audio_map)); /* Prepare GPIO8 for rear speaker amplifier */ reg = codec->driver->read(codec, AC97_GPIO_CFG);

2 1

[alsa-devel] [PATCH] ASoC: cs4271: fix dapm routing table
by Daniel Mack 11 Aug '13

11 Aug '13

The widgets are names "AINA" and "AINB", which is also what the datasheets calls the terminals. Fix the routing table to match these names. Signed-off-by: Daniel Mack <zonque(a)gmail.com> --- sound/soc/codecs/cs4271.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/codecs/cs4271.c b/sound/soc/codecs/cs4271.c index 35a6a67..a20f1bb 100644 --- a/sound/soc/codecs/cs4271.c +++ b/sound/soc/codecs/cs4271.c @@ -184,8 +184,8 @@ SND_SOC_DAPM_OUTPUT("AOUTB-"), }; static const struct snd_soc_dapm_route cs4271_dapm_routes[] = { - { "Capture", NULL, "AINL" }, - { "Capture", NULL, "AINR" }, + { "Capture", NULL, "AINA" }, + { "Capture", NULL, "AINB" }, { "AOUTA+", NULL, "Playback" }, { "AOUTA-", NULL, "Playback" }, -- 1.8.3.1

2 1

[alsa-devel] [PATCH] usb-audio: Fix invalid volume resolution for Logitech HD Webcam C525
by Maksim A. Boyko 11 Aug '13

11 Aug '13

Add the volume control quirk for avoiding the kernel warning for the Logitech HD Webcam C525 as in the similar commit 36691e1be6ec551eef4a5225f126a281f8c051c2 (ALSA: usb-audio: Fix invalid volume resolution for Logitech HD Webcam c310) for the Logitech HD Webcam C310. Reported-by: Maksim Boyko <maksim.a.boyko(a)gmail.com> Tested-by: Maksim Boyko <maksim.a.boyko(a)gmail.com> Cc: <stable(a)vger.kernel.org> # 3.10.5+ Signed-off-by: Maksim Boyko <maksim.a.boyko(a)gmail.com> -- sound/usb/mixer.c | 1 + 1 file changed, 1 insertion(+) --- linux-3.10.5-orig/sound/usb/mixer.c 2013-08-04 12:51:49.000000000 +0400 +++ linux-3.10.5/sound/usb/mixer.c 2013-08-06 21:12:42.274197287 +0400 @@ -888,6 +888,7 @@ static void volume_control_quirks(struct case USB_ID(0x046d, 0x081b): /* HD Webcam c310 */ case USB_ID(0x046d, 0x081d): /* HD Webcam c510 */ case USB_ID(0x046d, 0x0825): /* HD Webcam c270 */ + case USB_ID(0x046d, 0x0826): /* HD Webcam c525 */ case USB_ID(0x046d, 0x0991): /* Most audio usb devices lie about volume resolution. * Most Logitech webcams have res = 384. -- WBR, Maksim Boyko

2 1

[alsa-devel] [PATCH 1/2] ASoC: cs4271: Convert to table based control init
by Mark Brown 11 Aug '13

11 Aug '13

From: Mark Brown <broonie(a)linaro.org> Signed-off-by: Mark Brown <broonie(a)linaro.org> --- sound/soc/codecs/cs4271.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/sound/soc/codecs/cs4271.c b/sound/soc/codecs/cs4271.c index 03036b3..65ad56c 100644 --- a/sound/soc/codecs/cs4271.c +++ b/sound/soc/codecs/cs4271.c @@ -576,8 +576,7 @@ static int cs4271_probe(struct snd_soc_codec *codec) CS4271_MODE2_MUTECAEQUB, CS4271_MODE2_MUTECAEQUB); - return snd_soc_add_codec_controls(codec, cs4271_snd_controls, - ARRAY_SIZE(cs4271_snd_controls)); + return 0; } static int cs4271_remove(struct snd_soc_codec *codec) @@ -596,6 +595,9 @@ static struct snd_soc_codec_driver soc_codec_dev_cs4271 = { .remove = cs4271_remove, .suspend = cs4271_soc_suspend, .resume = cs4271_soc_resume, + + .controls = cs4271_snd_controls, + .num_controls = ARRAY_SIZE(cs4271_snd_controls), }; #if defined(CONFIG_SPI_MASTER) -- 1.8.4.rc1

3 5

[alsa-devel] [PATCH] ASoC: max9768: Add DAPM support
by Mark Brown 11 Aug '13

11 Aug '13

From: Mark Brown <broonie(a)linaro.org> This makes it possible to hook the device into a more complex board and ensures it will continue to work with non-DAPM support removed from the core. Signed-off-by: Mark Brown <broonie(a)linaro.org> --- sound/soc/codecs/max9768.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/sound/soc/codecs/max9768.c b/sound/soc/codecs/max9768.c index a6ac231..31f9156 100644 --- a/sound/soc/codecs/max9768.c +++ b/sound/soc/codecs/max9768.c @@ -118,6 +118,18 @@ static const struct snd_kcontrol_new max9768_mute[] = { SOC_SINGLE_BOOL_EXT("Playback Switch", 0, max9768_get_gpio, max9768_set_gpio), }; +static const struct snd_soc_dapm_widget max9768_dapm_widgets[] = { +SND_SOC_DAPM_INPUT("IN"), + +SND_SOC_DAPM_OUTPUT("OUT+"), +SND_SOC_DAPM_OUTPUT("OUT-"), +}; + +static const struct snd_soc_dapm_route max9768_dapm_routes[] = { + { "OUT+", NULL, "IN" }, + { "OUT-", NULL, "IN" }, +}; + static int max9768_probe(struct snd_soc_codec *codec) { struct max9768 *max9768 = snd_soc_codec_get_drvdata(codec); @@ -148,6 +160,10 @@ static struct snd_soc_codec_driver max9768_codec_driver = { .probe = max9768_probe, .controls = max9768_volume, .num_controls = ARRAY_SIZE(max9768_volume), + .dapm_widgets = max9768_dapm_widgets, + .num_dapm_widgets = ARRAY_SIZE(max9768_dapm_widgets), + .dapm_routes = max9768_dapm_routes, + .num_dapm_routes = ARRAY_SIZE(max9768_dapm_routes), }; static const struct regmap_config max9768_i2c_regmap_config = { -- 1.8.4.rc1

1 0

[alsa-devel] [PATCH] ASoC: ak5386: Add DAPM support
by Mark Brown 11 Aug '13

11 Aug '13

From: Mark Brown <broonie(a)linaro.org> This makes it possible to hook the device into a more complex board and ensures it will continue to work with non-DAPM support removed from the core. Signed-off-by: Mark Brown <broonie(a)linaro.org> --- sound/soc/codecs/ak5386.c | 17 ++++++++++++++++- 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/sound/soc/codecs/ak5386.c b/sound/soc/codecs/ak5386.c index 1f30398..72e953b 100644 --- a/sound/soc/codecs/ak5386.c +++ b/sound/soc/codecs/ak5386.c @@ -22,7 +22,22 @@ struct ak5386_priv { int reset_gpio; }; -static struct snd_soc_codec_driver soc_codec_ak5386; +static const struct snd_soc_dapm_widget ak5386_dapm_widgets[] = { +SND_SOC_DAPM_INPUT("AINL"), +SND_SOC_DAPM_INPUT("AINR"), +}; + +static const struct snd_soc_dapm_route ak5386_dapm_routes[] = { + { "Capture", NULL, "AINL" }, + { "Capture", NULL, "AINR" }, +}; + +static struct snd_soc_codec_driver soc_codec_ak5386 = { + .dapm_widgets = ak5386_dapm_widgets, + .num_dapm_widgets = ARRAY_SIZE(ak5386_dapm_widgets), + .dapm_routes = ak5386_dapm_routes, + .num_dapm_routes = ARRAY_SIZE(ak5386_dapm_routes), +}; static int ak5386_set_dai_fmt(struct snd_soc_dai *codec_dai, unsigned int format) -- 1.8.4.rc1

1 0