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

[alsa-devel] [PATCH] ASoC: img-parallel-out: Add pm_runtime_get/put to set_fmt callback
by Ed Blake 04 Oct '17

04 Oct '17

Add pm_runtime_get_sync and pm_runtime_put calls to set_fmt callback function. This fixes a bus error during boot when CONFIG_SUSPEND is defined when this function gets called while the device is runtime disabled and device registers are accessed while the clock is disabled. Signed-off-by: Ed Blake <ed.blake(a)sondrel.com> --- sound/soc/img/img-parallel-out.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/soc/img/img-parallel-out.c b/sound/soc/img/img-parallel-out.c index 23b0f0f..2fc8a63 100644 --- a/sound/soc/img/img-parallel-out.c +++ b/sound/soc/img/img-parallel-out.c @@ -164,9 +164,11 @@ static int img_prl_out_set_fmt(struct snd_soc_dai *dai, unsigned int fmt) return -EINVAL; } + pm_runtime_get_sync(prl->dev); reg = img_prl_out_readl(prl, IMG_PRL_OUT_CTL); reg = (reg & ~IMG_PRL_OUT_CTL_EDGE_MASK) | control_set; img_prl_out_writel(prl, reg, IMG_PRL_OUT_CTL); + pm_runtime_put(prl->dev); return 0; } -- 1.9.1

2 2

[alsa-devel] [PATCH] ASoC: codecs: msm8916-wcd-analog: fix micbias level
by Jean-François Têtu 04 Oct '17

04 Oct '17

The macro used to set the microphone bias level causes the snd_soc_write() call to overwrite other fields in the CDC_A_MICB_1_VAL register. The macro also does not return the proper level value to use. This fixes this by preserving all bits from the register that are not the level while setting the level. Signed-off-by: Jean-François Têtu <jean-francois.tetu(a)savoirfairelinux.com> --- sound/soc/codecs/msm8916-wcd-analog.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/sound/soc/codecs/msm8916-wcd-analog.c b/sound/soc/codecs/msm8916-wcd-analog.c index f562f2d..e1534fe 100644 --- a/sound/soc/codecs/msm8916-wcd-analog.c +++ b/sound/soc/codecs/msm8916-wcd-analog.c @@ -104,7 +104,7 @@ #define CDC_A_MICB_1_VAL (0xf141) #define MICB_MIN_VAL 1600 #define MICB_STEP_SIZE 50 -#define MICB_VOLTAGE_REGVAL(v) ((v - MICB_MIN_VAL)/MICB_STEP_SIZE) +#define MICB_VOLTAGE_REGVAL(v) (((v - MICB_MIN_VAL)/MICB_STEP_SIZE) << 3) #define MICB_1_VAL_MICB_OUT_VAL_MASK GENMASK(7, 3) #define MICB_1_VAL_MICB_OUT_VAL_V2P70V ((0x16) << 3) #define MICB_1_VAL_MICB_OUT_VAL_V1P80V ((0x4) << 3) @@ -349,8 +349,9 @@ static void pm8916_wcd_analog_micbias_enable(struct snd_soc_codec *codec) | MICB_1_CTL_EXT_PRECHARG_EN_ENABLE); if (wcd->micbias_mv) { - snd_soc_write(codec, CDC_A_MICB_1_VAL, - MICB_VOLTAGE_REGVAL(wcd->micbias_mv)); + snd_soc_update_bits(codec, CDC_A_MICB_1_VAL, + MICB_1_VAL_MICB_OUT_VAL_MASK, + MICB_VOLTAGE_REGVAL(wcd->micbias_mv)); /* * Special headset needs MICBIAS as 2.7V so wait for * 50 msec for the MICBIAS to reach 2.7 volts. -- 2.7.4

3 2

[alsa-devel] [PATCH 0/6 v3] ASoC: rsnd: add MIX Volume Ramp support
by Kuninori Morimoto 04 Oct '17

04 Oct '17

Hi Mark These are v3 of MIX Volume Ramp support on Renesas R-Car sound. 0001 - 0005 are prepare/cleanup patch for it. v2 -> v3 - tidyup typo: INITIALIZEED -> INITIALIZED - flags macro Kuninori Morimoto (6): ASoC: rsnd: add generic rsnd_flags_xxx() macro ASoC: rsnd: use generic rsnd_flags_xxx() macro on ADG ASoC: rsnd: DVC kctrl sets once ASoC: rsnd: CTU kctrl sets once ASoC: rsnd: makes volume ramp rate list generic ASoC: rsnd: add MIX Volume Ramp support sound/soc/sh/rcar/adg.c | 13 ++-- sound/soc/sh/rcar/core.c | 27 ++++++++ sound/soc/sh/rcar/ctu.c | 8 +++ sound/soc/sh/rcar/dvc.c | 41 ++++-------- sound/soc/sh/rcar/mix.c | 158 +++++++++++++++++++++++++++++++++++++++++++++-- sound/soc/sh/rcar/rsnd.h | 12 +++- sound/soc/sh/rcar/ssi.c | 27 ++++---- 7 files changed, 227 insertions(+), 59 deletions(-) -- 1.9.1

2 10

[alsa-devel] [PATCH 0/6] ASoC: rsnd: add MIX Volume Ramp support
by Kuninori Morimoto 04 Oct '17

04 Oct '17

Hi Mark These adds MIX Volume Ramp support on Renesas R-Car sound. 0001 - 0005 are prepare/cleanup patch for it. Kuninori Morimoto (6): ASoC: rsnd: add generic rsnd_flags_xxx() macro ASoC: rsnd: use generic rsnd_flags_xxx() macro on ADG ASoC: rsnd: DVC kctrl sets once ASoC: rsnd: CTU kctrl sets once ASoC: rsnd: makes volume ramp rate list generic ASoC: rsnd: add MIX Volume Ramp support sound/soc/sh/rcar/adg.c | 13 ++-- sound/soc/sh/rcar/core.c | 27 ++++++++ sound/soc/sh/rcar/ctu.c | 8 +++ sound/soc/sh/rcar/dvc.c | 41 ++++-------- sound/soc/sh/rcar/mix.c | 158 +++++++++++++++++++++++++++++++++++++++++++++-- sound/soc/sh/rcar/rsnd.h | 12 +++- sound/soc/sh/rcar/ssi.c | 27 ++++---- 7 files changed, 227 insertions(+), 59 deletions(-) -- 1.9.1

3 12

[alsa-devel] [PATCH 0/6 v2] ASoC: rsnd: add MIX Volume Ramp support
by Kuninori Morimoto 04 Oct '17

04 Oct '17

Hi Mark These are v2 of MIX Volume Ramp support on Renesas R-Car sound. 0001 - 0005 are prepare/cleanup patch for it. Kuninori Morimoto (6): ASoC: rsnd: add generic rsnd_flags_xxx() macro ASoC: rsnd: use generic rsnd_flags_xxx() macro on ADG ASoC: rsnd: DVC kctrl sets once ASoC: rsnd: CTU kctrl sets once ASoC: rsnd: makes volume ramp rate list generic ASoC: rsnd: add MIX Volume Ramp support sound/soc/sh/rcar/adg.c | 13 ++-- sound/soc/sh/rcar/core.c | 27 ++++++++ sound/soc/sh/rcar/ctu.c | 8 +++ sound/soc/sh/rcar/dvc.c | 41 ++++-------- sound/soc/sh/rcar/mix.c | 158 +++++++++++++++++++++++++++++++++++++++++++++-- sound/soc/sh/rcar/rsnd.h | 12 +++- sound/soc/sh/rcar/ssi.c | 27 ++++---- 7 files changed, 227 insertions(+), 59 deletions(-) -- 1.9.1

2 7

[alsa-devel] [PATCH 0/9] ASoC: cygnus: Various improvements and fixes
by Lori Hikichi 04 Oct '17

04 Oct '17

This patch series contains an number of improvements and refinements to the driver. There is also a fix for a problem when transferring four or more channels in TDM mode. Lori Hikichi (9): ASoC: cygnus: Add support for 384kHz frame rates ASoC: cygnus: Update bindings for audio clock changes ASoC: cygnus: Allow each port to select its clock source ASoC: cygnus: Only enable MCLK pins when in use ASoC: cygnus: Remove support for 8 bit audio and for mono ASoc: cygnus: Fix problems with multichannel transfers ASoC: cygnus: Remove set_fmt from SPDIF dai ops ASoC: cygnus: Add EXPORT_SYMBOL for helper function ASoC: cygnus: Tidy up of structure access .../bindings/sound/brcm,cygnus-audio.txt | 70 +- sound/soc/bcm/cygnus-pcm.c | 50 +- sound/soc/bcm/cygnus-ssp.c | 1336 ++++++++++---------- sound/soc/bcm/cygnus-ssp.h | 51 +- 4 files changed, 739 insertions(+), 768 deletions(-) -- 1.9.1

2 18

[alsa-devel] [PATCH v1 0/3] Minor updates to driver
by Lori Hikichi 04 Oct '17

04 Oct '17

This patchset contains a few small updates to the DAI driver. Lori Hikichi (3): ASoC: cygnus: Add EXPORT_SYMBOL for helper function ASoC: cygnus: Remove set_fmt from SPDIF dai ops ASoC: cygnus: Remove support for 8 bit audio and for mono sound/soc/bcm/cygnus-ssp.c | 32 +++++++++++++------------------- 1 file changed, 13 insertions(+), 19 deletions(-) -- 1.9.1

2 7

[alsa-devel] [PATCH 0/5] ASoC: soc-core: component functions cleanup
by Kuninori Morimoto 04 Oct '17

04 Oct '17

Hi Mark These are component functions customize. And is preparation for platform replace to component. I'm posting platform replace patch-set [RFC] to ML. These are preparation but independent from it. Kuninori Morimoto (5): ASoC: soc-core: rename "cmpnt" to "component" ASoC: soc-core: add component lookup functions ASoC: soc-core: add component /unregister_exp/remove functions ASoC: soc-core: add snd_soc_add_component() ASoC: soc-devres: use expanded unregister on devm register component include/sound/soc.h | 16 ++++++-- sound/soc/soc-core.c | 102 ++++++++++++++++++++++++++++++++++--------------- sound/soc/soc-devres.c | 6 +-- 3 files changed, 87 insertions(+), 37 deletions(-) -- 1.9.1

2 7

[alsa-devel] [PATCH] ASoC: codecs: msm8916-wcd-analog: fix module autoload
by Nicolas Dechesne 03 Oct '17

03 Oct '17

If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device with the corresponding module. Export the module alias information using the MODULE_DEVICE_TABLE() macro. Before this patch: $ modinfo snd_soc_msm8916_analog | grep alias $ After this patch: $ modinfo snd_soc_msm8916_analog | grep alias alias: of:N*T*Cqcom,pm8916-wcd-analog-codecC* alias: of:N*T*Cqcom,pm8916-wcd-analog-codec Signed-off-by: Nicolas Dechesne <nicolas.dechesne(a)linaro.org> --- sound/soc/codecs/msm8916-wcd-analog.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/soc/codecs/msm8916-wcd-analog.c b/sound/soc/codecs/msm8916-wcd-analog.c index 549c269acc7d..d79e5b35e003 100644 --- a/sound/soc/codecs/msm8916-wcd-analog.c +++ b/sound/soc/codecs/msm8916-wcd-analog.c @@ -1241,6 +1241,8 @@ static const struct of_device_id pm8916_wcd_analog_spmi_match_table[] = { { } }; +MODULE_DEVICE_TABLE(of, pm8916_wcd_analog_spmi_match_table); + static struct platform_driver pm8916_wcd_analog_spmi_driver = { .driver = { .name = "qcom,pm8916-wcd-spmi-codec", -- 2.11.0

2 1

[alsa-devel] "alsactl store" fails to save Echo Gina3G mixer
by S. Christian Collins 03 Oct '17

03 Oct '17

Greetings, I have two PCI sound cards in my PC: 1. Sound Blaster Audigy 2 ZS Gold (primary card) 2. Echo Gina3G If I save the mixer for the Audigy 2 via: alsactl -f ~/asound0.state store 0 ...the Audigy 2's mixer is properly saved. However, when I try to save the Gina3G mixer: alsactl -f ~/asound1.state store 1 ...I get the following error: alsactl: control.c:2513: snd_ctl_elem_value_get_integer: Assertion `idx < sizeof(obj->value.integer.value) / sizeof(obj->value.integer.value[0])' failed. Aborted I have tried to find any info on this error online, but to no avail. Does anybody know what is going on here? ** My System ** OS: KDE Neon 5.10.5 64-bit (Plasma Desktop 5.10.5, KDE Frameworks 5.37.0, Qt 5.9.1) Linux Kernel: 4.10.0.32-generic Motherboard: ASRock X58 Extreme3 (Intel X58 chipset) CPU: Intel Core i7 930 (2.8 GHz quad-core) RAM: 12GB DDR3 Video: NVIDIA GeForce GTX 970 w/ 4GB RAM (PCI Express) Sound Card #1: Sound Blaster Audigy 2 ZS Gold Sound Card #2: Echo Gina3G NVIDIA video driver: 375.66

3 11