Re: [alsa-devel] [RFC PATCH v2 0/5] snd-ice1712: Add Philips PSC724 Ultimate Edge

From: Ondrej Zary
Date: Wed Sep 19 2012 - 13:03:45 EST


On Wednesday 19 September 2012 17:28:21 Takashi Iwai wrote:
> At Tue, 18 Sep 2012 20:58:31 +0200,
>
> Ondrej Zary wrote:
> > Hello,
> > this is second attempt to add Philips PSC724 Ultimate Edge sound card
> > support to snd-ice1712 driver. Unlike other snd-ice1712 subdrivers, the
> > codec code is splitted into separate modules and thus reusable by other
> > ice1712 subdrivers.
>
> Well, the problem is that most of ice17xx boards use only a part of
> channels on codecs. Thus you don't want to expose controls for all
> channels, too. Otherwise user will get confusing dead controls.

The controls can be easily hidden by setting its name to NULL. It's even used
in the psc724 subdriver.

> If these new modules are supposed to be used by other ice1712/ice1724
> sub drivers, they should provide some good binding mechanism, too.
> (And that's one of the good reasons why ASoC has been nominated.)

They're easy to use, you just need to provide wm87x6_write() function. I think
that other subdrivers could be easily converted to use this (with hardware to
test).

> thanks,
>
> Takashi
>
> > Working: all analog outputs (front, rear, center+lfe, headphone) and
> > inputs (front mic, rear mic, line, cd, aux), mixer, headphone jack
> > detection, suspend/resume
> >
> > Currently untested: SPDIF
> >
> > Buggy: switching AGC mode in mixer causes alsamixer to crash - looks like
> > it does not like notifications about control activating/deactivating
> > _______________________________________________
> > Alsa-devel mailing list
> > Alsa-devel@xxxxxxxxxxxxxxxx
> > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/