* Felipe Balbi <balbi@xxxxxx> [140319 13:06]: > On Wed, Mar 19, 2014 at 02:59:13PM -0500, Felipe Balbi wrote: > > Hi, > > > > On Wed, Mar 19, 2014 at 08:29:20PM +0100, Arnd Bergmann wrote: > > > If we enable CONFIG_OMAP_MPU_TIMER and CONFIG_OMAP_DM_TIMER but > > > not CONFIG_OMAP_32K_TIMER on OMAP1, we currently get this build error: > > > > > > mach-omap1/pm.c: In function 'omap1_pm_idle': > > > mach-omap1/pm.c:123:9: error: 'enable_dyn_sleep' undeclared (first use in this function) > > > while (enable_dyn_sleep) { > > > ^ > > > mach-omap1/pm.c:123:9: note: each undeclared identifier is reported only once for each function it appears in > > > > > > This attempts to fix the #ifdef logic to deal with all combinations > > > of timers. > > > > > > Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> > > > Cc: linux-omap@xxxxxxxxxxxxxxx > > > Cc: Tony Lindgren <tony@xxxxxxxxxxx> > > > > I had sent a fix for months and months ago, what happened to it ? > > Here > > http://marc.info/?l=linux-omap&m=138963632408031&w=2 > > Tony, did you forget to send a pull request ? Hmm yes weird, can't see it in my omap-for-v3.14/fixes branch. I must have gotten interrupted while applying and then probably ran git reset --hard before committing. Arnd, maybe pick up Felipe's earlier patch instead? Acked-by: Tony Lindgren <tony@xxxxxxxxxxx> -- 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