January 2018 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [RESEND PATCH] ASoC: rt5514-spi: Check the validity of drvdata pointer on resume
by Marc Zyngier 24 Jan '18

24 Jan '18

The rt5514-spi driver seem to assume the validity of the drvdata pointer on resume, which it may not be populated, leading to a not-so-nice crash. This stems from the fact that rt5514_spi_pcm_probe() is never called on my system (a kevin Chromebook). No idea why, but if it can happen, it is worth fixing. Fixes: e9c50aa6bd39 ("ASoC: rt5514-spi: check irq status to schedule data copy in resume function") Signed-off-by: Marc Zyngier <marc.zyngier(a)arm.com> --- sound/soc/codecs/rt5514-spi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/rt5514-spi.c b/sound/soc/codecs/rt5514-spi.c index 2df91db765ac..9255afcf2c3a 100644 --- a/sound/soc/codecs/rt5514-spi.c +++ b/sound/soc/codecs/rt5514-spi.c @@ -482,7 +482,7 @@ static int __maybe_unused rt5514_resume(struct device *dev) if (device_may_wakeup(dev)) disable_irq_wake(irq); - if (rt5514_dsp->substream) { + if (rt5514_dsp && rt5514_dsp->substream) { rt5514_spi_burst_read(RT5514_IRQ_CTRL, (u8 *)&buf, sizeof(buf)); if (buf[0] & RT5514_IRQ_STATUS_BIT) rt5514_schedule_copy(rt5514_dsp); -- 2.14.2

1 0

[alsa-devel] [PATCH] ASoC: soc-pcm: don't call flush_delayed_work() many times in soc_pcm_private_free()
by Kuninori Morimoto 24 Jan '18

24 Jan '18

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> commit f523acebbb74 ("ASoC: add Component level pcm_new/pcm_free v2") added component level pcm_new/pcm_free, but flush_delayed_work() on soc_pcm_private_free() is called in for_each_rtdcom() loop. It doesn't need to be called many times. This patch moves it out of loop. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/soc-pcm.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c index 8075856..998800c 100644 --- a/sound/soc/soc-pcm.c +++ b/sound/soc/soc-pcm.c @@ -2831,10 +2831,9 @@ static void soc_pcm_private_free(struct snd_pcm *pcm) struct snd_soc_rtdcom_list *rtdcom; struct snd_soc_component *component; + /* need to sync the delayed work before releasing resources */ + flush_delayed_work(&rtd->delayed_work); for_each_rtdcom(rtd, rtdcom) { - /* need to sync the delayed work before releasing resources */ - - flush_delayed_work(&rtd->delayed_work); component = rtdcom->component; if (component->pcm_free) -- 1.9.1

2 1

[alsa-devel] [PATCH v5] ASoC: ak4613: call dummy write for PW_MGMT1/3 when Playback
by Kuninori Morimoto 24 Jan '18

24 Jan '18

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Power Down Release Command (PMVR, PMDAC, RSTN, PMDA1-PMDA6) which are located on PW_MGMT1 / PW_MGMT3 register must be write again after at least 5 LRCK cycle or later on each command. Otherwise, Playback volume will be 0dB. Basically, it should be 1. PowerDownRelease by Power Management1 <= call 1.x after 5LRCK 1.x Dummy write to Power Management1 2. PowerDownRelease by Power Management3 <= call 2.x after 5LRCK 2.x Dummy write to Power Management3 To avoid too many dummy write, this patch is merging these. 1. PowerDownRelease by Power Management1 2. PowerDownRelease by Power Management3 <= call after 5LRCK 2.x Dummy write to Power Management1/3 <= merge dummy write This patch adds dummy write when Playback Start timing. Tested-by: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx(a)renesas.com> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- v4 -> v5 - added Note on comment - tidyup comment (delayed work -> schedule_work) sound/soc/codecs/ak4613.c | 78 +++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 78 insertions(+) diff --git a/sound/soc/codecs/ak4613.c b/sound/soc/codecs/ak4613.c index ee9e822..d85c61e 100644 --- a/sound/soc/codecs/ak4613.c +++ b/sound/soc/codecs/ak4613.c @@ -15,6 +15,7 @@ */ #include <linux/clk.h> +#include <linux/delay.h> #include <linux/i2c.h> #include <linux/slab.h> #include <linux/of_device.h> @@ -95,6 +96,9 @@ struct ak4613_priv { struct mutex lock; const struct ak4613_interface *iface; struct snd_pcm_hw_constraint_list constraint; + struct work_struct dummy_write_work; + struct snd_soc_component *component; + unsigned int rate; unsigned int sysclk; unsigned int fmt; @@ -392,6 +396,7 @@ static int ak4613_dai_hw_params(struct snd_pcm_substream *substream, default: return -EINVAL; } + priv->rate = rate; /* * FIXME @@ -467,11 +472,83 @@ static int ak4613_set_bias_level(struct snd_soc_component *component, return 0; } +static void ak4613_dummy_write(struct work_struct *work) +{ + struct ak4613_priv *priv = container_of(work, + struct ak4613_priv, + dummy_write_work); + struct snd_soc_component *component = priv->component; + unsigned int mgmt1; + unsigned int mgmt3; + + /* + * PW_MGMT1 / PW_MGMT3 needs dummy write at least after 5 LR clocks + * + * Note + * + * To avoid extra delay, we want to avoid preemption here, + * but we can't. Because it uses I2C access which is using IRQ + * and sleep. Thus, delay might be more than 5 LR clocks + * see also + * ak4613_dai_trigger() + */ + udelay(5000000 / priv->rate); + + snd_soc_component_read(component, PW_MGMT1, &mgmt1); + snd_soc_component_read(component, PW_MGMT3, &mgmt3); + + snd_soc_component_write(component, PW_MGMT1, mgmt1); + snd_soc_component_write(component, PW_MGMT3, mgmt3); +} + +static int ak4613_dai_trigger(struct snd_pcm_substream *substream, int cmd, + struct snd_soc_dai *dai) +{ + struct snd_soc_codec *codec = dai->codec; + struct ak4613_priv *priv = snd_soc_codec_get_drvdata(codec); + + /* + * FIXME + * + * PW_MGMT1 / PW_MGMT3 needs dummy write at least after 5 LR clocks + * from Power Down Release. Otherwise, Playback volume will be 0dB. + * To avoid complex multiple delay/dummy_write method from + * ak4613_set_bias_level() / SND_SOC_DAPM_DAC_E("DACx", ...), + * call it once here. + * + * But, unfortunately, we can't "write" here because here is atomic + * context (It uses I2C access for writing). + * Thus, use schedule_work() to switching to normal context + * immediately. + * + * Note + * + * Calling ak4613_dummy_write() function might be delayed. + * In such case, ak4613 volume might be temporarily 0dB when + * beggining of playback. + * see also + * ak4613_dummy_write() + */ + + if ((cmd != SNDRV_PCM_TRIGGER_START) && + (cmd != SNDRV_PCM_TRIGGER_RESUME)) + return 0; + + if (substream->stream != SNDRV_PCM_STREAM_PLAYBACK) + return 0; + + priv->component = &codec->component; + schedule_work(&priv->dummy_write_work); + + return 0; +} + static const struct snd_soc_dai_ops ak4613_dai_ops = { .startup = ak4613_dai_startup, .shutdown = ak4613_dai_shutdown, .set_sysclk = ak4613_dai_set_sysclk, .set_fmt = ak4613_dai_set_fmt, + .trigger = ak4613_dai_trigger, .hw_params = ak4613_dai_hw_params, }; @@ -591,6 +668,7 @@ static int ak4613_i2c_probe(struct i2c_client *i2c, priv->iface = NULL; priv->cnt = 0; priv->sysclk = 0; + INIT_WORK(&priv->dummy_write_work, ak4613_dummy_write); mutex_init(&priv->lock); -- 1.9.1

2 1

[alsa-devel] [PATCH - JACK PCM plugin] jack: Support snd_pcm_drain()
by twischer＠de.adit-jv.com 24 Jan '18

24 Jan '18

From: Timo Wischer <twischer(a)de.adit-jv.com> Block on drain till available samples played Without this commit the JACK thread will be stopped before the ALSA buffer was completely forwarded to the JACK daemon. Signed-off-by: Timo Wischer <twischer(a)de.adit-jv.com> --- This patch is depending on http://mailman.alsa-project.org/pipermail/alsa-devel/2018-January/130991.ht… diff --git a/jack/pcm_jack.c b/jack/pcm_jack.c index e75aadc..ec94583 100644 --- a/jack/pcm_jack.c +++ b/jack/pcm_jack.c @@ -28,6 +28,17 @@ #include <alsa/asoundlib.h> #include <alsa/pcm_external.h> +/* ALSA supports up to 64 periods per buffer. + * Therefore at least 64 retries are valid and + * should not be handled as an error case + */ +#define MAX_DRAIN_RETRIES 100 +/* ALSA supports a a period with 8192 frames. + * This would result in ~170ms at 48kHz. + * Therefore a time out of 1 second is sufficient + */ +#define DRAIN_TIMEOUT 1000 + typedef enum _jack_format { SND_PCM_JACK_FORMAT_RAW } snd_pcm_jack_format_t; @@ -146,7 +157,13 @@ static int pcm_poll_unblock_check(snd_pcm_ioplug_t *io) snd_pcm_jack_t *jack = io->private_data; avail = snd_pcm_avail_update(io->pcm); - if (avail < 0 || avail >= jack->min_avail) { + /* In draining state poll_fd is used to wait + * till all pending frames are played. + * Therefore it has to be guarantee that a poll event is also generated + * if the buffer contains less than min_avail frames + */ + if (avail < 0 || avail >= jack->min_avail || + jack->state == SND_PCM_STATE_DRAINING) { write(jack->fd, &buf, 1); return 1; } @@ -242,7 +259,8 @@ snd_pcm_jack_process_cb(jack_nframes_t nframes, snd_pcm_ioplug_t *io) } hw_ptr = jack->hw_ptr; - if (jack->state == SND_PCM_STATE_RUNNING) { + if (jack->state == SND_PCM_STATE_RUNNING || + jack->state == SND_PCM_STATE_DRAINING) { const snd_pcm_channel_area_t *areas = snd_pcm_ioplug_mmap_areas(io); while (xfer < nframes) { @@ -402,6 +420,61 @@ static int snd_pcm_jack_start(snd_pcm_ioplug_t *io) return err; } +static int snd_pcm_jack_drain(snd_pcm_ioplug_t *io) +{ + snd_pcm_jack_t *jack = io->private_data; + const snd_pcm_state_t state = jack->state; + unsigned int retries = MAX_DRAIN_RETRIES; + char buf[32]; + + /* Immediately stop on capture device. + * snd_pcm_jack_stop() will be automatically called + * by snd_pcm_ioplug_drain() + */ + if (io->stream == SND_PCM_STREAM_CAPTURE) { + return 0; + } + + if (snd_pcm_jack_hw_avail(io, jack->hw_ptr, io->appl_ptr) <= 0) { + /* No data pending. Nothing to drain. */ + return 0; + } + + /* start device if not yet done */ + if (state == SND_PCM_STATE_PREPARED) { + snd_pcm_jack_start(io); + } + + /* FIXME: io->state will not be set to SND_PCM_STATE_DRAINING by the + * ALSA library before calling this function. + * Therefore this state has to be stored internally. + */ + jack->state = SND_PCM_STATE_DRAINING; + + struct pollfd pfd; + pfd.fd = io->poll_fd; + pfd.events = io->poll_events | POLLERR | POLLNVAL; + + while (snd_pcm_jack_hw_avail(io, jack->hw_ptr, io->appl_ptr) > 0) { + if (retries <= 0) { + SNDERR("Pending frames not yet processed."); + return -ETIMEDOUT; + } + + if (poll(&pfd, 1, DRAIN_TIMEOUT) < 0) { + SNDERR("Waiting for next JACK process callback failed (err %d)", + -errno); + return -errno; + } + + /* clean pending events. */ + while (read(io->poll_fd, &buf, sizeof(buf)) == sizeof(buf)) + ; + } + + return 0; +} + static int snd_pcm_jack_stop(snd_pcm_ioplug_t *io) { snd_pcm_jack_t *jack = io->private_data; @@ -433,6 +506,7 @@ static snd_pcm_ioplug_callback_t jack_pcm_callback = { .stop = snd_pcm_jack_stop, .pointer = snd_pcm_jack_pointer, .prepare = snd_pcm_jack_prepare, + .drain = snd_pcm_jack_drain, .poll_revents = snd_pcm_jack_poll_revents, }; -- 2.7.4

1 0

[alsa-devel] (no subject)
by twischer＠de.adit-jv.com 24 Jan '18

24 Jan '18

The following patch set will extend the JACK plugin to detect Xruns and signal them to the user application. This patch set is depending on http://mailman.alsa-project.org/pipermail/alsa-devel/2018-January/130987.ht…

1 3

[alsa-devel] Bug message caught in syslog
by Pavel Řezníček 24 Jan '18

24 Jan '18

Dear developers, I'm sorry if I write to the wrong list but I just catched several log messages that I was advised to report as a bug to you, the ALSA developers. I'm not experiencing any sound problems by ear, just found these messages in the syslog (via logcheck). Ran the alsa-info_alsa-info.sh shell script and its output can be found here: http://www.alsa-project.org/db/?f=d9b579a048b94cdc71e619c4deac60e4c8228dbb The log messages in question read as follows: Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: snd_pcm_avail() navrátil hodnotu, která je nezvykle vysoká: 570108 bajtů (3231 ms). Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: S největší pravděpodobností se jedná o chybu v ovladači ALSA "snd_hda_intel". Nahlaste prosím tento problém vývojářům ALSA. Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: snd_pcm_dump(): Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: Soft volume PCM Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: Control: PCM Playback Volume Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: min_dB: -51 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: max_dB: 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: resolution: 256 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: Its setup is: Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: stream : PLAYBACK Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: access : MMAP_INTERLEAVED Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: format : S16_LE Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: subformat : STD Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: channels : 2 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: rate : 44100 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: exact rate : 44100 (44100/1) Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: msbits : 16 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: buffer_size : 16384 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_size : 8192 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_time : 185759 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: tstamp_mode : ENABLE Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: tstamp_type : MONOTONIC Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_step : 1 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: avail_min : 15802 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_event : 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: start_threshold : -1 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: stop_threshold : 4611686018427387904 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: silence_threshold: 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: silence_size : 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: boundary : 4611686018427387904 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: Slave: Hardware PCM card 0 'HDA Intel PCH' device 0 subdevice 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: Its setup is: Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: stream : PLAYBACK Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: access : MMAP_INTERLEAVED Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: format : S16_LE Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: subformat : STD Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: channels : 2 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: rate : 44100 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: exact rate : 44100 (44100/1) Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: msbits : 16 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: buffer_size : 16384 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_size : 8192 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_time : 185759 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: tstamp_mode : ENABLE Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: tstamp_type : MONOTONIC Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_step : 1 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: avail_min : 15802 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_event : 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: start_threshold : -1 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: stop_threshold : 4611686018427387904 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: silence_threshold: 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: silence_size : 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: boundary : 4611686018427387904 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: appl_ptr : 95068169 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: hw_ptr : 95194312 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: snd_pcm_delay() navrátil hodnotu, která je nezvykle vysoká: -454064 bajtů (-2574 ms). Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: S největší pravděpodobností se jedná o chybu v ovladači ALSA "snd_hda_intel". Nahlaste prosím tento problém vývojářům ALSA. Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: snd_pcm_dump(): Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: Soft volume PCM Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: Control: PCM Playback Volume Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: min_dB: -51 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: max_dB: 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: resolution: 256 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: Its setup is: Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: stream : PLAYBACK Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: access : MMAP_INTERLEAVED Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: format : S16_LE Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: subformat : STD Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: channels : 2 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: rate : 44100 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: exact rate : 44100 (44100/1) Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: msbits : 16 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: buffer_size : 16384 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_size : 8192 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_time : 185759 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: tstamp_mode : ENABLE Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: tstamp_type : MONOTONIC Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_step : 1 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: avail_min : 15802 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_event : 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: start_threshold : -1 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: stop_threshold : 4611686018427387904 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: silence_threshold: 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: silence_size : 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: boundary : 4611686018427387904 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: Slave: Hardware PCM card 0 'HDA Intel PCH' device 0 subdevice 0 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: Its setup is: Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: stream : PLAYBACK Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: access : MMAP_INTERLEAVED Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: format : S16_LE Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: subformat : STD Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: channels : 2 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: rate : 44100 Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: exact rate : 44100 (44100/1) Jan 24 07:01:25 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: msbits : 16 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: buffer_size : 16384 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_size : 8192 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_time : 185759 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: tstamp_mode : ENABLE Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: tstamp_type : MONOTONIC Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_step : 1 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: avail_min : 15802 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: period_event : 0 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: start_threshold : -1 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: stop_threshold : 4611686018427387904 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: silence_threshold: 0 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: silence_size : 0 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: boundary : 4611686018427387904 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: appl_ptr : 95099921 Jan 24 07:01:26 ideapad-310 pulseaudio[2724]: [alsa-sink-CX20751/2 Analog] alsa-util.c: hw_ptr : 95213480 I'd be happy to help you solving this issue if not known already. Please feel free to ask for whatever information you'd need. Pavel Řezníček A. K. A. Cigydd

1 0

[alsa-devel] [PATCH][resend] ASoC: soc-pcm: don't call flush_delayed_work() many times in soc_pcm_private_free()
by Kuninori Morimoto 24 Jan '18

24 Jan '18

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> commit f523acebbb74 ("ASoC: add Component level pcm_new/pcm_free v2") added component level pcm_new/pcm_free, but flush_delayed_work() on soc_pcm_private_free() is called in for_each_rtdcom() loop. It doesn't need to be called many times. This patch moves it out of loop. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- 2weeks passed. resend sound/soc/soc-pcm.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c index 8075856..998800c 100644 --- a/sound/soc/soc-pcm.c +++ b/sound/soc/soc-pcm.c @@ -2831,10 +2831,9 @@ static void soc_pcm_private_free(struct snd_pcm *pcm) struct snd_soc_rtdcom_list *rtdcom; struct snd_soc_component *component; + /* need to sync the delayed work before releasing resources */ + flush_delayed_work(&rtd->delayed_work); for_each_rtdcom(rtd, rtdcom) { - /* need to sync the delayed work before releasing resources */ - - flush_delayed_work(&rtd->delayed_work); component = rtdcom->component; if (component->pcm_free) -- 1.9.1

1 0

[alsa-devel] [PATCH v5][RESEND v2] ASoC: ak4613: call dummy write for PW_MGMT1/3 when Playback
by Kuninori Morimoto 24 Jan '18

24 Jan '18

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Power Down Release Command (PMVR, PMDAC, RSTN, PMDA1-PMDA6) which are located on PW_MGMT1 / PW_MGMT3 register must be write again after at least 5 LRCK cycle or later on each command. Otherwise, Playback volume will be 0dB. Basically, it should be 1. PowerDownRelease by Power Management1 <= call 1.x after 5LRCK 1.x Dummy write to Power Management1 2. PowerDownRelease by Power Management3 <= call 2.x after 5LRCK 2.x Dummy write to Power Management3 To avoid too many dummy write, this patch is merging these. 1. PowerDownRelease by Power Management1 2. PowerDownRelease by Power Management3 <= call after 5LRCK 2.x Dummy write to Power Management1/3 <= merge dummy write This patch adds dummy write when Playback Start timing. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- 2week passed. resend sound/soc/codecs/ak4613.c | 78 +++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 78 insertions(+) diff --git a/sound/soc/codecs/ak4613.c b/sound/soc/codecs/ak4613.c index ee9e822..d85c61e 100644 --- a/sound/soc/codecs/ak4613.c +++ b/sound/soc/codecs/ak4613.c @@ -15,6 +15,7 @@ */ #include <linux/clk.h> +#include <linux/delay.h> #include <linux/i2c.h> #include <linux/slab.h> #include <linux/of_device.h> @@ -95,6 +96,9 @@ struct ak4613_priv { struct mutex lock; const struct ak4613_interface *iface; struct snd_pcm_hw_constraint_list constraint; + struct work_struct dummy_write_work; + struct snd_soc_component *component; + unsigned int rate; unsigned int sysclk; unsigned int fmt; @@ -392,6 +396,7 @@ static int ak4613_dai_hw_params(struct snd_pcm_substream *substream, default: return -EINVAL; } + priv->rate = rate; /* * FIXME @@ -467,11 +472,83 @@ static int ak4613_set_bias_level(struct snd_soc_component *component, return 0; } +static void ak4613_dummy_write(struct work_struct *work) +{ + struct ak4613_priv *priv = container_of(work, + struct ak4613_priv, + dummy_write_work); + struct snd_soc_component *component = priv->component; + unsigned int mgmt1; + unsigned int mgmt3; + + /* + * PW_MGMT1 / PW_MGMT3 needs dummy write at least after 5 LR clocks + * + * Note + * + * To avoid extra delay, we want to avoid preemption here, + * but we can't. Because it uses I2C access which is using IRQ + * and sleep. Thus, delay might be more than 5 LR clocks + * see also + * ak4613_dai_trigger() + */ + udelay(5000000 / priv->rate); + + snd_soc_component_read(component, PW_MGMT1, &mgmt1); + snd_soc_component_read(component, PW_MGMT3, &mgmt3); + + snd_soc_component_write(component, PW_MGMT1, mgmt1); + snd_soc_component_write(component, PW_MGMT3, mgmt3); +} + +static int ak4613_dai_trigger(struct snd_pcm_substream *substream, int cmd, + struct snd_soc_dai *dai) +{ + struct snd_soc_codec *codec = dai->codec; + struct ak4613_priv *priv = snd_soc_codec_get_drvdata(codec); + + /* + * FIXME + * + * PW_MGMT1 / PW_MGMT3 needs dummy write at least after 5 LR clocks + * from Power Down Release. Otherwise, Playback volume will be 0dB. + * To avoid complex multiple delay/dummy_write method from + * ak4613_set_bias_level() / SND_SOC_DAPM_DAC_E("DACx", ...), + * call it once here. + * + * But, unfortunately, we can't "write" here because here is atomic + * context (It uses I2C access for writing). + * Thus, use schedule_work() to switching to normal context + * immediately. + * + * Note + * + * Calling ak4613_dummy_write() function might be delayed. + * In such case, ak4613 volume might be temporarily 0dB when + * beggining of playback. + * see also + * ak4613_dummy_write() + */ + + if ((cmd != SNDRV_PCM_TRIGGER_START) && + (cmd != SNDRV_PCM_TRIGGER_RESUME)) + return 0; + + if (substream->stream != SNDRV_PCM_STREAM_PLAYBACK) + return 0; + + priv->component = &codec->component; + schedule_work(&priv->dummy_write_work); + + return 0; +} + static const struct snd_soc_dai_ops ak4613_dai_ops = { .startup = ak4613_dai_startup, .shutdown = ak4613_dai_shutdown, .set_sysclk = ak4613_dai_set_sysclk, .set_fmt = ak4613_dai_set_fmt, + .trigger = ak4613_dai_trigger, .hw_params = ak4613_dai_hw_params, }; @@ -591,6 +668,7 @@ static int ak4613_i2c_probe(struct i2c_client *i2c, priv->iface = NULL; priv->cnt = 0; priv->sysclk = 0; + INIT_WORK(&priv->dummy_write_work, ak4613_dummy_write); mutex_init(&priv->lock); -- 1.9.1 _______________________________________________ Alsa-devel mailing list Alsa-devel(a)alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

1 0

[alsa-devel] [PATCH] ASoC: soc-core: snd_soc_rtdcom_lookup() cares component driver name
by Kuninori Morimoto 23 Jan '18

23 Jan '18

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> snd_soc_rtdcom_lookup() look up component by uisng driver name. Then, it uses component->driver->name. Some driver might doesn't have it, thus it should care NULL pointer. This patch solve this issue. Reported-by: Mukunda,Vijendar <vijendar.mukunda(a)amd.com> Reported-by: Manuel Lauss <manuel.lauss(a)gmail.com> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Tested-by: Mukunda,Vijendar <vijendar.mukunda(a)amd.com> --- sound/soc/soc-core.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index 10f808e..c9fdcbb 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c @@ -358,9 +358,17 @@ struct snd_soc_component *snd_soc_rtdcom_lookup(struct snd_soc_pcm_runtime *rtd, { struct snd_soc_rtdcom_list *rtdcom; + if (!driver_name) + return NULL; + for_each_rtdcom(rtd, rtdcom) { - if ((rtdcom->component->driver->name == driver_name) || - strcmp(rtdcom->component->driver->name, driver_name) == 0) + const char *component_name = rtdcom->component->driver->name; + + if (!component_name) + continue; + + if ((component_name == driver_name) || + strcmp(component_name, driver_name) == 0) return rtdcom->component; } -- 1.9.1

2 1

[alsa-devel] [PATCH] sound: emu10k1: Replace GFP_ATOMIC with GFP_KERNEL in synth_alloc_pages
by Jia-Ju Bai 23 Jan '18

23 Jan '18

The function synth_alloc_pages is not called in atomic context. Thus GFP_ATOMIC is not necessary, and it can be replaced with GFP_KERNEL. Signed-off-by: Jia-Ju Bai <baijiaju1990(a)gmail.com> --- sound/pci/emu10k1/memory.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/pci/emu10k1/memory.c b/sound/pci/emu10k1/memory.c index 4f1f69b..a4dbae1 100644 --- a/sound/pci/emu10k1/memory.c +++ b/sound/pci/emu10k1/memory.c @@ -477,7 +477,7 @@ static int synth_alloc_pages(struct snd_emu10k1 *emu, struct snd_emu10k1_memblk if (p) __free_page(p); /* try to allocate from <16MB zone */ - p = alloc_page(GFP_ATOMIC | GFP_DMA | + p = alloc_page(GFP_KERNEL | GFP_DMA | __GFP_NORETRY | /* no OOM-killer */ __GFP_NOWARN); } -- 1.7.9.5

2 1