On Wednesday 15 July 2015 17:32:58 Yuan Yao wrote: > +#ifdef CONFIG_PM > +enum fsl_edma_pm_state { > + RUNNING = 0, > + SUSPENDED, > +}; > +#endif > > struct fsl_edma_hw_tcd { > The #ifdefs here seem unnecessary, at least most of them, better just do this all unconditionally. Arnd