Skip to content

kpb: stop key phrase detection during draining - #1363

Merged
tlauda merged 1 commit into
thesofproject:masterfrom
mrajwa:ghw_dev
May 12, 2019
Merged

tlauda merged 1 commit into
thesofproject:masterfrom
mrajwa:ghw_dev

Conversation

@mrajwa

@mrajwa mrajwa commented May 5, 2019

Copy link
Copy Markdown
Contributor

Once draining has been started we pause copy to
selector component which therefore also
blocks any consecutive key phrase detection.

Signed-off-by: Marcin Rajwa marcin.rajwa@linux.intel.com

Comment thread src/audio/kpb.c Resolved
@jajanusz

jajanusz commented May 7, 2019

Copy link
Copy Markdown
Contributor

@mrajwa please resolve conflicts

@jajanusz
jajanusz requested a review from lgirdwood May 7, 2019 11:42
Once draining has been started we pause copy to
selector component which therefore also
blocks any consecutive key phrase detection.

Signed-off-by: Marcin Rajwa <marcin.rajwa@linux.intel.com>
@mrajwa

mrajwa commented May 7, 2019

Copy link
Copy Markdown
Contributor Author

@jajanusz, resolved.

@mrajwa

mrajwa commented May 8, 2019

Copy link
Copy Markdown
Contributor Author

@lgirdwood, please re-review.

@tlauda

tlauda commented May 11, 2019

Copy link
Copy Markdown
Contributor

@lgirdwood Do you accept?

@lgirdwood

Copy link
Copy Markdown
Member

@mrajwa CI build failure block merging.

@michalgrodzicki

Copy link
Copy Markdown

Hi @lgirdwood,
I checked that one of the platforms (WHL) used in CI didn't respond.
I disabled test on this platform, and I retriggered all CI builds which failed.
Right now CI passed, please merge.

@tlauda
tlauda merged commit cf18562 into thesofproject:master May 12, 2019
@keyonjie

Copy link
Copy Markdown
Contributor

Looks this PR breaks KWD on my platform. @keqiaozhang also found the issue. @mrajwa can you confirm if it work on your side?

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.

6 participants