Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AEC support with Syntiant 112.3.6 SDK #6540

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

abhishek-samsung
Copy link
Contributor

No description provided.

abhishek-samsung and others added 9 commits November 27, 2024 06:47
when "SYNTIANT_NDP_ERROR_DATA_REREAD" error occurs, zero is
returned in the argument extract size as no bytes are extracted.
However, in the next iteration, zero is passed as extract size,
causing to extract the entire audio tank, hence corrupting the
heap. This exactly matches the crash we observed as the heap
corrupted (around 60KB) matches the size of the audio tank (60KB)

Signed-off-by: Abhishek Akkabathula <[email protected]>
use level based instead of edge based as its a more robust trigger for gpio interrupts

Signed-off-by: Abhishek Akkabathula <[email protected]>
added gaurd code to prevent infinite wait on sample ready interrupts.
In the case that we dont get any sample ready interrupt within given
time, return the apb to upper layer with zero bytes. In future when
async extraction is applied, we can again reuse the same apb. For now,
this case is not handled.

Signed-off-by: Abhishek Akkabathula <[email protected]>
ideally data extraction should happen every 20ms, however, during
testing, some times, it takes upto 60ms to extract a sample. As a
result, the delay keeps building up and resulting in fully filling
the circular audio tank. Hence, extract multiple sample when ever
possible to prevent the audio buildup in tank.

Signed-off-by: Abhishek Akkabathula <[email protected]>
enable barge-in via change of active flowset ID.
Implement host based recovery for NDP120. We simply reset the
device and reload the firmware.

Signed-off-by: Abhishek Akkabathula <[email protected]>
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.

3 participants