On Tue, Nov 20, 2007 at 11:40:25AM +0100, Takashi Iwai wrote: > Mark Brown wrote: > > +config SND_SOC_AK4535 > > + tristate > > + depends on SND_SOC > > + > Unless you add another interface support, this driver seems only for > I2C. Then let's add dependency on I2C (which will avoid a possible > unresolved symbol error). The idiom used by the rest of the ASoC codecs appears to be to omit the bus dependencies entirely from the codec drivers and rely on the machine driver pulling in the appropriate bus driver. For the sake of consistency I'd rather leave this just now and then separately go through and address this for all drivers at once - does that seem sensible to you? > > --- /dev/null > > +++ b/sound/soc/codecs/ak4535.c > ak4535.c has some obvious coding-style issues. > Try to run scripts/checkpatch.pl and fix the reported parts. I've fixed up the mechanical issues for next submission. _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
