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

[alsa-devel] [PATCH v5 0/6] ASoC: dwc: Add device tree support to designware I2S
by Andrew Jackson 30 Dec '14

30 Dec '14

From: Andrew Jackson <Andrew.Jackson(a)arm.com> This patch set extends the DesignWare I2S driver to provide device tree support and fixes a couple of small faults. Changes v4->v5 + Remove all clk_put calls [Andrew Jackson] + Call clk_disable_unprepare [Andrew Jackson] Changes v3->v4 + Drop applied patches + Use combined function to prepare clock [Mark Brown] + Use managed clock resources to avoid clk_put [Mark Brown] + Read configuration prameters from hardware for both platform data and device tree [Mark Brown] + Re-order patch sequence [Mark Brown] + Change union name to avoid future collisions [Mark Brown] + Check return code from clk_set_rate [Mark Brown] + Check parametsrs read from hardware agaist array limits and add further comments [Mark Brown] + Add of_match_ptr [Mark Brown] Changes v2->v3 + Drop applied patch + Flush FIFOs in prepare rather than hw_params [Lars-Peter Clausen] Changes v1->v2 + Drop negative use count patch [Mark Brown] + Remove unnecessary debug print messages [Lars-Peter Clausen] + Rewrite iteration as for loop rather than do...while [Mark Brown] + Reorder patches to send fixes first [Mark Brown] + Simplify device tree code [Mark Brown] + Split device tree patch in two [Mark Brown] + Expand explanatory comment on channel configuration [Rajeev Kumar] Arnd: I've not forgotten about updating the spear entries and will submit as a separate patch. Andrew Jackson (6): ASoC: dwc: Switch to managed clock resource ASoC: dwc: Prepare clock before use ASoC: dwc: Read I2S block configuration from registers ASoC: dwc: Register components with managed interface ASoC: dwc: Add documentation for I2S DT ASoC: dwc: Add devicetree support for Designware I2S .../devicetree/bindings/sound/designware-i2s.txt | 31 +++ sound/soc/dwc/Kconfig | 1 + sound/soc/dwc/designware_i2s.c | 283 ++++++++++++++++---- 3 files changed, 263 insertions(+), 52 deletions(-) create mode 100644 Documentation/devicetree/bindings/sound/designware-i2s.txt

3 11

[alsa-devel] [PATCH] ASoC: mxs-saif: Do not store the irq number in the private structure
by Fabio Estevam 30 Dec '14

30 Dec '14

From: Fabio Estevam <fabio.estevam(a)freescale.com> The irq number is only used inside the probe function, so there is really no need to store it in the private structure. Use a local 'irq' variable to hold the the irq number instead. Signed-off-by: Fabio Estevam <fabio.estevam(a)freescale.com> --- sound/soc/mxs/mxs-saif.c | 10 +++++----- sound/soc/mxs/mxs-saif.h | 1 - 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/sound/soc/mxs/mxs-saif.c b/sound/soc/mxs/mxs-saif.c index d986508..c866ade 100644 --- a/sound/soc/mxs/mxs-saif.c +++ b/sound/soc/mxs/mxs-saif.c @@ -710,7 +710,7 @@ static int mxs_saif_probe(struct platform_device *pdev) struct device_node *np = pdev->dev.of_node; struct resource *iores; struct mxs_saif *saif; - int ret = 0; + int irq, ret = 0; struct device_node *master; if (!np) @@ -763,16 +763,16 @@ static int mxs_saif_probe(struct platform_device *pdev) if (IS_ERR(saif->base)) return PTR_ERR(saif->base); - saif->irq = platform_get_irq(pdev, 0); - if (saif->irq < 0) { - ret = saif->irq; + irq = platform_get_irq(pdev, 0); + if (irq < 0) { + ret = irq; dev_err(&pdev->dev, "failed to get irq resource: %d\n", ret); return ret; } saif->dev = &pdev->dev; - ret = devm_request_irq(&pdev->dev, saif->irq, mxs_saif_irq, 0, + ret = devm_request_irq(&pdev->dev, irq, mxs_saif_irq, 0, dev_name(&pdev->dev), saif); if (ret) { dev_err(&pdev->dev, "failed to request irq\n"); diff --git a/sound/soc/mxs/mxs-saif.h b/sound/soc/mxs/mxs-saif.h index fbaf7ba..9a4c0b2 100644 --- a/sound/soc/mxs/mxs-saif.h +++ b/sound/soc/mxs/mxs-saif.h @@ -116,7 +116,6 @@ struct mxs_saif { unsigned int mclk; unsigned int mclk_in_use; void __iomem *base; - int irq; unsigned int id; unsigned int master_id; unsigned int cur_rate; -- 1.9.1

2 1

[alsa-devel] [PATCH] ASoC: davinci-mcasp: Support for combined tx/rx interrupt line
by Peter Ujfalusi 30 Dec '14

30 Dec '14

Some SoC, like da850/OMAP-L138 uses one common interrupt request for TX/RX events. Signed-off-by: Peter Ujfalusi <peter.ujfalusi(a)ti.com> --- sound/soc/davinci/davinci-mcasp.c | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) diff --git a/sound/soc/davinci/davinci-mcasp.c b/sound/soc/davinci/davinci-mcasp.c index 30b94d4f9c5d..95eef582331b 100644 --- a/sound/soc/davinci/davinci-mcasp.c +++ b/sound/soc/davinci/davinci-mcasp.c @@ -364,6 +364,20 @@ static irqreturn_t davinci_mcasp_rx_irq_handler(int irq, void *data) return IRQ_RETVAL(handled_mask); } +static irqreturn_t davinci_mcasp_common_irq_handler(int irq, void *data) +{ + struct davinci_mcasp *mcasp = (struct davinci_mcasp *)data; + irqreturn_t ret = IRQ_NONE; + + if (mcasp->substreams[SNDRV_PCM_STREAM_PLAYBACK]) + ret = davinci_mcasp_tx_irq_handler(irq, data); + + if (mcasp->substreams[SNDRV_PCM_STREAM_CAPTURE]) + ret |= davinci_mcasp_rx_irq_handler(irq, data); + + return ret; +} + static int davinci_mcasp_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt) { @@ -1441,6 +1455,22 @@ static int davinci_mcasp_probe(struct platform_device *pdev) mcasp->dev = &pdev->dev; + irq = platform_get_irq_byname(pdev, "common"); + if (irq >= 0) { + irq_name = devm_kasprintf(&pdev->dev, GFP_KERNEL, "%s_common\n", + dev_name(&pdev->dev)); + ret = devm_request_threaded_irq(&pdev->dev, irq, NULL, + davinci_mcasp_common_irq_handler, + IRQF_ONESHOT, irq_name, mcasp); + if (ret) { + dev_err(&pdev->dev, "common IRQ request failed\n"); + goto err; + } + + mcasp->irq_request[SNDRV_PCM_STREAM_PLAYBACK] = XUNDRN; + mcasp->irq_request[SNDRV_PCM_STREAM_CAPTURE] = ROVRN; + } + irq = platform_get_irq_byname(pdev, "rx"); if (irq >= 0) { irq_name = devm_kasprintf(&pdev->dev, GFP_KERNEL, "%s_rx\n", -- 2.2.1

2 1

[alsa-devel] [RFC] ASoC: wm8962: Allow codec to work when CONFIG_PM=n
by Fabio Estevam 30 Dec '14

30 Dec '14

From: Fabio Estevam <fabio.estevam(a)freescale.com> Currently the codec does not work in the CONFIG_PM=n case because the regulators and clock are kept disabled. Allow the codec to be functional by calling wm8962_runtime_resume() when CONFIG_PM=n. Tested on a imx6sl-evk in the CONFIG_PM=y and CONFIG_PM=n cases. Signed-off-by: Fabio Estevam <fabio.estevam(a)freescale.com> --- Probably there is a better solution, hence I am sending it as RFC. Thanks sound/soc/codecs/wm8962.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/sound/soc/codecs/wm8962.c b/sound/soc/codecs/wm8962.c index d32d554..0b041c9 100644 --- a/sound/soc/codecs/wm8962.c +++ b/sound/soc/codecs/wm8962.c @@ -3546,6 +3546,9 @@ static int wm8962_set_pdata_from_of(struct i2c_client *i2c, return 0; } +static int wm8962_runtime_resume(struct device *dev); +static int wm8962_runtime_suspend(struct device *dev); + static int wm8962_i2c_probe(struct i2c_client *i2c, const struct i2c_device_id *id) { @@ -3771,6 +3774,9 @@ static int wm8962_i2c_probe(struct i2c_client *i2c, /* The drivers should power up as needed */ regulator_bulk_disable(ARRAY_SIZE(wm8962->supplies), wm8962->supplies); + if (!IS_ENABLED(CONFIG_PM)) + wm8962_runtime_resume(&i2c->dev); + return 0; err_enable: @@ -3781,11 +3787,12 @@ err: static int wm8962_i2c_remove(struct i2c_client *client) { + if (!IS_ENABLED(CONFIG_PM)) + wm8962_runtime_suspend(&client->dev); snd_soc_unregister_codec(&client->dev); return 0; } -#ifdef CONFIG_PM static int wm8962_runtime_resume(struct device *dev) { struct wm8962_priv *wm8962 = dev_get_drvdata(dev); @@ -3860,7 +3867,6 @@ static int wm8962_runtime_suspend(struct device *dev) return 0; } -#endif static struct dev_pm_ops wm8962_pm = { SET_RUNTIME_PM_OPS(wm8962_runtime_suspend, wm8962_runtime_resume, NULL) -- 1.9.1

2 1

[alsa-devel] [PATCH 1/9] ALSA: pcm_params: Remove unused add/sub functions
by Lars-Peter Clausen 30 Dec '14

30 Dec '14

Those two functions are not used anywhere and also their name is a bit to generic to be in a global header, so remove them. Signed-off-by: Lars-Peter Clausen <lars(a)metafoo.de> --- include/sound/pcm_params.h | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/include/sound/pcm_params.h b/include/sound/pcm_params.h index 6b1c78f..15760f5 100644 --- a/include/sound/pcm_params.h +++ b/include/sound/pcm_params.h @@ -325,20 +325,6 @@ static inline int snd_interval_eq(const struct snd_interval *i1, const struct sn i1->max == i2->max && i1->openmax == i2->openmax; } -static inline unsigned int add(unsigned int a, unsigned int b) -{ - if (a >= UINT_MAX - b) - return UINT_MAX; - return a + b; -} - -static inline unsigned int sub(unsigned int a, unsigned int b) -{ - if (a > b) - return a - b; - return 0; -} - #define params_access(p) ((__force snd_pcm_access_t)\ snd_mask_min(hw_param_mask_c((p), SNDRV_PCM_HW_PARAM_ACCESS))) #define params_format(p) ((__force snd_pcm_format_t)\ -- 1.8.0

2 9

[alsa-devel] [GIT PULL] ASoC updates for v3.20
by Mark Brown 30 Dec '14

30 Dec '14

The following changes since commit 97bf6af1f928216fd6c5a66e8a57bfa95a659672: Linux 3.19-rc1 (2014-12-20 17:08:50 -0800) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-v3.19-rc2 for you to fetch changes up to 1b9f1ae8907cb13d3d2538d98385b2744c9490d5: Merge remote-tracking branches 'asoc/topic/samsung' and 'asoc/topic/wm8750' into asoc-next (2014-12-24 12:56:48 +0000) ---------------------------------------------------------------- ASoC: Updates for v3.20 Nothing too exciting here yet, a small optimization for DAPM from Lars-Peter and a few small bits and pieces for drivers but nothing that really stands out. ---------------------------------------------------------------- Andrew Jackson (4): ASoC: dwc: Remove unnecessary debug messages and tests ASoC: dwc: Reorder code in preparation for DT support ASoC: dwc: Ensure FIFOs are flushed to prevent channel swap ASoC: dwc: Iterate over all channels Bard Liao (3): ASoC: rt5670: set platform data by dmi ASoC: rt5670: Add runtime PM support ASoC: rt5670: Keep sysclk on if JD func is used Fang, Yang A (1): ASoC: rt5677: fixed rt5677_dsp_vad_put rt5677_dsp_vad_get panic Jarkko Nikula (4): ASoC: Intel: Add I2C dependency to two new machines ASoC: Intel: Fix BYTCR firmware name ASoC: Intel: Fix BYTCR machine driver MODULE_ALIAS ASoC: pcm: Fix vague codec and cpu DAI prepare error messages Jianqun Xu (3): ASoC: rockchip: i2s: fix error defination of transmit data level ASoC: rockchip: i2s: fix maxburst of dma data to 4 ASoC: rockchip: i2s: set TDL and RDL to 16 samples Jie Yang (2): ASoC: Intel: correct the fixed free block allocation ASoC: Intel: Store the entry_point read from FW file Lars-Peter Clausen (4): ASoC: wm8750: Make VREF a VMID instead of OUTPUT widget ASoC: wm8750: Cleanup manual bias level transitions ASoC: dapm: Don't mark MICBIAS widgets as auto non-connected ASoC: dapm: Simplify fully route card handling Mark Brown (9): ASoC: dapm: Remove snd_soc_of_parse_audio_routing() due to deferred probe Merge remote-tracking branch 'asoc/fix/dapm' into asoc-linus Merge remote-tracking branches 'asoc/fix/dwc', 'asoc/fix/intel', 'asoc/fix/rockchip' and 'asoc/fix/rt5677' into asoc-linus Merge remote-tracking branch 'asoc/topic/dapm' into asoc-next Merge remote-tracking branch 'asoc/topic/pcm' into asoc-next Merge remote-tracking branch 'asoc/topic/pcm512x' into asoc-next Merge remote-tracking branches 'asoc/topic/dwc', 'asoc/topic/intel' and 'asoc/topic/omap' into asoc-next Merge remote-tracking branches 'asoc/topic/pxa', 'asoc/topic/rockchip', 'asoc/topic/rt5670' and 'asoc/topic/rt5677' into asoc-next Merge remote-tracking branches 'asoc/topic/samsung' and 'asoc/topic/wm8750' into asoc-next Oder Chiou (3): ASoC: rt5677: Revise the filter powers ASoC: rt5677: Add the ASRC support ASoC: rt5677: Adjust the routing of "PLL1" Peter Rosin (1): ASoC: pcm512x: Also support PCM514x devices Takashi Iwai (1): ASoC: intel: Remove superfluous backslash in Kconfig Wei Yongjun (1): ASoC: Intel: fix sparse non static symbol warnings Wolfram Sang (4): ASoC: intel: sst: drop owner assignment from platform_drivers ASoC: omap: drop owner assignment from platform_drivers ASoC: pxa: drop owner assignment from platform_drivers ASoC: samsung: drop owner assignment from platform_drivers kbuild test robot (2): ASoC: Intel: fix platform_no_drv_owner.cocci warnings ASoC: Intel: fix platform_no_drv_owner.cocci warnings .../devicetree/bindings/sound/pcm512x.txt | 3 +- include/sound/soc-dapm.h | 1 - sound/soc/codecs/pcm512x-i2c.c | 4 + sound/soc/codecs/pcm512x-spi.c | 4 + sound/soc/codecs/rt5670.c | 38 ++++ sound/soc/codecs/rt5677.c | 204 ++++++++++++++++++--- sound/soc/codecs/wm8750.c | 2 +- sound/soc/dwc/designware_i2s.c | 135 +++++++------- sound/soc/intel/Kconfig | 8 +- sound/soc/intel/bytcr_dpcm_rt5640.c | 3 +- sound/soc/intel/cht_bsw_rt5672.c | 1 - sound/soc/intel/sst-firmware.c | 7 +- sound/soc/intel/sst/sst_acpi.c | 7 +- sound/soc/omap/omap-hdmi-audio.c | 1 - sound/soc/pxa/spitz.c | 1 - sound/soc/rockchip/rockchip_i2s.c | 8 +- sound/soc/rockchip/rockchip_i2s.h | 2 +- sound/soc/samsung/arndale_rt5631.c | 1 - sound/soc/soc-core.c | 17 +- sound/soc/soc-dapm.c | 105 ++--------- sound/soc/soc-pcm.c | 7 +- 21 files changed, 349 insertions(+), 210 deletions(-)

2 1

[alsa-devel] ASoC fixes for v3.19
by Mark Brown 30 Dec '14

30 Dec '14

The following changes since commit 97bf6af1f928216fd6c5a66e8a57bfa95a659672: Linux 3.19-rc1 (2014-12-20 17:08:50 -0800) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-fix-v3.19-rc2 for you to fetch changes up to f2ec1d3b25776051a9205289dab12e9401d33f8a: Merge remote-tracking branches 'asoc/fix/dwc', 'asoc/fix/intel', 'asoc/fix/rockchip' and 'asoc/fix/rt5677' into asoc-linus (2014-12-24 12:56:32 +0000) ---------------------------------------------------------------- ASoC: Fixes for v3.19 A few fixes for v3.19, a few driver specifics and one core fix which fixes a boot crash on OMAP if deferred probing kicks in due to attempting to modify static data. ---------------------------------------------------------------- Andrew Jackson (2): ASoC: dwc: Ensure FIFOs are flushed to prevent channel swap ASoC: dwc: Iterate over all channels Fang, Yang A (1): ASoC: rt5677: fixed rt5677_dsp_vad_put rt5677_dsp_vad_get panic Jarkko Nikula (3): ASoC: Intel: Add I2C dependency to two new machines ASoC: Intel: Fix BYTCR firmware name ASoC: Intel: Fix BYTCR machine driver MODULE_ALIAS Jianqun Xu (2): ASoC: rockchip: i2s: fix error defination of transmit data level ASoC: rockchip: i2s: fix maxburst of dma data to 4 Jie Yang (1): ASoC: Intel: correct the fixed free block allocation Mark Brown (3): ASoC: dapm: Remove snd_soc_of_parse_audio_routing() due to deferred probe Merge remote-tracking branch 'asoc/fix/dapm' into asoc-linus Merge remote-tracking branches 'asoc/fix/dwc', 'asoc/fix/intel', 'asoc/fix/rockchip' and 'asoc/fix/rt5677' into asoc-linus sound/soc/codecs/rt5677.c | 9 ++++--- sound/soc/dwc/designware_i2s.c | 49 +++++++++++++++++++++++-------------- sound/soc/intel/Kconfig | 4 +-- sound/soc/intel/bytcr_dpcm_rt5640.c | 2 +- sound/soc/intel/sst-firmware.c | 6 ++++- sound/soc/intel/sst/sst_acpi.c | 2 +- sound/soc/rockchip/rockchip_i2s.c | 4 +-- sound/soc/rockchip/rockchip_i2s.h | 2 +- sound/soc/soc-core.c | 14 ++++------- 9 files changed, 52 insertions(+), 40 deletions(-)

2 1

[alsa-devel] [PATCH 0/2 v7] Digidesign Mbox 1 enhancements
by Damien Zammit 30 Dec '14

30 Dec '14

Revision 7. Damien Zammit (2): ALSA: usb-audio: Add mixer control for Digidesign Mbox 1 clock source ALSA: usb-audio: Add duplex mode for Digidesign Mbox 1 and enable mixer sound/usb/mixer_maps.c | 9 ++++ sound/usb/mixer_quirks.c | 129 +++++++++++++++++++++++++++++++++++++++++++++++ sound/usb/quirks-table.h | 41 +++++++++++---- 3 files changed, 170 insertions(+), 9 deletions(-) -- 1.9.1

3 5

[alsa-devel] [PATCH 1/2] [ALSA] add SNDRV_PCM_TRIGGER_DRAIN trigger
by libin.yang＠intel.com 30 Dec '14

30 Dec '14

From: Libin Yang <libin.yang(a)intel.com> Add SNDRV_PCM_TRIGGER_DRAIN trigger for pcm drain. Some audio devices require notification of drain events in order to properly drain and shutdown an audio stream. Signed-off-by: Libin Yang <libin.yang(a)intel.com> --- include/sound/pcm.h | 1 + sound/core/pcm_native.c | 5 +++++ 2 files changed, 6 insertions(+) diff --git a/include/sound/pcm.h b/include/sound/pcm.h index 8bb00a2..bb77aa3 100644 --- a/include/sound/pcm.h +++ b/include/sound/pcm.h @@ -109,6 +109,7 @@ struct snd_pcm_ops { #define SNDRV_PCM_TRIGGER_PAUSE_RELEASE 4 #define SNDRV_PCM_TRIGGER_SUSPEND 5 #define SNDRV_PCM_TRIGGER_RESUME 6 +#define SNDRV_PCM_TRIGGER_DRAIN 7 #define SNDRV_PCM_POS_XRUN ((snd_pcm_uframes_t)-1) diff --git a/sound/core/pcm_native.c b/sound/core/pcm_native.c index 166d59c..15fe682 100644 --- a/sound/core/pcm_native.c +++ b/sound/core/pcm_native.c @@ -1517,6 +1517,11 @@ static int snd_pcm_do_drain_init(struct snd_pcm_substream *substream, int state) { struct snd_pcm_runtime *runtime = substream->runtime; if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) { + + if (substream->runtime->trigger_master == substream) + substream->ops->trigger(substream, + SNDRV_PCM_TRIGGER_DRAIN); + switch (runtime->status->state) { case SNDRV_PCM_STATE_PREPARED: /* start playback stream if possible */ -- 1.9.1

3 8

[alsa-devel] [PATCH] mfd: wm8994: set mfd id-base for regulator devs creation to avoid conflicts
by Inha Song 30 Dec '14

30 Dec '14

After commit: 6e3f62f0793e ("mfd: core: Fix platform-device id generation") We must set the id base when register a duplicate name of mfd_cell. If not, duplicate filename error was reported. - sysfs: cannot create duplicate filename '/devices/.../wm8994-ldo' Signed-off-by: Inha Song <ideal.song(a)samsung.com> --- drivers/mfd/wm8994-core.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/mfd/wm8994-core.c b/drivers/mfd/wm8994-core.c index e6fab94..47cd3cc 100644 --- a/drivers/mfd/wm8994-core.c +++ b/drivers/mfd/wm8994-core.c @@ -36,12 +36,12 @@ static const struct mfd_cell wm8994_regulator_devs[] = { { .name = "wm8994-ldo", - .id = 1, + .id = 0, .pm_runtime_no_callbacks = true, }, { .name = "wm8994-ldo", - .id = 2, + .id = 1, .pm_runtime_no_callbacks = true, }, }; @@ -344,7 +344,7 @@ static int wm8994_device_init(struct wm8994 *wm8994, int irq) dev_set_drvdata(wm8994->dev, wm8994); /* Add the on-chip regulators first for bootstrapping */ - ret = mfd_add_devices(wm8994->dev, -1, + ret = mfd_add_devices(wm8994->dev, 0, wm8994_regulator_devs, ARRAY_SIZE(wm8994_regulator_devs), NULL, 0, NULL); -- 2.0.0.390.gcb682f8

1 0