[PATCH 2/2] ARM: perf: make name of arm_pmu_type consistent

Russell King - ARM Linux linux at arm.linux.org.uk
Fri Aug 12 05:06:09 EDT 2011


On Wed, Aug 10, 2011 at 01:58:58PM +0100, Mark Rutland wrote:
> Commit f12482c9 ("ARM: 6974/1: pmu: refactor reservation") changed
> {release,reserve}_pmu to take an enum arm_pmu_type as a parameter, but
> inconsistently named the parameter `type' or `device'. It would be nice
> if these were consistent.
> 
> This patch makes use of enum arm_pmu_type consistent, always using
> 'device' as this is more common in usage.

This is confusing.  It's called arm_pmu_TYPE, and it's referred to as a
type in printks.  Yet you're renaming the variable to device.  That just
adds to the problem.

What is it, a type number or a device number?

If it's a device number, then the printks also need fixing, and maybe the
enumerated type name too.



More information about the linux-arm-kernel mailing list