On Wed, Jun 13, 2012 at 11:43 AM, Pandita, Vikram <[email protected]> wrote: > On Tue, Jun 12, 2012 at 11:07 PM, Pandita, Vikram <[email protected]> > wrote: >> On Thu, Jun 7, 2012 at 2:22 PM, Jon Hunter <[email protected]> wrote: >>> In order to use the CTI interrupts inconjunction with the DEBUGSS we need to >>> re-map the CTI IRQs to the DEBUGSS HWMOD. The purpose for doing this is so >>> we >>> can create a PMU device based upon the DEBUGSS HWMOD and use the CTI >>> interrupts >>> for routing ARM PMU events for OMAP4430 devices. >>> >>> This is based upon Benoit Cousson's patch [1]. >>> >>> [1] >>> http://lists.infradead.org/pipermail/linux-arm-kernel/2011-November/073319.html >>> >>> Cc: Ming Lei <[email protected]> >>> Cc: Will Deacon <[email protected]> >>> Cc: Benoit Cousson <[email protected]> >>> Cc: Paul Walmsley <[email protected]> >>> Cc: Kevin Hilman <[email protected]> >>> >>> Reviewed-by: Santosh Shilimkar <[email protected]> >>> Signed-off-by: Jon Hunter <[email protected]> >>> --- >>> arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 9 +++++++-- >>> 1 file changed, 7 insertions(+), 2 deletions(-) >>> >>> diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >>> b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >>> index 950454a..faf5a6d 100644 >>> --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >>> +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >>> @@ -482,10 +482,17 @@ static struct omap_hwmod_class >>> omap44xx_debugss_hwmod_class = { >>> }; >>> >>> /* debugss */ >>> +static struct omap_hwmod_irq_info omap44xx_debugss_irqs[] = { >>> + { .name = "cti0", .irq = 1 + OMAP44XX_IRQ_GIC_START }, >>> + { .name = "cti1", .irq = 2 + OMAP44XX_IRQ_GIC_START }, >> >> Data manual assigns cit0 == MA_IRQ_1 and cti1 == MA_IRQ_2 >> Why do you add 32 base GIC_START ? > > Looks we do the same for pl310: > { .name = "pl310", .irq = 0 + OMAP44XX_IRQ_GIC_START }, > > so u can ignore my comment .. excuse the noise... > All the SPI ( shared Peripheral IRQ's) have offset since first 32 IRQs are dedicated for SGI and PPI's.
CTIx and PL310 interrupts are mapped on SPI and hence they need that offset. Regards Santosh -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
