Hello.
Hans J. Koch wrote:
-----Original Message-----
From: Hans J. Koch [mailto:[email protected]]
Sent: Sunday, February 20, 2011 12:04 AM
To: TK, Pratheesh Gangadhar
Cc: Thomas Gleixner; Arnd Bergmann; [email protected];
[email protected]; [email protected];
Chatterjee, Amit; Hans J. Koch; LKML
Subject: Re: [PATCH 1/2] PRUSS UIO driver support
On Sat, Feb 19, 2011 at 09:10:23PM +0530, TK, Pratheesh Gangadhar wrote:
For my understanding - if the interrupt is not shared and not level triggered -
is this okay to have empty handler?
Greg already said he won't accept that. And I'm quite sure these
interrupts
are level triggered, since that is the default on arch/omap.
E.g. in arch/arm/mach-omap1/irq.c, a loop sets all irqs to level triggered
handling: set_irq_handler(j, handle_level_irq); (line 234)
You should be looking at arch/arm/mach-davinci/irq.c
Into arch/arm/mach-davinci/cp_intc.c actually as we're not talking about a
"real" DaVincis here, but about OMAP-L138.
Hmpf. I alway get lost in the directory structure of TI SoCs...
At least I learned that OMAP3 is in the omap2 directory :)
It's probably even more convoluted with DaVinci and friends. :-)
and all interrupts except IRQ_TINT1_TINT34 is set to edge trigger mode (line
160).
Is that really needed? Level triggered interrupts should certainly be
preferred if possible.
Don't look there, this code is for "real" DaVincis only.
Thanks,
Hans
WBR, Sergei
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
