diff options
Diffstat (limited to 'target/linux/brcm2708/patches-4.19/950-0396-staging-bcm2835-audio-Remove-redundant-function-call.patch')
-rw-r--r-- | target/linux/brcm2708/patches-4.19/950-0396-staging-bcm2835-audio-Remove-redundant-function-call.patch | 95 |
1 files changed, 95 insertions, 0 deletions
diff --git a/target/linux/brcm2708/patches-4.19/950-0396-staging-bcm2835-audio-Remove-redundant-function-call.patch b/target/linux/brcm2708/patches-4.19/950-0396-staging-bcm2835-audio-Remove-redundant-function-call.patch new file mode 100644 index 0000000000..482c9ba5cc --- /dev/null +++ b/target/linux/brcm2708/patches-4.19/950-0396-staging-bcm2835-audio-Remove-redundant-function-call.patch @@ -0,0 +1,95 @@ +From 79a3c1a4419b2bf04f6ff5ef84cd74b0456fdd9a Mon Sep 17 00:00:00 2001 +From: Takashi Iwai <tiwai@suse.de> +Date: Tue, 4 Sep 2018 17:58:35 +0200 +Subject: [PATCH] staging: bcm2835-audio: Remove redundant function + calls + +commit 124950ebe9fa8547c59e8d4acc8d6c59e6278ed6 upstream. + +bcm2835_audio_setup(), bcm2835_audio_flush_buffers() and +bcm2835_audio_flush_playback_buffers() functions do implement +nothing. + +Also, bcm2835_audio_set_ctls() is already called inside +bcm2835_audio_set_params(), so the later call is superfluous. + +This patch removes these superfluous implementations. + +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 | 5 ----- + .../bcm2835-audio/bcm2835-vchiq.c | 21 ------------------- + .../vc04_services/bcm2835-audio/bcm2835.h | 3 --- + 3 files changed, 29 deletions(-) + +--- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c ++++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c +@@ -277,11 +277,6 @@ static int snd_bcm2835_pcm_prepare(struc + if (err < 0) + audio_error(" error setting hw params\n"); + +- bcm2835_audio_setup(alsa_stream); +- +- /* in preparation of the stream, set the controls (volume level) of the stream */ +- bcm2835_audio_set_ctls(alsa_stream); +- + memset(&alsa_stream->pcm_indirect, 0, sizeof(alsa_stream->pcm_indirect)); + + alsa_stream->pcm_indirect.hw_buffer_size = +--- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c ++++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c +@@ -580,12 +580,6 @@ unlock: + return ret; + } + +-int bcm2835_audio_setup(struct bcm2835_alsa_stream *alsa_stream) +-{ +- +- return 0; +-} +- + static int bcm2835_audio_start_worker(struct bcm2835_alsa_stream *alsa_stream) + { + struct vc_audio_msg m; +@@ -774,21 +768,6 @@ unlock: + return ret; + } + +-/** +- * Returns all buffers from arm->vc +- */ +-void bcm2835_audio_flush_buffers(struct bcm2835_alsa_stream *alsa_stream) +-{ +-} +- +-/** +- * Forces VC to flush(drop) its filled playback buffers and +- * return them the us. (VC->ARM) +- */ +-void bcm2835_audio_flush_playback_buffers(struct bcm2835_alsa_stream *alsa_stream) +-{ +-} +- + unsigned int bcm2835_audio_retrieve_buffers(struct bcm2835_alsa_stream *alsa_stream) + { + unsigned int count = atomic_read(&alsa_stream->retrieved); +--- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835.h ++++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835.h +@@ -158,7 +158,6 @@ int bcm2835_audio_close(struct bcm2835_a + int bcm2835_audio_set_params(struct bcm2835_alsa_stream *alsa_stream, + unsigned int channels, unsigned int samplerate, + unsigned int bps); +-int bcm2835_audio_setup(struct bcm2835_alsa_stream *alsa_stream); + int bcm2835_audio_start(struct bcm2835_alsa_stream *alsa_stream); + int bcm2835_audio_stop(struct bcm2835_alsa_stream *alsa_stream); + int bcm2835_audio_set_ctls(struct bcm2835_alsa_stream *alsa_stream); +@@ -167,7 +166,5 @@ int bcm2835_audio_write(struct bcm2835_a + void *src); + void bcm2835_playback_fifo(struct bcm2835_alsa_stream *alsa_stream); + unsigned int bcm2835_audio_retrieve_buffers(struct bcm2835_alsa_stream *alsa_stream); +-void bcm2835_audio_flush_buffers(struct bcm2835_alsa_stream *alsa_stream); +-void bcm2835_audio_flush_playback_buffers(struct bcm2835_alsa_stream *alsa_stream); + + #endif /* __SOUND_ARM_BCM2835_H */ |