Re: [RFC PATCH 01/20] coresight: etm3x: splitting 'etm_enable_hw()' operations

2015-10-01 Thread Alexander Shishkin
Mathieu Poirier writes: > On 30 September 2015 at 03:58, Alexander Shishkin > wrote: >> Most of these things can also be bypassed, as at least initially perf >> events won't be using trigger/sequencer configurations, so we could >> simply clear all these things out when a first perf event is

Re: [RFC PATCH 01/20] coresight: etm3x: splitting 'etm_enable_hw()' operations

2015-10-01 Thread Mathieu Poirier
On 30 September 2015 at 03:58, Alexander Shishkin wrote: > Mathieu Poirier writes: > >> -static void etm_enable_hw(void *info) >> +static void etm_power_up_cpu(void *info) >> { >> - int i; >> - u32 etmcr; >> - struct etm_drvdata *drvdata = info; >> + struct coresight_device

Re: [RFC PATCH 01/20] coresight: etm3x: splitting 'etm_enable_hw()' operations

2015-10-01 Thread Mathieu Poirier
On 30 September 2015 at 03:58, Alexander Shishkin wrote: > Mathieu Poirier writes: > >> -static void etm_enable_hw(void *info) >> +static void etm_power_up_cpu(void *info) >> { >> - int i; >> - u32 etmcr; >> - struct

Re: [RFC PATCH 01/20] coresight: etm3x: splitting 'etm_enable_hw()' operations

2015-10-01 Thread Alexander Shishkin
Mathieu Poirier writes: > On 30 September 2015 at 03:58, Alexander Shishkin > wrote: >> Most of these things can also be bypassed, as at least initially perf >> events won't be using trigger/sequencer configurations, so we could >>

Re: [RFC PATCH 01/20] coresight: etm3x: splitting 'etm_enable_hw()' operations

2015-09-30 Thread Alexander Shishkin
Mathieu Poirier writes: > -static void etm_enable_hw(void *info) > +static void etm_power_up_cpu(void *info) > { > - int i; > - u32 etmcr; > - struct etm_drvdata *drvdata = info; > + struct coresight_device *csdev = info; > + struct etm_drvdata *drvdata =

Re: [RFC PATCH 01/20] coresight: etm3x: splitting 'etm_enable_hw()' operations

2015-09-30 Thread Alexander Shishkin
Mathieu Poirier writes: > -static void etm_enable_hw(void *info) > +static void etm_power_up_cpu(void *info) > { > - int i; > - u32 etmcr; > - struct etm_drvdata *drvdata = info; > + struct coresight_device *csdev = info; > + struct etm_drvdata