[mrajwa]

This patch fixes the reset procedure of cadence codecs.

Potential fix for #4104, not tested.

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

[lgirdwood]

@mrajwa looks like this should be for TGL13 and rc3 ?
@dbaluta any comments ?

[lgirdwood]

@slawblauciak fyi - potential for TGL13

[mrajwa]

@lgirdwood yes lets include this patch

[dbaluta]

@lgirdwood this should be OK.

@paulstelian97 can you please test this patch with imx-stable-v1.8 branch?

[lgirdwood]

@paulstelian97 any update ?
@slawblauciak @mwasko @mrajwa fyi - could be needed for TGL13
@keyonjie for rc3.

[paulstelian97]

@paulstelian97 any update ?

Not quite yet, but from visual inspection I'm thinking on whether this needs some stress testing (I don't know if codec reset will reuse existing memory or not).

[keyonjie]

@paulstelian97 any update ?
@slawblauciak @mwasko @mrajwa fyi - could be needed for TGL13
@keyonjie for rc3.

Okay, will back port it once merged.

[dbaluta]

@lgirdwood @paulstelian97 @keyonjie I tested this patch and it doesn't do any harm. So, we can merge it. I have nothing against.

Anyhow, it doesn't fix #4104 or a related IMX issue. Where second cplay run fails. I'm investigating that on my side.

@paulstelian97 I will get back to you and hand over some codec_adapter IMX issues as next 2 weeks I will be in vacation.

[lgirdwood]

@mrajwa do we need another fix for #4104 ?

[mrajwa]

@lgirdwood, I need to sync with @zrombel on that as the issue is only reproducable on a slim driver at the moment. I have it on my todo list so if the issue is not covered by this one I will submit another patch.

[lgirdwood]

@lgirdwood, I need to sync with @zrombel on that as the issue is only reproducable on a slim driver at the moment. I have it on my todo list so if the issue is not covered by this one I will submit another patch.

Ok, could be another cause for #4104 but this is freeing the codec correctly so can be merged.