May 2015 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH] ASoC: Intel: fix broadwell module removing failed issue
by Jie Yang 30 May '15

30 May '15

From: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> In haswell-pcm module unloading, we can't free runtime modules directly, for they may be already freed in runtime suspend. Here add executing suspend call to unload runtime modules, only for status not equal to RPM_SUSPEND, to fix broadwell module removing failed issue. Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> Signed-off-by: Jie Yang <yang.jie(a)intel.com> --- sound/soc/intel/haswell/sst-haswell-pcm.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sound/soc/intel/haswell/sst-haswell-pcm.c b/sound/soc/intel/haswell/sst-haswell-pcm.c index 23ae040..bd96629 100644 --- a/sound/soc/intel/haswell/sst-haswell-pcm.c +++ b/sound/soc/intel/haswell/sst-haswell-pcm.c @@ -1118,8 +1118,10 @@ static int hsw_pcm_remove(struct snd_soc_platform *platform) snd_soc_platform_get_drvdata(platform); int i; + /* execute a suspend call to unload all FW resources */ + if (!pm_runtime_status_suspended(platform->dev)) + pm_runtime_put_sync_suspend(platform->dev); pm_runtime_disable(platform->dev); - hsw_pcm_free_modules(priv_data); for (i = 0; i < ARRAY_SIZE(hsw_dais); i++) { if (hsw_dais[i].playback.channels_min) -- 1.9.1

5 8

[alsa-devel] [PATCH - alsa-plugins/arcam-av ] arcam-av: Include sys/select.h for fd_set definition
by Khem Raj 30 May '15

30 May '15

fixes build errors on non-glibc based systems arcam-av/arcam_av.c:389:2: error: unknown type name 'fd_set' Signed-off-by: Khem Raj <raj.khem(a)gmail.com> --- arcam-av/arcam_av.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arcam-av/arcam_av.c b/arcam-av/arcam_av.c index 0491fc6..fff96f9 100644 --- a/arcam-av/arcam_av.c +++ b/arcam-av/arcam_av.c @@ -31,6 +31,7 @@ #include <unistd.h> #include <sys/ipc.h> +#include <sys/select.h> #include <sys/shm.h> #include <sys/stat.h> #include <sys/stat.h> -- 2.1.4

2 1

[alsa-devel] [RFC][PATCH 0/2] ALSA: control: add dimen validator
by Takashi Sakamoto 30 May '15

30 May '15

Hi, This patchset adds a validator for dimen field in control element information. The dimen information is used to represent channel matrix of the control element, thus the number of elements in the matrix should be within the number of channels in the control element. Current implementation allows each drivers to set arbitrary number to the information. When the information is wrong, it can cause buffer-over-run to userspace applications depending on application implementation. Actually, the dimen information is scarcely used by current drivers and applications, while it's better for ALSA core to ensure the information is enough safe. Takashi Sakamoto (2): ALSA: control: add dimension validator for userspace element ALSA: control: add dimension validator for kernel driver sound/core/control.c | 62 ++++++++++++++++++++++++++++++++++++++-------------- 1 file changed, 46 insertions(+), 16 deletions(-) -- 2.1.4

2 5

[alsa-devel] [PATCH - alsa-plugins/arcam-av ] arcam-av: Include sys/select.h for fd_set definition
by Khem Raj 30 May '15

30 May '15

fixes build errors on non-glibc based systems arcam-av/arcam_av.c:389:2: error: unknown type name 'fd_set' Signed-off-by: Khem Raj <raj.khem(a)gmail.com> --- arcam-av/arcam_av.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arcam-av/arcam_av.c b/arcam-av/arcam_av.c index 0491fc6..fff96f9 100644 --- a/arcam-av/arcam_av.c +++ b/arcam-av/arcam_av.c @@ -31,6 +31,7 @@ #include <unistd.h> #include <sys/ipc.h> +#include <sys/select.h> #include <sys/shm.h> #include <sys/stat.h> #include <sys/stat.h> -- 2.1.4

1 0

[alsa-devel] randconfig build error with next-20150529, in sound/soc/qcom/lpass-ipq806x
by Jim Davis 29 May '15

29 May '15

Building with the attached random configuration file, warning: (SND_SOC_STORM) selects SND_SOC_LPASS_IPQ806X which has unmet direct dependencies (SOUND && !M68K && !UML && SND && SND_SOC && SND_SOC_QCOM) warning: (INTEL_SOC_DTS_IOSF_CORE && PUNIT_ATOM_DEBUG) selects IOSF_MBI which has unmet direct dependencies (PCI) ERROR: "asoc_qcom_lpass_cpu_dai_probe" [sound/soc/qcom/snd-soc-lpass-ipq806x.ko] undefined! ERROR: "asoc_qcom_lpass_cpu_platform_probe" [sound/soc/qcom/snd-soc-lpass-ipq806x.ko] undefined! ERROR: "asoc_qcom_lpass_cpu_platform_remove" [sound/soc/qcom/snd-soc-lpass-ipq806x.ko] undefined! ERROR: "asoc_qcom_lpass_cpu_dai_ops" [sound/soc/qcom/snd-soc-lpass-ipq806x.ko] undefined!

1 0

[alsa-devel] [PATCH v3 0/3] ASoC topology
by Liam Girdwood 29 May '15

29 May '15

ASoC topology (formerly called dynamic firmware) allows audio DSP topology to be defined as part of the firmware package and not hard coded into the driver. This allows DSP vendors to ship multiple different firmwares or firmwares with programmable topologies for different devices (i.e. phones, tablets, TVs) using a single DSP driver. The DSP topology can be contained as part of the firmware file or exist as separate files. The topology core and file format is generic text and not tied to any single vendor or device. The topology core consists of kernel and userspace components :- Kernel. The kernel core reads in the topology data and builds the driver capabilities and audio topology based on this data. This includes building the DAPM graph, setting PCM and DAI capabilities, registering kcontrols and registering DAPM widgets. The userspace tool allows the creation of binary DSP FW topology files based on a topology text files. The tool is temporarily available on github and its intended to be merged into the alsa-utils package after a clean up. https://github.com/lgirdwood/topology.git Changes since V2 :- o Moved TLV channel maps into TLV UAPI header. o Report error in logs when no handler exists for vendor data types. o Fix comments on kcontrol/widget ops to match core code. Changes since RFC :- o Fixed patch series to survive bisection builds. o Removed widget ID enum from UAPI. o Added channel mappings to controls. o Added TLV macro patch to UAPI. o Fixed locking for runtime and init time topology loading. o Moved the UAPI group index field to the block header. Changes since V1 (back in 2012) :- o Name changed to ASoC Topology o Structures from soc.h and soc-dapm.h no longer exposed as UAPIs o Moved to component model from codec/platform model. o Added reserved fields in each UAPI structure. o UAPI structures packed and endianess defined for each structure. o Added support for PCM, DAI and codec <-> codec link capabilities. o Added support for client drivers manifests. o Added support for private data for each dynamic object type. o Single list of dynamic objects rather than list of each object type. Liam

1 0

[alsa-devel] [PATCH] sound/oss: use schedule_timeout_interruptible()
by Nicholas Mc Guire 29 May '15

29 May '15

API consolidation with coccinelle found: ./sound/oss/msnd_pinnacle.c:1292:2-18: consolidation with schedule_timeout_*() recommended This is a 1:1 conversion of the current calls to an available helper only - so only an API consolidation to improve readability. Patch was compile tested with x86_64_defconfig Patch is against 4.1-rc5 (localversion-next is -next-20150529) Signed-off-by: Nicholas Mc Guire <hofrat(a)osadl.org> --- sound/oss/msnd_pinnacle.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/sound/oss/msnd_pinnacle.c b/sound/oss/msnd_pinnacle.c index a8ceef8..a8bb4a0 100644 --- a/sound/oss/msnd_pinnacle.c +++ b/sound/oss/msnd_pinnacle.c @@ -1288,8 +1288,7 @@ static int __init calibrate_adc(WORD srate) & ~0x0001, dev.SMA + SMA_wCurrHostStatusFlags); if (msnd_send_word(&dev, 0, 0, HDEXAR_CAL_A_TO_D) == 0 && chk_send_dsp_cmd(&dev, HDEX_AUX_REQ) == 0) { - __set_current_state(TASK_INTERRUPTIBLE); - schedule_timeout(HZ / 3); + schedule_timeout_interruptible(HZ / 3); return 0; } printk(KERN_WARNING LOGNAME ": ADC calibration failed\n"); -- 1.7.10.4

2 1

[alsa-devel] [PATCH] ALSA: jack: Fix the id uniqueness check
by Takashi Iwai 29 May '15

29 May '15

snd_kctl_jack_new() tries to assign a unique index number when a name string that has been already registered is passed. However, it checks with the base string without "Jack" suffix, so it never hits. Fix the call with the properly processed name string instead. Fixes: b8dd086674cf 'ALSA: Jack: handle jack embedded kcontrol creating within ctljack') Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/core/ctljack.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/core/ctljack.c b/sound/core/ctljack.c index 8f8d1033425c..9149a4aefa95 100644 --- a/sound/core/ctljack.c +++ b/sound/core/ctljack.c @@ -70,7 +70,7 @@ snd_kctl_jack_new(const char *name, struct snd_card *card) return NULL; jack_kctl_name_gen(kctl->id.name, name, sizeof(kctl->id.name)); - kctl->id.index = get_available_index(card, name); + kctl->id.index = get_available_index(card, kctl->id.name); kctl->private_value = 0; return kctl; } -- 2.4.2

1 0

[alsa-devel] [PATCH] ALSA: usb-audio: Add mic volume fix quirk for Logitech Quickcam Fusion
by Wolfram Sang 29 May '15

29 May '15

Fix this from the logs: usb 7-1: New USB device found, idVendor=046d, idProduct=08ca ... usb 7-1: Warning! Unlikely big volume range (=3072), cval->res is probably wrong. usb 7-1: [5] FU [Mic Capture Volume] ch = 1, val = 4608/7680/1 Signed-off-by: Wolfram Sang <wsa(a)the-dreams.de> --- sound/usb/mixer.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c index 3e2ef61c627b83..4256fdedbd8009 100644 --- a/sound/usb/mixer.c +++ b/sound/usb/mixer.c @@ -918,6 +918,7 @@ static void volume_control_quirks(struct usb_mixer_elem_info *cval, case USB_ID(0x046d, 0x081d): /* HD Webcam c510 */ case USB_ID(0x046d, 0x0825): /* HD Webcam c270 */ case USB_ID(0x046d, 0x0826): /* HD Webcam c525 */ + case USB_ID(0x046d, 0x08ca): /* Logitech Quickcam Fusion */ case USB_ID(0x046d, 0x0991): /* Most audio usb devices lie about volume resolution. * Most Logitech webcams have res = 384. -- 2.1.4

2 1

Re: [alsa-devel] Add dell new platform support
by Takashi Iwai 29 May '15

29 May '15

At Fri, 29 May 2015 07:46:36 +0000, Kailang wrote: > > Hi Takashi, > > Dell new platform with ALC256 codec. SSID is 0x0706. > Attach patch is add the verb table quirk to enable headset mode. Thanks, applied. Takashi

1 0