All of lore.kernel.org
 help / color / mirror / Atom feed
* linux-next: manual merge of the arm-perf tree with the arm tree
@ 2014-07-02  1:17 Stephen Rothwell
  2014-07-02  9:15 ` Will Deacon
  0 siblings, 1 reply; 2+ messages in thread
From: Stephen Rothwell @ 2014-07-02  1:17 UTC (permalink / raw)
  To: Will Deacon; +Cc: linux-next, linux-kernel, Russell King, Mark Rutland

[-- Attachment #1: Type: text/plain, Size: 2543 bytes --]

Hi Will,

Today's linux-next merge of the arm-perf tree got a conflict in
arch/arm/kernel/perf_event_cpu.c between commit 324bbc0804ac ("ARM:
make it easier to check the CPU part number correctly") from the arm
tree and commit 6f73e8ababdd ("arm: perf: clean up PMU names") from the
arm-perf tree.

I fixed it up (see below) and can carry the fix as necessary (no action
is required).

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
diff --cc arch/arm/kernel/perf_event_cpu.c
index c02c2e8c877d,c63dc4201a7f..000000000000
--- a/arch/arm/kernel/perf_event_cpu.c
+++ b/arch/arm/kernel/perf_event_cpu.c
@@@ -254,35 -259,38 +257,39 @@@ static int probe_current_pmu(struct arm
  
  	pr_info("probing PMU on CPU %d\n", cpu);
  
 +	switch (read_cpuid_part()) {
  	/* ARM Ltd CPUs. */
 -	if (implementor == ARM_CPU_IMP_ARM) {
 -		switch (part_number) {
 -		case ARM_CPU_PART_ARM1136:
 -			ret = armv6_1136_pmu_init(pmu);
 -			break;
 -		case ARM_CPU_PART_ARM1156:
 -			ret = armv6_1156_pmu_init(pmu);
 -			break;
 -		case ARM_CPU_PART_ARM1176:
 -			ret = armv6_1176_pmu_init(pmu);
 -			break;
 -		case ARM_CPU_PART_ARM11MPCORE:
 -			ret = armv6mpcore_pmu_init(pmu);
 -			break;
 -		case ARM_CPU_PART_CORTEX_A8:
 -			ret = armv7_a8_pmu_init(pmu);
 -			break;
 -		case ARM_CPU_PART_CORTEX_A9:
 -			ret = armv7_a9_pmu_init(pmu);
 -			break;
 -		}
 -	/* Intel CPUs [xscale]. */
 -	} else if (implementor == ARM_CPU_IMP_INTEL) {
 -		switch (xscale_cpu_arch_version()) {
 -		case ARM_CPU_XSCALE_ARCH_V1:
 -			ret = xscale1pmu_init(pmu);
 -			break;
 -		case ARM_CPU_XSCALE_ARCH_V2:
 -			ret = xscale2pmu_init(pmu);
 -			break;
 +	case ARM_CPU_PART_ARM1136:
++		ret = armv6_1136_pmu_init(pmu);
++		break;
 +	case ARM_CPU_PART_ARM1156:
++		ret = armv6_1156_pmu_init(pmu);
++		break;
 +	case ARM_CPU_PART_ARM1176:
- 		ret = armv6pmu_init(pmu);
++		ret = armv6_1176_pmu_init(pmu);
 +		break;
 +	case ARM_CPU_PART_ARM11MPCORE:
 +		ret = armv6mpcore_pmu_init(pmu);
 +		break;
 +	case ARM_CPU_PART_CORTEX_A8:
 +		ret = armv7_a8_pmu_init(pmu);
 +		break;
 +	case ARM_CPU_PART_CORTEX_A9:
 +		ret = armv7_a9_pmu_init(pmu);
 +		break;
 +
 +	default:
 +		if (read_cpuid_implementor() == ARM_CPU_IMP_INTEL) {
 +			switch (xscale_cpu_arch_version()) {
 +			case ARM_CPU_XSCALE_ARCH_V1:
 +				ret = xscale1pmu_init(pmu);
 +				break;
 +			case ARM_CPU_XSCALE_ARCH_V2:
 +				ret = xscale2pmu_init(pmu);
 +				break;
 +			}
  		}
 +		break;
  	}
  
  	put_cpu();

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: linux-next: manual merge of the arm-perf tree with the arm tree
  2014-07-02  1:17 linux-next: manual merge of the arm-perf tree with the arm tree Stephen Rothwell
@ 2014-07-02  9:15 ` Will Deacon
  0 siblings, 0 replies; 2+ messages in thread
From: Will Deacon @ 2014-07-02  9:15 UTC (permalink / raw)
  To: Stephen Rothwell; +Cc: linux-next, linux-kernel, Russell King, Mark Rutland

On Wed, Jul 02, 2014 at 02:17:08AM +0100, Stephen Rothwell wrote:
> Hi Will,

Hi Stephen,

> Today's linux-next merge of the arm-perf tree got a conflict in
> arch/arm/kernel/perf_event_cpu.c between commit 324bbc0804ac ("ARM:
> make it easier to check the CPU part number correctly") from the arm
> tree and commit 6f73e8ababdd ("arm: perf: clean up PMU names") from the
> arm-perf tree.
> 
> I fixed it up (see below) and can carry the fix as necessary (no action
> is required).

Looks good, thanks.

Russell -- could you put your CPU part number patches on a stable branch
please? That way I can send you a pull request based on that (or you can fix
the conflicts up yourself, just let me know what you prefer).

Will

> diff --cc arch/arm/kernel/perf_event_cpu.c
> index c02c2e8c877d,c63dc4201a7f..000000000000
> --- a/arch/arm/kernel/perf_event_cpu.c
> +++ b/arch/arm/kernel/perf_event_cpu.c
> @@@ -254,35 -259,38 +257,39 @@@ static int probe_current_pmu(struct arm
>   
>   	pr_info("probing PMU on CPU %d\n", cpu);
>   
>  +	switch (read_cpuid_part()) {
>   	/* ARM Ltd CPUs. */
>  -	if (implementor == ARM_CPU_IMP_ARM) {
>  -		switch (part_number) {
>  -		case ARM_CPU_PART_ARM1136:
>  -			ret = armv6_1136_pmu_init(pmu);
>  -			break;
>  -		case ARM_CPU_PART_ARM1156:
>  -			ret = armv6_1156_pmu_init(pmu);
>  -			break;
>  -		case ARM_CPU_PART_ARM1176:
>  -			ret = armv6_1176_pmu_init(pmu);
>  -			break;
>  -		case ARM_CPU_PART_ARM11MPCORE:
>  -			ret = armv6mpcore_pmu_init(pmu);
>  -			break;
>  -		case ARM_CPU_PART_CORTEX_A8:
>  -			ret = armv7_a8_pmu_init(pmu);
>  -			break;
>  -		case ARM_CPU_PART_CORTEX_A9:
>  -			ret = armv7_a9_pmu_init(pmu);
>  -			break;
>  -		}
>  -	/* Intel CPUs [xscale]. */
>  -	} else if (implementor == ARM_CPU_IMP_INTEL) {
>  -		switch (xscale_cpu_arch_version()) {
>  -		case ARM_CPU_XSCALE_ARCH_V1:
>  -			ret = xscale1pmu_init(pmu);
>  -			break;
>  -		case ARM_CPU_XSCALE_ARCH_V2:
>  -			ret = xscale2pmu_init(pmu);
>  -			break;
>  +	case ARM_CPU_PART_ARM1136:
> ++		ret = armv6_1136_pmu_init(pmu);
> ++		break;
>  +	case ARM_CPU_PART_ARM1156:
> ++		ret = armv6_1156_pmu_init(pmu);
> ++		break;
>  +	case ARM_CPU_PART_ARM1176:
> - 		ret = armv6pmu_init(pmu);
> ++		ret = armv6_1176_pmu_init(pmu);
>  +		break;
>  +	case ARM_CPU_PART_ARM11MPCORE:
>  +		ret = armv6mpcore_pmu_init(pmu);
>  +		break;
>  +	case ARM_CPU_PART_CORTEX_A8:
>  +		ret = armv7_a8_pmu_init(pmu);
>  +		break;
>  +	case ARM_CPU_PART_CORTEX_A9:
>  +		ret = armv7_a9_pmu_init(pmu);
>  +		break;
>  +
>  +	default:
>  +		if (read_cpuid_implementor() == ARM_CPU_IMP_INTEL) {
>  +			switch (xscale_cpu_arch_version()) {
>  +			case ARM_CPU_XSCALE_ARCH_V1:
>  +				ret = xscale1pmu_init(pmu);
>  +				break;
>  +			case ARM_CPU_XSCALE_ARCH_V2:
>  +				ret = xscale2pmu_init(pmu);
>  +				break;
>  +			}
>   		}
>  +		break;
>   	}
>   
>   	put_cpu();



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-07-02  9:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-02  1:17 linux-next: manual merge of the arm-perf tree with the arm tree Stephen Rothwell
2014-07-02  9:15 ` Will Deacon

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.