On Tue, Mar 14, 2017 at 10:16:28AM +0530, Shubhrajyoti Datta wrote: > On Tue, Mar 14, 2017 at 10:04 AM, Vinod Koul <vinod.koul@xxxxxxxxx> wrote: > > On Thu, Mar 09, 2017 at 12:49:17PM +0530, Shubhrajyoti Datta wrote: > >> Add runtime support for the dma driver. > > > > runtime pm right? > yes then make it explicit > >> + pm_runtime_set_active(&pdev->dev); > >> + pm_runtime_enable(&pdev->dev); > > > > This makes no sense, runtime pm means actively managing device power which > > is not the case, so what exactly are you trying here > > Yes however some of the power domain driver stuff relies on truthfully > reporting the runtime state > so I was trying to set the state to active when using it. > > > However you are right we are working on actively managing the device > power patch also. > will get back. ok -- ~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