Re: [PATCH V7 04/24] coresight: moving PM runtime operations to core framework

2015-12-24 Thread Rabin Vincent
On Wed, Dec 23, 2015 at 09:27:28AM -0700, Mathieu Poirier wrote: > On 19 December 2015 at 10:13, Rabin Vincent wrote: > > It should presumably be using pm_runtime_put() instead. > > That's a first - what platform did you test on? If I send you fixes > will you be able to help me with the verific

Re: [PATCH V7 04/24] coresight: moving PM runtime operations to core framework

2015-12-23 Thread Mathieu Poirier
On 19 December 2015 at 10:13, Rabin Vincent wrote: > On Fri, Dec 18, 2015 at 01:59:00PM -0700, Mathieu Poirier wrote: >> @@ -415,9 +418,13 @@ struct list_head *coresight_build_path(struct >> coresight_device *csdev) >> */ >> void coresight_release_path(struct list_head *path) >> { >> + st

Re: [PATCH V7 04/24] coresight: moving PM runtime operations to core framework

2015-12-19 Thread Rabin Vincent
On Fri, Dec 18, 2015 at 01:59:00PM -0700, Mathieu Poirier wrote: > @@ -415,9 +418,13 @@ struct list_head *coresight_build_path(struct > coresight_device *csdev) > */ > void coresight_release_path(struct list_head *path) > { > + struct coresight_device *csdev; > struct coresight_node

[PATCH V7 04/24] coresight: moving PM runtime operations to core framework

2015-12-18 Thread Mathieu Poirier
Moving PM runtime operations in Coresight devices enable() and disable() API to the framework core when a path is setup. That way the runtime core doesn't have to be involved everytime a path is enabled. It also avoids calling runtime PM operations in IRQ context. Signed-off-by: Mathieu Poirier