All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Nelson <eric.nelson@boundarydevices.com>
To: Fabio Estevam <festevam@gmail.com>,
	 Troy Kisky <troy.kisky@boundarydevices.com>
Cc: "meta-freescale@yoctoproject.org" <meta-freescale@yoctoproject.org>
Subject: Re: No sound on nitrogen6x + linux-boundary 3.10.17
Date: Tue, 06 May 2014 07:04:10 -0700	[thread overview]
Message-ID: <5368EBDA.5080903@boundarydevices.com> (raw)
In-Reply-To: <CAOMZO5AAqW=rLwrJjmxCHRA5Rkgg94m=n+94vT658+MbbiEwLw@mail.gmail.com>

Hi Fabio,

On 05/05/2014 04:27 PM, Fabio Estevam wrote:
> On Mon, May 5, 2014 at 8:22 PM, Troy Kisky
> <troy.kisky@boundarydevices.com> wrote:
>> On 5/5/2014 12:31 PM, Eric Nelson wrote:
>>> Thanks for the detailed report, Erik.
>>>
>>> And thanks for the info, Fabio and John.
>>>
>>> I'm out of the office today, but CC'ing Troy to see if he can reproduce things.
>>>
>> I just did a similar fix for the problem, but I also reset registers to power on
>> default value.
>
> Like this one, I guess:
> https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/sound/soc/codecs/sgtl5000.c?id=af8ee11209e749c75eabf32b2a4ca631f396acf8
>

Essentially yes.

Troy pulled in an older patch that also has some retry logic
around I2C reads and writes:
	https://github.com/boundarydevices/linux-imx6/commit/c8e4e642

Erik, I'll forward a patch against linux-boundary_3.10.17.bb later
today, but you can test by changing SRCREV to
	fe41f1e14c1769c2012b21ac6375ddb1ab15ea52

Regards,


Eric



  parent reply	other threads:[~2014-05-06 14:04 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-05  6:51 No sound on nitrogen6x + linux-boundary 3.10.17 Erik Botö
2014-05-05 13:24 ` Erik Botö
2014-05-05 14:31   ` Otavio Salvador
2014-05-05 14:32   ` Erik Botö
2014-05-05 17:40 ` Fabio Estevam
2014-05-05 17:55   ` Fabio Estevam
2014-05-05 18:02     ` John Weber
2014-05-05 19:31       ` Eric Nelson
2014-05-05 23:22         ` Troy Kisky
2014-05-05 23:27           ` Fabio Estevam
2014-05-05 23:59             ` Troy Kisky
2014-05-06 14:04             ` Eric Nelson [this message]
2014-05-06 14:39               ` Erik Botö
2014-05-06 18:16                 ` Eric Nelson
2014-05-07  8:02                   ` Erik Botö
2014-05-07 13:32                     ` Eric Nelson
2014-05-28 17:26       ` Fabio Estevam
2014-05-28 19:12         ` John Weber
2014-05-06 12:36     ` Erik Botö

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5368EBDA.5080903@boundarydevices.com \
    --to=eric.nelson@boundarydevices.com \
    --cc=festevam@gmail.com \
    --cc=meta-freescale@yoctoproject.org \
    --cc=troy.kisky@boundarydevices.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.