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

[alsa-devel] [PATCH] ASoC: cs35l35: Disable regulators on driver removal
by Charles Keepax 05 Apr '19

05 Apr '19

The chips main power supplies VA and VP are enabled during probe but then never disabled, this will cause warnings from the regulator framework on driver removal. Fix this by adding a remove callback and disabling the supplies, whilst doing so follow best practice and put the chip back into reset as well. Signed-off-by: Charles Keepax <ckeepax(a)opensource.cirrus.com> --- sound/soc/codecs/cs35l35.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/sound/soc/codecs/cs35l35.c b/sound/soc/codecs/cs35l35.c index 9f4a59871cee7..c71696146c5ec 100644 --- a/sound/soc/codecs/cs35l35.c +++ b/sound/soc/codecs/cs35l35.c @@ -1635,6 +1635,16 @@ static int cs35l35_i2c_probe(struct i2c_client *i2c_client, return ret; } +static int cs35l35_i2c_remove(struct i2c_client *i2c_client) +{ + struct cs35l35_private *cs35l35 = i2c_get_clientdata(i2c_client); + + regulator_bulk_disable(cs35l35->num_supplies, cs35l35->supplies); + gpiod_set_value_cansleep(cs35l35->reset_gpio, 0); + + return 0; +} + static const struct of_device_id cs35l35_of_match[] = { {.compatible = "cirrus,cs35l35"}, {}, @@ -1655,6 +1665,7 @@ static struct i2c_driver cs35l35_i2c_driver = { }, .id_table = cs35l35_id, .probe = cs35l35_i2c_probe, + .remove = cs35l35_i2c_remove, }; module_i2c_driver(cs35l35_i2c_driver); -- 2.11.0

2 1

[alsa-devel] [RFC PATCH 0/3] Fixes for component unregister and remove
by Ranjani Sridharan 05 Apr '19

05 Apr '19

This set of patches aims to fix the problems seen during module unloading with the skylake driver and also with SOF. The sequence seen during snd_soc_skl module unload is that the machine driver in unregistered first which frees the snd_card. When the component driver is unregistered following this and the topology is unloaded, a null pointer dereference is encountered while removing the kcontrols because the snd_card has already been freed. The first 2 patches fix this by removing the topology unloading from component_unregister() and moving it to the component remove() callback. This also makes the register_component() and unregister_component() calls balanced. But this still doesnt solve the problem entirely because the component driver's remove() callback is invoked after soc_cleanup_card_resources() which frees the snd_card. And we encounter the same null pointer dereference seen while remving the kcontrols added to the snd_card. In order to avoid this problem, the third patch proposes to remove the link components in snd_soc_unbind_card() before soc_cleanup_card_resources(). This way, the snd_card() will remain valid when the topology is unloaded. Ranjani Sridharan (3): ASoC: core: do not unload topology in unregister_component() ASoC: intel: skylake: add remove() callback for component driver ASoC: core: remove link components before cleaning up card resources sound/soc/intel/skylake/skl-pcm.c | 7 +++++++ sound/soc/soc-core.c | 13 +++++++++++-- 2 files changed, 18 insertions(+), 2 deletions(-) -- 2.17.1

2 10

[alsa-devel] [PATCH 0/6] ASoC: meson: add g12a SoC family support
by Jerome Brunet 05 Apr '19

05 Apr '19

The Amlogic g12a SoC family audio subsystem derives from the one found in the axg SoC family. To add support for the g12a in the existing elements, this patchset deals with following differences compared to the axg family: * fifos must set an initial start address within the buffer. The fifo pointer shall be reset to this address on startup. * frddrs may now play on up to 3 output interfaces simultaneously. * tdmout formatters need a different signal skew offset. Jerome Brunet (6): ASoC: meson: add g12a compatibles ASoC: meson: axg-fifo: add g12a support ASoC: meson: axg-toddr: add g12a support ASoC: meson: axg-frddr: add g12a support ASoC: meson: axg-tdm-formatter: rework quirks settings ASoC: meson: axg-tdmout: add g12a support .../bindings/sound/amlogic,axg-fifo.txt | 4 +- .../bindings/sound/amlogic,axg-pdm.txt | 3 +- .../bindings/sound/amlogic,axg-spdifin.txt | 3 +- .../bindings/sound/amlogic,axg-spdifout.txt | 3 +- .../sound/amlogic,axg-tdm-formatters.txt | 4 +- sound/soc/meson/axg-fifo.c | 34 ++++- sound/soc/meson/axg-fifo.h | 2 + sound/soc/meson/axg-frddr.c | 143 +++++++++++++++++- sound/soc/meson/axg-tdm-formatter.c | 6 +- sound/soc/meson/axg-tdm-formatter.h | 11 +- sound/soc/meson/axg-tdmin.c | 16 +- sound/soc/meson/axg-tdmout.c | 29 +++- sound/soc/meson/axg-toddr.c | 53 +++++++ 13 files changed, 286 insertions(+), 25 deletions(-) -- 2.20.1

2 12

[alsa-devel] [PATCH 0/2] ASoC: topology: fix endianness issues
by Pierre-Louis Bossart 05 Apr '19

05 Apr '19

Sparse reports dozens of issues with the topology code due to endianness. Fix as needed. There are still a handful of issues that I could not figure out (see below), but at least now the logs become usable. The only remaining problematic files are the SST loader and Skylake topology handling, which are likely to remain as is without any updates. sound/soc/soc-ops.c:657:33: warning: invalid assignment: &= sound/soc/soc-ops.c:657:33: left side has type unsigned short sound/soc/soc-ops.c:657:33: right side has type restricted __be16 sound/soc/soc-ops.c:661:33: warning: invalid assignment: &= sound/soc/soc-ops.c:661:33: left side has type unsigned int sound/soc/soc-ops.c:661:33: right side has type restricted __be32 sound/soc/soc-topology.c:616:70: warning: restricted __le32 degrades to integer sound/soc/soc-topology.c:618:70: warning: restricted __le32 degrades to integer sound/soc/soc-topology.c:633:60: warning: restricted __le32 degrades to integer sound/soc/soc-topology.c:635:60: warning: restricted __le32 degrades to integer sound/soc/soc-topology.c:637:61: warning: restricted __le32 degrades to integer sound/soc/soc-topology.c:650:60: warning: restricted __le32 degrades to integer sound/soc/soc-topology.c:652:60: warning: restricted __le32 degrades to integer sound/soc/soc-topology.c:654:61: warning: restricted __le32 degrades to integer sound/soc/soc-topology.c:2507:16: warning: restricted __le32 degrades to integer Pierre-Louis Bossart (2): ASoC: topology: fix endianness issues ASoC: topology: fix big-endian check sound/soc/soc-topology.c | 295 +++++++++++++++++++++++---------------- 1 file changed, 173 insertions(+), 122 deletions(-) -- 2.17.1

2 4

[alsa-devel] [PATCH 0/3] soundwire: code cleanup
by Pierre-Louis Bossart 04 Apr '19

04 Apr '19

SoundWire support will be provided in Linux with the Sound Open Firmware (SOF) on Intel platforms. Before we start adding the missing pieces, there are a number of warnings and style issues reported by checkpatch, cppcheck and Coccinelle that need to be cleaned-up. While I am at it, I could use feedback from maintainers. We are ready to release a number of required patches that were not provided earlier, but the initial contributors have either left Intel or moved on to other domains. While I'd like to give them credit for their work, I don't think leaving the email addresses as is is a good solution (email bounces, etc). I also don't think providing a non-Intel address is correct since the work was funded by Intel. Is there a recommended practice here? Pierre-Louis Bossart (3): soundwire: fix style issues soundwire: bus: remove useless initializations soundwire: stream: remove useless initialization of local variable drivers/soundwire/Kconfig | 2 +- drivers/soundwire/bus.c | 89 ++++++++-------- drivers/soundwire/bus.h | 16 +-- drivers/soundwire/bus_type.c | 4 +- drivers/soundwire/cadence_master.c | 87 +++++++-------- drivers/soundwire/cadence_master.h | 22 ++-- drivers/soundwire/intel.c | 89 ++++++++-------- drivers/soundwire/intel.h | 4 +- drivers/soundwire/intel_init.c | 12 +-- drivers/soundwire/mipi_disco.c | 116 +++++++++++--------- drivers/soundwire/slave.c | 10 +- drivers/soundwire/stream.c | 163 +++++++++++++++-------------- 12 files changed, 315 insertions(+), 299 deletions(-) -- 2.17.1

2 5

[alsa-devel] [PATCH v4 00/20] ASoC: Sound Open Firmware (SOF) - Intel support
by Pierre-Louis Bossart 04 Apr '19

04 Apr '19

This patch series introduces the support for Intel devices (Baytrail, CherryTrail, Broadwell, ApolloLake, GeminiLake, CannonLake, IceLake). Support for Haswell, Skylake and KabyLake is incomplete at the moment and is not shared for now. The series starts with the 'legacy' devices, then all the more recent ones and concludes with ACPI/PCI hooks. SKL+ platforms can be compiled without HDaudio link/codec support, in which case the hdac library is not used. When support for HDMI or HDaudio codecs is selected, parts of this library are used. A small set of machine drivers supported by SOF is now provided as part of this series, more to follow. Support for SPI-based devices, compressed streams is also not provided for now since it's not ready for upstream just yet. Thank you for reviews and comments, we appreciate the time spent commenting on this large patchset. Thanks in particular to Alan Cox and Andy Shevchenko for their comments on an earlier version. Changes since v3: Removed Haswell, Skylake and Kabylake support (code is not tested or is still under development). These devices will be re-added when there is functional firmware and for Skylake/Kabylake a signed version of the firmware usable on existing platforms. Added support for existing machine drivers when relevant (more machine drivers will be added in next batch) Fixed hardware programming sequences (ROM init timeout, HDaudio capture, simultaneous capture and playback, DMA preload) Avoid reading IPC reply on CTX_SAVE reply since memory windows might be powered off. Simplified NOCODEC support, now mutually exclusive with HDaudio link to avoid suspend/resume issues. Changes since v2: Addressed dozens of comments from Takashi Iwai, Mark Brown, Andy Shevchenko, Daniel Baluta (Thanks!) Transition to new display power management Fixed suspend issues Optimized register polling, better handling of timeouts and firmware download retries Detect number of HDaudio codec and select machine driver accordingly Added workqueue based probe for HDaudio (dependency on modules) Fixed platform name override and conflicts with DT deferred probe changes Added prefix for firmware and topology files and module arguments Split Baytrail and Edison Fixed warnings (smatch, cppcheck) Keyon Jie (2): ASoC: SOF: Intel: Add hda-bus support and initialization ASoC: SOF: Intel: add SKL+ platform DAIs Liam Girdwood (15): ASoC: SOF: Intel: Add BYT, CHT and BSW DSP HW support. ASoC: SOF: Intel: Add BDW HW DSP support ASoC: SOF: Intel: Add APL/CNL HW DSP support ASoC: SOF: Intel: Add HDA controller for Intel DSP ASoC: SOF: Intel: Add Intel specific HDA DSP HW operations ASoC: SOF: Intel: Add Intel specific HDA IPC mechanisms. ASoC: SOF: Intel: Add Intel specific HDA firmware loader ASoC: SOF: Intel: Add Intel specific HDA PCM operations ASoC: SOF: Intel: Add Intel specific HDA stream operations ASoC: SOF: Intel: Add Intel specific HDA trace operations ASoC: SOF: Intel: Add platform differentiation for APL and CNL ASoC: SOF: Add ACPI device support ASoC: SOF: Add PCI device support ASoC: SOF: Add Build support for SOF core and Intel drivers ASoC: Intel: Make sure HSW/BDW based machine drivers build for SOF Pierre-Louis Bossart (3): ASoC: SOF: Intel: Add support for HDAudio codecs ASoC: Intel: Kconfig: expose common option between SST and SOF drivers ASoC: Intel: select relevant machine drivers for SOF sound/soc/Kconfig | 1 + sound/soc/Makefile | 1 + sound/soc/intel/Kconfig | 7 +- sound/soc/intel/boards/Kconfig | 22 +- sound/soc/intel/boards/bdw-rt5677.c | 4 + sound/soc/intel/boards/broadwell.c | 4 + sound/soc/intel/boards/haswell.c | 4 + sound/soc/sof/Kconfig | 141 +++++ sound/soc/sof/Makefile | 18 + sound/soc/sof/intel/Kconfig | 222 ++++++++ sound/soc/sof/intel/Makefile | 15 + sound/soc/sof/intel/apl.c | 112 ++++ sound/soc/sof/intel/bdw.c | 694 ++++++++++++++++++++++ sound/soc/sof/intel/byt.c | 853 ++++++++++++++++++++++++++++ sound/soc/sof/intel/cnl.c | 257 +++++++++ sound/soc/sof/intel/hda-bus.c | 108 ++++ sound/soc/sof/intel/hda-codec.c | 171 ++++++ sound/soc/sof/intel/hda-ctrl.c | 165 ++++++ sound/soc/sof/intel/hda-dai.c | 318 +++++++++++ sound/soc/sof/intel/hda-dsp.c | 428 ++++++++++++++ sound/soc/sof/intel/hda-ipc.c | 402 +++++++++++++ sound/soc/sof/intel/hda-loader.c | 366 ++++++++++++ sound/soc/sof/intel/hda-pcm.c | 242 ++++++++ sound/soc/sof/intel/hda-stream.c | 682 ++++++++++++++++++++++ sound/soc/sof/intel/hda-trace.c | 98 ++++ sound/soc/sof/intel/hda.c | 672 ++++++++++++++++++++++ sound/soc/sof/intel/hda.h | 546 ++++++++++++++++++ sound/soc/sof/intel/shim.h | 179 ++++++ sound/soc/sof/sof-acpi-dev.c | 310 ++++++++++ sound/soc/sof/sof-pci-dev.c | 373 ++++++++++++ 30 files changed, 7407 insertions(+), 8 deletions(-) create mode 100644 sound/soc/sof/Kconfig create mode 100644 sound/soc/sof/Makefile create mode 100644 sound/soc/sof/intel/Kconfig create mode 100644 sound/soc/sof/intel/Makefile create mode 100644 sound/soc/sof/intel/apl.c create mode 100644 sound/soc/sof/intel/bdw.c create mode 100644 sound/soc/sof/intel/byt.c create mode 100644 sound/soc/sof/intel/cnl.c create mode 100644 sound/soc/sof/intel/hda-bus.c create mode 100644 sound/soc/sof/intel/hda-codec.c create mode 100644 sound/soc/sof/intel/hda-ctrl.c create mode 100644 sound/soc/sof/intel/hda-dai.c create mode 100644 sound/soc/sof/intel/hda-dsp.c create mode 100644 sound/soc/sof/intel/hda-ipc.c create mode 100644 sound/soc/sof/intel/hda-loader.c create mode 100644 sound/soc/sof/intel/hda-pcm.c create mode 100644 sound/soc/sof/intel/hda-stream.c create mode 100644 sound/soc/sof/intel/hda-trace.c create mode 100644 sound/soc/sof/intel/hda.c create mode 100644 sound/soc/sof/intel/hda.h create mode 100644 sound/soc/sof/intel/shim.h create mode 100644 sound/soc/sof/sof-acpi-dev.c create mode 100644 sound/soc/sof/sof-pci-dev.c -- 2.17.1

5 34

[alsa-devel] [PATCH] ASoC: wm_adsp: Avoid calling snd_compr_stop_error from WDT expiry
by Charles Keepax 04 Apr '19

04 Apr '19

It is unsafe to call snd_compr_stop_error from outside of the compressed ops. Firstly the compressed device lock needs to be held and secondly it queues error work to issue a trigger stop which should not happen after the stream has been freed. To avoid these issues use the same trick used for the IRQ handling, simply send a snd_compr_fragment_elapsed to cause user-space to wake on the poll, then report the error when user-space issues the pointer request after it wakes. Fixes: 56bc22bd1dac ("ASoC: wm_adsp: Shutdown any compressed streams on DSP watchdog timeout") Signed-off-by: Charles Keepax <ckeepax(a)opensource.cirrus.com> --- sound/soc/codecs/wm_adsp.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/sound/soc/codecs/wm_adsp.c b/sound/soc/codecs/wm_adsp.c index c8c49d5b8ac95..a9298bfddd9c7 100644 --- a/sound/soc/codecs/wm_adsp.c +++ b/sound/soc/codecs/wm_adsp.c @@ -4092,7 +4092,7 @@ int wm_adsp_compr_pointer(struct snd_compr_stream *stream, buf = compr->buf; - if (!buf || buf->error) { + if (dsp->fatal_error || !buf || buf->error) { snd_compr_stop_error(stream, SNDRV_PCM_STATE_XRUN); ret = -EIO; goto out; @@ -4196,12 +4196,13 @@ static int wm_adsp_buffer_capture_block(struct wm_adsp_compr *compr, int target) static int wm_adsp_compr_read(struct wm_adsp_compr *compr, char __user *buf, size_t count) { + struct wm_adsp *dsp = compr->dsp; int ntotal = 0; int nwords, nbytes; compr_dbg(compr, "Requested read of %zu bytes\n", count); - if (!compr->buf || compr->buf->error) { + if (dsp->fatal_error || !compr->buf || compr->buf->error) { snd_compr_stop_error(compr->stream, SNDRV_PCM_STATE_XRUN); return -EIO; } @@ -4262,11 +4263,8 @@ static void wm_adsp_fatal_error(struct wm_adsp *dsp) dsp->fatal_error = true; list_for_each_entry(compr, &dsp->compr_list, list) { - if (compr->stream) { - snd_compr_stop_error(compr->stream, - SNDRV_PCM_STATE_XRUN); + if (compr->stream) snd_compr_fragment_elapsed(compr->stream); - } } } -- 2.11.0

2 3

[alsa-devel] [PATCH 1/2] ASoC: tlv320aic32x4: Add Switch for Setting Common Mode Voltage
by Annaliese McDermond 04 Apr '19

04 Apr '19

Add a switch for setting common mode voltage. This can allow for higher drive levels on the amplifier outputs. Signed-off-by: Annaliese McDermond <nh6z(a)nh6z.net> --- sound/soc/codecs/tlv320aic32x4.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/sound/soc/codecs/tlv320aic32x4.c b/sound/soc/codecs/tlv320aic32x4.c index aec86fd6ac15..77df2cbdf5f4 100644 --- a/sound/soc/codecs/tlv320aic32x4.c +++ b/sound/soc/codecs/tlv320aic32x4.c @@ -242,6 +242,12 @@ static DECLARE_TLV_DB_SCALE(tlv_driver_gain, -600, 100, 0); /* -12dB min, 0.5dB steps */ static DECLARE_TLV_DB_SCALE(tlv_adc_vol, -1200, 50, 0); +static const char * const lo_cm_text[] = { + "Full Chip", "1.65V", +}; + +static SOC_ENUM_SINGLE_DECL(lo_cm_enum, AIC32X4_CMMODE, 3, lo_cm_text); + static const struct snd_kcontrol_new aic32x4_snd_controls[] = { SOC_DOUBLE_R_S_TLV("PCM Playback Volume", AIC32X4_LDACVOL, AIC32X4_RDACVOL, 0, -0x7f, 0x30, 7, 0, tlv_pcm), @@ -255,6 +261,7 @@ static const struct snd_kcontrol_new aic32x4_snd_controls[] = { AIC32X4_HPRGAIN, 6, 0x01, 1), SOC_DOUBLE_R("LO DAC Playback Switch", AIC32X4_LOLGAIN, AIC32X4_LORGAIN, 6, 0x01, 1), + SOC_ENUM("LO Playback Common Mode Switch", lo_cm_enum), SOC_DOUBLE_R("Mic PGA Switch", AIC32X4_LMICPGAVOL, AIC32X4_RMICPGAVOL, 7, 0x01, 1), -- 2.19.1

2 5

[alsa-devel] [PATCH] ASoC: max98357a: add missing supported rates
by Jerome Brunet 04 Apr '19

04 Apr '19

According the publicly available datasheet (and some test) the max98357a also supports 32, 44.1 and 88.2 kHz sample rate. Signed-off-by: Jerome Brunet <jbrunet(a)baylibre.com> --- Datasheet is available here: https://datasheets.maximintegrated.com/en/ds/MAX98357A-MAX98357B.pdf sound/soc/codecs/max98357a.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/soc/codecs/max98357a.c b/sound/soc/codecs/max98357a.c index d469576b5a7b..d037a3e4d323 100644 --- a/sound/soc/codecs/max98357a.c +++ b/sound/soc/codecs/max98357a.c @@ -97,7 +97,10 @@ static struct snd_soc_dai_driver max98357a_dai_driver = { SNDRV_PCM_FMTBIT_S32, .rates = SNDRV_PCM_RATE_8000 | SNDRV_PCM_RATE_16000 | + SNDRV_PCM_RATE_32000 | + SNDRV_PCM_RATE_44100 | SNDRV_PCM_RATE_48000 | + SNDRV_PCM_RATE_88200 | SNDRV_PCM_RATE_96000, .rate_min = 8000, .rate_max = 96000, -- 2.20.1

2 1

[alsa-devel] [Xen-devel][PATCH] ALSA: xen-front: Do not use stream buffer size before it is set
by Oleksandr Andrushchenko 04 Apr '19

04 Apr '19

From: Oleksandr Andrushchenko <oleksandr_andrushchenko(a)epam.com> This fixes the regression introduced while moving to Xen shared buffer implementation. Fixes: 58f9d806d16a ("ALSA: xen-front: Use Xen common shared buffer implementation") Signed-off-by: Oleksandr Andrushchenko <oleksandr_andrushchenko(a)epam.com> --- sound/xen/xen_snd_front_alsa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/xen/xen_snd_front_alsa.c b/sound/xen/xen_snd_front_alsa.c index a7f413cb704d..b14ab512c2ce 100644 --- a/sound/xen/xen_snd_front_alsa.c +++ b/sound/xen/xen_snd_front_alsa.c @@ -441,7 +441,7 @@ static int shbuf_setup_backstore(struct xen_snd_front_pcm_stream_info *stream, { int i; - stream->buffer = alloc_pages_exact(stream->buffer_sz, GFP_KERNEL); + stream->buffer = alloc_pages_exact(buffer_sz, GFP_KERNEL); if (!stream->buffer) return -ENOMEM; -- 2.21.0

3 3