Re: Audio glitch with SGTL5000

[Eric Nelson <eric.nelson@boundarydevices.com>]

Hi Nikolay,

On 03/01/2015 02:29 PM, Nikolay Dimitrov wrote:
> Hi Eric,
> 
> On 02/28/2015 08:53 PM, 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
> 
> Can you confirm tha this fix solves the pop on line-out? Just want to
> be sure, as most of the related patch-work fixes mostly the headphones-
> related pop.
> 

Our boards don't support line-out, so I can't confirm that, though
it should (VAG applies to both HP and LO).

Also, Gary reported earlier that the patch doesn't fix the
pop on 3.10.53.

I presume that's because we also have Fabio's SMALL_POP fix applied on
our 3.10.17 tree.

Regards,


Eric