sof icon indicating copy to clipboard operation
sof copied to clipboard

ipc4: fix for mixout not working after reset

Open serhiy-katsyuba-intel opened this issue 2 years ago • 5 comments

Fixes a bug in mixout_reset(): mixin pointers in mixed data info should not be cleared on reset. They should only be set on bind and cleared on unbind.

Signed-off-by: Serhiy Katsyuba [email protected]

serhiy-katsyuba-intel avatar Oct 17 '22 18:10 serhiy-katsyuba-intel

@serhiy-katsyuba-intel Can you undraft this, this seems very good. Together with kernel PR https://github.com/thesofproject/linux/pull/3917/ , this is getting much better results with mixer topologies.

kv2019i avatar Oct 18 '22 05:10 kv2019i

@serhiy-katsyuba-intel Can you undraft this, this seems very good. Together with kernel PR thesofproject/linux#3917 , this is getting much better results with mixer topologies.

The PR is not draft since yesterday. Probably some refresh issue on your side. The PR is ready to merge. The failure reported by the CI looks like the CI problem and is unrelated to modifications made.

serhiy-katsyuba-intel avatar Oct 18 '22 13:10 serhiy-katsyuba-intel

@serhiy-katsyuba-intel wrote:

The PR is not draft since yesterday. Probably some refresh issue on your side. The PR is ready to merge. The failure reported by the CI looks like the CI problem and is unrelated to modifications made.

Ack, this is good to now! We need to wait for the few mandatory CI runs to complete and then this should be good to merge. The already reported CI issues are known ones.

kv2019i avatar Oct 18 '22 13:10 kv2019i

@wszypelt sorry, need to bug you again. I couldn't find this PR at all in Intel CI dashboards, so it appears it was not queued. Need a retrigger?

kv2019i avatar Oct 19 '22 08:10 kv2019i

@kv2019i It's good to let me know. in fact qb did not queue this pr correctly, I started it manually, within 30 minutes there should be results

wszypelt avatar Oct 19 '22 08:10 wszypelt

Thank you @wszypelt ! Now all looks good. One known failure (fix in review at https://github.com/thesofproject/sof-test/pull/971 ), so proceeding with merge.

kv2019i avatar Oct 19 '22 10:10 kv2019i