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

[alsa-devel] ASoC fixes for v4.1
by Mark Brown 30 Apr '15

30 Apr '15

The following changes since commit b787f68c36d49bb1d9236f403813641efa74a031: Linux 4.1-rc1 (2015-04-26 17:59:10 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-v4.1-rc1 for you to fetch changes up to d839c98f98826f0c13e46d5a4cecb46dd357b50e: Merge remote-tracking branches 'asoc/fix/rt5677', 'asoc/fix/samsung' and 'asoc/fix/tfa9879' into asoc-linus (2015-04-29 13:37:31 +0100) ---------------------------------------------------------------- ASoC: Fixes for v4.1 A few fixes for v4.1, none earth shattering and mostly driver related except for one change to fix !PM builds for Intel platforms which is done by adding stubs in the core so other platforms don't run into the same issue. ---------------------------------------------------------------- Axel Lin (1): ASoC: rt5645: Fix mask for setting RT5645_DMIC_2_DP_GPIO12 bit Bard Liao (2): ASoC: rt5677: add register patch for PLL ASoC: rt5677: fixed wrong DMIC ref clock Charles Keepax (1): ASoC: dapm: Enable autodisable on SOC_DAPM_SINGLE_TLV_AUTODISABLE Fabio Estevam (1): ASoC: fsl_ssi: Fix platform_get_irq() error handling Fang, Yang A (1): ASoC: rt5645: Add ACPI match ID Jie Yang (1): ASoC: add static inline funcs to fix a compiling issue Jin Yao (1): ASoC: Intel: Remove invalid kfree of devm allocated data Kuninori Morimoto (1): ASoC: rsnd: set dmaen->chan = NULL when error case Mark Brown (5): Merge remote-tracking branch 'asoc/fix/dapm' into asoc-linus Merge remote-tracking branch 'asoc/fix/intel' into asoc-linus Merge remote-tracking branch 'asoc/fix/rt5645' into asoc-linus Merge remote-tracking branches 'asoc/fix/email', 'asoc/fix/fsl-ssi', 'asoc/fix/pm', 'asoc/fix/qcom' and 'asoc/fix/rcar' into asoc-linus Merge remote-tracking branches 'asoc/fix/rt5677', 'asoc/fix/samsung' and 'asoc/fix/tfa9879' into asoc-linus Rajeev Kumar (1): ASoC: Update email-id of Rajeev Kumar Takashi Iwai (1): ASoC: qcom: Return an error for invalid PCM trigger command Vinod Koul (1): ASoC: Intel: fix the makefile for atom code Wei Yongjun (3): ASoC: tfa9879: Fix return value check in tfa9879_i2c_probe() ASoC: samsung: s3c24xx-i2s: Fix return value check in s3c24xx_iis_dev_probe() ASoC: Intel: sst_byt: remove kfree for memory allocated with devm_kzalloc include/sound/designware_i2s.h | 2 +- include/sound/soc-dapm.h | 2 +- include/sound/soc.h | 12 ++++++++++++ include/sound/spear_dma.h | 2 +- sound/soc/codecs/rt5645.c | 13 ++++++++++++- sound/soc/codecs/rt5677.c | 5 ++++- sound/soc/codecs/tfa9879.c | 4 ++-- sound/soc/fsl/fsl_ssi.c | 2 +- sound/soc/intel/Makefile | 2 +- sound/soc/intel/baytrail/sst-baytrail-ipc.c | 1 - sound/soc/intel/haswell/sst-haswell-ipc.c | 1 - sound/soc/qcom/lpass-cpu.c | 2 +- sound/soc/samsung/s3c24xx-i2s.c | 4 ++-- sound/soc/sh/rcar/dma.c | 1 + 14 files changed, 39 insertions(+), 14 deletions(-)

2 1

[alsa-devel] [PATCH 1/5] ASoC: Correct typo in SOC_VALUE_ENUM_SINGLE macro
by Charles Keepax 30 Apr '15

30 Apr '15

xnitmes is clearly intended to be xnitems, but all other macros just refer to this as xitems, so change it to that. Signed-off-by: Charles Keepax <ckeepax(a)opensource.wolfsonmicro.com> --- include/sound/soc.h | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/sound/soc.h b/include/sound/soc.h index a73d855..45cfd69 100644 --- a/include/sound/soc.h +++ b/include/sound/soc.h @@ -190,8 +190,8 @@ #define SOC_VALUE_ENUM_DOUBLE(xreg, xshift_l, xshift_r, xmask, xitems, xtexts, xvalues) \ { .reg = xreg, .shift_l = xshift_l, .shift_r = xshift_r, \ .mask = xmask, .items = xitems, .texts = xtexts, .values = xvalues} -#define SOC_VALUE_ENUM_SINGLE(xreg, xshift, xmask, xnitmes, xtexts, xvalues) \ - SOC_VALUE_ENUM_DOUBLE(xreg, xshift, xshift, xmask, xnitmes, xtexts, xvalues) +#define SOC_VALUE_ENUM_SINGLE(xreg, xshift, xmask, xitems, xtexts, xvalues) \ + SOC_VALUE_ENUM_DOUBLE(xreg, xshift, xshift, xmask, xitems, xtexts, xvalues) #define SOC_ENUM_SINGLE_VIRT(xitems, xtexts) \ SOC_ENUM_SINGLE(SND_SOC_NOPM, 0, xitems, xtexts) #define SOC_ENUM(xname, xenum) \ -- 1.7.2.5

3 8

[alsa-devel] [RFC 0/7] ASoC: hda - ASoC DSP widget event handlers
by Vinod Koul 30 Apr '15

30 Apr '15

The aDSP is triggered by DAPM. The DAPM code is atm dependent upon the DFW aka tplg RFC send by Liam. We will post missing tplg to driver links once that series is accepted This series is dependent upon a) Takashi topic/hda which has core hdac splitup b) SKL PCM driver series, which adds the SKL HDA Platform driver c) SKL IPC series which sends IPCs to aDSP This last series adds the missing link between b) and c). Comments most welcome -- ~Vinod Jeeja KP (7): ASoC: hda: Add skl dsp init and registering with SST IPC lib ASoC: hda: add hepler to configure module params ASoC: hda: add FW module init/bind IPC ASoC: hda: add FW pipe create/delete/set_pipe_state IPC ASOC: hda: add DSP platfrom controls widget event handlers ASoC: hda: Add support for SSP register settings ASoC: hda: Apply dai params_fixup for DSP widgets include/sound/soc-hda-sst-dsp.h | 3 + sound/soc/hda/Kconfig | 1 + sound/soc/hda/Makefile | 3 +- sound/soc/hda/hda_dsp_controls.c | 913 ++++++++++++++++++++++++++++++++++++ sound/soc/hda/hda_dsp_controls.h | 338 +++++++++++++ sound/soc/hda/hda_dsp_ssp_config.c | 290 ++++++++++++ sound/soc/hda/hda_dsp_ssp_config.h | 423 +++++++++++++++++ sound/soc/hda/hda_skl.h | 108 ++++- sound/soc/hda/hda_soc_dsp.c | 702 +++++++++++++++++++++++++++ 9 files changed, 2767 insertions(+), 14 deletions(-) create mode 100644 sound/soc/hda/hda_dsp_controls.c create mode 100644 sound/soc/hda/hda_dsp_controls.h create mode 100644 sound/soc/hda/hda_dsp_ssp_config.c create mode 100644 sound/soc/hda/hda_dsp_ssp_config.h create mode 100644 sound/soc/hda/hda_soc_dsp.c -- 1.7.9.5

3 21

[alsa-devel] Simple Mixer help
by Bram Bos 30 Apr '15

30 Apr '15

I want to get access to the ALSA Simple Mixer interface, but rather than the 'default' sound device I want to target a specific USB sound device. I'm at a loss how to refer to it... I am using a modified version of the code below but I have no idea what to replace the "card" and "selem_name" strings with. The device name? The 'hwX.Y' string? And do I need to change the index#? Too many variables to solve this by trial and error. Any hints appreciated! void SetAlsaMasterVolume(long volume){ long min, max; snd_mixer_t *handle; snd_mixer_selem_id_t *sid; const char *card = "default"; const char *selem_name = "Master"; snd_mixer_open(&handle, 0); snd_mixer_attach(handle, card); snd_mixer_selem_register(handle, NULL, NULL); snd_mixer_load(handle); snd_mixer_selem_id_alloca(&sid); snd_mixer_selem_id_set_index(sid, 0); snd_mixer_selem_id_set_name(sid, selem_name); snd_mixer_elem_t* elem = snd_mixer_find_selem(handle, sid); snd_mixer_selem_get_playback_volume_range(elem, &min, &max); snd_mixer_selem_set_playback_volume_all(elem, volume * max / 100); snd_mixer_close(handle);}

3 4

[alsa-devel] [for-next] NULL pointer dereference in codec probe
by Jani Nikula 30 Apr '15

30 Apr '15

Hi all, our graphics QA found an issue [1] with the for-next branch of [2] merged on top of v4.1-rc1. Backtrace in the bug. I didn't really have time to look into the details, but just letting you know. BR, Jani. [1] https://bugs.freedesktop.org/show_bug.cgi?id=90229 [2] git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git -- Jani Nikula, Intel Open Source Technology Center

3 5

[alsa-devel] [PATCH lib 0/5] alsa-lib namehint fixes
by Takashi Iwai 30 Apr '15

30 Apr '15

Hi, here is a series of fixes for alsa-lib namehint stuff. Currently snd_device_name_hint() doesn't return proper lists except for PCM. Also it may do bogus free() or segfault for wrong operations. This patchset addresses such issues and also adds the missing hint descriptions to ctl, seq, hwdep, etc. Takashi === Takashi Iwai (5): namehint: Fix invalid list access in snd_device_name_hint() namehint: Fix the listing without device number namehint: Fix bad free with invalid iface name Allow hint for ctl, hwdep, timer and seq conf: Add hint descriptions to ctl, hwdep, seq and timer devices include/local.h | 2 ++ src/conf/alsa.conf | 15 ++++++++++++++- src/confmisc.c | 13 +++++++++++++ src/control/control_hw.c | 4 +--- src/control/control_shm.c | 7 ++----- src/control/namehint.c | 26 ++++++++++++-------------- src/hwdep/hwdep_hw.c | 4 +--- src/pcm/pcm.c | 12 ------------ src/pcm/pcm_local.h | 5 ++--- src/rawmidi/rawmidi.c | 18 ------------------ src/rawmidi/rawmidi_local.h | 2 +- src/seq/seq_hw.c | 4 +--- src/timer/timer_hw.c | 4 +--- src/timer/timer_query_hw.c | 4 +--- 14 files changed, 51 insertions(+), 69 deletions(-) -- 2.3.6

1 5

[alsa-devel] [PATCH v4 0/2] Add support for select accessory detect mode to HPDETL or HPDETR
by Inha Song 30 Apr '15

30 Apr '15

This set of patches adds support for select accessory detect mode to HPDETL or HPDETR. Changes in v4: - Set the hpdet_channel default value to HPDETL in variable declaration. - Fix the indentation in binding documentation. Changes in v3: - Set the hpdet_channel default value to HPDETL If the value is unknown or invalid. Changes in v2: - Use the value in pdata instead of hpdet_channel in extcon_info. - Wrap arizona_extcon_of_get in IS_ENABLED(CONFIG_OF). - Change hpdet_channel type to unsigned from signed in pdata. - Move ARIZONA_ACCDET_MODE_* define to dt-binding header and directly set it to pdata. Inha Song (2): extcon: arizona: Add support for select accessory detect mode when headphone detection mfd: arizona: Update DT binding to support hpdet channel Documentation/devicetree/bindings/mfd/arizona.txt | 6 ++++ drivers/extcon/extcon-arizona.c | 38 ++++++++++++++++++----- include/dt-bindings/mfd/arizona.h | 8 +++++ include/linux/mfd/arizona/pdata.h | 3 ++ 4 files changed, 47 insertions(+), 8 deletions(-) create mode 100644 include/dt-bindings/mfd/arizona.h -- 2.0.0.390.gcb682f8

4 7

[alsa-devel] [PATCH] ASoC: rt5645: fix implicit declaration error
by Bard Liao 30 Apr '15

30 Apr '15

kbuild robot reports a implicit declaration of function 'rt5645_irq_detection' error. Signed-off-by: Bard Liao <bardliao(a)realtek.com> --- sound/soc/codecs/rt5645.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/soc/codecs/rt5645.c b/sound/soc/codecs/rt5645.c index a81a24f..8bad9fe 100644 --- a/sound/soc/codecs/rt5645.c +++ b/sound/soc/codecs/rt5645.c @@ -2535,6 +2535,8 @@ static int rt5645_jack_detect(struct snd_soc_codec *codec, int jack_insert) return rt5645->jack_type; } +static int rt5645_irq_detection(struct rt5645_priv *rt5645); + int rt5645_set_jack_detect(struct snd_soc_codec *codec, struct snd_soc_jack *hp_jack, struct snd_soc_jack *mic_jack, struct snd_soc_jack *btn_jack) -- 1.8.1.1.439.g50a6b54

2 1

[alsa-devel] [PATCH] ALSA: hda - remove check on CONFIG_SND_HDA_I915 in azx_probe_continue()
by mengdong.lin＠intel.com 30 Apr '15

30 Apr '15

From: Mengdong Lin <mengdong.lin(a)intel.com> This is to avoid a build warning that label 'i915_power_fail' is defined but not used. There are already fake hda_i915 functions when CONFIG_SND_HDA_I915 is not defined, and error message for missing CONFIG_SND_HDA_I915 for the HD-A controller or codec that depends on i915 power well. Signed-off-by: Mengdong Lin <mengdong.lin(a)intel.com> diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index 5aa5cfa..87df90d 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -1907,8 +1907,6 @@ static int azx_probe_continue(struct azx *chip) if (pci->device != 0x0f04 && pci->device != 0x2284) hda->need_i915_power = 1; - -#ifdef CONFIG_SND_HDA_I915 err = hda_i915_init(hda); if (err < 0) goto i915_power_fail; @@ -1919,7 +1917,6 @@ static int azx_probe_continue(struct azx *chip) "Cannot turn on display power on i915\n"); goto i915_power_fail; } -#endif } err = azx_first_init(chip); -- 1.9.1

2 1

[alsa-devel] [PATCH v3 0/5] ALSA: hda - remove unnecessary controller dependency on i915 gfx power
by mengdong.lin＠intel.com 30 Apr '15

30 Apr '15

From: Mengdong Lin <mengdong.lin(a)intel.com> Mengdong Lin (5): ALSA: hda - implement a refcount for i915 power well switch ALSA: hda - allow a codec to control the link power ALSA: hda - implement link_power ops for i915 display power control ALSA: hda - divide controller and codec dependency on i915 gfx power well ALSA: hda - remove controller dependency on i915 power well for Baytrail/Braswell include/sound/hdaudio.h | 4 ++++ sound/hda/hdac_device.c | 15 ++++++++++++++ sound/pci/hda/hda_codec.c | 4 ++++ sound/pci/hda/hda_controller.c | 11 ++++++++++ sound/pci/hda/hda_controller.h | 2 ++ sound/pci/hda/hda_i915.c | 18 +++++++++++++---- sound/pci/hda/hda_intel.c | 46 ++++++++++++++++++++++++++++++++++-------- sound/pci/hda/hda_intel.h | 2 ++ sound/pci/hda/patch_hdmi.c | 9 +++++++++ 9 files changed, 99 insertions(+), 12 deletions(-) -- 1.9.1

3 8