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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux