Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-19 Thread Suzuki K Poulose
On 19/01/17 11:40, Greg KH wrote: Fixes: commit 237483aa5cf43 ("coresight: stm: adding driver for CoreSight STM component") Cc: Pratik Patel Cc: Greg Kroah-Hartman Cc: sta...@vger.kernel.org # 4.7+ Acked-by: Mathieu Poirier

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-19 Thread Suzuki K Poulose
On 19/01/17 11:40, Greg KH wrote: Fixes: commit 237483aa5cf43 ("coresight: stm: adding driver for CoreSight STM component") Cc: Pratik Patel Cc: Greg Kroah-Hartman Cc: sta...@vger.kernel.org # 4.7+ Acked-by: Mathieu Poirier Reviewed-by: Chunyan Zhang Reported-by: Robert Walker

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-19 Thread Greg KH
On Mon, Jan 16, 2017 at 06:00:00PM +, Suzuki K Poulose wrote: > The stm is automatically enabled when an application sets the policy > via ->link() call back by using coresight_enable(), which keeps the > refcount of the current users of the STM. However, the unlink() callback > issues

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-19 Thread Greg KH
On Mon, Jan 16, 2017 at 06:00:00PM +, Suzuki K Poulose wrote: > The stm is automatically enabled when an application sets the policy > via ->link() call back by using coresight_enable(), which keeps the > refcount of the current users of the STM. However, the unlink() callback > issues

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-16 Thread Mathieu Poirier
On 13 January 2017 at 10:11, Suzuki K Poulose wrote: > On 13/01/17 16:48, Mathieu Poirier wrote: >> >> On 10 January 2017 at 04:21, Suzuki K Poulose >> wrote: >>> >>> The stm is automatically enabled when an application sets the policy >>> via

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-16 Thread Mathieu Poirier
On 13 January 2017 at 10:11, Suzuki K Poulose wrote: > On 13/01/17 16:48, Mathieu Poirier wrote: >> >> On 10 January 2017 at 04:21, Suzuki K Poulose >> wrote: >>> >>> The stm is automatically enabled when an application sets the policy >>> via ->link() call back by using coresight_enable(),

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-13 Thread Suzuki K Poulose
On 13/01/17 16:48, Mathieu Poirier wrote: On 10 January 2017 at 04:21, Suzuki K Poulose wrote: The stm is automatically enabled when an application sets the policy via ->link() call back by using coresight_enable(), which keeps the refcount of the current users of the

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-13 Thread Suzuki K Poulose
On 13/01/17 16:48, Mathieu Poirier wrote: On 10 January 2017 at 04:21, Suzuki K Poulose wrote: The stm is automatically enabled when an application sets the policy via ->link() call back by using coresight_enable(), which keeps the refcount of the current users of the STM. However, the

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-13 Thread Mathieu Poirier
On 10 January 2017 at 04:21, Suzuki K Poulose wrote: > The stm is automatically enabled when an application sets the policy > via ->link() call back by using coresight_enable(), which keeps the > refcount of the current users of the STM. However, the unlink() callback >

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-13 Thread Mathieu Poirier
On 10 January 2017 at 04:21, Suzuki K Poulose wrote: > The stm is automatically enabled when an application sets the policy > via ->link() call back by using coresight_enable(), which keeps the > refcount of the current users of the STM. However, the unlink() callback > issues stm_disable()

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-12 Thread Chunyan Zhang
On 11 January 2017 at 21:59, Suzuki K Poulose wrote: > On 11/01/17 11:41, Chunyan Zhang wrote: >> >> On 11 January 2017 at 01:36, Mathieu Poirier >> wrote: >>> >>> On Tue, Jan 10, 2017 at 11:21:55AM +, Suzuki K Poulose wrote: The

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-12 Thread Chunyan Zhang
On 11 January 2017 at 21:59, Suzuki K Poulose wrote: > On 11/01/17 11:41, Chunyan Zhang wrote: >> >> On 11 January 2017 at 01:36, Mathieu Poirier >> wrote: >>> >>> On Tue, Jan 10, 2017 at 11:21:55AM +, Suzuki K Poulose wrote: The stm is automatically enabled when an application

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-11 Thread Suzuki K Poulose
On 11/01/17 11:41, Chunyan Zhang wrote: On 11 January 2017 at 01:36, Mathieu Poirier wrote: On Tue, Jan 10, 2017 at 11:21:55AM +, Suzuki K Poulose wrote: The stm is automatically enabled when an application sets the policy via ->link() call back by using

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-11 Thread Suzuki K Poulose
On 11/01/17 11:41, Chunyan Zhang wrote: On 11 January 2017 at 01:36, Mathieu Poirier wrote: On Tue, Jan 10, 2017 at 11:21:55AM +, Suzuki K Poulose wrote: The stm is automatically enabled when an application sets the policy via ->link() call back by using coresight_enable(), which keeps

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-11 Thread Chunyan Zhang
On 11 January 2017 at 01:36, Mathieu Poirier wrote: > On Tue, Jan 10, 2017 at 11:21:55AM +, Suzuki K Poulose wrote: >> The stm is automatically enabled when an application sets the policy >> via ->link() call back by using coresight_enable(), which keeps the >>

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-11 Thread Chunyan Zhang
On 11 January 2017 at 01:36, Mathieu Poirier wrote: > On Tue, Jan 10, 2017 at 11:21:55AM +, Suzuki K Poulose wrote: >> The stm is automatically enabled when an application sets the policy >> via ->link() call back by using coresight_enable(), which keeps the >> refcount of the current users

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-10 Thread Mathieu Poirier
On Tue, Jan 10, 2017 at 11:21:55AM +, Suzuki K Poulose wrote: > The stm is automatically enabled when an application sets the policy > via ->link() call back by using coresight_enable(), which keeps the > refcount of the current users of the STM. However, the unlink() callback > issues

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-10 Thread Mathieu Poirier
On Tue, Jan 10, 2017 at 11:21:55AM +, Suzuki K Poulose wrote: > The stm is automatically enabled when an application sets the policy > via ->link() call back by using coresight_enable(), which keeps the > refcount of the current users of the STM. However, the unlink() callback > issues