diff options
author | Álvaro Fernández Rojas <noltari@gmail.com> | 2019-08-09 19:50:30 +0200 |
---|---|---|
committer | Álvaro Fernández Rojas <noltari@gmail.com> | 2019-08-27 11:10:01 +0200 |
commit | 2340c646e6a5da658e114be2f721ff90853588d4 (patch) | |
tree | df9744d2344e71e8c5d1b75c3e1528bc474313fe /target/linux/brcm2708/patches-4.19/950-0445-staging-bcm2835-audio-Fix-incorrect-draining-handlin.patch | |
parent | 81d0da118694feb874f992093063a2852285d194 (diff) | |
download | upstream-2340c646e6a5da658e114be2f721ff90853588d4.tar.gz upstream-2340c646e6a5da658e114be2f721ff90853588d4.tar.bz2 upstream-2340c646e6a5da658e114be2f721ff90853588d4.zip |
brcm2708: update to latest patches from RPi foundation
Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
Diffstat (limited to 'target/linux/brcm2708/patches-4.19/950-0445-staging-bcm2835-audio-Fix-incorrect-draining-handlin.patch')
-rw-r--r-- | target/linux/brcm2708/patches-4.19/950-0445-staging-bcm2835-audio-Fix-incorrect-draining-handlin.patch | 69 |
1 files changed, 0 insertions, 69 deletions
diff --git a/target/linux/brcm2708/patches-4.19/950-0445-staging-bcm2835-audio-Fix-incorrect-draining-handlin.patch b/target/linux/brcm2708/patches-4.19/950-0445-staging-bcm2835-audio-Fix-incorrect-draining-handlin.patch deleted file mode 100644 index 5cd0af06ed..0000000000 --- a/target/linux/brcm2708/patches-4.19/950-0445-staging-bcm2835-audio-Fix-incorrect-draining-handlin.patch +++ /dev/null @@ -1,69 +0,0 @@ -From 52a5b0e8e24f31fdef175e76405fd4ee08936f26 Mon Sep 17 00:00:00 2001 -From: Takashi Iwai <tiwai@suse.de> -Date: Tue, 4 Sep 2018 17:58:38 +0200 -Subject: [PATCH 445/725] staging: bcm2835-audio: Fix incorrect draining - handling - -commit 7d2a91f5f1bcf08ca257bcf1ed9721fcd341f834 upstream. - -The handling of SNDRV_PCM_TRIGGER_STOP at the trigger callback is -incorrect: when the STOP is issued, the driver is supposed to drop the -stream immediately. Meanwhile bcm2835 driver checks the DRAINING -state and tries to issue some different command. - -This patch straightens things a bit, dropping the incorrect state -checks. The draining behavior would be still not perfect at this -point, but will be improved in a later patch. - -Signed-off-by: Takashi Iwai <tiwai@suse.de> -Tested-by: Stefan Wahren <stefan.wahren@i2se.com> -Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> ---- - .../vc04_services/bcm2835-audio/bcm2835-pcm.c | 18 ++++++------------ - 1 file changed, 6 insertions(+), 12 deletions(-) - ---- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c -+++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c -@@ -153,7 +153,6 @@ static int snd_bcm2835_playback_open_gen - chip->alsa_stream[idx] = alsa_stream; - - chip->opened |= (1 << idx); -- alsa_stream->draining = 1; - - out: - mutex_unlock(&chip->audio_mutex); -@@ -268,6 +267,7 @@ static int snd_bcm2835_pcm_prepare(struc - alsa_stream->buffer_size = snd_pcm_lib_buffer_bytes(substream); - alsa_stream->period_size = snd_pcm_lib_period_bytes(substream); - alsa_stream->pos = 0; -+ alsa_stream->draining = false; - - audio_debug("buffer_size=%d, period_size=%d pos=%d frame_bits=%d\n", - alsa_stream->buffer_size, alsa_stream->period_size, -@@ -312,21 +312,15 @@ static int snd_bcm2835_pcm_trigger(struc - switch (cmd) { - case SNDRV_PCM_TRIGGER_START: - err = bcm2835_audio_start(alsa_stream); -- if (!err) -- alsa_stream->draining = 1; -- else -+ if (err) - audio_error(" Failed to START alsa device (%d)\n", err); - break; -+ case SNDRV_PCM_TRIGGER_DRAIN: -+ alsa_stream->draining = true; -+ break; - case SNDRV_PCM_TRIGGER_STOP: -- if (runtime->status->state == SNDRV_PCM_STATE_DRAINING) { -- audio_info("DRAINING\n"); -- alsa_stream->draining = 1; -- } else { -- audio_info("DROPPING\n"); -- alsa_stream->draining = 0; -- } - err = bcm2835_audio_stop(alsa_stream); -- if (err != 0) -+ if (err) - audio_error(" Failed to STOP alsa device (%d)\n", err); - break; - default: |