diff options
author | Jaroslav Kysela <perex@perex.cz> | 2023-05-05 17:52:41 +0200 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2023-05-05 18:23:51 +0200 |
commit | 190cb66a4ee0ac4419e1f8a9de46074a8c8e4c23 (patch) | |
tree | de4332f87af73b053df8dd586d3a8ad2028c4ab7 /sound | |
parent | 781b4da64b9e3dafb356b7365e7ecf273e879602 (diff) | |
download | lwn-190cb66a4ee0ac4419e1f8a9de46074a8c8e4c23.tar.gz lwn-190cb66a4ee0ac4419e1f8a9de46074a8c8e4c23.zip |
ALSA: pcm: playback silence - remove extra code
The removed condition handles de facto only one situation where
runtime->silence_filled variable is equal to runtime->buffer_size,
because this variable cannot go over the buffer size. This case is
implicitly caught by the required comparison of the noise distance
with the threshold.
Suggested-by: Oswald Buddenhagen <oswald.buddenhagen@gmx.de>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
Signed-off-by: Oswald Buddenhagen <oswald.buddenhagen@gmx.de>
Link: https://lore.kernel.org/r/20230505155244.2312199-4-oswald.buddenhagen@gmx.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/core/pcm_lib.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sound/core/pcm_lib.c b/sound/core/pcm_lib.c index 5ddb74a12030..a1838130c830 100644 --- a/sound/core/pcm_lib.c +++ b/sound/core/pcm_lib.c @@ -61,8 +61,6 @@ void snd_pcm_playback_silence(struct snd_pcm_substream *substream, snd_pcm_ufram runtime->silence_filled = 0; runtime->silence_start = appl_ptr; } - if (runtime->silence_filled >= runtime->buffer_size) - return; /* initialization outside pointer updates */ if (new_hw_ptr == ULONG_MAX) new_hw_ptr = runtime->status->hw_ptr; |