February 2018 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH v2] ASoC: tlv320aic31xx: Fix master mode clock I2S bus clocks
by Peter Ujfalusi 13 Feb '18

13 Feb '18

In the reset state of the codec we do not have complete playback or capture routes. The audio playback/capture will not work due to missing clock signals on the I2S bus if PLL, MDAC/NDAC/DAC MADC/NADC/ADC is powered down. To make sure that even if all output/input is disconnected the codec is generating clocks, we need to have valid DAPM route in every case to power up the must needed parts of the codec. I have verified that switching DAC (during playback) or ADC (during capture) will stop the I2S clocks, so the only solution is to connect the 'Off' routes as well to output/input. The routes only going to be added if the codec is configured as clock master. Tested on am43x-epos-evm with aic3111 codec in master mode. Signed-off-by: Peter Ujfalusi <peter.ujfalusi(a)ti.com> --- Hi, Changes since v1: - Only apply the master mode DAPM routes when the codec is clock master - comments added to explain the need. Regards, Peter sound/soc/codecs/tlv320aic31xx.c | 46 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 46 insertions(+) diff --git a/sound/soc/codecs/tlv320aic31xx.c b/sound/soc/codecs/tlv320aic31xx.c index 858cb8be445f..15de89318ff5 100644 --- a/sound/soc/codecs/tlv320aic31xx.c +++ b/sound/soc/codecs/tlv320aic31xx.c @@ -166,6 +166,7 @@ struct aic31xx_priv { unsigned int sysclk; u8 p_div; int rate_div_line; + bool master_dapm_route_applied; }; struct aic31xx_rate_divs { @@ -670,6 +671,29 @@ aic310x_audio_map[] = { {"SPK", NULL, "SPK ClassD"}, }; +/* + * Always connected DAPM routes for codec clock master modes. + * If the codec is the master on the I2S bus, we need to power on components + * to have valid DAC_CLK and also the DACs and ADC for playback/capture. + * Otherwise the codec will not generate clocks on the bus. + */ +static const struct snd_soc_dapm_route +common31xx_cm_audio_map[] = { + {"DAC Left Input", "Off", "DAC IN"}, + {"DAC Right Input", "Off", "DAC IN"}, + + {"HPL", NULL, "DAC Left"}, + {"HPR", NULL, "DAC Right"}, +}; + +static const struct snd_soc_dapm_route +aic31xx_cm_audio_map[] = { + {"MIC1LP P-Terminal", "Off", "MIC1LP"}, + {"MIC1RP P-Terminal", "Off", "MIC1RP"}, + {"MIC1LM P-Terminal", "Off", "MIC1LM"}, + {"MIC1LM M-Terminal", "Off", "MIC1LM"}, +}; + static int aic31xx_add_controls(struct snd_soc_codec *codec) { int ret = 0; @@ -916,6 +940,7 @@ static int aic31xx_set_dai_fmt(struct snd_soc_dai *codec_dai, unsigned int fmt) { struct snd_soc_codec *codec = codec_dai->codec; + struct aic31xx_priv *aic31xx = snd_soc_codec_get_drvdata(codec); u8 iface_reg1 = 0; u8 iface_reg2 = 0; u8 dsp_a_val = 0; @@ -992,6 +1017,27 @@ static int aic31xx_set_dai_fmt(struct snd_soc_dai *codec_dai, AIC31XX_BCLKINV_MASK, iface_reg2); + /* Add the needed DAPM route(s) for codec clock master modes, once */ + if (iface_reg1 & (AIC31XX_BCLK_MASTER | AIC31XX_WCLK_MASTER) && + !aic31xx->master_dapm_route_applied) { + struct snd_soc_dapm_context *dapm = snd_soc_codec_get_dapm(codec); + int ret; + + pr_err("%s: trying to add master routes\n", __func__); + ret = snd_soc_dapm_add_routes(dapm, common31xx_cm_audio_map, + ARRAY_SIZE(common31xx_cm_audio_map)); + if (!ret && !(aic31xx->codec_type & DAC31XX_BIT)) + ret = snd_soc_dapm_add_routes(dapm, + aic31xx_cm_audio_map, + ARRAY_SIZE(aic31xx_cm_audio_map)); + + pr_err("%s: master routes added with %d\n", __func__, ret); + if (ret) + return ret; + + aic31xx->master_dapm_route_applied = true; + } + return 0; } -- Peter Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

1 1

[alsa-devel] pyalsa support for Python 3
by Maurizio Berti 13 Feb '18

13 Feb '18

Hello, I used pyalsa's alsaseq module for a project of mine which used Python 2. Now I'm switching to Python 3 and I'd need to use the alsaseq module with that version. Unfortunally, it seems that Python 3 support has not been added yet. I found this message: http://linux-audio.4202.n7.nabble.com/Python3-PyAlsa-Build-fails-td83590.ht… It is older than the latest release of pyalsa, but, even after trying to follow the instructions, I still get errors and cannot build pyalsa for Python3: while the "pyalsa/alsacontrol.c:184:6: error: ‘struct pyalsacontrol’ has no member named ‘ob_type’" error is gone, it still doesn't build. The latest message reports: error: command 'i686-pc-linux-gnu-gcc' failed with exit status 1 Is there any chance that pyalsa gets an update for it? Thank you, Maurizio -- È difficile avere una convinzione precisa quando si parla delle ragioni del cuore. - "Sostiene Pereira", Antonio Tabucchi http://www.jidesk.net

2 1

[alsa-devel] [asoc:topic/rt5514 1/1] sound/soc/codecs/rt5514-spi.c:477:70: error: 'DRV_NAME' undeclared; did you mean 'IRQ_NONE'?
by kbuild test robot 13 Feb '18

13 Feb '18

tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/rt5514 head: 04e8262ac8c3f59599ea6dfe5cd1a039828ea4ad commit: 04e8262ac8c3f59599ea6dfe5cd1a039828ea4ad [1/1] ASoC: rt5514: replace codec to component config: xtensa-allyesconfig (attached as .config) compiler: xtensa-linux-gcc (GCC) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout 04e8262ac8c3f59599ea6dfe5cd1a039828ea4ad # save the attached .config to linux build tree make.cross ARCH=xtensa All errors (new ones prefixed by >>): sound/soc/codecs/rt5514-spi.c: In function 'rt5514_resume': >> sound/soc/codecs/rt5514-spi.c:477:70: error: 'DRV_NAME' undeclared (first use in this function); did you mean 'IRQ_NONE'? struct snd_soc_component *component = snd_soc_lookup_component(dev, DRV_NAME); ^~~~~~~~ IRQ_NONE sound/soc/codecs/rt5514-spi.c:477:70: note: each undeclared identifier is reported only once for each function it appears in vim +477 sound/soc/codecs/rt5514-spi.c 474 475 static int __maybe_unused rt5514_resume(struct device *dev) 476 { > 477 struct snd_soc_component *component = snd_soc_lookup_component(dev, DRV_NAME); 478 struct rt5514_dsp *rt5514_dsp = 479 snd_soc_component_get_drvdata(component); 480 int irq = to_spi_device(dev)->irq; 481 u8 buf[8]; 482 483 if (device_may_wakeup(dev)) 484 disable_irq_wake(irq); 485 486 if (rt5514_dsp) { 487 if (rt5514_dsp->substream) { 488 rt5514_spi_burst_read(RT5514_IRQ_CTRL, (u8 *)&buf, 489 sizeof(buf)); 490 if (buf[0] & RT5514_IRQ_STATUS_BIT) 491 rt5514_schedule_copy(rt5514_dsp); 492 } 493 } 494 495 return 0; 496 } 497 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

2 1

[alsa-devel] [PATCH v2 0/9] add UniPhier audio system support
by Katsuhiro Suzuki 13 Feb '18

13 Feb '18

This series adds support for Socionext audio system for UniPhier LD11/LD20 SoCs. This driver supports I2S output for Line-In, Line-Out and S/PDIF output. UniPhier AIO DAI driver provides sound devices such as I2S, S/PDIF. Since the AIO has mixed register map for those I/Os, it is hard to split register areas for each sound devices. --- Changes in v2: - Add comments to aiodma_irq() - Add members to struct uniphier_aio_sub and add methods for compress audio - Expose clocking to userspace - Fix bad name 'srcport' to 'src' - Split DMA, DAI patches from large one - Validate parameters in hw_params() - Add error checks - Fix typo in error messages - Change license comment style to C++ from C Katsuhiro Suzuki (9): ASoC: uniphier: add DT bindings documentation for UniPhier AIO ASoC: uniphier: add support for UniPhier AIO common driver ASoC: uniphier: add support for UniPhier AIO DMA driver ASoC: uniphier: add support for UniPhier AIO CPU DAI driver ASoC: uniphier: add support for UniPhier AIO compress audio ASoC: uniphier: add support for UniPhier LD11/LD20 AIO driver arm64: dts: uniphier: add sound node for UniPhier arm64: dts: uniphier: add speaker out for UniPhier LD11/LD20 boards arm64: dts: uniphier: add compress audio out for UniPhier LD11/LD20 .../devicetree/bindings/sound/uniphier,aio.txt | 36 + .../boot/dts/socionext/uniphier-ld11-global.dts | 120 +++ arch/arm64/boot/dts/socionext/uniphier-ld11.dtsi | 31 + .../boot/dts/socionext/uniphier-ld20-global.dts | 120 +++ arch/arm64/boot/dts/socionext/uniphier-ld20.dtsi | 35 + sound/soc/uniphier/Kconfig | 22 + sound/soc/uniphier/Makefile | 6 + sound/soc/uniphier/aio-compress.c | 440 ++++++++ sound/soc/uniphier/aio-core.c | 1104 ++++++++++++++++++++ sound/soc/uniphier/aio-cpu.c | 570 ++++++++++ sound/soc/uniphier/aio-dma.c | 318 ++++++ sound/soc/uniphier/aio-ld11.c | 431 ++++++++ sound/soc/uniphier/aio-reg.h | 462 ++++++++ sound/soc/uniphier/aio.h | 355 +++++++ 14 files changed, 4050 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/uniphier,aio.txt create mode 100644 sound/soc/uniphier/aio-compress.c create mode 100644 sound/soc/uniphier/aio-core.c create mode 100644 sound/soc/uniphier/aio-cpu.c create mode 100644 sound/soc/uniphier/aio-dma.c create mode 100644 sound/soc/uniphier/aio-ld11.c create mode 100644 sound/soc/uniphier/aio-reg.h create mode 100644 sound/soc/uniphier/aio.h -- 2.15.0

3 16

[alsa-devel] [asoc:topic/wm9712 1/1] sound/soc//codecs/wm9712.c:648:18: error: implicit declaration of function 'snd_soc_new_ac97_component'; did you mean 'snd_soc_new_ac97_codec'?
by kbuild test robot 13 Feb '18

13 Feb '18

tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/wm9712 head: 164c976a9493991ebe7e838931ff895e6e04d7c2 commit: 164c976a9493991ebe7e838931ff895e6e04d7c2 [1/1] ASoC: wm9712: replace codec to component config: sparc64-allyesconfig (attached as .config) compiler: sparc64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout 164c976a9493991ebe7e838931ff895e6e04d7c2 # save the attached .config to linux build tree make.cross ARCH=sparc64 All error/warnings (new ones prefixed by >>): sound/soc//codecs/wm9712.c: In function 'wm9712_soc_probe': >> sound/soc//codecs/wm9712.c:648:18: error: implicit declaration of function 'snd_soc_new_ac97_component'; did you mean 'snd_soc_new_ac97_codec'? [-Werror=implicit-function-declaration] wm9712->ac97 = snd_soc_new_ac97_component(component, WM9712_VENDOR_ID, ^~~~~~~~~~~~~~~~~~~~~~~~~~ snd_soc_new_ac97_codec >> sound/soc//codecs/wm9712.c:648:16: warning: assignment makes pointer from integer without a cast [-Wint-conversion] wm9712->ac97 = snd_soc_new_ac97_component(component, WM9712_VENDOR_ID, ^ >> sound/soc//codecs/wm9712.c:659:4: error: implicit declaration of function 'snd_soc_free_ac97_component'; did you mean 'snd_soc_free_ac97_codec'? [-Werror=implicit-function-declaration] snd_soc_free_ac97_component(wm9712->ac97); ^~~~~~~~~~~~~~~~~~~~~~~~~~~ snd_soc_free_ac97_codec cc1: some warnings being treated as errors vim +648 sound/soc//codecs/wm9712.c 636 637 static int wm9712_soc_probe(struct snd_soc_component *component) 638 { 639 struct wm9712_priv *wm9712 = snd_soc_component_get_drvdata(component); 640 struct regmap *regmap; 641 int ret; 642 643 if (wm9712->mfd_pdata) { 644 wm9712->ac97 = wm9712->mfd_pdata->ac97; 645 regmap = wm9712->mfd_pdata->regmap; 646 } else { 647 #ifdef CONFIG_SND_SOC_AC97_BUS > 648 wm9712->ac97 = snd_soc_new_ac97_component(component, WM9712_VENDOR_ID, 649 WM9712_VENDOR_ID_MASK); 650 if (IS_ERR(wm9712->ac97)) { 651 ret = PTR_ERR(wm9712->ac97); 652 dev_err(component->dev, 653 "Failed to register AC97 codec: %d\n", ret); 654 return ret; 655 } 656 657 regmap = regmap_init_ac97(wm9712->ac97, &wm9712_regmap_config); 658 if (IS_ERR(regmap)) { > 659 snd_soc_free_ac97_component(wm9712->ac97); 660 return PTR_ERR(regmap); 661 } 662 #endif 663 } 664 665 snd_soc_component_init_regmap(component, regmap); 666 667 /* set alc mux to none */ 668 snd_soc_component_update_bits(component, AC97_VIDEO, 0x3000, 0x3000); 669 670 return 0; 671 } 672 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

1 0

[alsa-devel] [asoc:topic/wm9713 1/1] sound/soc/codecs/wm9713.c:1219:18: error: implicit declaration of function 'snd_soc_new_ac97_component'; did you mean 'snd_soc_new_ac97_codec'?
by kbuild test robot 12 Feb '18

12 Feb '18

tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/wm9713 head: 807cef2d327a5148d456d9c4966bfb9b2785ffc8 commit: 807cef2d327a5148d456d9c4966bfb9b2785ffc8 [1/1] ASoC: wm9713: replace codec to component config: cris-allmodconfig (attached as .config) compiler: cris-linux-gcc (GCC) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout 807cef2d327a5148d456d9c4966bfb9b2785ffc8 # save the attached .config to linux build tree make.cross ARCH=cris All errors (new ones prefixed by >>): sound/soc/codecs/wm9713.c: In function 'wm9713_soc_probe': >> sound/soc/codecs/wm9713.c:1219:18: error: implicit declaration of function 'snd_soc_new_ac97_component'; did you mean 'snd_soc_new_ac97_codec'? [-Werror=implicit-function-declaration] wm9713->ac97 = snd_soc_new_ac97_component(component, WM9713_VENDOR_ID, ^~~~~~~~~~~~~~~~~~~~~~~~~~ snd_soc_new_ac97_codec sound/soc/codecs/wm9713.c:1219:16: warning: assignment makes pointer from integer without a cast [-Wint-conversion] wm9713->ac97 = snd_soc_new_ac97_component(component, WM9713_VENDOR_ID, ^ >> sound/soc/codecs/wm9713.c:1225:4: error: implicit declaration of function 'snd_soc_free_ac97_component'; did you mean 'snd_soc_free_ac97_codec'? [-Werror=implicit-function-declaration] snd_soc_free_ac97_component(wm9713->ac97); ^~~~~~~~~~~~~~~~~~~~~~~~~~~ snd_soc_free_ac97_codec cc1: some warnings being treated as errors vim +1219 sound/soc/codecs/wm9713.c 1208 1209 static int wm9713_soc_probe(struct snd_soc_component *component) 1210 { 1211 struct wm9713_priv *wm9713 = snd_soc_component_get_drvdata(component); 1212 struct regmap *regmap = NULL; 1213 1214 if (wm9713->mfd_pdata) { 1215 wm9713->ac97 = wm9713->mfd_pdata->ac97; 1216 regmap = wm9713->mfd_pdata->regmap; 1217 } else { 1218 #ifdef CONFIG_SND_SOC_AC97_BUS > 1219 wm9713->ac97 = snd_soc_new_ac97_component(component, WM9713_VENDOR_ID, 1220 WM9713_VENDOR_ID_MASK); 1221 if (IS_ERR(wm9713->ac97)) 1222 return PTR_ERR(wm9713->ac97); 1223 regmap = regmap_init_ac97(wm9713->ac97, &wm9713_regmap_config); 1224 if (IS_ERR(regmap)) { > 1225 snd_soc_free_ac97_component(wm9713->ac97); 1226 return PTR_ERR(regmap); 1227 } 1228 #endif 1229 } 1230 1231 snd_soc_component_init_regmap(component, regmap); 1232 1233 /* unmute the adc - move to kcontrol */ 1234 snd_soc_component_update_bits(component, AC97_CD, 0x7fff, 0x0000); 1235 1236 return 0; 1237 } 1238 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

1 0

[alsa-devel] [PATCH v2 4/5] ALSA: emu10k1: make sure synth DMA pages are allocated with DMA functions
by Maciej S. Szmigiero 12 Feb '18

12 Feb '18

Commit a5003fc04113 ("[ALSA] emu10k1 - simplify page allocation for synth") switched from using the DMA allocator for synth DMA pages to manually calling alloc_page(). However, this usage has an implicit assumption that the DMA address space for the emu10k1-family chip is the same as the CPU physical address space which is not true for a system with a IOMMU. Since this made the synth part of the driver non-functional on such systems let's effectively revert that commit. Signed-off-by: Maciej S. Szmigiero <mail(a)maciej.szmigiero.name> --- Changes from v1: None in this patch. sound/pci/emu10k1/memory.c | 70 ++++++++++++++++++++++++++-------------------- 1 file changed, 39 insertions(+), 31 deletions(-) diff --git a/sound/pci/emu10k1/memory.c b/sound/pci/emu10k1/memory.c index fcb04cbbc9ab..5cdffe2d31e1 100644 --- a/sound/pci/emu10k1/memory.c +++ b/sound/pci/emu10k1/memory.c @@ -457,49 +457,44 @@ static void get_single_page_range(struct snd_util_memhdr *hdr, *last_page_ret = last_page; } -/* release allocated pages */ -static void __synth_free_pages(struct snd_emu10k1 *emu, int first_page, - int last_page) -{ - int page; - - for (page = first_page; page <= last_page; page++) { - free_page((unsigned long)emu->page_ptr_table[page]); - emu->page_addr_table[page] = 0; - emu->page_ptr_table[page] = NULL; - } -} - /* * allocate kernel pages */ static int synth_alloc_pages(struct snd_emu10k1 *emu, struct snd_emu10k1_memblk *blk) { int page, first_page, last_page; + struct snd_dma_buffer dmab; emu10k1_memblk_init(blk); get_single_page_range(emu->memhdr, blk, &first_page, &last_page); /* allocate kernel pages */ for (page = first_page; page <= last_page; page++) { - /* first try to allocate from <4GB zone */ - struct page *p = alloc_page(GFP_KERNEL | GFP_DMA32 | - __GFP_NOWARN); - if (!p || (page_to_pfn(p) & ~(emu->dma_mask >> PAGE_SHIFT))) { - if (p) - __free_page(p); - /* try to allocate from <16MB zone */ - p = alloc_page(GFP_ATOMIC | GFP_DMA | - __GFP_NORETRY | /* no OOM-killer */ - __GFP_NOWARN); - } - if (!p) { - __synth_free_pages(emu, first_page, page - 1); - return -ENOMEM; + if (snd_dma_alloc_pages(SNDRV_DMA_TYPE_DEV, + snd_dma_pci_data(emu->pci), + PAGE_SIZE, &dmab) < 0) + goto __fail; + if (!is_valid_page(emu, dmab.addr)) { + snd_dma_free_pages(&dmab); + goto __fail; } - emu->page_addr_table[page] = page_to_phys(p); - emu->page_ptr_table[page] = page_address(p); + emu->page_addr_table[page] = dmab.addr; + emu->page_ptr_table[page] = dmab.area; } return 0; + +__fail: + /* release allocated pages */ + last_page = page - 1; + for (page = first_page; page <= last_page; page++) { + dmab.area = emu->page_ptr_table[page]; + dmab.addr = emu->page_addr_table[page]; + dmab.bytes = PAGE_SIZE; + snd_dma_free_pages(&dmab); + emu->page_addr_table[page] = 0; + emu->page_ptr_table[page] = NULL; + } + + return -ENOMEM; } /* @@ -507,10 +502,23 @@ static int synth_alloc_pages(struct snd_emu10k1 *emu, struct snd_emu10k1_memblk */ static int synth_free_pages(struct snd_emu10k1 *emu, struct snd_emu10k1_memblk *blk) { - int first_page, last_page; + int page, first_page, last_page; + struct snd_dma_buffer dmab; get_single_page_range(emu->memhdr, blk, &first_page, &last_page); - __synth_free_pages(emu, first_page, last_page); + dmab.dev.type = SNDRV_DMA_TYPE_DEV; + dmab.dev.dev = snd_dma_pci_data(emu->pci); + for (page = first_page; page <= last_page; page++) { + if (emu->page_ptr_table[page] == NULL) + continue; + dmab.area = emu->page_ptr_table[page]; + dmab.addr = emu->page_addr_table[page]; + dmab.bytes = PAGE_SIZE; + snd_dma_free_pages(&dmab); + emu->page_addr_table[page] = 0; + emu->page_ptr_table[page] = NULL; + } + return 0; }

2 2

[alsa-devel] usleep_range without a range
by Joe Perches 12 Feb '18

12 Feb '18

scheduling can generally be better when these values are not identical. Perhaps these ranges should be expanded. $ git grep -P -n "usleep_range\s*$\s*([\w\.\>\-]+)\s*,\s*\1\s*$" drivers/clk/ux500/clk-sysctrl.c:45: usleep_range(clk->enable_delay_us, clk->enable_delay_us); drivers/cpufreq/pmac64-cpufreq.c:140: usleep_range(1000, 1000); drivers/cpufreq/pmac64-cpufreq.c:239: usleep_range(10000, 10000); /* should be faster , to fix */ drivers/cpufreq/pmac64-cpufreq.c:284: usleep_range(500, 500); drivers/media/i2c/smiapp/smiapp-core.c:1228: usleep_range(1000, 1000); drivers/media/i2c/smiapp/smiapp-core.c:1235: usleep_range(1000, 1000); drivers/media/i2c/smiapp/smiapp-core.c:1240: usleep_range(sleep, sleep); drivers/media/i2c/smiapp/smiapp-core.c:1387: usleep_range(5000, 5000); drivers/media/i2c/smiapp/smiapp-quirk.c:205: usleep_range(2000, 2000); drivers/media/i2c/smiapp/smiapp-regs.c:279: usleep_range(2000, 2000); drivers/power/supply/ab8500_fg.c:643: usleep_range(100, 100); drivers/staging/rtl8192u/r819xU_phy.c:180: usleep_range(1000, 1000); drivers/staging/rtl8192u/r819xU_phy.c:736: usleep_range(1000, 1000); drivers/staging/rtl8192u/r819xU_phy.c:740: usleep_range(1000, 1000); sound/soc/codecs/ab8500-codec.c:1065: usleep_range(AB8500_ANC_SM_DELAY, AB8500_ANC_SM_DELAY); sound/soc/codecs/ab8500-codec.c:1068: usleep_range(AB8500_ANC_SM_DELAY, AB8500_ANC_SM_DELAY);

1 0

[alsa-devel] [asoc:topic/wm9713 1/1] sound/soc/codecs/wm9713.c:1219:18: error: implicit declaration of function 'snd_soc_new_ac97_component'
by kbuild test robot 12 Feb '18

12 Feb '18

tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/wm9713 head: 807cef2d327a5148d456d9c4966bfb9b2785ffc8 commit: 807cef2d327a5148d456d9c4966bfb9b2785ffc8 [1/1] ASoC: wm9713: replace codec to component config: openrisc-allmodconfig (attached as .config) compiler: or1k-linux-gcc (GCC) 6.0.0 20160327 (experimental) reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout 807cef2d327a5148d456d9c4966bfb9b2785ffc8 # save the attached .config to linux build tree make.cross ARCH=openrisc All error/warnings (new ones prefixed by >>): sound/soc/codecs/wm9713.c: In function 'wm9713_soc_probe': >> sound/soc/codecs/wm9713.c:1219:18: error: implicit declaration of function 'snd_soc_new_ac97_component' [-Werror=implicit-function-declaration] wm9713->ac97 = snd_soc_new_ac97_component(component, WM9713_VENDOR_ID, ^~~~~~~~~~~~~~~~~~~~~~~~~~ >> sound/soc/codecs/wm9713.c:1219:16: warning: assignment makes pointer from integer without a cast [-Wint-conversion] wm9713->ac97 = snd_soc_new_ac97_component(component, WM9713_VENDOR_ID, ^ >> sound/soc/codecs/wm9713.c:1225:4: error: implicit declaration of function 'snd_soc_free_ac97_component' [-Werror=implicit-function-declaration] snd_soc_free_ac97_component(wm9713->ac97); ^~~~~~~~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors vim +/snd_soc_new_ac97_component +1219 sound/soc/codecs/wm9713.c 1208 1209 static int wm9713_soc_probe(struct snd_soc_component *component) 1210 { 1211 struct wm9713_priv *wm9713 = snd_soc_component_get_drvdata(component); 1212 struct regmap *regmap = NULL; 1213 1214 if (wm9713->mfd_pdata) { 1215 wm9713->ac97 = wm9713->mfd_pdata->ac97; 1216 regmap = wm9713->mfd_pdata->regmap; 1217 } else { 1218 #ifdef CONFIG_SND_SOC_AC97_BUS > 1219 wm9713->ac97 = snd_soc_new_ac97_component(component, WM9713_VENDOR_ID, 1220 WM9713_VENDOR_ID_MASK); 1221 if (IS_ERR(wm9713->ac97)) 1222 return PTR_ERR(wm9713->ac97); 1223 regmap = regmap_init_ac97(wm9713->ac97, &wm9713_regmap_config); 1224 if (IS_ERR(regmap)) { > 1225 snd_soc_free_ac97_component(wm9713->ac97); 1226 return PTR_ERR(regmap); 1227 } 1228 #endif 1229 } 1230 1231 snd_soc_component_init_regmap(component, regmap); 1232 1233 /* unmute the adc - move to kcontrol */ 1234 snd_soc_component_update_bits(component, AC97_CD, 0x7fff, 0x0000); 1235 1236 return 0; 1237 } 1238 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

1 0

[alsa-devel] [asoc:topic/uniphier 6/7] sound/soc/uniphier/aio-compress.c:314:34: sparse: cast to restricted __be16
by kbuild test robot 12 Feb '18

12 Feb '18

tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/uniphier head: db4cb3d04eba91db48da0032c44b83ed7185973c commit: e98131222ff931a670739cede00d1fb9d6f83763 [6/7] ASoC: uniphier: add support for UniPhier AIO compress audio reproduce: # apt-get install sparse git checkout e98131222ff931a670739cede00d1fb9d6f83763 make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) >> sound/soc/uniphier/aio-compress.c:314:34: sparse: cast to restricted __be16 >> sound/soc/uniphier/aio-compress.c:314:34: sparse: cast to restricted __be16 >> sound/soc/uniphier/aio-compress.c:314:34: sparse: cast to restricted __be16 >> sound/soc/uniphier/aio-compress.c:314:34: sparse: cast to restricted __be16 vim +314 sound/soc/uniphier/aio-compress.c 290 291 static int aio_compr_send_to_hw(struct uniphier_aio_sub *sub, 292 char __user *buf, size_t dstsize) 293 { 294 u32 __user *srcbuf = (u32 __user *)buf; 295 u32 *dstbuf = (u32 *)(sub->compr_area + sub->wr_offs); 296 int src = 0, dst = 0, ret; 297 u32 frm, frm_a, frm_b; 298 299 while (dstsize > 0) { 300 ret = get_user(frm, srcbuf + src); 301 if (ret) 302 return ret; 303 src++; 304 305 frm_a = frm & 0xffff; 306 frm_b = (frm >> 16) & 0xffff; 307 308 if (frm == IEC61937_HEADER_SIGN) { 309 frm_a |= 0x01000000; 310 311 /* Next data is Pc and Pd */ 312 sub->iec_header = true; 313 } else { > 314 u16 pc = be16_to_cpu((__be16)frm_a); 315 316 if (sub->iec_header && sub->iec_pc != pc) { 317 /* Force overwrite IEC frame type */ 318 sub->iec_pc = pc; 319 ret = aio_oport_set_stream_type(sub, pc); 320 if (ret) 321 return ret; 322 } 323 sub->iec_header = false; 324 } 325 dstbuf[dst++] = frm_a; 326 dstbuf[dst++] = frm_b; 327 328 dstsize -= sizeof(u32) * 2; 329 } 330 331 return 0; 332 } 333 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

1 0