Re: [PATCH 2/2] ARM: perf: Add support for Scorpion PMUs

2015-02-12 Thread Stephen Boyd
On 02/12/15 04:49, Mark Rutland wrote: > Hi, > > I haven't given this a thorough review, but I spotted a couple of items > below. > > On Wed, Feb 11, 2015 at 01:05:24AM +, Stephen Boyd wrote: >> Scorpion supports a set of local performance monitor event >> selection registers (LPM) sitting behi

Re: [PATCH 2/2] ARM: perf: Add support for Scorpion PMUs

2015-02-12 Thread Mark Rutland
Hi, I haven't given this a thorough review, but I spotted a couple of items below. On Wed, Feb 11, 2015 at 01:05:24AM +, Stephen Boyd wrote: > Scorpion supports a set of local performance monitor event > selection registers (LPM) sitting behind a cp15 based interface > that extend the archite

Re: [PATCH 2/2] ARM: perf: Add support for Scorpion PMUs

2015-02-11 Thread Ashwin Chaugule
On 11 February 2015 at 13:27, Stephen Boyd wrote: > On 02/10, Ashwin Chaugule wrote: >> Hi Stephen, >> >> On 10 February 2015 at 20:05, Stephen Boyd wrote: >> > Scorpion supports a set of local performance monitor event >> > selection registers (LPM) sitting behind a cp15 based interface >> > tha

Re: [PATCH 2/2] ARM: perf: Add support for Scorpion PMUs

2015-02-11 Thread Stephen Boyd
On 02/10, Stephen Boyd wrote: > Scorpion supports a set of local performance monitor event > selection registers (LPM) sitting behind a cp15 based interface > that extend the architected PMU events to include Scorpion CPU > and Venum VFP specific events. To use these events the user is > expected t

Re: [PATCH 2/2] ARM: perf: Add support for Scorpion PMUs

2015-02-11 Thread Stephen Boyd
On 02/10, Ashwin Chaugule wrote: > Hi Stephen, > > On 10 February 2015 at 20:05, Stephen Boyd wrote: > > Scorpion supports a set of local performance monitor event > > selection registers (LPM) sitting behind a cp15 based interface > > that extend the architected PMU events to include Scorpion CP

Re: [PATCH 2/2] ARM: perf: Add support for Scorpion PMUs

2015-02-10 Thread Ashwin Chaugule
Hi Stephen, On 10 February 2015 at 20:05, Stephen Boyd wrote: > Scorpion supports a set of local performance monitor event > selection registers (LPM) sitting behind a cp15 based interface > that extend the architected PMU events to include Scorpion CPU > and Venum VFP specific events. To use the

[PATCH 2/2] ARM: perf: Add support for Scorpion PMUs

2015-02-10 Thread Stephen Boyd
Scorpion supports a set of local performance monitor event selection registers (LPM) sitting behind a cp15 based interface that extend the architected PMU events to include Scorpion CPU and Venum VFP specific events. To use these events the user is expected to program the lpm register with the even