From: marek.vasut@gmail.com (Marek Vasut)
To: linux-arm-kernel@lists.infradead.org
Subject: AC97 problems with pxa...
Date: Wed, 7 Apr 2010 09:50:37 +0200 [thread overview]
Message-ID: <201004070950.37809.marek.vasut@gmail.com> (raw)
In-Reply-To: <4BBBACBE.7040505@bitsim.com>
Dne ?t 6. dubna 2010 23:50:54 Jakob Viketoft napsal(a):
> Hello!
>
> I'm trying to add sound support to a Toradex Colibri 270 v2 card, but
> have run into some strangeness which I don't quite understand. The v2 of
> this card uses the WM9712 codec and I've written a machine driver to tie
> the codec and the pxa2xx-ac97 driver together. I'm using the 2.6.33.1
> kernel and my machine driver is a simplified version of the tosa driver
> (no power management or headphone jack logic). However, I get two problems:
> 1. Internal clock and headphone output is turned off in the
> AC97_POWERDOWN register and I can't see where to (properly) turn it on,
> neither in kernel space or in userland. No-one else using this codec
> seem to be doing it.
> 2. The init function defined in my snd_soc_card struct doesn't seem to
> get called, resulting in way too many mixer settings (I have a number of
> snd_soc_dapm_nc_pin calls) with alsa, but might it also have something
> to do with problem 1?
>
> When explicitly writing 0 (the ugly way) to the AC97_POWERDOWN register
> I get perfect sound output, but I would like to do it the right way.
>
> Two error reports from the dmesg output that might be relevant:
> * pxa2xx_ac97_try_cold_reset: cold reset timeout (GSR=0x44)
> * Error: Driver 'pxa2xx-ac97' is already registered, aborting...
>
Aren't you mixing ASoC driver with non-ASoC driver? Check how this is done on
the palmtx for example (and the pxa27x.c asoc driver)
> The AC97 interface is added in the machine initialization through the
> pxa_set_ac97_info() in the same way as tosa and many others and this
> could explain the second error message...
>
> Any insight would be appreciated!
>
> Regards,
>
> /Jakob
>
>
CC Mark Brown
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
next prev parent reply other threads:[~2010-04-07 7:50 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-06 21:50 AC97 problems with pxa Jakob Viketoft
2010-04-07 2:09 ` Eric Miao
2010-04-07 9:13 ` Jakob Viketoft
2010-04-07 19:08 ` Marek Vasut
2010-04-07 7:50 ` Marek Vasut [this message]
2010-04-07 9:37 ` Jakob Viketoft
2010-04-07 9:54 ` Mark Brown
2010-04-08 9:17 ` Jakob Viketoft
2010-04-08 9:54 ` Mark Brown
2010-04-08 13:05 ` Jakob Viketoft
2010-04-08 13:49 ` Mark Brown
2010-04-08 14:05 ` Jakob Viketoft
2010-04-08 14:12 ` Mark Brown
2010-04-08 14:35 ` Jakob Viketoft
2010-04-08 14:44 ` Mark Brown
2010-04-08 15:07 ` Jakob Viketoft
2010-04-08 14:06 ` Jakob Viketoft
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=201004070950.37809.marek.vasut@gmail.com \
--to=marek.vasut@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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.