June 2017 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH 0/3] ASoC: add/use asoc_simple_card_of_canonicalize_cpu()
by Kuninori Morimoto 22 Jun '17

22 Jun '17

Hi Mark These fixes audio graph cards DAI counting bug. Simple Card side doesn't have this issue, but Audio Graph Card side has it. Kuninori Morimoto (3): ASoC: simple-card-utils: add asoc_simple_card_of_canonicalize_cpu() ASoC: audio-graph-card: use asoc_simple_card_of_canonicalize_cpu() ASoC: audio-graph-scu-card: use asoc_simple_card_of_canonicalize_cpu() include/sound/simple_card_utils.h | 1 + sound/soc/generic/audio-graph-card.c | 4 +--- sound/soc/generic/audio-graph-scu-card.c | 4 +--- sound/soc/generic/simple-card-utils.c | 20 ++++++++++++++++++++ 4 files changed, 23 insertions(+), 6 deletions(-) -- 1.9.1

1 4

[alsa-devel] [asoc:topic/pxa 2/2] /tmp/ccrgbnNE.s:1411: Error: unknown mnemonic `ldmia' -- `ldmia x26, {r0, r1, r2, r3}'
by kbuild test robot 21 Jun '17

21 Jun '17

tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/pxa head: d89e69c0f985392e708b54c92b322163da820cb9 commit: d89e69c0f985392e708b54c92b322163da820cb9 [2/2] ASoC: pxa: add COMPILE_TEST on SND_MMP_SOC config: arm64-allmodconfig (attached as .config) compiler: aarch64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 reproduce: wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout d89e69c0f985392e708b54c92b322163da820cb9 # save the attached .config to linux build tree make.cross ARCH=arm64 All errors (new ones prefixed by >>): /tmp/ccrgbnNE.s: Assembler messages: >> /tmp/ccrgbnNE.s:1411: Error: unknown mnemonic `ldmia' -- `ldmia x26,{r0,r1,r2,r3}' >> /tmp/ccrgbnNE.s:1412: Error: unknown mnemonic `stmia' -- `stmia x19!,{r0,r1,r2,r3}' >> /tmp/ccrgbnNE.s:1618: Error: unknown mnemonic `ldmia' -- `ldmia x19!,{r0,r1,r2,r3}' >> /tmp/ccrgbnNE.s:1619: Error: unknown mnemonic `stmia' -- `stmia x26,{r0,r1,r2,r3}' --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

2 1

[alsa-devel] [PATCH 0/2] Add ZTE ZTE zx-aud96p22 audio codec driver
by Shawn Guo 21 Jun '17

21 Jun '17

From: Shawn Guo <shawn.guo(a)linaro.org> This is a pick-up of Baoyou's zx-aud96p22 audio codec driver submission [1], with quite some significant rework as below. - Instead of hard-coding the GPIO used to control external amplifier, use generic devm_gpiod_get_optional() call to retrieve the GPIO defined in device tree. - Rather than directly using I2C bus calls, use REGMAP_I2C interface which is more common for accessing codec register via I2C bus. - Use snd_kcontrol_new infrastructural for Volume and Mute control. - Use DAPM widget and route for various power bits control. - A lot of code cleanups [1] https://lkml.org/lkml/2017/2/15/170 Baoyou Xie (2): ASoC: add bindings for ZTE zx-aud96p22 audio codec ASoC: zx_aud96p22: add ZTE ZX AUD96P22 codec driver .../devicetree/bindings/sound/zte,zx-aud96p22.txt | 28 ++ sound/soc/codecs/Kconfig | 5 + sound/soc/codecs/Makefile | 2 + sound/soc/codecs/zx_aud96p22.c | 425 +++++++++++++++++++++ 4 files changed, 460 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/zte,zx-aud96p22.txt create mode 100644 sound/soc/codecs/zx_aud96p22.c -- 1.9.1

2 11

[alsa-devel] [asoc:topic/pxa 2/2] sound/soc/pxa/mmp-pcm.c:130:3: error: implicit declaration of function '__phys_to_pfn'
by kbuild test robot 21 Jun '17

21 Jun '17

tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/pxa head: d89e69c0f985392e708b54c92b322163da820cb9 commit: d89e69c0f985392e708b54c92b322163da820cb9 [2/2] ASoC: pxa: add COMPILE_TEST on SND_MMP_SOC config: ia64-allyesconfig (attached as .config) compiler: ia64-linux-gcc (GCC) 6.2.0 reproduce: wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout d89e69c0f985392e708b54c92b322163da820cb9 # save the attached .config to linux build tree make.cross ARCH=ia64 All errors (new ones prefixed by >>): sound/soc/pxa/mmp-pcm.c: In function 'mmp_pcm_mmap': >> sound/soc/pxa/mmp-pcm.c:130:3: error: implicit declaration of function '__phys_to_pfn' [-Werror=implicit-function-declaration] __phys_to_pfn(runtime->dma_addr) + off, ^~~~~~~~~~~~~ cc1: some warnings being treated as errors vim +/__phys_to_pfn +130 sound/soc/pxa/mmp-pcm.c 7a824e214 Zhangfei Gao 2012-06-11 114 ac581e60d Lars-Peter Clausen 2013-03-22 115 dma_data.dma_res = r; ac581e60d Lars-Peter Clausen 2013-03-22 116 dma_data.ssp_id = cpu_dai->id; 7a824e214 Zhangfei Gao 2012-06-11 117 7c1c1d4a7 Lars-Peter Clausen 2013-04-15 118 return snd_dmaengine_pcm_open_request_chan(substream, filter, 7c1c1d4a7 Lars-Peter Clausen 2013-04-15 119 &dma_data); 7a824e214 Zhangfei Gao 2012-06-11 120 } 7a824e214 Zhangfei Gao 2012-06-11 121 7a824e214 Zhangfei Gao 2012-06-11 122 static int mmp_pcm_mmap(struct snd_pcm_substream *substream, 7a824e214 Zhangfei Gao 2012-06-11 123 struct vm_area_struct *vma) 7a824e214 Zhangfei Gao 2012-06-11 124 { 7a824e214 Zhangfei Gao 2012-06-11 125 struct snd_pcm_runtime *runtime = substream->runtime; 7a824e214 Zhangfei Gao 2012-06-11 126 unsigned long off = vma->vm_pgoff; 7a824e214 Zhangfei Gao 2012-06-11 127 7a824e214 Zhangfei Gao 2012-06-11 128 vma->vm_page_prot = pgprot_noncached(vma->vm_page_prot); 7a824e214 Zhangfei Gao 2012-06-11 129 return remap_pfn_range(vma, vma->vm_start, 7a824e214 Zhangfei Gao 2012-06-11 @130 __phys_to_pfn(runtime->dma_addr) + off, 7a824e214 Zhangfei Gao 2012-06-11 131 vma->vm_end - vma->vm_start, vma->vm_page_prot); 7a824e214 Zhangfei Gao 2012-06-11 132 } 7a824e214 Zhangfei Gao 2012-06-11 133 60e10d2fb Lars-Peter Clausen 2013-05-14 134 static struct snd_pcm_ops mmp_pcm_ops = { 7a824e214 Zhangfei Gao 2012-06-11 135 .open = mmp_pcm_open, 7c1c1d4a7 Lars-Peter Clausen 2013-04-15 136 .close = snd_dmaengine_pcm_close_release_chan, 7a824e214 Zhangfei Gao 2012-06-11 137 .ioctl = snd_pcm_lib_ioctl, 7a824e214 Zhangfei Gao 2012-06-11 138 .hw_params = mmp_pcm_hw_params, :::::: The code at line 130 was first introduced by commit :::::: 7a824e214e25a49442fe868dac0af8a904b24f58 ASoC: mmp: add audio dma support :::::: TO: Zhangfei Gao <zhangfei.gao(a)marvell.com> :::::: CC: Mark Brown <broonie(a)opensource.wolfsonmicro.com> --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

1 0

[alsa-devel] [PATCH 1/3] ASoC: fsl: mpc5200_dma: remove unused psc_dma
by Kuninori Morimoto 21 Jun '17

21 Jun '17

linux/sound/soc/fsl/mpc5200_dma.c:305:18: warning: unused variable \ psc_dma’ [-Wunused-variable] Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/fsl/mpc5200_dma.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/fsl/mpc5200_dma.c b/sound/soc/fsl/mpc5200_dma.c index 93885d9..cd8ccaba 100644 --- a/sound/soc/fsl/mpc5200_dma.c +++ b/sound/soc/fsl/mpc5200_dma.c @@ -302,7 +302,6 @@ static int psc_dma_new(struct snd_soc_pcm_runtime *rtd) struct snd_card *card = rtd->card->snd_card; struct snd_soc_dai *dai = rtd->cpu_dai; struct snd_pcm *pcm = rtd->pcm; - struct psc_dma *psc_dma = snd_soc_dai_get_drvdata(rtd->cpu_dai); size_t size = psc_dma_hardware.buffer_bytes_max; int rc; -- 1.9.1

2 1

[alsa-devel] [PATCH] ASoC: pxa: add COMPILE_TEST on SND_PXA2XX_SOC
by Kuninori Morimoto 21 Jun '17

21 Jun '17

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> It doesn't use asm header. We can add COMPILE_TEST Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/pxa/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/pxa/Kconfig b/sound/soc/pxa/Kconfig index 823b5a2..960744e 100644 --- a/sound/soc/pxa/Kconfig +++ b/sound/soc/pxa/Kconfig @@ -1,6 +1,6 @@ config SND_PXA2XX_SOC tristate "SoC Audio for the Intel PXA2xx chip" - depends on ARCH_PXA + depends on ARCH_PXA || COMPILE_TEST select SND_PXA2XX_LIB help Say Y or M if you want to add support for codecs attached to -- 1.9.1

2 1

[alsa-devel] [PATCH v2 0/2] ASoC: stm32: add SPDIFRX driver
by olivier moysan 21 Jun '17

21 Jun '17

This patch-set handles the SPDIFRX on STM32 platforms. The SPDIFRX peripheral, is designed to receive an S/PDIF flow compliant with IEC-60958 and IEC-61937 standards. SPDIFRX uses two DMA channels: - one DMA channel for S/PDIF data stream. - one DMA channel for control flow (channel status and user data bits). DMA is used for control flow instead of interrupts to get better performances, as channel status and user data are retrieved byte per byte. v1 -> v2: - fix merge issue olivier moysan (2): ASoC: stm32: Add DT bindings for SPDIFRX interface ASoC: stm32: Add SPDIFRX support .../devicetree/bindings/sound/st,stm32-spdifrx.txt | 56 ++ sound/soc/stm/Kconfig | 10 + sound/soc/stm/Makefile | 4 + sound/soc/stm/stm32_spdifrx.c | 998 +++++++++++++++++++++ 4 files changed, 1068 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/st,stm32-spdifrx.txt create mode 100644 sound/soc/stm/stm32_spdifrx.c -- 1.9.1

2 3

[alsa-devel] [PATCH] ASoC: rsnd: fixup unsigned expression compared with zero: main_rate
by Kuninori Morimoto 21 Jun '17

21 Jun '17

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> This patch fixes this WARNING sound/soc/sh/rcar/ssi.c:285:5-14: WARNING: Unsigned expression\ compared with zero: main_rate < 0 Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- Mark, Julia I think this patch solves reported issue, but my compiler and smatch doesn't indicate it. Thus, I couldn't confirm it. sound/soc/sh/rcar/core.c | 2 +- sound/soc/sh/rcar/rsnd.h | 2 +- sound/soc/sh/rcar/ssi.c | 8 ++++---- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/sound/soc/sh/rcar/core.c b/sound/soc/sh/rcar/core.c index 4892c0a..409bac3 100644 --- a/sound/soc/sh/rcar/core.c +++ b/sound/soc/sh/rcar/core.c @@ -763,7 +763,7 @@ static int rsnd_soc_hw_rule(struct rsnd_priv *priv, struct snd_interval *baseline, struct snd_interval *iv) { struct snd_interval p; - int rate; + unsigned int rate; int i; snd_interval_any(&p); diff --git a/sound/soc/sh/rcar/rsnd.h b/sound/soc/sh/rcar/rsnd.h index 9428d4e..99c5761 100644 --- a/sound/soc/sh/rcar/rsnd.h +++ b/sound/soc/sh/rcar/rsnd.h @@ -680,7 +680,7 @@ void rsnd_ssi_parse_hdmi_connection(struct rsnd_priv *priv, void rsnd_parse_connect_ssi(struct rsnd_dai *rdai, struct device_node *playback, struct device_node *capture); -int rsnd_ssi_clk_query(struct rsnd_priv *priv, +unsigned int rsnd_ssi_clk_query(struct rsnd_priv *priv, int param1, int param2, int *idx); /* diff --git a/sound/soc/sh/rcar/ssi.c b/sound/soc/sh/rcar/ssi.c index c8956c3..46feddd 100644 --- a/sound/soc/sh/rcar/ssi.c +++ b/sound/soc/sh/rcar/ssi.c @@ -208,14 +208,14 @@ u32 rsnd_ssi_multi_slaves_runtime(struct rsnd_dai_stream *io) return 0; } -int rsnd_ssi_clk_query(struct rsnd_priv *priv, +unsigned int rsnd_ssi_clk_query(struct rsnd_priv *priv, int param1, int param2, int *idx) { int ssi_clk_mul_table[] = { 1, 2, 4, 8, 16, 6, 12, }; int j, ret; - int main_rate; + unsigned int main_rate; for (j = 0; j < ARRAY_SIZE(ssi_clk_mul_table); j++) { @@ -245,7 +245,7 @@ int rsnd_ssi_clk_query(struct rsnd_priv *priv, return main_rate; } - return -EINVAL; + return 0; } static int rsnd_ssi_master_clk_start(struct rsnd_mod *mod, @@ -282,7 +282,7 @@ static int rsnd_ssi_master_clk_start(struct rsnd_mod *mod, } main_rate = rsnd_ssi_clk_query(priv, rate, chan, &idx); - if (main_rate < 0) { + if (!main_rate) { dev_err(dev, "unsupported clock rate\n"); return -EIO; } -- 1.9.1

2 1

[alsa-devel] [PATCH alsa-lib 0/4] Add the support for explicit sync of appl_ptr
by Takashi Iwai 21 Jun '17

21 Jun '17

Hi, this is a patchset for alsa-lib corresponding to the patchset I've sent for kernel. The first two patches do the actual fixes, while the last two are merely cleanups / optimizations. thanks, Takashi === Takashi Iwai (4): pcm: hw: Add the support for explicit sync of appl_ptr pcm: hw: Call USER_PVERSION ioctl at open pcm: hw: Remove superfluous call of snd_pcm_set_appl_ptr() pcm: hw: Reduce redundant sync_ptr() calls include/sound/asound.h | 4 +++- src/pcm/pcm_hw.c | 47 ++++++++++++++++++++++++++++++++--------------- 2 files changed, 35 insertions(+), 16 deletions(-) -- 2.13.1

2 5

Re: [alsa-devel] Default procedure for sespend and resume state
by Takashi Iwai 21 Jun '17

21 Jun '17

On Wed, 21 Jun 2017 09:13:23 +0200, Kailang wrote: > > Hi Takashi, > > This patch will be more stable during sespend and resume state. It's "suspend" :) The code changes look OK, but please be more specific what problem this patch is trying to address. I suppose here "stability" meaning the pop-noise issue or such, not about the functionality of suspend/resume? thanks, Takashi

1 0