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