All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: anish kumar <anish198519851985@gmail.com>
Cc: tiwai@suse.de, alsa-devel@alsa-project.org, lrg@ti.com
Subject: Re: commit: dpcm: Allow FE to be opened without valid BE routes
Date: Wed, 7 Nov 2012 15:28:10 +0100	[thread overview]
Message-ID: <20121107142809.GD20844@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <1352292999.1574.6.camel@anish-Inspiron-N5050>


[-- Attachment #1.1: Type: text/plain, Size: 1318 bytes --]

On Wed, Nov 07, 2012 at 09:56:39PM +0900, anish kumar wrote:
> On Wed, 2012-11-07 at 09:48 +0100, Mark Brown wrote:

> > You shouldn't be relying on parsing dmesg for anything, that's never
> > going to be robust as you've just found.  dmesg is for humans. 

> Probably I couldn't explain the problem clearly.Problem is that in our
> case we get sometimes(very random) audio mute in android devices due to
> UCM commands from the platform not sent properly or sent in such a way
> that FE and BE don't get connected.
> This issues are very hard to reproduce and only gets reproduced randomly
> by the test engineer and when it does happen, test engineer takes crash
> dump and provides us the log.So it is almost impossible to know what
> caused this audio mute at that time and all we have is this very
> log(dev_warn(FE and BE not connected)) to let us know that this mute was
> due to platform and not codec or any other problem with the kernel.
> Anyway looks like this idea is not liked by the community so I won't
> push further.

This really just sounds like a basic debugging issue, you can either add
additional trace locally or work to improve the trace infrastructure -
for example, make sure there's tracepoints showing what DPCM does in the
same way there are tracepoints for DAPM.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



      reply	other threads:[~2012-11-07 14:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <mailman.495.1352127790.820.alsa-devel@alsa-project.org>
2012-11-06 17:03 ` commit: dpcm: Allow FE to be opened without valid BE routes anish kumar
2012-11-06 17:32   ` Liam Girdwood
2012-11-07  8:48   ` Mark Brown
2012-11-07 12:56     ` anish kumar
2012-11-07 14:28       ` Mark Brown [this message]

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=20121107142809.GD20844@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=anish198519851985@gmail.com \
    --cc=lrg@ti.com \
    --cc=tiwai@suse.de \
    /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.