On Mon, Oct 09, 2017 at 03:40:27PM +0100, Ed Blake wrote: > +static int img_mdc_runtime_resume(struct device *dev) > +{ > + struct mdc_dma *mdma = dev_get_drvdata(dev); > + > + return clk_prepare_enable(mdma->clk); > +} This one is same as... On Mon, Oct 09, 2017 at 03:40:26PM +0100, Ed Blake wrote: > +static int img_mdc_resume_early(struct device *dev) > +{ > + struct mdc_dma *mdma = dev_get_drvdata(dev); > + > + return clk_prepare_enable(mdma->clk); > +} ... this one. why not use a same fn for both... -- ~Vinod -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html