On Sunday 02 October 2011 16:53:31 Russell King - ARM Linux wrote: > On Sun, Oct 02, 2011 at 04:45:41PM +0200, Arnd Bergmann wrote: > > -#if defined(CONFIG_MENELAUS) && \ > > - (defined(CONFIG_MMC_OMAP) || defined(CONFIG_MMC_OMAP_MODULE)) > > +#if defined(CONFIG_MENELAUS) && defined(CONFIG_MMC_OMAP) > > +/* || defined(CONFIG_MMC_OMAP_MODULE)) */ > > +/* FIXME: cannot call omap_mmc_notify_cover_event for ONFIG_MMC_OMAP_MODULE */ > > #ifdef CONFIG_MMC_OMAP_MODULE > #warning FIXME: cannot call omap_mmc_notify_cover_event for CONFIG_MMC_OMAP_MODULE > #endif > #if defined(CONFIG_MENELAUS) && defined(CONFIG_MMC_OMAP) > > So that it can be seen at build time? > > Also note the 'ONFIG' typo... Ok, good idea. I've updated the patch in the git tree accordingly. Depending on what Tony wants, I might send out the entire series again once there are no more new comments. Arnd -- 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