On 01/18/2017 03:02 PM, Sekhar Nori wrote: > On Wednesday 18 January 2017 07:21 PM, Alexandre Bailon wrote: >>>> +static int da8xx_runtime_suspend(struct device *dev) >>>> +{ >>>> + struct da8xx_glue *glue = dev_get_drvdata(dev); >>>> + >>>> + clk_disable_unprepare(glue->clk); > >>> I thought RPM would do that for you... > >> DA8xx doesn't support yet the common clock framework. >> So for now, we have to manage clock in the driver. > > We do have arch/arm/mach-davinci/pm_domain.c which should help with > pm_runtime. OK. I will take a look. > > Did you already try without the explicit clock enables and it did not > work? Please do use _sync() version of get() and put() calls also. Yes, without the clock enables, it doesn't work. > > Thanks, > Sekhar > Thanks, Alexandre -- 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