2016-10-12 13:59:07

by Mika Penttilä

[permalink] [raw]
Subject: pre 4.9-rc dma regression, imx6 sound etc

Hi!

Bisected that commit 5881826 - " dmaengine: imx-sdma - update the residue calculation for cyclic channels" is first bad commit to cause audio regression on imx6q, sgtl5000 codec. It causes audible disturbing background noise when playing wavs.

Unfortunately, reverting only 5881826 causes uarts to fail, so another fix is needed.

Thanks,

Mika






2016-10-13 07:18:07

by Nandor Han

[permalink] [raw]
Subject: Re: EXT: pre 4.9-rc dma regression, imx6 sound etc



On 12/10/2016 16:25, Mika Penttilä wrote:
> Hi!
>
> Bisected that commit 5881826 - " dmaengine: imx-sdma - update the residue calculation for cyclic channels" is first bad commit to cause audio regression on imx6q, sgtl5000 codec. It causes audible disturbing background noise when playing wavs.
>
> Unfortunately, reverting only 5881826 causes uarts to fail, so another fix is needed.
>
> Thanks,
>
> Mika
>
>
>
>
>

Hi Mika,
Thanks for info. I have already posted a patch that will fix this issue.

https://lkml.org/lkml/2016/10/11/319

It was already tested on imx6 and imx53. Let me know how it works.

--
Regards,
Nandor

2016-10-13 08:14:56

by Mika Penttilä

[permalink] [raw]
Subject: Re: EXT: pre 4.9-rc dma regression, imx6 sound etc

On 10/13/2016 08:25 AM, Han, Nandor (GE Healthcare) wrote:
>
>
> On 12/10/2016 16:25, Mika Penttilä wrote:
>> Hi!
>>
>> Bisected that commit 5881826 - " dmaengine: imx-sdma - update the residue calculation for cyclic channels" is first bad commit to cause audio regression on imx6q, sgtl5000 codec. It causes audible disturbing background noise when playing wavs.
>>
>> Unfortunately, reverting only 5881826 causes uarts to fail, so another fix is needed.
>>
>> Thanks,
>>
>> Mika
>>
>>
>>
>>
>>
>
> Hi Mika,
> Thanks for info. I have already posted a patch that will fix this issue.
>
> https://lkml.org/lkml/2016/10/11/319
>
> It was already tested on imx6 and imx53. Let me know how it works.
>

Hi,

Great, with this patch audio is ok again! This patch should go in for 4.9!

Thanks,
Mika