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

[alsa-devel] [PATCH 1/3] ASoC: rt5651: add Volume to the name of volume control
by Bard Liao 25 Jul '18

25 Jul '18

add Volume to the name of volume control Signed-off-by: Bard Liao <bardliao(a)realtek.com> --- sound/soc/codecs/rt5651.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/soc/codecs/rt5651.c b/sound/soc/codecs/rt5651.c index 985852f..fe970ed 100644 --- a/sound/soc/codecs/rt5651.c +++ b/sound/soc/codecs/rt5651.c @@ -332,11 +332,11 @@ static const struct snd_kcontrol_new rt5651_snd_controls[] = { RT5651_L_VOL_SFT, RT5651_R_VOL_SFT, 175, 0, dac_vol_tlv), /* IN1/IN2/IN3 Control */ - SOC_SINGLE_TLV("IN1 Boost", RT5651_IN1_IN2, + SOC_SINGLE_TLV("IN1 Boost Volume", RT5651_IN1_IN2, RT5651_BST_SFT1, 8, 0, bst_tlv), - SOC_SINGLE_TLV("IN2 Boost", RT5651_IN1_IN2, + SOC_SINGLE_TLV("IN2 Boost Volume", RT5651_IN1_IN2, RT5651_BST_SFT2, 8, 0, bst_tlv), - SOC_SINGLE_TLV("IN3 Boost", RT5651_IN3, + SOC_SINGLE_TLV("IN3 Boost Volume", RT5651_IN3, RT5651_BST_SFT1, 8, 0, bst_tlv), /* INL/INR Volume Control */ SOC_DOUBLE_TLV("IN Capture Volume", RT5651_INL1_INR1_VOL, -- 2.7.4

2 3

[alsa-devel] [PATCH 0/6] Fix Sparse warnings on sound/soc/intel
by Pierre-Louis Bossart 25 Jul '18

25 Jul '18

Since we found quite a few issues when checking the SOF code with Sparse, I figured I'd run a quick check on the rest of the Intel audio drivers. Some of the issues are trivial but in the end there are a couple of improvements identified by reducing the volume of warnings, so it's worth it overall. If there are concerns about the code readability, I would still recommend merging the last 3-4 patches that are really the interesting ones exposed by Sparse checks. Pierre-Louis Bossart (6): ASoC: Intel: boards: cleanup snd_mask_set usage ASoC: Intel: Haswell: fix endianness handling ASoC: Intel: Skylake: BDL definitions should be __le32 ASoC: Intel: common: make sst_dma functions static ASoC: Intel: Atom: simplify iomem address and casts ASoC: Intel: Atom: fix inversion between __iowrite32 and __ioread32 sound/soc/intel/atom/sst/sst_drv_interface.c | 29 ++++++------ sound/soc/intel/atom/sst/sst_loader.c | 6 +-- sound/soc/intel/boards/bdw-rt5677.c | 2 +- sound/soc/intel/boards/bxt_da7219_max98357a.c | 2 +- sound/soc/intel/boards/bxt_rt298.c | 2 +- sound/soc/intel/boards/glk_rt5682_max98357a.c | 2 +- sound/soc/intel/boards/kbl_da7219_max98357a.c | 2 +- sound/soc/intel/boards/kbl_rt5663_max98927.c | 6 ++- .../soc/intel/boards/kbl_rt5663_rt5514_max98927.c | 6 ++- sound/soc/intel/boards/skl_nau88l25_max98357a.c | 2 +- sound/soc/intel/boards/skl_nau88l25_ssm4567.c | 2 +- sound/soc/intel/boards/skl_rt286.c | 2 +- sound/soc/intel/common/sst-firmware.c | 6 +-- sound/soc/intel/haswell/sst-haswell-dsp.c | 53 ++++++++++++---------- sound/soc/intel/skylake/skl-sst-cldma.c | 8 ++-- sound/soc/intel/skylake/skl-sst-cldma.h | 2 +- 16 files changed, 70 insertions(+), 62 deletions(-) -- 2.14.1

3 13

[alsa-devel] Applied "ASoC: Intel: Skylake: fix widget handling" to the asoc tree
by Mark Brown 25 Jul '18

25 Jul '18

The patch ASoC: Intel: Skylake: fix widget handling has been applied to the asoc tree at https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark >From fe65324e3f5205072a2d55ac9c63ec77155fa528 Mon Sep 17 00:00:00 2001 From: Rakesh Ughreja <rakesh.a.ughreja(a)intel.com> Date: Tue, 24 Jul 2018 19:50:48 -0500 Subject: [PATCH] ASoC: Intel: Skylake: fix widget handling include DAPM Mux and output widgets into the list. Signed-off-by: Rakesh Ughreja <rakesh.a.ughreja(a)intel.com> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> Signed-off-by: Mark Brown <broonie(a)kernel.org> --- sound/soc/intel/skylake/skl-topology.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/soc/intel/skylake/skl-topology.c b/sound/soc/intel/skylake/skl-topology.c index 647e52aecdc3..9845d853644c 100644 --- a/sound/soc/intel/skylake/skl-topology.c +++ b/sound/soc/intel/skylake/skl-topology.c @@ -108,6 +108,9 @@ static int is_skl_dsp_widget_type(struct snd_soc_dapm_widget *w, case snd_soc_dapm_aif_out: case snd_soc_dapm_dai_out: case snd_soc_dapm_switch: + case snd_soc_dapm_output: + case snd_soc_dapm_mux: + return false; default: return true; -- 2.18.0

1 0

[alsa-devel] [PATCH] Added support for Encore mDSD and Topping DX7s XMOS USB DACs
by JCruk 25 Jul '18

25 Jul '18

Signed-off-by: JCruk <jcrukley(a)gmail.com> --- sound/usb/quirks.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c index 02b6cc0..7683d33 100644 --- a/sound/usb/quirks.c +++ b/sound/usb/quirks.c @@ -1373,6 +1373,8 @@ u64 snd_usb_interface_dsd_format_quirks(struct snd_usb_audio *chip, return SNDRV_PCM_FMTBIT_DSD_U32_BE; break; + case USB_ID(0x152a, 0x8750): /* Topping DX7s */ + case USB_ID(0x16d0, 0x09dd): /* Encore mDSD */ case USB_ID(0x0d8c, 0x0316): /* Hegel HD12 DSD */ case USB_ID(0x16b0, 0x06b2): /* NuPrime DAC-10 */ case USB_ID(0x16d0, 0x0733): /* Furutech ADL Stratos */ -- 2.7.4

1 0

Re: [alsa-devel] [PATCH 1/3] ASoC: rt5651: add Volume to the name of volume control
by Mark Brown 25 Jul '18

25 Jul '18

On Wed, Jul 25, 2018 at 02:01:21AM +0000, Bard Liao wrote: > > The RT5631 doesn't appear to have any users except Arndale, certainly no > > x86 users? > As far as I know, there is no x86 users using RT5631. OK, I think it's probably safe to apply that one then as I'm not seeing any in tree users except that Arndale expansion.

1 0

[alsa-devel] [PATCH] ASoC: AMD: Add a fix voltage regulator for DA7219 and ADAU7002
by Akshu Agrawal 25 Jul '18

25 Jul '18

DA7219 for our platform need to be configured for 1.8V. Hence, we add a fixed volatge regulator with supplies of 1.8V in the machine driver. Signed-off-by: Akshu Agrawal <akshu.agrawal(a)amd.com> --- sound/soc/amd/Kconfig | 2 ++ sound/soc/amd/acp-da7219-max98357a.c | 45 ++++++++++++++++++++++++++++++++++++ 2 files changed, 47 insertions(+) diff --git a/sound/soc/amd/Kconfig b/sound/soc/amd/Kconfig index 6cbf9cf..c447a51 100644 --- a/sound/soc/amd/Kconfig +++ b/sound/soc/amd/Kconfig @@ -8,6 +8,8 @@ config SND_SOC_AMD_CZ_DA7219MX98357_MACH select SND_SOC_DA7219 select SND_SOC_MAX98357A select SND_SOC_ADAU7002 + select REGULATOR + select REGULATOR_FIXED_VOLTAGE depends on SND_SOC_AMD_ACP && I2C help This option enables machine driver for DA7219 and MAX9835. diff --git a/sound/soc/amd/acp-da7219-max98357a.c b/sound/soc/amd/acp-da7219-max98357a.c index f42606e..fdf8972 100644 --- a/sound/soc/amd/acp-da7219-max98357a.c +++ b/sound/soc/amd/acp-da7219-max98357a.c @@ -31,7 +31,10 @@ #include <sound/jack.h> #include <linux/clk.h> #include <linux/gpio.h> +#include <linux/platform_device.h> #include <linux/module.h> +#include <linux/regulator/fixed.h> +#include <linux/regulator/machine.h> #include <linux/i2c.h> #include <linux/input.h> #include <linux/acpi.h> @@ -320,11 +323,53 @@ static void cz_dmic_shutdown(struct snd_pcm_substream *substream) .num_controls = ARRAY_SIZE(cz_mc_controls), }; +static struct regulator_consumer_supply acp_da7219_supplies[] = { + REGULATOR_SUPPLY("VDD", "i2c-DLGS7219:00"), + REGULATOR_SUPPLY("VDDMIC", "i2c-DLGS7219:00"), + REGULATOR_SUPPLY("VDDIO", "i2c-DLGS7219:00"), + REGULATOR_SUPPLY("IOVDD", "ADAU7002:00"), +}; + +static struct regulator_init_data acp_da7219_data = { + .constraints = { + .always_on = 1, + }, + .num_consumer_supplies = ARRAY_SIZE(acp_da7219_supplies), + .consumer_supplies = acp_da7219_supplies, +}; + +static struct fixed_voltage_config acp_da7219 = { + .supply_name = "reg-fixed-1.8V", + .microvolts = 1800000, /* 1.8V */ + .gpio = -EINVAL, + .enabled_at_boot = 1, + .init_data = &acp_da7219_data, +}; + +static struct platform_device acp_da7219_regulator = { + .name = "reg-fixed-voltage", + .id = PLATFORM_DEVID_AUTO, + .dev = { + .platform_data = &acp_da7219, + }, +}; + static int cz_probe(struct platform_device *pdev) { int ret; struct snd_soc_card *card; struct acp_platform_info *machine; + static bool regulators_registered; + + if (!regulators_registered) { + ret = platform_device_register(&acp_da7219_regulator); + if (ret) { + dev_err(&pdev->dev, "Failed to register regulator: %d\n", + ret); + return ret; + } + regulators_registered = true; + } machine = devm_kzalloc(&pdev->dev, sizeof(struct acp_platform_info), GFP_KERNEL); -- 1.9.1

3 4

Re: [alsa-devel] 答复: [PATCH 3/4] drm/radeon: Add audio component support
by Takashi Iwai 25 Jul '18

25 Jul '18

On Wed, 25 Jul 2018 05:32:52 +0200, Qu, Jim wrote: > @@ -269,6 +271,10 @@ static void radeon_audio_enable(struct radeon_device *rdev, > > if (rdev->audio.funcs->enable) > rdev->audio.funcs->enable(rdev, pin, enable_mask); > + > + if (acomp && acomp->audio_ops && acomp->audio_ops->pin_eld_notify) > + acomp->audio_ops->pin_eld_notify(acomp->audio_ops->audio_ptr, > + pin->id, -1); > > >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> > Jim: radeon_audio_enable() can enable audios and also disable them, so eld noitfy callback should be called when enable_mask is true. It's intentional. The notifier needs to be called at disablement time, too. The audio driver has to follow the HDMI audio disablement, and notifier receives both on and off case. The actual state is inquired via get_eld call by HD-audio followed after the notification. thanks, Takashi

2 1

[alsa-devel] Building alsa-utils, adding changes
by Jamey Kirby 25 Jul '18

25 Jul '18

I have been working with alsa for a while. We use it to archive content from our radio station. I am using arecord. Build problem: 1) I cloned alsa-utils from github 2) Spent two hours on Google trying to figure out how to build the darn thing. 3) I am running Ubuntu Server 18.0 with build-essentials installed. 4) I get errors that I do not know how to fix when I try to setup ./configure. 5) I just need to make a build. What do I need to do? Feature: I am adding a feature. In the loop for capture(), when a new file is opened, I want to close the old file (when doing continuous recording). I added a new CLI parameter -K. When this is set, the capture() loop will close the old file before opening the new file. This is an issue for me for two reasons: 1) When will i run out of file handles? I capture one hour files continuously. After 10 hours, I see that all the capture files are still open? 2) I need to be notified when the file is closed so that I can do post-processing of rht capture. I've tried inotifywait in a script, and in another script I tried lsof. lsof works great, but the files are open,. See, the main issue is that I need to know when each file completes so that I can post process. Here is my post process script outline: #!/bin/bash for f in $HOME/vboxshare/jkirby/newarc/*.raw do filename=$(basename $f) open=$(lsof -t $f | wc -w) if [ $open -lt 1 ] then echo "$filename is closed" echo "Do post processing here" else echo "$filename is open by $open processe(s)" fi done If I could close the file before or after the previous file is closed, I can do my work. Otherwise, I have no way to know if the file is open and being processed or not. So, I've made the changes, I just need to build the darn thing. Thanks for any advice or help. -- Jamey Kirby Disrupting the establishment since 1964 *This is a personal email account and as such, emails are not subject to archiving. Nothing else really matters.*

1 0

[alsa-devel] [PATCH v2] ASoC: da7219: Allow pdata to specify a VDDIO
by Daniel Kurtz 24 Jul '18

24 Jul '18

Some systems do not have software controllable regulators driving the DA7219's supplies, nor can they use device tree to create "always-on fixed regulators" to easily pretend like they do. On these systems the call to devm_regulator_bulk_get() just creates a set of dummy registers. Calling regulator_get_voltage() on a dummy regulator just returns -EINVAL, in which case the DA7219 is always set up to use the default VDDIO voltage range of 2.5-3.6V. Provide a new device property to let such systems specify a different VDDIO if needed (e.g., 1.8V). Signed-off-by: Daniel Kurtz <djkurtz(a)chromium.org> --- Changes for v2: - fix to use device_property_read_u32() include/sound/da7219.h | 2 ++ sound/soc/codecs/da7219.c | 19 +++++++++++++------ 2 files changed, 15 insertions(+), 6 deletions(-) diff --git a/include/sound/da7219.h b/include/sound/da7219.h index 1bfcb16f2d10ab..16ab125ad4adbf 100644 --- a/include/sound/da7219.h +++ b/include/sound/da7219.h @@ -38,6 +38,8 @@ struct da7219_pdata { const char *dai_clks_name; + u32 vddio; + /* Mic */ enum da7219_micbias_voltage micbias_lvl; enum da7219_mic_amp_in_sel mic_amp_in_sel; diff --git a/sound/soc/codecs/da7219.c b/sound/soc/codecs/da7219.c index 980a6a8bf56d38..9893920b26f41f 100644 --- a/sound/soc/codecs/da7219.c +++ b/sound/soc/codecs/da7219.c @@ -1634,6 +1634,9 @@ static struct da7219_pdata *da7219_fw_to_pdata(struct snd_soc_component *compone else pdata->mic_amp_in_sel = DA7219_MIC_AMP_IN_SEL_DIFF; + if (device_property_read_u32(dev, "dlg,vddio", &of_val32) >= 0) + pdata->vddio = of_val32; + return pdata; } @@ -1717,8 +1720,12 @@ static int da7219_handle_supplies(struct snd_soc_component *component) /* Determine VDDIO voltage provided */ vddio = da7219->supplies[DA7219_SUPPLY_VDDIO].consumer; ret = regulator_get_voltage(vddio); + /* If regulator_get_voltage() fails, try to use vddio from pdata. */ + if (ret < 0 && da7219->pdata) + ret = da7219->pdata->vddio; if (ret < 1200000) - dev_warn(component->dev, "Invalid VDDIO voltage\n"); + dev_warn(component->dev, "Invalid VDDIO voltage: %d mV\n", + ret); else if (ret < 2800000) io_voltage_lvl = DA7219_IO_VOLTAGE_LEVEL_1_2V_2_8V; @@ -1872,6 +1879,11 @@ static int da7219_probe(struct snd_soc_component *component) mutex_init(&da7219->ctrl_lock); mutex_init(&da7219->pll_lock); + /* Handle DT/ACPI/Platform data */ + da7219->pdata = dev_get_platdata(component->dev); + if (!da7219->pdata) + da7219->pdata = da7219_fw_to_pdata(component); + /* Regulator configuration */ ret = da7219_handle_supplies(component); if (ret) @@ -1897,11 +1909,6 @@ static int da7219_probe(struct snd_soc_component *component) break; } - /* Handle DT/ACPI/Platform data */ - da7219->pdata = dev_get_platdata(component->dev); - if (!da7219->pdata) - da7219->pdata = da7219_fw_to_pdata(component); - da7219_handle_pdata(component); /* Check if MCLK provided */ -- 2.18.0.233.g985f88cf7e-goog

3 2

[alsa-devel] [PATCH 1/4] ASoC: fsl-asoc-card: Switch to SPDX identifier
by Fabio Estevam 24 Jul '18

24 Jul '18

Adopt the SPDX license identifier headers to ease license compliance management. Signed-off-by: Fabio Estevam <fabio.estevam(a)nxp.com> --- sound/soc/fsl/fsl-asoc-card.c | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/sound/soc/fsl/fsl-asoc-card.c b/sound/soc/fsl/fsl-asoc-card.c index 4a6750a..07808c6 100644 --- a/sound/soc/fsl/fsl-asoc-card.c +++ b/sound/soc/fsl/fsl-asoc-card.c @@ -1,14 +1,10 @@ -/* - * Freescale Generic ASoC Sound Card driver with ASRC - * - * Copyright (C) 2014 Freescale Semiconductor, Inc. - * - * Author: Nicolin Chen <nicoleotsuka(a)gmail.com> - * - * This file is licensed under the terms of the GNU General Public License - * version 2. This program is licensed "as is" without any warranty of any - * kind, whether express or implied. - */ +// SPDX-License-Identifier: GPL-2.0 +// +// Freescale Generic ASoC Sound Card driver with ASRC +// +// Copyright (C) 2014 Freescale Semiconductor, Inc. +// +// Author: Nicolin Chen <nicoleotsuka(a)gmail.com> #include <linux/clk.h> #include <linux/i2c.h> -- 2.7.4

2 7