Hello Peter, one very minor comment: On Tue, 14 Oct 2008, Peter 'p2' De Schrijver wrote: > diff --git a/arch/arm/plat-omap/include/mach/powerdomain.h b/arch/arm/plat-omap/include/mach/powerdomain.h > index 52663fc..6271d85 100644 > --- a/arch/arm/plat-omap/include/mach/powerdomain.h > +++ b/arch/arm/plat-omap/include/mach/powerdomain.h > @@ -119,6 +119,11 @@ struct powerdomain { > > int state; > unsigned state_counter[4]; > + > +#ifdef CONFIG_PM_DEBUG > + s64 timer; > + s64 state_timer[4]; > +#endif > }; this probably goes best in patch 2/3? Other than that, looks good to me. - Paul -- 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