Re: Audio glitch with SGTL5000

[Gary Thomas <gary@mlbassoc.com>]

On 2015-02-28 11:53, Eric Nelson wrote:
> Hi all,
>
> On 01/22/2015 12:52 PM, Fabio Estevam wrote:
>> Hi Nikolay,
>>
>> On Thu, Jan 22, 2015 at 5:49 PM, Nikolay Dimitrov <picmaster@mail.bg> wrote:
>>
>>> Sorry, I didn't mean that at all.
>>
>> No problem. I am glad you spotted the problem with the SMALL_POP bit.
>> I will also check if there is an error in the SMALL_POP bit definition
>> in the manual.
>>
>>>> This is off-topic here on a Yocto list though, so I suggest we move
>>>> such discussion to the alsa-devel list which is the appropriate place.
>>>
>>>
>>> OK, understood.
>>
>> Maybe sending your workaround patch as a RFC could be a start for the
>> discussion there.
>>
>
> I think I found the problem, and it was fixed in main-line in this
> commit by Marek:
>
> 	https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/sound/soc/codecs/sgtl5000.c?id=dd4d2d6dfb49e8916064f2cb07f0ad7b32a82fb7
>
> I pushed a 3.10.17 version of it to our Github tree:
> 	https://github.com/boundarydevices/linux-imx6/commit/ff27114

I still get the large pops on my board with this change included.
My kernel is currently based on the linux-imx:3.10.53_1.1.0_ga
I'd love to check it on my SabreLite, but I need that running
3.10.53 as well, so it will have to wait.

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------