[alsa-devel] [PATCH v2 1/9] include: platform_data: Platform data header for OMAP4 ASoC audio

Liam Girdwood lrg at ti.com
Thu Dec 22 16:42:53 CET 2011


On Thu, 2011-12-22 at 13:04 +0000, Mark Brown wrote:
> On Thu, Dec 22, 2011 at 02:42:40PM +0200, Peter Ujfalusi wrote:
> 
> >  include/linux/platform_data/omap-abe-twl6040.h |   48 ++++++++++++++++++++++++
> 
> omap-abe-mcpdm-twl6040 please.  Seriously, just drop the abe - it's not
> an optional feature of the OMAP and the names are getting quite long.

ABE can be an optional feature on OMAP (and quite an important one when
it is used). Fwiw, we do have a mixture of users, most use the ABE but
some don't, so it's best to specify ABE in the naming to avoid any
confusion.

This patch looks fine to me.

Regards

Liam




More information about the Alsa-devel mailing list