[alsa-devel] ALSA: fix excessive background noise introduced by OSS emulation rate shrink

Takashi Iwai tiwai at suse.de
Mon Feb 23 08:46:14 CET 2009


At Sat, 21 Feb 2009 08:05:04 -0600,
Steve Chen wrote:
> 
> 
> Incorrect variable was used to get the next sample which caused S2
> to be stuck with the same value resulting in loud background noise.
> 
> Signed-off-by: Steve Chen <schen at mvista.com>

Oh what a long-living bug.
Applied now.  Thanks!


Takashi

> 
> diff --git a/sound/core/oss/rate.c b/sound/core/oss/rate.c
> index a466443..2fa9299 100644
> --- a/sound/core/oss/rate.c
> +++ b/sound/core/oss/rate.c
> @@ -157,7 +157,7 @@ static void resample_shrink(struct snd_pcm_plugin *plugin,
>  		while (dst_frames1 > 0) {
>  			S1 = S2;
>  			if (src_frames1-- > 0) {
> -				S1 = *src;
> +				S2 = *src;
>  				src += src_step;
>  			}
>  			if (pos & ~R_MASK) {
> 
> 


More information about the Alsa-devel mailing list