On Wed, Mar 02, 2011 at 12:58:31AM +0800, tom.leiming@xxxxxxxxx wrote: > From: Ming Lei <tom.leiming@xxxxxxxxx> > > This patch supports pmu irq routed from CTI, so > make pmu/perf working on OMAP4. > > The idea is from Woodruff Richard in the disscussion > "Oprofile on Pandaboard / Omap4" of pandaboard@xxxxxxxxxxxxxxxxx > > Cc: Woodruff Richard <r-woodruff2@xxxxxx> > Cc: Tony Lindgren <tony@xxxxxxxxxxx> > Cc: linux-omap@xxxxxxxxxxxxxxx > Signed-off-by: Ming Lei <tom.leiming@xxxxxxxxx> > --- > arch/arm/mach-omap2/dbg44xx.h | 18 +++++++++++++ > arch/arm/mach-omap2/devices.c | 55 ++++++++++++++++++++++++++++++++++++++++- > 2 files changed, 72 insertions(+), 1 deletions(-) > create mode 100644 arch/arm/mach-omap2/dbg44xx.h > > diff --git a/arch/arm/mach-omap2/dbg44xx.h b/arch/arm/mach-omap2/dbg44xx.h > new file mode 100644 > index 0000000..e447ad5 > --- /dev/null > +++ b/arch/arm/mach-omap2/dbg44xx.h > @@ -0,0 +1,18 @@ > +/* > + * OMAP44xx on-chip debug support > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + * > + * XXX This file needs to be updated to align on one of "OMAP4", "OMAP44XX", > + * or "OMAP4430". > + */ > + > +#ifndef __ARCH_ARM_MACH_OMAP2_DBG44XX_H > +#define __ARCH_ARM_MACH_OMAP2_DBG44XX_H > + > +#define OMAP44XX_CTI0_BASE 0x54148000 > +#define OMAP44XX_CTI1_BASE 0x54149000 > + > +#endif > diff --git a/arch/arm/mach-omap2/devices.c b/arch/arm/mach-omap2/devices.c > index d216976..0420a58 100644 > --- a/arch/arm/mach-omap2/devices.c > +++ b/arch/arm/mach-omap2/devices.c > @@ -35,6 +35,7 @@ > > #include "mux.h" > #include "control.h" > +#include "dbg44xx.h" > > #if defined(CONFIG_VIDEO_OMAP2) || defined(CONFIG_VIDEO_OMAP2_MODULE) > > @@ -322,19 +323,71 @@ static struct resource omap3_pmu_resource = { > .flags = IORESOURCE_IRQ, > }; > > +static struct resource omap4_pmu_resource[] = { > + { > + .start = OMAP44XX_IRQ_CTI0, > + .end = OMAP44XX_IRQ_CTI0, > + .flags = IORESOURCE_IRQ, > + }, > + { > + .start = OMAP44XX_IRQ_CTI1, > + .end = OMAP44XX_IRQ_CTI1, > + .flags = IORESOURCE_IRQ, > + } > +}; indentation is wrong. > static void omap_init_pmu(void) > { > if (cpu_is_omap24xx()) > omap_pmu_device.resource = &omap2_pmu_resource; > else if (cpu_is_omap34xx()) > omap_pmu_device.resource = &omap3_pmu_resource; missing add braces to these two. -- balbi -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html