Skip to content

SRC: Add attenuation of -1 dB into processing to avoid saturation - #1069

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
singalsu:src_attenuate_coefficients
Mar 4, 2019
Merged

lgirdwood merged 1 commit into
thesofproject:masterfrom
singalsu:src_attenuate_coefficients

Conversation

@singalsu

@singalsu singalsu commented Mar 1, 2019

Copy link
Copy Markdown
Collaborator

This patch replaces all the filter coefficients to avoid the
unpleasant effect of samples saturation that the previous 0 dB max.
gain scaled coefficients could cause. The saturation of transients
may still happen but it should be more rare.

The filter coefficients scaling is changed from fixed pass band peak
align (to 0 dB) into DC gain controllable via new parameter gain in
src_param.m.

Signed-off-by: Seppo Ingalsuo seppo.ingalsuo@linux.intel.com

This patch replaces all the filter coefficients to avoid the
unpleasant effect of samples saturation that the previous 0 dB max.
gain scaled coefficients could cause. The saturation of transients
may still happen but it should be more rare.

The filter coefficients scaling is changed from fixed pass band peak
align (to 0 dB) into DC gain controllable via new parameter gain in
src_param.m.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
@singalsu

singalsu commented Mar 1, 2019

Copy link
Copy Markdown
Collaborator Author

This addresses issue #619 though I could find no overflow to happen in firmware. The change for SRC tools is in the end of the diff listing.

@lgirdwood
lgirdwood merged commit ee5c247 into thesofproject:master Mar 4, 2019
@singalsu
singalsu deleted the src_attenuate_coefficients branch May 18, 2020 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants