[alsa-devel] [patch 2/2] alsa: s6105 IP camera machine specific ASoC code

Mark Brown broonie at sirena.org.uk
Fri Mar 27 17:39:03 CET 2009


On Fri, Mar 27, 2009 at 05:33:21PM +0100, Daniel Glöckner wrote:
> On 03/26/2009 03:26 PM, Mark Brown wrote:

> > Are you sure you're applying the correct configuration to the correct
> > DAI here - S6105_CODEC_CLOCK is a CPU define, not a CODEC define?

> In this case S6105 was referring to the S6105 reference design. It's a machine
> define at the top of the file.

Hrm, OK.

> >> +static int snd_soc_dapm_set_endpoint(struct snd_soc_codec *codec, char *name,

> > This shouldn't be in a driver!

> I know, but it was made static in soc-dapm.c when it was renamed to
> snd_soc_dapm_set_pin.

In cases like this you should submit a patch adding the API back to the
core rather than adding a generic API as local code in your machine
driver.


More information about the Alsa-devel mailing list