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

[alsa-devel] [PATCH] ASoC: audio-graph-scu-card: fix balance of of_node_get/put()
by Kuninori Morimoto 31 Jul '17

31 Jul '17

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> audio-graph-scu-card with CONFIG_OF_DYNAMIC triggers several errors at kernel boot, like the following one OF: ERROR: Bad of_node_put() on /soc/sound@ec500000/ports/port@1 This patch doesn't call unnecessary of_node_put() which was called in other function (= of_graph_get_port_parent(), of_phandle_iterator_next()) Reported-by: Antonio Borneo <borneo.antonio(a)gmail.com> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/generic/audio-graph-scu-card.c | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/sound/soc/generic/audio-graph-scu-card.c b/sound/soc/generic/audio-graph-scu-card.c index dcd2df3..bff1e6a 100644 --- a/sound/soc/generic/audio-graph-scu-card.c +++ b/sound/soc/generic/audio-graph-scu-card.c @@ -215,7 +215,6 @@ static int asoc_graph_card_parse_of(struct graph_card_data *priv) codec_ep = of_graph_get_remote_endpoint(cpu_ep); rcpu_ep = of_graph_get_remote_endpoint(codec_ep); - of_node_put(cpu_port); of_node_put(cpu_ep); of_node_put(codec_ep); of_node_put(rcpu_ep); @@ -248,11 +247,9 @@ static int asoc_graph_card_parse_of(struct graph_card_data *priv) cpu_port = it.node; cpu_ep = of_get_next_child(cpu_port, NULL); codec_ep = of_graph_get_remote_endpoint(cpu_ep); - codec_port = of_graph_get_port_parent(codec_ep); + codec_port = of_graph_get_port_parent(codec_ep); /* call put */ - of_node_put(cpu_port); of_node_put(cpu_ep); - of_node_put(codec_ep); of_node_put(codec_port); if (codec) { @@ -304,11 +301,9 @@ static int asoc_graph_get_dais_count(struct device *dev) cpu_port = it.node; cpu_ep = of_get_next_child(cpu_port, NULL); codec_ep = of_graph_get_remote_endpoint(cpu_ep); - codec_port = of_graph_get_port_parent(codec_ep); + codec_port = of_graph_get_port_parent(codec_ep); /* call put */ - of_node_put(cpu_port); of_node_put(cpu_ep); - of_node_put(codec_ep); of_node_put(codec_port); if (cpu_ep) -- 1.9.1

1 1

[alsa-devel] snd_ctl_add_enumerated_elem_set and TLV
by Fulup Ar Foll 31 Jul '17

31 Jul '17

Hi, I'm using snd_ctl_add_enumerated_elem_set to create user controls, I do not understand why it set "TLV readable" option at true by default. I have a simple enumeration without TLV. I use snd_ctl_elem_write but do not call snd_ctl_elem_tlv_write. I do no understand how to prevent snd_ctl_elem_info_is_tlv_readable to return true for this control, obviously even if TLV is set as readable, it is empty and reading fail. Example fulup@fulup-desktop:~/Workspace/AlsaPulse> amixer -D hw:USB cget numid=237 numid=237,iface=MIXER,name='Hal-VolRamp-Mode' ; type=ENUMERATED,access=rw---RW-,values=1,items=4 ; Item #0 'None' ; Item #1 'Normal' ; Item #2 'Smooth' ; Item #3 'Emergency' : values=1 amixer: Control hw:USB element TLV read error: No such device or address Question: is there something special to set when a control does not have a TLV ? Fulup PS: if I write a fake TLV everything goes fine, but this is not what I expect for ENUM.

2 2

[alsa-devel] [PATCH] ASoC: fix balance of of_node_get()/of_node_put()
by Antonio Borneo 30 Jul '17

30 Jul '17

On Hikey target board, enabling CONFIG_OF_DYNAMIC triggers several errors at kernel boot, like the following one OF: ERROR: Bad of_node_put() on /soc/i2s@f7118000/ports/port@0/endpoint each followed by stack dump. Fixed by: - removing of_node_put() in the body of of_for_each_phandle(){}, since already provided at each iteration. Add it in case the loop is break out; - adding of_node_get() before calling of_graph_get_port_parent() or asoc_graph_card_dai_link_of(). Tested with kernel v4.13-rc1 with hikey_defconfig taken from https://git.linaro.org/people/john.stultz/android-dev.git branch dev/hikey-mainline-WIP Signed-off-by: Antonio Borneo <borneo.antonio(a)gmail.com> --- To: Liam Girdwood <lgirdwood(a)gmail.com> To: Mark Brown <broonie(a)kernel.org> To: Jaroslav Kysela <perex(a)perex.cz> To: Takashi Iwai <tiwai(a)suse.com> To: alsa-devel(a)alsa-project.org Cc: linux-kernel(a)vger.kernel.org Cc: Wei Xu <xuwei5(a)hisilicon.com> Cc: John Stultz <john.stultz(a)linaro.org> Cc: linux-arm-kernel(a)lists.infradead.org --- sound/soc/generic/audio-graph-card.c | 14 +++++++++----- sound/soc/generic/simple-card-utils.c | 5 +++++ sound/soc/soc-core.c | 5 +++++ 3 files changed, 19 insertions(+), 5 deletions(-) diff --git a/sound/soc/generic/audio-graph-card.c b/sound/soc/generic/audio-graph-card.c index c0f08a6..7574c5f 100644 --- a/sound/soc/generic/audio-graph-card.c +++ b/sound/soc/generic/audio-graph-card.c @@ -228,10 +228,16 @@ static int asoc_graph_card_parse_of(struct graph_card_data *priv) */ of_for_each_phandle(&it, rc, node, "dais", NULL, 0) { + /* + * asoc_graph_card_dai_link_of() will call + * of_node_put(). So, call of_node_get() here + */ + of_node_get(it.node); ret = asoc_graph_card_dai_link_of(it.node, priv, idx++); - of_node_put(it.node); - if (ret < 0) + if (ret < 0) { + of_node_put(it.node); return ret; + } } return asoc_simple_card_parse_card_name(card, NULL); @@ -244,10 +250,8 @@ static int asoc_graph_get_dais_count(struct device *dev) int count = 0; int rc; - of_for_each_phandle(&it, rc, node, "dais", NULL, 0) { + of_for_each_phandle(&it, rc, node, "dais", NULL, 0) count++; - of_node_put(it.node); - } return count; } diff --git a/sound/soc/generic/simple-card-utils.c b/sound/soc/generic/simple-card-utils.c index 26d64fa..144954b 100644 --- a/sound/soc/generic/simple-card-utils.c +++ b/sound/soc/generic/simple-card-utils.c @@ -250,6 +250,11 @@ static int asoc_simple_card_get_dai_id(struct device_node *ep) if (ret != -ENOTSUPP) return ret; + /* + * of_graph_get_port_parent() will call + * of_node_put(). So, call of_node_get() here + */ + of_node_get(ep); node = of_graph_get_port_parent(ep); /* diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index 921622a..a0f39de 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c @@ -4087,6 +4087,11 @@ int snd_soc_get_dai_id(struct device_node *ep) struct device_node *node; int ret; + /* + * of_graph_get_port_parent() will call + * of_node_put(). So, call of_node_get() here + */ + of_node_get(ep); node = of_graph_get_port_parent(ep); /* -- 1.9.1

3 9

[alsa-devel] Fwd: Fwd: Re: Basic questions
by tsepp2001＠t-online.de 30 Jul '17

30 Jul '17

Hi Clemens, thank you. In blocking-mode it works. Then i will follow the non-blocking mode. I modified first the callback: void playback_callback (snd_pcm_sframes_t nframes) { snd_pcm_sframes_t avail; avail = snd_pcm_avail_update(handle); while (avail >= period_size) { snd_pcm_writei(handle, buffer_alsa , nframes); avail = snd_pcm_avail_update(handle); std::cout << " Avail: " << avail << std::endl; } } ... called it at the end: playback_callback(period_size); and I get (for "avail"): Frames: 6894888 buffer_size (frames): 2048 buffer_time (frames): 46439 period_size (frames): 256 period_time (usec): 5804 periods (count): 8 Avail: 1282 Avail: 1027 Avail: 772 Avail: 517 Avail: 261 Avail: 6 the same mistake... Regards Thomas tsepp2001(a)t-online.de wrote: > Am I right to use async mode No. It is not supported by all devices, and almost impossible to use correctly. > to get lowest latency times Latency depends on the buffer size, and how fast your code can react to the notification that some space is available. > and using pause/play/stop? You probably want to use non-blocking mode. Regards, Clemens _______________________________________________ Alsa-devel mailing list Alsa-devel(a)alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

1 0

[alsa-devel] Gina 3g and Echomixer issue
by Britt Yazel 30 Jul '17

30 Jul '17

Hey guys, I work in an auditory neuroscience research lab, and we are using a PCI Gina 3g to communicate with a Fireface 800 over ADAT. My issue is this, 1) each time I reboot the system the digital mode is set to Coaxial digital, rather than ADAT, and I must set it after each reboot. Is there a way to make this setting persist through reboot, or perhaps a parameter that can be set when the kernel module is loaded to force ADAT? 2) The first time I switch from Coaxial digital to ADAT upon a reboot, I consistently get a loud squeeling sound coming from my studio monitors. I don't think this is the intended behaviuor, and it makes me worried when switching into ADAT mode. Granted, after switching the first time to ADAT, I can switch between coaxial digital and ADAT without the squeeling happeneing, it is just that first time after the reboot. Thanks for the help guys. The Gina 3g, even though it is an old audio device, is still our most reliable sound card. We have tried many others with horrible success, and most other ADAT capable PCI/PCIe sound cards are far too expensive, and don't have robust Linux support. However, if you guys have a good reccomendation of an ADAT capable PCIe sound card with full Linux support, please let me know. -Britt

2 1

[alsa-devel] Basic questions
by tsepp2001＠t-online.de 30 Jul '17

30 Jul '17

Hi everybody, I am new to ALSA-API and would be glad, if someone give me an advice to the following questions. Up to now I used Rtaudio for cross compiling the player for windows and Linux. But know I need to go more into ALSA... - Am I right to use async mode to get lowest latency times and using pause/play/stop? - I used the following piece of code. But don't get any output. The callback output shows "avail" with very short count of frames... I write a whole file (raw) to buffer, called buffer_alsa with 6894888 frames. What is going wrong? I get this output: Frames: 6894888 buffer_size (frames): 2048 buffer_time (frames): 46439 period_size (frames): 256 period_time (usec): 5804 periods (count): 8 Avail: 1540 Avail: 1285 Avail: 1030 Avail: 775 Avail: 520 Avail: 265 Avail: 10 Avail: 2 Avail: 2 Avail: 3 Avail: 2 ... The code follows ... Best regards Thomas ----------------------------- void MyCallback(snd_async_handler_t *pcm_callback) { snd_pcm_t *handle = snd_async_handler_get_pcm(pcm_callback); snd_pcm_sframes_t avail; //std::cout << "Period_size: " << period_size << std::endl; avail = snd_pcm_avail_update(handle); while (avail >= period_size) { snd_pcm_writei(handle, buffer_alsa , period_size); avail = snd_pcm_avail_update(handle); std::cout << " Avail: " << avail << std::endl; } } //---------------------------------------------------- snd_pcm_hw_params_alloca(&params); if ((err = snd_pcm_hw_params_any(handle, params)) < 0) std::cout << " Error (reading configuration): " << snd_strerror(err) << std::endl; /* Set parameters */ if ((err = snd_pcm_hw_params_set_access(handle, params,SND_PCM_ACCESS_RW_INTERLEAVED)) < 0) //printf("ERROR: Can't set interleaved mode. %s\n", snd_strerror(err)); std::cout << " Error (can't set interleaved mode): " << snd_strerror(err) << std::endl; if ((err = snd_pcm_hw_params_set_format(handle, params, FORMAT)) < 0) //printf("ERROR: Can't set format. %s\n", snd_strerror(err)); std::cout << " Error (can't set format): " << snd_strerror(err) << std::endl; if ((err = snd_pcm_hw_params_set_channels(handle, params, data.channels)) < 0) //printf("ERROR: Can't set channels number. %s\n", snd_strerror(err)); std::cout << " Error (can't set channels number): " << snd_strerror(err) << std::endl; if ((err = snd_pcm_hw_params_set_rate_near(handle, params, &fs, 0)) < 0) //printf("ERROR: Can't set rate. %s\n", snd_strerror(err)); std::cout << " Error (can't set rate): " << snd_strerror(err) << std::endl; if ((err = snd_pcm_hw_params_set_buffer_size_near(handle, params, &buffer_size)) < 0) std::cout << " Error (can't set buffer size): " << snd_strerror(err) << std::endl; if ((err = snd_pcm_hw_params_set_period_size_near (handle, params, &period_size, NULL)) < 0) std::cout << " Error (can't set period size): " << snd_strerror(err) << std::endl; /* Write parameters */ if ((err = snd_pcm_hw_params(handle, params)) < 0) //printf("ERROR: Can't set hardware parameters. %s\n", snd_strerror(err)); std::cout << " Error (can't set harware params): " << snd_strerror(err) << std::endl; //---------------------- ------------------------------ // Get Parameters snd_pcm_hw_params_get_buffer_size(params, &buffer_sz); std::cout << " buffer_size (frames): " << buffer_sz << std::endl; snd_pcm_hw_params_get_buffer_time(params, &buffer_tm, NULL); std::cout << " buffer_time (frames): " << buffer_tm << std::endl; snd_pcm_hw_params_get_period_size(params, &aframes, 0); std::cout << " period_size (frames): " << aframes << std::endl; snd_pcm_hw_params_get_period_time(params, &data.channels, NULL); // tmp Zeit in us std::cout << " period_time (usec): " << data.channels << std::endl; snd_pcm_hw_params_get_periods(params, &piods, 0); std::cout << " periods (count): " << piods << std::endl; // ---------------------------------------------------------------------------- //snd_pcm_hw_params_free (params); // --------------------- Hardware-Params End ------------------------------ snd_pcm_sw_params_t *sw_params; if ((err = snd_pcm_sw_params_malloc (&sw_params)) < 0) { std::cout << " Error (can't allocate sw params): " << snd_strerror(err) << std::endl; return; } if ((err = snd_pcm_sw_params_current(handle, sw_params)) < 0) { std::cout << "Error (can't return curr sw config): " << snd_strerror(err) << std::endl; return; } if ((err = snd_pcm_sw_params_set_start_threshold (handle, sw_params, buffer_size)) < 0) // 0U oder 4096 ? { // This parameter controls the wake up point. If the count of available samples is equal or greater than this value, then the application will be activated. std::cout << "Error (can't set start thresh mode): " << snd_strerror(err) << std::endl; return; } /* if ((err = snd_pcm_sw_params_set_silence_threshold( handle, sw_params, 0 )) < 0) { std::cout << " Error (can't set silence thres): " << snd_strerror(err) << std::endl; return; } */ if ((err = snd_pcm_sw_params_set_avail_min (handle, sw_params, period_size)) < 0) { std::cout << " Error (can't set min avail count): " << snd_strerror(err) << std::endl; return; } // here are two options for a fix //snd_pcm_sw_params_set_silence_size( phandle, sw_params, ULONG_MAX ); /* snd_pcm_uframes_t val; if ((err = snd_pcm_sw_params_get_boundary(sw_params, &val)) < 0) { std::cout << " Error (can't get boundary): " << snd_strerror(err) << std::endl; return; } if ((err = snd_pcm_sw_params_set_silence_size(handle, sw_params, val)) < 0) { std::cout << " Error (can't set silence): " << snd_strerror(err) << std::endl; return; } */ if ((err = snd_pcm_sw_params(handle, sw_params)) < 0) { std::cout << " Error (can't set sw params): " << snd_strerror(err) << std::endl; return; } snd_pcm_sw_params_free(sw_params); // ------------------------- SW-Params End --------------------------------- if ((err = snd_pcm_prepare(handle)) < 0) { std::cout << " Error (can't prepare audio): " << snd_strerror(err) << std::endl; return; } snd_async_handler_t *pcm_callback; if ((err = snd_async_add_pcm_handler(&pcm_callback, handle, MyCallback, NULL)) < 0) { std::cout << " Error (can't add pcm handler): " << snd_strerror(err) << std::endl; return; } if ((err = snd_pcm_writei(handle, buffer_alsa, 2 * period_size) == -EPIPE)) { std::cout << " Error: XRUN" << std::endl; err = snd_pcm_recover(handle, err, 0); if ((err = snd_pcm_prepare (handle)) < 0) { std::cout << " Error (can't prepare audio): " << snd_strerror(err) << std::endl; return; } } if ((err = snd_pcm_start(handle)) < 0) { std::cout << " Error (can't start audio): " << snd_strerror(err) << std::endl; return; } void MyCallback(snd_async_handler_t *pcm_callback);

2 1

[alsa-devel] [PATCH] ASoC: soc-pcm: Remove unused 'debugfs_dpcm_state' entry
by Fabio Estevam 29 Jul '17

29 Jul '17

From: Fabio Estevam <fabio.estevam(a)nxp.com> 'debugfs_dpcm_state' member from structure snd_soc_pcm_runtime is never used at all, so it is safe to remove it. Signed-off-by: Fabio Estevam <fabio.estevam(a)nxp.com> --- include/sound/soc.h | 1 - sound/soc/soc-pcm.c | 5 ++--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/include/sound/soc.h b/include/sound/soc.h index 8f9376f..9d6e1db 100644 --- a/include/sound/soc.h +++ b/include/sound/soc.h @@ -1233,7 +1233,6 @@ struct snd_soc_pcm_runtime { struct delayed_work delayed_work; #ifdef CONFIG_DEBUG_FS struct dentry *debugfs_dpcm_root; - struct dentry *debugfs_dpcm_state; #endif unsigned int num; /* 0-based and monotonic increasing */ diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c index 7d3859e..24148de 100644 --- a/sound/soc/soc-pcm.c +++ b/sound/soc/soc-pcm.c @@ -3000,8 +3000,7 @@ void soc_dpcm_debugfs_add(struct snd_soc_pcm_runtime *rtd) return; } - rtd->debugfs_dpcm_state = debugfs_create_file("state", 0444, - rtd->debugfs_dpcm_root, - rtd, &dpcm_state_fops); + debugfs_create_file("state", 0444, rtd->debugfs_dpcm_root, + rtd, &dpcm_state_fops); } #endif -- 2.7.4

1 0

Re: [alsa-devel] [PATCH] ASoC: rt5514: Use the local variable to get the data from SPI device
by Mark Brown 28 Jul '17

28 Jul '17

On Mon, Jul 24, 2017 at 03:06:36AM +0000, Oder Chiou wrote: > > On Fri, Jul 21, 2017 at 04:00:47PM +0800, Oder Chiou wrote: > > > The patch uses the local variable to get the data from SPI device to > > > prevent to get all zero data. > > How do you end up with all zero data from reading directly into rxbuf? > While the "runtime->dma_area" put into the rxbuf directly on the Intel > platform, it seemed that had the limitation to let the memory cannot be > written the data using the driver of the SPI controller. We verified the > original code on the platforms of Nvidia and Rockchip, and these are > working fine. Do we understand why that is? Note that IIRC you're not really supposed to DMA to or from the stack, there are some platforms with restrictions there so it's possible you're just moving the problem around here if rxbuf can be on the stack.

1 0

[alsa-devel] [PATCH 0/5] ALSA: constify snd_timer_hardware structures
by Bhumika Goyal 28 Jul '17

28 Jul '17

Declare snd_timer_hardware structures as const. Bhumika Goyal (5): ALSA: sound/core: constify snd_timer_hardware structures ALSA: isa: constify snd_timer_hardware structures ALSA: pci: constify snd_timer_hardware structures ALSA: sparc: constify snd_timer_hardware structure ALSA: opl3: constify snd_timer_hardware structures sound/core/hrtimer.c | 2 +- sound/core/pcm_timer.c | 2 +- sound/core/timer.c | 2 +- sound/drivers/opl3/opl3_lib.c | 4 ++-- sound/isa/ad1816a/ad1816a_lib.c | 2 +- sound/isa/gus/gus_timer.c | 4 ++-- sound/isa/wss/wss_lib.c | 2 +- sound/pci/emu10k1/timer.c | 2 +- sound/pci/ymfpci/ymfpci_main.c | 2 +- sound/sparc/cs4231.c | 2 +- 10 files changed, 12 insertions(+), 12 deletions(-) -- 1.9.1

1 6

[alsa-devel] [PATCH] ASoC: Intel: Reset hw_ptr on resume trigger
by Cheng-Yi Chiang 28 Jul '17

28 Jul '17

From: "U. Artie Eoff" <ullysses.a.eoff(a)intel.com> Reset the hw_ptr before queuing the restore_stream_context work to eradicate a nasty white audio noise on resume. Signed-off-by: U. Artie Eoff <ullysses.a.eoff(a)intel.com> --- sound/soc/intel/baytrail/sst-baytrail-pcm.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sound/soc/intel/baytrail/sst-baytrail-pcm.c b/sound/soc/intel/baytrail/sst-baytrail-pcm.c index 4765ad474544..e0db7070cd42 100644 --- a/sound/soc/intel/baytrail/sst-baytrail-pcm.c +++ b/sound/soc/intel/baytrail/sst-baytrail-pcm.c @@ -187,8 +187,10 @@ static int sst_byt_pcm_trigger(struct snd_pcm_substream *substream, int cmd) sst_byt_stream_start(byt, pcm_data->stream, 0); break; case SNDRV_PCM_TRIGGER_RESUME: - if (pdata->restore_stream == true) + if (pdata->restore_stream == true) { + pcm_data->hw_ptr = 0; schedule_work(&pcm_data->work); + } else sst_byt_stream_resume(byt, pcm_data->stream); break; -- 2.12.2

2 1

[alsa-devel] [v2 0/3] ASoC: Intel: Fixes in Kabylake machine driver for clear playback and recording
by Harsha Priya 28 Jul '17

28 Jul '17

This patch series includes fixes for clean playback and recording in kabylake machine driver. It fixes the below issues in the corresponding patches and the details of each fix is provided in the patch's commit message. 1. Fix cracking noise in rt5663 headphones 2. Fix pop noise of dmic recording 3. Enable 4 slot IV feedback for maxim amps Harsha Priya (3): ASoC: Intel: Fix cracking noise in rt5663 headphones for kabylake platform ASoC: Intel: Fix pop noise of dmic recording in kabylake platform using rt5514 ASoC: Intel: Enabling 4 slot IV feedback for max98927 on Kabylake platform .../soc/intel/boards/kbl_rt5663_rt5514_max98927.c | 28 +++++++++++----------- 1 file changed, 14 insertions(+), 14 deletions(-) -- 1.9.1

4 10

[alsa-devel] [PATCH] ASoC: codecs: add wm8524 codec driver
by Mihai Serban 28 Jul '17

28 Jul '17

WM8524 is a 24-bit 192KHz stereo digital/analog converter (DAC) with integral charge pump and a simple hardware control interface. Product information can be found at: https://www.cirrus.com/products/wm8524/ Signed-off-by: Mihai Serban <mihai.serban(a)nxp.com> Signed-off-by: Shengjiu Wang <shengjiu.wang(a)nxp.com> --- Documentation/devicetree/bindings/sound/wm8524.txt | 16 ++ sound/soc/codecs/Kconfig | 5 + sound/soc/codecs/Makefile | 2 + sound/soc/codecs/wm8524.c | 265 +++++++++++++++++++++ 4 files changed, 288 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/wm8524.txt create mode 100644 sound/soc/codecs/wm8524.c diff --git a/Documentation/devicetree/bindings/sound/wm8524.txt b/Documentation/devicetree/bindings/sound/wm8524.txt new file mode 100644 index 0000000..20c6200 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/wm8524.txt @@ -0,0 +1,16 @@ +WM8524 audio CODEC + +This device does not use I2C or SPI but a simple Hardware Control Interface. + +Required properties: + + - compatible : "wlf,wm8524" + + - wlf,mute-gpios: a GPIO spec for the MUTE pin. + +Example: + +codec: wm8524@0 { + compatible = "wlf,wm8524"; + wlf,mute-gpios = <&gpio1 8 GPIO_ACTIVE_LOW>; +}; diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig index 024ddc9..010811e 100644 --- a/sound/soc/codecs/Kconfig +++ b/sound/soc/codecs/Kconfig @@ -174,6 +174,7 @@ config SND_SOC_ALL_CODECS select SND_SOC_WM8400 if MFD_WM8400 select SND_SOC_WM8510 if SND_SOC_I2C_AND_SPI select SND_SOC_WM8523 if I2C + select SND_SOC_WM8524 if GPIOLIB select SND_SOC_WM8580 if I2C select SND_SOC_WM8711 if SND_SOC_I2C_AND_SPI select SND_SOC_WM8727 @@ -974,6 +975,10 @@ config SND_SOC_WM8523 tristate "Wolfson Microelectronics WM8523 DAC" depends on I2C +config SND_SOC_WM8524 + tristate "Wolfson Microelectronics WM8524 DAC" + depends on GPIOLIB + config SND_SOC_WM8580 tristate "Wolfson Microelectronics WM8580 and WM8581 CODECs" depends on I2C diff --git a/sound/soc/codecs/Makefile b/sound/soc/codecs/Makefile index a8a4b07..12dc2f7 100644 --- a/sound/soc/codecs/Makefile +++ b/sound/soc/codecs/Makefile @@ -183,6 +183,7 @@ snd-soc-wm8350-objs := wm8350.o snd-soc-wm8400-objs := wm8400.o snd-soc-wm8510-objs := wm8510.o snd-soc-wm8523-objs := wm8523.o +snd-soc-wm8524-objs := wm8524.o snd-soc-wm8580-objs := wm8580.o snd-soc-wm8711-objs := wm8711.o snd-soc-wm8727-objs := wm8727.o @@ -416,6 +417,7 @@ obj-$(CONFIG_SND_SOC_WM8350) += snd-soc-wm8350.o obj-$(CONFIG_SND_SOC_WM8400) += snd-soc-wm8400.o obj-$(CONFIG_SND_SOC_WM8510) += snd-soc-wm8510.o obj-$(CONFIG_SND_SOC_WM8523) += snd-soc-wm8523.o +obj-$(CONFIG_SND_SOC_WM8524) += snd-soc-wm8524.o obj-$(CONFIG_SND_SOC_WM8580) += snd-soc-wm8580.o obj-$(CONFIG_SND_SOC_WM8711) += snd-soc-wm8711.o obj-$(CONFIG_SND_SOC_WM8727) += snd-soc-wm8727.o diff --git a/sound/soc/codecs/wm8524.c b/sound/soc/codecs/wm8524.c new file mode 100644 index 0000000..f42ec66 --- /dev/null +++ b/sound/soc/codecs/wm8524.c @@ -0,0 +1,265 @@ +/* + * wm8524.c -- WM8524 ALSA SoC Audio driver + * + * Copyright 2009 Wolfson Microelectronics plc + * Copyright 2017 NXP + * + * Based on WM8523 ALSA SoC Audio driver written by Mark Brown + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + */ + +#include <linux/module.h> +#include <linux/moduleparam.h> +#include <linux/init.h> +#include <linux/delay.h> +#include <linux/slab.h> +#include <linux/gpio/consumer.h> +#include <linux/of_device.h> +#include <sound/core.h> +#include <sound/pcm.h> +#include <sound/pcm_params.h> +#include <sound/soc.h> +#include <sound/initval.h> + +#define WM8524_NUM_RATES 7 + +/* codec private data */ +struct wm8524_priv { + struct gpio_desc *mute; + unsigned int sysclk; + unsigned int rate_constraint_list[WM8524_NUM_RATES]; + struct snd_pcm_hw_constraint_list rate_constraint; +}; + + +static const struct snd_soc_dapm_widget wm8524_dapm_widgets[] = { +SND_SOC_DAPM_DAC("DAC", "Playback", SND_SOC_NOPM, 0, 0), +SND_SOC_DAPM_OUTPUT("LINEVOUTL"), +SND_SOC_DAPM_OUTPUT("LINEVOUTR"), +}; + +static const struct snd_soc_dapm_route wm8524_dapm_routes[] = { + { "LINEVOUTL", NULL, "DAC" }, + { "LINEVOUTR", NULL, "DAC" }, +}; + +static const struct { + int value; + int ratio; +} lrclk_ratios[WM8524_NUM_RATES] = { + { 1, 128 }, + { 2, 192 }, + { 3, 256 }, + { 4, 384 }, + { 5, 512 }, + { 6, 768 }, + { 7, 1152 }, +}; + +static int wm8524_startup(struct snd_pcm_substream *substream, + struct snd_soc_dai *dai) +{ + struct snd_soc_codec *codec = dai->codec; + struct wm8524_priv *wm8524 = snd_soc_codec_get_drvdata(codec); + + /* The set of sample rates that can be supported depends on the + * MCLK supplied to the CODEC - enforce this. + */ + if (!wm8524->sysclk) { + dev_err(codec->dev, + "No MCLK configured, call set_sysclk() on init\n"); + return -EINVAL; + } + + snd_pcm_hw_constraint_list(substream->runtime, 0, + SNDRV_PCM_HW_PARAM_RATE, + &wm8524->rate_constraint); + + gpiod_set_value_cansleep(wm8524->mute, 1); + + return 0; +} + +static void wm8524_shutdown(struct snd_pcm_substream *substream, + struct snd_soc_dai *dai) +{ + struct snd_soc_codec *codec = dai->codec; + struct wm8524_priv *wm8524 = snd_soc_codec_get_drvdata(codec); + + gpiod_set_value_cansleep(wm8524->mute, 0); +} + +static int wm8524_set_dai_sysclk(struct snd_soc_dai *codec_dai, + int clk_id, unsigned int freq, int dir) +{ + struct snd_soc_codec *codec = codec_dai->codec; + struct wm8524_priv *wm8524 = snd_soc_codec_get_drvdata(codec); + unsigned int val; + int i, j = 0; + + wm8524->sysclk = freq; + + wm8524->rate_constraint.count = 0; + for (i = 0; i < ARRAY_SIZE(lrclk_ratios); i++) { + val = freq / lrclk_ratios[i].ratio; + /* Check that it's a standard rate since core can't + * cope with others and having the odd rates confuses + * constraint matching. + */ + switch (val) { + case 8000: + case 32000: + case 44100: + case 48000: + case 88200: + case 96000: + case 176400: + case 192000: + dev_err(codec->dev, "Supported sample rate: %dHz\n", + val); + wm8524->rate_constraint_list[j++] = val; + wm8524->rate_constraint.count++; + break; + default: + dev_dbg(codec->dev, "Skipping sample rate: %dHz\n", + val); + } + } + + /* Need at least one supported rate... */ + if (wm8524->rate_constraint.count == 0) + return -EINVAL; + + return 0; +} + +static int wm8524_set_fmt(struct snd_soc_dai *codec_dai, unsigned int fmt) +{ + fmt &= (SND_SOC_DAIFMT_FORMAT_MASK | SND_SOC_DAIFMT_INV_MASK | + SND_SOC_DAIFMT_MASTER_MASK); + + if (fmt != (SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF | + SND_SOC_DAIFMT_CBS_CFS)) { + dev_err(codec_dai->dev, "Invalid DAI format\n"); + return -EINVAL; + } + + return 0; +} + +static int wm8524_mute_stream(struct snd_soc_dai *dai, int mute, int stream) +{ + struct wm8524_priv *wm8524 = snd_soc_codec_get_drvdata(dai->codec); + + if (wm8524->mute) + gpiod_set_value_cansleep(wm8524->mute, mute); + + return 0; +} + +#define WM8524_RATES SNDRV_PCM_RATE_8000_192000 + +#define WM8524_FORMATS (SNDRV_PCM_FMTBIT_S16_LE | SNDRV_PCM_FMTBIT_S24_LE) + +static const struct snd_soc_dai_ops wm8524_dai_ops = { + .startup = wm8524_startup, + .shutdown = wm8524_shutdown, + .set_sysclk = wm8524_set_dai_sysclk, + .set_fmt = wm8524_set_fmt, + .mute_stream = wm8524_mute_stream, +}; + +static struct snd_soc_dai_driver wm8524_dai = { + .name = "wm8524-hifi", + .playback = { + .stream_name = "Playback", + .channels_min = 2, + .channels_max = 2, + .rates = WM8524_RATES, + .formats = WM8524_FORMATS, + }, + .ops = &wm8524_dai_ops, +}; + +static int wm8524_probe(struct snd_soc_codec *codec) +{ + struct wm8524_priv *wm8524 = snd_soc_codec_get_drvdata(codec); + + wm8524->rate_constraint.list = &wm8524->rate_constraint_list[0]; + wm8524->rate_constraint.count = + ARRAY_SIZE(wm8524->rate_constraint_list); + + return 0; +} + +static const struct snd_soc_codec_driver soc_codec_dev_wm8524 = { + .probe = wm8524_probe, + + .component_driver = { + .dapm_widgets = wm8524_dapm_widgets, + .num_dapm_widgets = ARRAY_SIZE(wm8524_dapm_widgets), + .dapm_routes = wm8524_dapm_routes, + .num_dapm_routes = ARRAY_SIZE(wm8524_dapm_routes), + }, +}; + +#ifdef CONFIG_OF +static const struct of_device_id wm8524_of_match[] = { + { .compatible = "wlf,wm8524" }, + { /* sentinel*/ } +}; +MODULE_DEVICE_TABLE(of, wm8524_of_match); +#endif + +static int wm8524_codec_probe(struct platform_device *pdev) +{ + struct wm8524_priv *wm8524; + int ret; + + wm8524 = devm_kzalloc(&pdev->dev, sizeof(struct wm8524_priv), + GFP_KERNEL); + if (wm8524 == NULL) + return -ENOMEM; + + platform_set_drvdata(pdev, wm8524); + + wm8524->mute = devm_gpiod_get(&pdev->dev, "wlf,mute", GPIOD_OUT_LOW); + if (IS_ERR(wm8524->mute)) { + ret = PTR_ERR(wm8524->mute); + dev_err(&pdev->dev, "Failed to get mute line: %d\n", ret); + return ret; + } + + ret = snd_soc_register_codec(&pdev->dev, + &soc_codec_dev_wm8524, &wm8524_dai, 1); + if (ret < 0) { + dev_err(&pdev->dev, "Failed to register codec: %d\n", ret); + snd_soc_unregister_platform(&pdev->dev); + } + + return ret; +} + +static int wm8524_codec_remove(struct platform_device *pdev) +{ + snd_soc_unregister_codec(&pdev->dev); + return 0; +} + +static struct platform_driver wm8524_codec_driver = { + .probe = wm8524_codec_probe, + .remove = wm8524_codec_remove, + .driver = { + .name = "wm8524-codec", + .of_match_table = wm8524_of_match, + }, +}; +module_platform_driver(wm8524_codec_driver); + +MODULE_DESCRIPTION("ASoC WM8524 driver"); +MODULE_AUTHOR("Mihai Serban <mihai.serban(a)nxp.com>"); +MODULE_ALIAS("platform:wm8524-codec"); +MODULE_LICENSE("GPL"); -- 2.7.4

2 1

[alsa-devel] [PATCH v2 0/2] ASoC: Intel: Add Kabylake machine driver for RT5663
by Kevin Cheng 28 Jul '17

28 Jul '17

Add kabylake machine driver & corresponding driver entry in machine table for rt5663 codec. For this particular machine, the RT5663 is a headset codec connected on ssp1. there is no dmic and speaker connected. Modify the machine driver in kbl_rt5663_max98927 to reuse the common codes. V2: adjust the code order. Kevin Cheng (2): ASoC: Intel: Add Kabylake machine driver for RT5663 ASoC: Intel: Add Kabylake RT5663 machine driver entry sound/soc/intel/boards/kbl_rt5663_max98927.c | 246 +++++++++++++++++++++++---- sound/soc/intel/skylake/skl.c | 5 + 2 files changed, 214 insertions(+), 37 deletions(-) -- 2.12.2

3 4

[alsa-devel] [PATCH 0/2] ASoC: Intel: Add Kabylake machine driver for RT5663
by Kevin Cheng 28 Jul '17

28 Jul '17

Add kabylake machine driver & corresponding driver entry in machine table for rt5663 codec. For this particular machine, the RT5663 is a headset codec connected on ssp1. there is no dmic and speaker connected. Modify the machine driver to kbl_rt5663_max98927 to reuse the common codes. Kevin Cheng (2): ASoC: Intel: Add Kabylake machine driver for RT5663 ASoC: Intel: Add Kabylake RT5663 machine driver entry sound/soc/intel/boards/kbl_rt5663_max98927.c | 246 +++++++++++++++++++++++---- sound/soc/intel/skylake/skl.c | 5 + 2 files changed, 214 insertions(+), 37 deletions(-) -- 1.9.1

2 3

[alsa-devel] [RFT 1/2] ASoC: samsung: Fix possible double iounmap on s3c24xx driver probe failure
by Krzysztof Kozlowski 28 Jul '17

28 Jul '17

Commit 87b132bc0315 ("ASoC: samsung: s3c24{xx,12}-i2s: port to use generic dmaengine API") moved ioremap() call from s3c-i2s-v2.c:s3c_i2sv2_probe() to s3c2412-i2s.c:s3c2412_iis_dev_probe() and converted it to devm- resource managed interface. However the error path in first of them - s3c_i2sv2_probe() - was not updated. If getting a iis clock in s3c_i2sv2_probe() failed, the address space would be unmapped there. This could lead to: 1. double iounmap() later from devm-interface of s3c2412_iis_dev_probe()), 2. accessing the memory by other functions in s3c2412-i2s.c unit. Anyway, the owner of this mapped region should be s3c2412-i2s.c because it starts the mapping. Affected are drivers for S3C24xx family although issue was not reproduced. Fixes: 87b132bc0315 ("ASoC: samsung: s3c24{xx,12}-i2s: port to use generic dmaengine API") Signed-off-by: Krzysztof Kozlowski <krzk(a)kernel.org> --- Not marking as Cc-stable because this is theoretical problem, not reproduced and also not tested. Please, kindly test on S3C24xx hardware. --- sound/soc/samsung/s3c-i2s-v2.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/samsung/s3c-i2s-v2.c b/sound/soc/samsung/s3c-i2s-v2.c index ca522a95160b..9b28046eea8e 100644 --- a/sound/soc/samsung/s3c-i2s-v2.c +++ b/sound/soc/samsung/s3c-i2s-v2.c @@ -634,7 +634,6 @@ int s3c_i2sv2_probe(struct snd_soc_dai *dai, i2s->iis_pclk = clk_get(dev, "iis"); if (IS_ERR(i2s->iis_pclk)) { dev_err(dev, "failed to get iis_clock\n"); - iounmap(i2s->regs); return -ENOENT; } -- 2.11.0

3 5

[alsa-devel] [RFT v2 0/3] ASoC: samsung: s3c24xx: Fixes for error paths
by Krzysztof Kozlowski 28 Jul '17

28 Jul '17

Hi, Changes since v1: 1. Add patch 2/3. 2. Drop assignment of iis_cclk=iis_pclk to have balance with clk disables error and remove paths (pointed by Arvind). Best regards, Krzysztof Krzysztof Kozlowski (3): ASoC: samsung: Fix possible double iounmap on s3c24xx driver probe failure ASoC: samsung: Add missing prepare for iis clock of s3c24xx ASoC: samsung: Add proper error paths to s3c24xx I2S driver sound/soc/samsung/s3c-i2s-v2.c | 12 ++++++++++-- sound/soc/samsung/s3c-i2s-v2.h | 7 +++++++ sound/soc/samsung/s3c2412-i2s.c | 13 +++++++++---- 3 files changed, 26 insertions(+), 6 deletions(-) -- 2.11.0

3 10

[alsa-devel] [PATCH] ASoC: rsnd: merge snd_pcm_ops::open and snd_soc_dai_ops::startup
by Kuninori Morimoto 28 Jul '17

28 Jul '17

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Current rsnd driver has snd_pcm_ops::open and snd_soc_dai_ops::startup, but, these are called from soc_pcm_open in same time. This patch merge these into rsnd_soc_dai_startup() Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/sh/rcar/core.c | 80 +++++++++++++++++++----------------------------- 1 file changed, 31 insertions(+), 49 deletions(-) diff --git a/sound/soc/sh/rcar/core.c b/sound/soc/sh/rcar/core.c index 8dcf1a8..6fbbcbf 100644 --- a/sound/soc/sh/rcar/core.c +++ b/sound/soc/sh/rcar/core.c @@ -844,12 +844,28 @@ static int rsnd_soc_hw_rule_channels(struct snd_pcm_hw_params *params, ir, &ic); } -static void rsnd_soc_hw_constraint(struct snd_pcm_runtime *runtime, - struct snd_soc_dai *dai) +static struct snd_pcm_hardware rsnd_pcm_hardware = { + .info = SNDRV_PCM_INFO_INTERLEAVED | + SNDRV_PCM_INFO_MMAP | + SNDRV_PCM_INFO_MMAP_VALID, + .buffer_bytes_max = 64 * 1024, + .period_bytes_min = 32, + .period_bytes_max = 8192, + .periods_min = 1, + .periods_max = 32, + .fifo_size = 256, +}; + +static int rsnd_soc_dai_startup(struct snd_pcm_substream *substream, + struct snd_soc_dai *dai) { struct rsnd_dai *rdai = rsnd_dai_to_rdai(dai); + struct rsnd_priv *priv = rsnd_rdai_to_priv(rdai); + struct rsnd_dai_stream *io = rsnd_rdai_to_io(rdai, substream); struct snd_pcm_hw_constraint_list *constraint = &rdai->constraint; + struct snd_pcm_runtime *runtime = substream->runtime; unsigned int max_channels = rsnd_rdai_channels_get(rdai); + int ret; int i; /* @@ -866,34 +882,26 @@ static void rsnd_soc_hw_constraint(struct snd_pcm_runtime *runtime, constraint->count = i + 1; } + snd_soc_set_runtime_hwparams(substream, &rsnd_pcm_hardware); + snd_pcm_hw_constraint_list(runtime, 0, SNDRV_PCM_HW_PARAM_CHANNELS, constraint); + snd_pcm_hw_constraint_integer(runtime, + SNDRV_PCM_HW_PARAM_PERIODS); + /* * Sampling Rate / Channel Limitation * It depends on Clock Master Mode */ - if (!rsnd_rdai_is_clk_master(rdai)) - return; - - snd_pcm_hw_rule_add(runtime, 0, SNDRV_PCM_HW_PARAM_RATE, - rsnd_soc_hw_rule_rate, dai, - SNDRV_PCM_HW_PARAM_CHANNELS, -1); - snd_pcm_hw_rule_add(runtime, 0, SNDRV_PCM_HW_PARAM_CHANNELS, - rsnd_soc_hw_rule_channels, dai, - SNDRV_PCM_HW_PARAM_RATE, -1); -} - -static int rsnd_soc_dai_startup(struct snd_pcm_substream *substream, - struct snd_soc_dai *dai) -{ - struct rsnd_dai *rdai = rsnd_dai_to_rdai(dai); - struct rsnd_priv *priv = rsnd_rdai_to_priv(rdai); - struct rsnd_dai_stream *io = rsnd_rdai_to_io(rdai, substream); - int ret; - - /* rsnd_io_to_runtime() is not yet enabled here */ - rsnd_soc_hw_constraint(substream->runtime, dai); + if (rsnd_rdai_is_clk_master(rdai)) { + snd_pcm_hw_rule_add(runtime, 0, SNDRV_PCM_HW_PARAM_RATE, + rsnd_soc_hw_rule_rate, dai, + SNDRV_PCM_HW_PARAM_CHANNELS, -1); + snd_pcm_hw_rule_add(runtime, 0, SNDRV_PCM_HW_PARAM_CHANNELS, + rsnd_soc_hw_rule_channels, dai, + SNDRV_PCM_HW_PARAM_RATE, -1); + } /* * call rsnd_dai_call without spinlock @@ -1105,31 +1113,6 @@ static int rsnd_dai_probe(struct rsnd_priv *priv) /* * pcm ops */ -static struct snd_pcm_hardware rsnd_pcm_hardware = { - .info = SNDRV_PCM_INFO_INTERLEAVED | - SNDRV_PCM_INFO_MMAP | - SNDRV_PCM_INFO_MMAP_VALID, - .buffer_bytes_max = 64 * 1024, - .period_bytes_min = 32, - .period_bytes_max = 8192, - .periods_min = 1, - .periods_max = 32, - .fifo_size = 256, -}; - -static int rsnd_pcm_open(struct snd_pcm_substream *substream) -{ - struct snd_pcm_runtime *runtime = substream->runtime; - int ret = 0; - - snd_soc_set_runtime_hwparams(substream, &rsnd_pcm_hardware); - - ret = snd_pcm_hw_constraint_integer(runtime, - SNDRV_PCM_HW_PARAM_PERIODS); - - return ret; -} - static int rsnd_hw_params(struct snd_pcm_substream *substream, struct snd_pcm_hw_params *hw_params) { @@ -1159,7 +1142,6 @@ static snd_pcm_uframes_t rsnd_pointer(struct snd_pcm_substream *substream) } static struct snd_pcm_ops rsnd_pcm_ops = { - .open = rsnd_pcm_open, .ioctl = snd_pcm_lib_ioctl, .hw_params = rsnd_hw_params, .hw_free = snd_pcm_lib_free_pages, -- 1.9.1

2 1

[alsa-devel] [PATCH v3 0/3] ASoC: Intel: Fixes in Kabylake machine driver for clear playback and recording
by Harsha Priya 28 Jul '17

28 Jul '17

This patch series includes fixes for clean playback and recording in kabylake machine driver. It fixes the below issues in the corresponding patches and the details of each fix is provided in the patch's commit message. 1. Fix cracking noise in rt5663 headphones 2. Fix pop noise of dmic recording 3. Enable 4 slot IV feedback for maxim amps Harsha Priya (3): ASoC: Intel: Fix cracking noise in rt5663 headphones for kabylake platform ASoC: Intel: Fix pop noise of dmic recording in kabylake platform using rt5514 ASoC: Intel: Enabling 4 slot IV feedback for max98927 on Kabylake platform .../soc/intel/boards/kbl_rt5663_rt5514_max98927.c | 28 +++++++++++----------- 1 file changed, 14 insertions(+), 14 deletions(-) -- 1.9.1

2 4

[alsa-devel] [PATCH v2 0/2] ASoC: Intel: Add Kabylake machine driver for RT5663
by Kevin Cheng 28 Jul '17

28 Jul '17

Add kabylake machine driver & corresponding driver entry in machine table for rt5663 codec. For this particular machine, the RT5663 is a headset codec connected on ssp1. there is no dmic and speaker connected. Modify the machine driver in kbl_rt5663_max98927 to reuse the common codes. V2: adjust the code order. Kevin Cheng (2): ASoC: Intel: Add Kabylake machine driver for RT5663 ASoC: Intel: Add Kabylake RT5663 machine driver entry sound/soc/intel/boards/kbl_rt5663_max98927.c | 246 +++++++++++++++++++++++---- sound/soc/intel/skylake/skl.c | 5 + 2 files changed, 214 insertions(+), 37 deletions(-) -- 2.12.2

2 3

[alsa-devel] [PATCH v1 0/3] add jack detection to MSM8916 analog
by Damien Riegel 27 Jul '17

27 Jul '17

This series adds support for the MBHC (Multi-Button Headset Control) of the PM8916's analog codec. This IP is capable to identify up to five buttons on a headset, but for now only the jack detection is supported. A complete implementation exists in the Android kernel, but it's quite complex and I'd rather go step by step. First patch fixes a define that is squatting a bit used by the MBHC. Second patch is the actual implementation of the jack detection, and third patch plugs the jack detection in the sound card driver. Damien Riegel (3): ASoC: codecs: msm8916-analog: fix DIG_CLK_CTL_RXD3_CLK_EN define ASoC: codecs: msm8916-analog: support jack detection ASoC: qcom: apq8016-sbc: enable jack detection sound/soc/codecs/msm8916-wcd-analog.c | 105 +++++++++++++++++++++++++++++++++- sound/soc/codecs/msm8916-wcd-analog.h | 18 ++++++ sound/soc/qcom/apq8016_sbc.c | 16 ++++++ 3 files changed, 138 insertions(+), 1 deletion(-) create mode 100644 sound/soc/codecs/msm8916-wcd-analog.h -- 2.13.3

3 10

[alsa-devel] [PATCH v3 0/2] Add support for TI TAS6424 digital amplifier
by Michael Stecklein 27 Jul '17

27 Jul '17

Hello all, This two-patch series adds the dt bindings, driver, and other related changes for the TAS6424 digital amplifier. I am upstreaming this driver on behalf of Andreas. Regards, Michael Stecklein <m-stecklein(a)ti.com> Changes since v2: - fixed binding as per Rob Herring's comments on v2 Changes since v1: - rebased on 4.13-rc1 Original v1 patches: - https://patchwork.kernel.org/patch/9795235/ - https://patchwork.kernel.org/patch/9795237/ Andreas Dannenberg (1): ASoC: add support for TAS6424 digital amplifier Michael Stecklein (1): dt-bindings: sound: add bindings for TAS6424 .../devicetree/bindings/sound/ti,tas6424.txt | 28 + sound/soc/codecs/Kconfig | 8 + sound/soc/codecs/Makefile | 2 + sound/soc/codecs/tas6424.c | 755 +++++++++++++++++++++ sound/soc/codecs/tas6424.h | 127 ++++ 5 files changed, 920 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/ti,tas6424.txt create mode 100644 sound/soc/codecs/tas6424.c create mode 100644 sound/soc/codecs/tas6424.h -- 2.7.4

3 6

[alsa-devel] [PATCH v3 00/17] Add support for Cirrus Logic CS47L35/L85/L90/L91 codecs
by Richard Fitzgerald 27 Jul '17

27 Jul '17

The Cirrus Logic CS47L35, CS47L85, CS47L90/91 codecs are complex audio SoC devices. In addition to the core audio capability they have onboard GPIO, regulators, DSPs and interrupt controller and a large register map space accessed over SPI or I2C. This family of codecs is based around common IP blocks and they are managed by a set of common drivers referred to as "Madera". Summary of main changes since V1: - Removed child DT nodes and of_compatibles to make the DT flat - Fixed review comments on irqchip driver - Used new field names for IRQ1_STATUS_33 register - Moved the gpio driver bindings into the main MFD binding doc - Some code improvements to the pinctrl driver Richard Fitzgerald (17): mfd: madera: Add register definitions for Cirrus Logic Madera codecs mfd: madera: Add DT bindings for Cirrus Logic Madera codecs mfd: madera: Add common support for Cirrus Logic Madera codecs mfd: madera: Register map tables for Cirrus Logic CS47L35 mfd: madera: Register map tables for Cirrus Logic CS47L85 mfd: madera: Register map tables for Cirrus Logic CS47L90/91 regulator: arizona-micsupp: Add support for Cirrus Logic Madera codecs regulator: arizona-ldo1: Add support for Cirrus Logic Madera codecs irqchip: Add driver for Cirrus Logic Madera codecs pinctrl: madera: Add DT bindings for Cirrus Logic Madera codecs pinctrl: madera: Add driver for Cirrus Logic Madera codecs gpio: madera: Support Cirrus Logic Madera class codecs ASoC: madera: Add DT bindings for Cirrus Logic Madera codecs ASoC: madera: Add common support for Cirrus Logic Madera codecs ASoC: cs47l35: Add codec driver for Cirrus Logic CS47L35 ASoC: cs47l85: Add codec driver for Cirrus Logic CS47L85 ASoC: cs47l90: Add codec driver for Cirrus Logic CS47L90 Documentation/devicetree/bindings/mfd/madera.txt | 102 + .../bindings/pinctrl/cirrus,madera-pinctrl.txt | 99 + .../bindings/regulator/arizona-regulator.txt | 3 +- Documentation/devicetree/bindings/sound/madera.txt | 67 + MAINTAINERS | 23 + drivers/gpio/Kconfig | 6 + drivers/gpio/Makefile | 1 + drivers/gpio/gpio-madera.c | 196 + drivers/irqchip/Kconfig | 3 + drivers/irqchip/Makefile | 1 + drivers/irqchip/irq-madera.c | 301 + drivers/mfd/Kconfig | 46 + drivers/mfd/Makefile | 13 + drivers/mfd/cs47l35-tables.c | 1688 ++++ drivers/mfd/cs47l85-tables.c | 3169 +++++++ drivers/mfd/cs47l90-tables.c | 2830 +++++++ drivers/mfd/madera-core.c | 632 ++ drivers/mfd/madera-i2c.c | 131 + drivers/mfd/madera-spi.c | 132 + drivers/mfd/madera.h | 52 + drivers/pinctrl/Kconfig | 1 + drivers/pinctrl/Makefile | 1 + drivers/pinctrl/cirrus/Kconfig | 15 + drivers/pinctrl/cirrus/Makefile | 11 + drivers/pinctrl/cirrus/pinctrl-cs47l35.c | 46 + drivers/pinctrl/cirrus/pinctrl-cs47l85.c | 60 + drivers/pinctrl/cirrus/pinctrl-cs47l90.c | 58 + drivers/pinctrl/cirrus/pinctrl-madera.c | 1074 +++ drivers/pinctrl/cirrus/pinctrl-madera.h | 40 + drivers/regulator/Kconfig | 15 +- drivers/regulator/arizona-ldo1.c | 82 +- drivers/regulator/arizona-micsupp.c | 71 +- include/dt-bindings/sound/madera.h | 27 + include/linux/irqchip/irq-madera-pdata.h | 23 + include/linux/irqchip/irq-madera.h | 96 + include/linux/mfd/madera/core.h | 180 + include/linux/mfd/madera/pdata.h | 61 + include/linux/mfd/madera/registers.h | 8832 ++++++++++++++++++++ include/sound/madera-pdata.h | 61 + sound/soc/codecs/Kconfig | 23 + sound/soc/codecs/Makefile | 8 + sound/soc/codecs/cs47l35.c | 1747 ++++ sound/soc/codecs/cs47l85.c | 2706 ++++++ sound/soc/codecs/cs47l90.c | 2645 ++++++ sound/soc/codecs/madera.c | 4430 ++++++++++ sound/soc/codecs/madera.h | 470 ++ 46 files changed, 32268 insertions(+), 10 deletions(-) create mode 100644 Documentation/devicetree/bindings/mfd/madera.txt create mode 100644 Documentation/devicetree/bindings/pinctrl/cirrus,madera-pinctrl.txt create mode 100644 Documentation/devicetree/bindings/sound/madera.txt create mode 100644 drivers/gpio/gpio-madera.c create mode 100644 drivers/irqchip/irq-madera.c create mode 100644 drivers/mfd/cs47l35-tables.c create mode 100644 drivers/mfd/cs47l85-tables.c create mode 100644 drivers/mfd/cs47l90-tables.c create mode 100644 drivers/mfd/madera-core.c create mode 100644 drivers/mfd/madera-i2c.c create mode 100644 drivers/mfd/madera-spi.c create mode 100644 drivers/mfd/madera.h create mode 100644 drivers/pinctrl/cirrus/Kconfig create mode 100644 drivers/pinctrl/cirrus/Makefile create mode 100644 drivers/pinctrl/cirrus/pinctrl-cs47l35.c create mode 100644 drivers/pinctrl/cirrus/pinctrl-cs47l85.c create mode 100644 drivers/pinctrl/cirrus/pinctrl-cs47l90.c create mode 100644 drivers/pinctrl/cirrus/pinctrl-madera.c create mode 100644 drivers/pinctrl/cirrus/pinctrl-madera.h create mode 100644 include/dt-bindings/sound/madera.h create mode 100644 include/linux/irqchip/irq-madera-pdata.h create mode 100644 include/linux/irqchip/irq-madera.h create mode 100644 include/linux/mfd/madera/core.h create mode 100644 include/linux/mfd/madera/pdata.h create mode 100644 include/linux/mfd/madera/registers.h create mode 100644 include/sound/madera-pdata.h create mode 100644 sound/soc/codecs/cs47l35.c create mode 100644 sound/soc/codecs/cs47l85.c create mode 100644 sound/soc/codecs/cs47l90.c create mode 100644 sound/soc/codecs/madera.c create mode 100644 sound/soc/codecs/madera.h -- 1.9.1

7 42

[alsa-devel] [PATCH 0/2] ASoC: hdmi-codec: Allow driver to restrict sample size
by srinivas.kandagatla＠linaro.org 27 Jul '17

27 Jul '17

From: Srinivas Kandagatla <srinivas.kandagatla(a)linaro.org> This patch adds option to allow hdmi-codec drivers to restrict the audio sample size based on the values that are suppored in hardware. First patch is adding that formats option to hdmi_codec_pdata and second patch is a fix in adv7511 codec driver which only supports 16 and 24 bit samples. Srinivas Kandagatla (2): ASoC: hdmi-codec: Allow drivers to restrict sample sizes. drm/bridge: adv7511: restrict audio sample sizes drivers/gpu/drm/bridge/adv7511/adv7511_audio.c | 2 ++ include/sound/hdmi-codec.h | 1 + sound/soc/codecs/hdmi-codec.c | 3 +++ 3 files changed, 6 insertions(+) -- 2.11.0

3 4

[alsa-devel] [PATCHv2 0/6] Motorola Droid 4 Audio Support
by Sebastian Reichel 27 Jul '17

27 Jul '17

Hi, This adds audio support to Motorola Droid 4. Tested: - Playing via both DACs using Speaker, Earpiece, Headphone - Recording using internal Mic - Volume Controls Known not to work: - 3.5mm detection support (requires closed source firmware, needs further investigation) - Modem / Bluetooth Audio (actually untested, but probably needs some quirks) Changes since PATCHv1: * Add patch from Tony exporting soc_dpcm_runtime_update * Integrate DT patch for vaudio initial mode * Split dt-binding from codec patch and add Ack from Rob * Fix CPCAP position in Kconfig/Makefile * Avoid "err +=" constructs * Simplify reset function * Drop cpcap_audio_read/write helpers * Do not use tertiary operator for mute register value * Update Input Mux logic * Switch from simple-audio-card to audio-graph-card -- Sebastian Sebastian Reichel (5): dt-bindings: sound: add motorola,cpcap-audio-codec ASoC: codec: cpcap: new codec ARM: dts: motorola-cpcap-mapphone: add audio-codec ARM: dts: motorola-cpcap-mapphone: set initial mode for vaudio ARM: dts: omap4-droid4: add soundcard Tony Lindgren (1): ALSA: pcm: Export soc_dpcm_runtime_update .../bindings/sound/motorola,cpcap-audio-codec.txt | 19 + arch/arm/boot/dts/motorola-cpcap-mapphone.dtsi | 7 + arch/arm/boot/dts/omap4-droid4-xt894.dts | 84 + sound/soc/codecs/Kconfig | 5 + sound/soc/codecs/Makefile | 2 + sound/soc/codecs/cpcap.c | 1607 ++++++++++++++++++++ sound/soc/soc-pcm.c | 2 + 7 files changed, 1726 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/motorola,cpcap-audio-codec.txt create mode 100644 sound/soc/codecs/cpcap.c -- 2.13.2

4 19

Re: [alsa-devel] [PATCH v1 5/6] ASoC: codecs: msm8916-wcd-analog: add MBHC support
by Damien Riegel 27 Jul '17

27 Jul '17

[Sorry for breaking the thread, but I don't have the initial message to reply to.] I tried your patchset and faced some issues regarding removal detection: the driver detects headphone and microphone removal as soon as I press a button on the headset. evtest logs: [Headset insertion] Event: time 10207.841157, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 1 Event: time 10207.841157, -------------- SYN_REPORT ------------ Event: time 10208.039105, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 0 Event: time 10208.039105, -------------- SYN_REPORT ------------ Event: time 10208.075519, type 5 (EV_SW), code 2 (SW_HEADPHONE_INSERT), value 1 Event: time 10208.075519, type 5 (EV_SW), code 4 (SW_MICROPHONE_INSERT), value 1 Event: time 10208.075519, -------------- SYN_REPORT ------------ [Pressing Volume up Button] Event: time 10210.590403, type 1 (EV_KEY), code 115 (KEY_VOLUMEUP), value 1 Event: time 10210.590403, type 5 (EV_SW), code 2 (SW_HEADPHONE_INSERT), value 0 Event: time 10210.590403, type 5 (EV_SW), code 4 (SW_MICROPHONE_INSERT), value 0 Event: time 10210.590403, -------------- SYN_REPORT ------------ Event: time 10210.719560, type 1 (EV_KEY), code 115 (KEY_VOLUMEUP), value 0 Event: time 10210.719560, -------------- SYN_REPORT ------------ [Unplugging] Event: time 10221.926156, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 1 Event: time 10221.926156, -------------- SYN_REPORT ------------ Event: time 10222.026466, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 0 Event: time 10222.026466, -------------- SYN_REPORT ------------ Also, the KEY_MEDIA is wrongly reported but I don't see that every time, I think it depends on how fast I plug in the headset. Anyway, I think that can easily be filtered out by testing the presence of the microphone. Regards, -- Damien

2 1