[alsa-devel] [PATCH] NUC900/audio: add nuc900 audio driver support
broonie at opensource.wolfsonmicro.com
Mon May 17 15:46:22 CEST 2010
On Mon, May 17, 2010 at 02:26:12PM +0800, Wan ZongShun wrote:
> >On Sun, May 16, 2010 at 11:34:01PM +0800, Wan ZongShun wrote:
> >>+ if (!(AUDIO_READ(nuc900_audio->mmio + ACTL_ACIS0) & CODEC_READY))
> >>+ dev_err(nuc900_audio->dev, "AC97 codec cold reset failed!\n");
> >What is this actually checking in the hardware? Not all CODECs enable
> >the AC97 link by default after a cold reset, the standard allows them to
> >power up in a low power state which will
> The bit 'CODEC_READY' indicates the external AC97 audio CODEC is ready,
> I just forget to implement the warm reset operation, if need, I will do
> it. In addition, Can I add the checking CODEC_READY operation to
> warm reset function?
That sounds fine. For defensiveness it might be worth checking the
CODEC on register reads and writes too (it'd make diagnostics a bit
> >>+#define IIS_EN 0x0002
> >Looks like there's I2S support to come?
> Yes,nuc900 supports both IIS and AC97,but my board only enable AC97 support,
> what's up here?
Nothing, just curious.
More information about the Alsa-devel