alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Timur Tabi <timur@freescale.com>
Cc: Alsa-devel@alsa-project.org
Subject: Re: Need help with WM8960
Date: Thu, 13 Sep 2012 11:13:03 +0800	[thread overview]
Message-ID: <20120913024245.GA4467@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <5051160C.2010802@freescale.com>

On Wed, Sep 12, 2012 at 06:09:00PM -0500, Timur Tabi wrote:

> The first problem is that I get this:

> 	wm8960 0-001a: No platform data supplied

> What kind of platform data is the driver expecting, and how do I get it
> that data?  I don't seem to need any platform data to make the code work.

So, it should be fairly clear from the code that its looking for a
struct wm8960_data.  You can actually leave this totally blank but it's
important that the capless setting is correct one way or another - you
need to look at the schematic and check to see if there are capacitors
on the output not.

> You can see that he disables the DAPM controls for DACL, DACR, ADCL, and
> ADCR, and forces those bits set in the wm8960_probe function.  There's
> also a comment that documents some limitations.  What is the proper way to
> implement this change?

I'm not entirely sure why the user is doing this - unfortuantely they
just bodged the driver without reporting the issue.  The first issue
does not make much sense, the issues described are system level issues
that are unrelated to the ADC for the most part (for example, things
related to the micbias generally depend on the micbias enable).  Without
more information it's hard to comment.  The second sounds like they
missed the LRCM bit, I'll just add the ability to control that from
platform data.

> Third, in order to actually hear audio, I had to unmute the "Left Output
> Mixer PCM" and "Right Output Mixer PCM".  Is this something that the
> machine driver should do automatically?  And if so, how?

No, it's something that the application should do.  

  reply	other threads:[~2012-09-13  3:13 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-12 23:09 Need help with WM8960 Timur Tabi
2012-09-13  3:13 ` Mark Brown [this message]
2012-09-13  3:26   ` Tabi Timur-B04825
2012-09-13  3:31     ` Mark Brown
2012-09-13 14:43   ` Timur Tabi
2012-09-13 15:44     ` Timur Tabi
2012-09-13 16:29       ` Timur Tabi
2012-09-19  2:59         ` Mark Brown

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=20120913024245.GA4467@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=Alsa-devel@alsa-project.org \
    --cc=timur@freescale.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).