All of lore.kernel.org
 help / color / mirror / Atom feed
From: jakob.viketoft@bitsim.com (Jakob Viketoft)
To: linux-arm-kernel@lists.infradead.org
Subject: AC97 problems with pxa...
Date: Tue, 06 Apr 2010 23:50:54 +0200	[thread overview]
Message-ID: <4BBBACBE.7040505@bitsim.com> (raw)

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...

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

             reply	other threads:[~2010-04-06 21:50 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-06 21:50 Jakob Viketoft [this message]
2010-04-07  2:09 ` AC97 problems with pxa Eric Miao
2010-04-07  9:13   ` Jakob Viketoft
2010-04-07 19:08     ` Marek Vasut
2010-04-07  7:50 ` Marek Vasut
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=4BBBACBE.7040505@bitsim.com \
    --to=jakob.viketoft@bitsim.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.