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

[alsa-devel] [PATCH] Revert "ALSA: hda - Shut up pins at power-saving mode with Conexnat codecs"
by David Henningsson 09 Jan '13

09 Jan '13

This reverts commit 697c373e34613609cb5450f98b91fefb6e910588. The original patch was meant to remove clicking, but in fact caused even more clicking instead. Thanks to c4pp4 for doing most of the work with this bug. BugLink: https://bugs.launchpad.net/bugs/886975 Signed-off-by: David Henningsson <david.henningsson(a)canonical.com> --- @c4pp4: Is this consistent with what you have done to reduce the clicking? Will you recommend upstream Linux to take the patch below? sound/pci/hda/patch_conexant.c | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/sound/pci/hda/patch_conexant.c b/sound/pci/hda/patch_conexant.c index 60890bf..dd798c3 100644 --- a/sound/pci/hda/patch_conexant.c +++ b/sound/pci/hda/patch_conexant.c @@ -558,24 +558,12 @@ static int conexant_build_controls(struct hda_codec *codec) return 0; } -#ifdef CONFIG_PM -static int conexant_suspend(struct hda_codec *codec) -{ - snd_hda_shutup_pins(codec); - return 0; -} -#endif - static const struct hda_codec_ops conexant_patch_ops = { .build_controls = conexant_build_controls, .build_pcms = conexant_build_pcms, .init = conexant_init, .free = conexant_free, .set_power_state = conexant_set_power, -#ifdef CONFIG_PM - .suspend = conexant_suspend, -#endif - .reboot_notify = snd_hda_shutup_pins, }; #ifdef CONFIG_SND_HDA_INPUT_BEEP @@ -4405,10 +4393,6 @@ static const struct hda_codec_ops cx_auto_patch_ops = { .init = cx_auto_init, .free = conexant_free, .unsol_event = snd_hda_jack_unsol_event, -#ifdef CONFIG_PM - .suspend = conexant_suspend, -#endif - .reboot_notify = snd_hda_shutup_pins, }; /* -- 1.7.9.5

2 7

[alsa-devel] [PATCH v5 2/2] ASoC: mxs-saif: Remove platform data
by Fabio Estevam 08 Jan '13

08 Jan '13

From: Fabio Estevam <fabio.estevam(a)freescale.com> All MXS users have been converted to device tree and the board files have been removed. No need to keep platform data in the driver. Signed-off-by: Fabio Estevam <fabio.estevam(a)freescale.com> Acked-by: Dong Aisheng <dong.aisheng(a)linaro.org> Acked-by: Shawn Guo <shawn.guo(a)linaro.org> --- Changes since v4: - Rebased against fix/mxs branch and fix file removal Changes since v3: - Remove pdev->id check Changes since v2: - Also remove 'if (np)' as dt is the only probing mechanism. Changes since v1: - Also remove mxs_saif_platform_data and sound/saif.h, which was used only to include mxs_saif_platform_data definition. include/sound/saif.h | 16 ---------------- sound/soc/mxs/mxs-saif.c | 44 ++++++++++++++++---------------------------- 2 files changed, 16 insertions(+), 44 deletions(-) delete mode 100644 include/sound/saif.h diff --git a/include/sound/saif.h b/include/sound/saif.h deleted file mode 100644 index f22f3e1..0000000 --- a/include/sound/saif.h +++ /dev/null @@ -1,16 +0,0 @@ -/* - * Copyright 2011 Freescale Semiconductor, Inc. All Rights Reserved. - * - * 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. - */ - -#ifndef __SOUND_SAIF_H__ -#define __SOUND_SAIF_H__ - -struct mxs_saif_platform_data { - bool master_mode; /* if true use master mode */ - int master_id; /* id of the master if in slave mode */ -}; -#endif diff --git a/sound/soc/mxs/mxs-saif.c b/sound/soc/mxs/mxs-saif.c index c294fbb..05dbadb 100644 --- a/sound/soc/mxs/mxs-saif.c +++ b/sound/soc/mxs/mxs-saif.c @@ -32,7 +32,6 @@ #include <sound/pcm.h> #include <sound/pcm_params.h> #include <sound/soc.h> -#include <sound/saif.h> #include <asm/mach-types.h> #include <mach/hardware.h> #include <mach/mxs.h> @@ -660,43 +659,32 @@ static int __devinit mxs_saif_probe(struct platform_device *pdev) struct device_node *np = pdev->dev.of_node; struct resource *iores, *dmares; struct mxs_saif *saif; - struct mxs_saif_platform_data *pdata; struct pinctrl *pinctrl; int ret = 0; + struct device_node *master; - - if (!np && pdev->id >= ARRAY_SIZE(mxs_saif)) + if (!np) return -EINVAL; saif = devm_kzalloc(&pdev->dev, sizeof(*saif), GFP_KERNEL); if (!saif) return -ENOMEM; - if (np) { - struct device_node *master; - saif->id = of_alias_get_id(np, "saif"); - if (saif->id < 0) - return saif->id; - /* - * If there is no "fsl,saif-master" phandle, it's a saif - * master. Otherwise, it's a slave and its phandle points - * to the master. - */ - master = of_parse_phandle(np, "fsl,saif-master", 0); - if (!master) { - saif->master_id = saif->id; - } else { - saif->master_id = of_alias_get_id(master, "saif"); - if (saif->master_id < 0) - return saif->master_id; - } + saif->id = of_alias_get_id(np, "saif"); + if (saif->id < 0) + return saif->id; + /* + * If there is no "fsl,saif-master" phandle, it's a saif + * master. Otherwise, it's a slave and its phandle points + * to the master. + */ + master = of_parse_phandle(np, "fsl,saif-master", 0); + if (!master) { + saif->master_id = saif->id; } else { - saif->id = pdev->id; - pdata = pdev->dev.platform_data; - if (pdata && !pdata->master_mode) - saif->master_id = pdata->master_id; - else - saif->master_id = saif->id; + saif->master_id = of_alias_get_id(master, "saif"); + if (saif->master_id < 0) + return saif->master_id; } if (saif->master_id < 0 || saif->master_id >= ARRAY_SIZE(mxs_saif)) { -- 1.7.9.5

2 5

[alsa-devel] [PATCH] ALSA: pxa2xx: fix ac97 cold reset for pxa27x
by Mike Dunn 07 Jan '13

07 Jan '13

Currently, ac97 reset is broken on PXA27x. Through trial-and-error (the pxa270 developer's manual is mostly incoherent on the topic of ac97 reset), I fixed it by setting the WARM_RST bit in the GCR register at the end of the cold reset, and then skipping the warm reset if the link is already up and running. It appears that setting the WARM_RST bit is a necessary final step during cold reset. I think that the PXA25x and PXA3xx may currently be working correctly because WARM_RST is set within the warm reset routine, and the codec drivers always follow a cold reset with a warm reset during their initialization. so this combination effectively completes a cold reset. This doesn't work on the PXA27x because its warm reset routine contains additional code for working around a hardware bug in the warm reset sequence, which causes the reset sequence to fail. I only have a PXA27x platform for testing, so this patch only affects the PXA27x. Signed-off-by: Mike Dunn <mikedunn(a)newsguy.com> --- As an aside... I don't understand how always following a cold reset with a warm reset (as the codec drivers do) makes sense. Unless I'm mistaken, a warm reset can only occur on a link that's currently down. All the timing diagrams I've seen for reset (warm and cold) show the link down (BITCLK stopped) at the start of the reset sequence. The SYNC line is used to effect the warm reset, and SYNC is already pulsing every frame on a link that's up and running, so it seems physically impossible to do a warm reset on a link that's currently up. But I'm no ac97 expert, so I may just be embarassing myself. Any insight gratefully received. Thanks! sound/arm/pxa2xx-ac97-lib.c | 15 +++++++++++++++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/sound/arm/pxa2xx-ac97-lib.c b/sound/arm/pxa2xx-ac97-lib.c index 6fc0ae9..aa242ff 100644 --- a/sound/arm/pxa2xx-ac97-lib.c +++ b/sound/arm/pxa2xx-ac97-lib.c @@ -138,6 +138,10 @@ static inline void pxa_ac97_warm_pxa27x(void) { gsr_bits = 0; + /* don't attempt warm reset if link is up */ + if (GSR & (GSR_SCR | GSR_PCR)) + return; + /* warm reset broken on Bulverde, so manually keep AC97 reset high */ pxa27x_assert_ac97reset(reset_gpio, 1); udelay(10); @@ -148,6 +152,8 @@ static inline void pxa_ac97_warm_pxa27x(void) static inline void pxa_ac97_cold_pxa27x(void) { + unsigned int timeout; + GCR &= GCR_COLD_RST; /* clear everything but nCRST */ GCR &= ~GCR_COLD_RST; /* then assert nCRST */ @@ -159,6 +165,15 @@ static inline void pxa_ac97_cold_pxa27x(void) clk_disable(ac97conf_clk); GCR = GCR_COLD_RST; udelay(50); + + /* + * Setting the WARM_RST bit is a necessary part of a cold reset, at + * least on the buggy and poorly documented PXA27x ac97 controller. + */ + GCR |= GCR_WARM_RST; + timeout = 100; /* wait for the codec-ready bit to be set */ + while (!((GSR | gsr_bits) & (GSR_PCR | GSR_SCR)) && timeout--) + mdelay(1); } #endif -- 1.7.8.6

3 6

Re: [alsa-devel] Missing exactly 3 of 8 audio packets?
by Daniel Griscom 06 Jan '13

06 Jan '13

Dear Daniel (M) and Clemens, I'd like to conclude this thread if I could. It seems clear that configuring our USB audio card's descriptor with a bInterval value of 1 (a packet every microframe) was too aggressive, and was part of what caused the packet stuttering issue. (Right?) We almost certainly should increase bInterval; I'll try 2, and if that seems stable will probably go to 3 (4 microframes) for some margin. However, I'm still concerned that the symptoms don't sound like a simple starvation issue, where I'd expect to have a missing frame every once in a while. Instead, we were seeing reliable operation interspersed with periods where lots of packets were being consistently omitted. To me, this suggests that there's some other issue going on, either in the USB stack or in the USB hardware itself. Given that, I'm worried that increasing bInterval will just move the symptoms around, making it harder to duplicate but not truly becoming robust. Any thoughts on this? Should I try chasing the issue into the Linux USB development mailing list, or will they just say "set bInterval to a larger value!"? Thanks, Dan -- Daniel T. Griscom griscom(a)suitable.com Suitable Systems http://www.suitable.com/ 1 Centre Street, Suite 204 (781) 665-0053 Wakefield, MA 01880-2400

2 1

Re: [alsa-devel] sound: usb-audio: [PATCH]x2 full capture/playback/spdif support for Digidesign Mbox 2
by Damien Zammit 04 Jan '13

04 Jan '13

On 23 December 2012 14:36, Damien Zammit <damien.zammit(a)gmail.com> wrote: > I have fixed both the kernel panic and the broken pipe issue with midi. > I believe this completes the patch, I just don't know how to submit it to you. Please find attached the patch you accepted previously: (Version 5) Plus an incremental patch (Version 5-2) ***Please ignore patch (Version 6)*** I hope this is the easiest way for you to review it. It compiles, runs, and works on the hardware now without errors. Thanks, Damien

2 6

[alsa-devel] [PATCH 0/2] ASoC: palm27x: machine driver fixes and updates
by Mike Dunn 02 Jan '13

02 Jan '13

Hi, A fix and an update to the palm27x machine driver. Happy New Year! Mike Dunn (2): ASoC: palm27x: fix widgets and routes in dai_link init ASoC: palm27x: register card in platform_driver probe sound/soc/pxa/palm27x.c | 36 +++++++++--------------------------- 1 files changed, 9 insertions(+), 27 deletions(-) -- 1.7.8.6

2 3

[alsa-devel] [PATCH 0/8] ASoC: OMAP-twl4030: Add support for more3 boards
by Peter Ujfalusi 02 Jan '13

02 Jan '13

Hello, This series will update the omap-twl4030 machine driver and the twl4030 codec driver to enable support for more boards. With this change we can remove two more machine drivers: zoom2 and sdp3430 since they will be supported by the common omap-twl4030 machine driver. The arch/arm/mach-omap2/ patches will go as a separate series to avoid cross tree issues. Regards, Peter --- Peter Ujfalusi (8): ASoC: twl4030: Correct the support for Voice port ASoC: zoom2: No need to configure the Voice port anymore ASoC: sdp3430: No need to configure the Voice port anymore ASoC: twl4030: Convert MICBIAS to SUPPLY widget ASoC: twl4030: Configure extmute pinmux when the dedicated pin is in use ASoC: sdp3430: No need to configure pin mux for extmute ASoC: omap-twl4030: Add support for routing, voice port and jack detect ASoC: OMAP: Remove obsolete machine drivers for Zoom2 and SDP3430 .../devicetree/bindings/sound/omap-twl4030.txt | 46 ++++ sound/soc/codecs/twl4030.c | 71 ++++-- sound/soc/omap/Kconfig | 19 +- sound/soc/omap/Makefile | 4 - sound/soc/omap/omap-twl4030.c | 204 ++++++++++++++- sound/soc/omap/omap3pandora.c | 8 +- sound/soc/omap/sdp3430.c | 278 --------------------- sound/soc/omap/zoom2.c | 207 --------------- 8 files changed, 304 insertions(+), 533 deletions(-) delete mode 100644 sound/soc/omap/sdp3430.c delete mode 100644 sound/soc/omap/zoom2.c -- 1.8.0.2

2 9

[alsa-devel] [PATCH] ASoC: pxa2xx-ac97: fix oops when codec_driver probe runs
by Mike Dunn 02 Jan '13

02 Jan '13

This patch fixes a kernel oops that occurs when the snd_soc_codec_driver probe function runs. The exception occurs when the codec calls soc_ac97_ops.reset(). The pxa2xx-ac97's clocks have not been initialized at this point, but the cpu's reset function (pxa_ac97_cold_pxa27x()) expects them to be, resulting in a NULL pointer dereference in the pxa's clk_enable(). The pxa2xx-ac97's clocks are initialized in pxa2xx_ac97_hw_probe(). To ensure that the clocks are initialized when the codec_driver probe runs, the calls to pxa2xx_ac97_hw_{probe,remove}() are moved from the cpu's dai_driver probe/remove functions to those of its platform_driver. This is necessary because snd_soc_instantiate_card() always calls the codec_driver probe before any of the dai_drivers' probes. This change leaves nothing in the cpu's dai_driver probe/remove functions, so those functions are eliminated. Tested on the pxa27x; palmtreo680 machine using the wm9712 codec. Signed-off-by: Mike Dunn <mikedunn(a)newsguy.com> --- I'm new to the ASoC code. Experts, please have a look. Thanks! sound/soc/pxa/pxa2xx-ac97.c | 38 +++++++++++++++++++------------------- 1 files changed, 19 insertions(+), 19 deletions(-) diff --git a/sound/soc/pxa/pxa2xx-ac97.c b/sound/soc/pxa/pxa2xx-ac97.c index 4b0a009..940e9f4 100644 --- a/sound/soc/pxa/pxa2xx-ac97.c +++ b/sound/soc/pxa/pxa2xx-ac97.c @@ -104,17 +104,6 @@ static int pxa2xx_ac97_resume(struct snd_soc_dai *dai) #define pxa2xx_ac97_resume NULL #endif -static int pxa2xx_ac97_probe(struct snd_soc_dai *dai) -{ - return pxa2xx_ac97_hw_probe(to_platform_device(dai->dev)); -} - -static int pxa2xx_ac97_remove(struct snd_soc_dai *dai) -{ - pxa2xx_ac97_hw_remove(to_platform_device(dai->dev)); - return 0; -} - static int pxa2xx_ac97_hw_params(struct snd_pcm_substream *substream, struct snd_pcm_hw_params *params, struct snd_soc_dai *cpu_dai) @@ -184,8 +173,6 @@ static struct snd_soc_dai_driver pxa_ac97_dai_driver[] = { { .name = "pxa2xx-ac97", .ac97_control = 1, - .probe = pxa2xx_ac97_probe, - .remove = pxa2xx_ac97_remove, .suspend = pxa2xx_ac97_suspend, .resume = pxa2xx_ac97_resume, .playback = { @@ -236,21 +223,34 @@ EXPORT_SYMBOL_GPL(soc_ac97_ops); static int pxa2xx_ac97_dev_probe(struct platform_device *pdev) { + int ret; + if (pdev->id != -1) { dev_err(&pdev->dev, "PXA2xx has only one AC97 port.\n"); return -ENXIO; } - /* Punt most of the init to the SoC probe; we may need the machine - * driver to do interesting things with the clocking to get us up - * and running. - */ - return snd_soc_register_dais(&pdev->dev, pxa_ac97_dai_driver, - ARRAY_SIZE(pxa_ac97_dai_driver)); + ret = snd_soc_register_dais(&pdev->dev, pxa_ac97_dai_driver, + ARRAY_SIZE(pxa_ac97_dai_driver)); + if (ret < 0) { + dev_err(&pdev->dev, "snd_soc_register_dais failed with %d\n", + ret); + return ret; + } + + ret = pxa2xx_ac97_hw_probe(pdev); + if (ret < 0) { + dev_err(&pdev->dev, "pxa2xx_ac97_hw_probe failed with %d\n", + ret); + snd_soc_unregister_dais(&pdev->dev, + ARRAY_SIZE(pxa_ac97_dai_driver)); + } + return ret; } static int pxa2xx_ac97_dev_remove(struct platform_device *pdev) { + pxa2xx_ac97_hw_remove(pdev); snd_soc_unregister_dais(&pdev->dev, ARRAY_SIZE(pxa_ac97_dai_driver)); return 0; } -- 1.7.8.6

2 1

[alsa-devel] [PATCH 0/6] ARM: OMAP3: Audio support update for omap-twl4030
by Peter Ujfalusi 31 Dec '12

31 Dec '12

Hello, omap-twl4030 ASoC machine driver can support more boards with the following series: http://mailman.alsa-project.org/pipermail/alsa-devel/2012-December/058119.h… Rework the omap-twl4030 audio support code in arch/arm/mach-omap2/ to take advantages of this. The same machine driver can support the zoom2 and sdp3430 boards as well. Regards, Peter --- Peter Ujfalusi (6): ARM: OMAP: 3430sdp: Enable extmute functionality for audio ARM: OMAP: zoom: Zoom2 does not have extmute functionality ARM: OMAP2+: twl-common: Add default twl4030 audio configuration ARM: OMAP2+: twl-common: Allow boards to customize the twl4030 audio setup ARM: OMAP: zoom: Audio support via the common omap-twl4030 machine driver ARM: OMAP: sdp3430: Audio support via the common omap-twl4030 machine driver arch/arm/mach-omap2/board-3430sdp.c | 20 +++++++++++++++++++ arch/arm/mach-omap2/board-cm-t35.c | 2 +- arch/arm/mach-omap2/board-devkit8000.c | 2 +- arch/arm/mach-omap2/board-igep0020.c | 2 +- arch/arm/mach-omap2/board-omap3beagle.c | 2 +- arch/arm/mach-omap2/board-omap3evm.c | 2 +- arch/arm/mach-omap2/board-overo.c | 2 +- arch/arm/mach-omap2/board-zoom-peripherals.c | 29 ++++++++++++++++++++-------- arch/arm/mach-omap2/twl-common.c | 17 ++++++++++------ arch/arm/mach-omap2/twl-common.h | 3 ++- 10 files changed, 60 insertions(+), 21 deletions(-) -- 1.8.0.2

1 6

Re: [alsa-devel] Help with adding mic bias to omap-twl4030 audio driver
by Peter Ujfalusi 31 Dec '12

31 Dec '12

Hi Neil, On 12/18/2012 12:18 AM, NeilBrown wrote: > Hi Peter, > I recently discovered your omap-twl4030 driver and thought to use it to > replace the hacked-around driver I'm using for the GTA04 mobile phone > platform. > > It works fine for playing sound out the speaker, but the microphone doesn't > work - I think because there is no mechanism to enable the microphone bias. > > I copied some bits out of the old driver - just enough to make it work. > However this probably makes the code specific to my platform. > > So I'm wondering if you could give me some guidelines on how this "should" > be done. Should these tables be passed in with the platform_data (I haven't > graduated to device-tree yet)? I'm going to send a series soon to update the omap-twl4030 machine driver. This update going to add support for microphones and all other routings as well. With the upcoming series the omap-twl4030 will support zoom2 and sdp3430. I'll CC you with the series so you can take a look if it is going to help with your board. > > My 'hacked-up' driver has a few more entries in this table than just those > few, but I wanted to strip it down to what is needed for the main microphone > to work. > > Thanks for any help you can provide, > NeilBrown > > P.S is there some good reason to use the name > struct omap_tw4030_pdata > rather than > struct omap_twl4030_pdata > > (i.e. with the 'l')?? > > diff --git a/sound/soc/omap/omap-twl4030.c b/sound/soc/omap/omap-twl4030.c > index 3b97b879..aa4d866 100644 > --- a/sound/soc/omap/omap-twl4030.c > +++ b/sound/soc/omap/omap-twl4030.c > @@ -87,6 +87,32 @@ static struct snd_soc_ops omap_twl4030_ops = { > .hw_params = omap_twl4030_hw_params, > }; > > +static const struct snd_soc_dapm_widget gta04_dapm_widgets[] = { > + SND_SOC_DAPM_MIC("Internal Mic", NULL), > +}; > + > +static const struct snd_soc_dapm_route audio_map[] = { > + {"MAINMIC", NULL, "Mic Bias 1"}, > + {"Mic Bias 1", NULL, "Internal Mic"}, > +}; > + > +static int omap3gta04_init(struct snd_soc_pcm_runtime *runtime) > +{ > + int ret; > + struct snd_soc_codec *codec = runtime->codec; > + struct snd_soc_dapm_context *dapm = &codec->dapm; > + > + ret = snd_soc_dapm_new_controls(dapm, gta04_dapm_widgets, > + ARRAY_SIZE(gta04_dapm_widgets)); > + if (ret < 0) > + return ret; > + > + snd_soc_dapm_add_routes(dapm, audio_map, > + ARRAY_SIZE(audio_map)); > + > + return snd_soc_dapm_sync(dapm); > +} > + > /* Digital audio interface glue - connects codec <--> CPU */ > static struct snd_soc_dai_link omap_twl4030_dai_links[] = { > { > @@ -97,6 +123,7 @@ static struct snd_soc_dai_link omap_twl4030_dai_links[] = { > .platform_name = "omap-pcm-audio", > .codec_name = "twl4030-codec", > .ops = &omap_twl4030_ops, > + .init = &omap3gta04_init, > }, > }; > > -- Péter

1 0