sof icon indicating copy to clipboard operation
sof copied to clipboard

Published 20 hours ago • verified publisher icon thesofproject

cherry pick module_adapter changes from main to mt8186/v0.2

Open johnylin76 opened this issue 2 years ago • 4 comments

https://github.com/thesofproject/sof/commit/b3106c396e540c260bba3b08c218f6598ea74726 module_adapter: get the actual period_bytes https://github.com/thesofproject/sof/commit/b9889d52d0e1b6002ee016b55fa0487544fc0593 module_adapter: Modify reset API https://github.com/thesofproject/sof/commit/53b3bc6a956ae5957966336f88980438f1f85a88 module_adapter:Fix dangling pointer issue in module reset

johnylin76 avatar Sep 27 '22 01:09 johnylin76

Back-port https://github.com/thesofproject/sof/commit/b9889d52d0e1b6002ee016b55fa0487544fc0593 for https://github.com/thesofproject/sof/issues/6331

johnylin76 avatar Sep 27 '22 01:09 johnylin76

@thesofproject/mediatek pls review.

lgirdwood avatar Sep 28 '22 11:09 lgirdwood

Hi @Zames-Chang , Please be noted that we may want this PR be merged along with the fix on 3P vendors' side (DTS/Waves). Merging this without 3P fixes may cause other issues when 3P module is applied.

johnylin76 avatar Sep 30 '22 04:09 johnylin76

I see. Thanks for the info

Zames-Chang avatar Oct 01 '22 03:10 Zames-Chang

DTS provided v1.0.8 for the fix on vendor's side. This PR is able to merge for the updated firmware build with v1.0.8 DTS lib

johnylin76 avatar Dec 14 '22 05:12 johnylin76

Hi Johny

Will this affect SOF without (DTS/Waves) ?

Zames-Chang avatar Dec 18 '22 06:12 Zames-Chang

Hi Johny

Will this affect SOF without (DTS/Waves) ?

No, the changes are all under module_adapter which is applied by DTS/Waves only in our cases.

johnylin76 avatar Dec 18 '22 23:12 johnylin76