Re: question in request_threaded_irq
On Mon, Aug 15, 2011 at 7:03 PM, radhika bhaskaran wrote: > Hi javier, > > Thanks for the help. > > I have tried the suggestion made by you. But unfortunately it did not work. > > For my debugging purpose i tried to register two isr's on the same number > IRQ no in the same driver. But the isr which is registerd first is being > hit. > > The other isr funciton which is registered after the first isr is never > executed. > > Any more suggestions? > > IRQ_SHARED flag could work if IRQ_ONESHOT isn't enabled. In your request_threaded_irq(), you assigned thread_func() and NULL for handler_func(). It means that IRQ_ONESHOT is enabled. You either registered it to handler_func(), or abandon irq handler sharing between multiple drivers. > > On Sun, Aug 14, 2011 at 6:31 AM, Javier Martinez Canillas < > martinez.jav...@gmail.com> wrote: > >> On Sat, Aug 13, 2011 at 9:25 PM, radhika bhaskaran >> wrote: >> > Hi, >> > I have a question with respect to request_threaded_irq. >> > Assume that one driver has registered and ISR on some particular irq >> number. >> > Can i use the same irq number and register and isr in another driver. >> > Because that is my requirement. >> > when i try to do that is an error message with an error no as -19. >> > err = request_threaded_irq(pdata->irq , NULL, testing_fucntion, >> > IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING, >> > "testing_function", dev); >> > Can some one please help me whether it is possible. >> > Regards, >> > RB >> >> You have to use the IRQF_SHARED flag that specifies the IRQ will be >> shared for two devices: >> >> err = request_threaded_irq(pdata->irq , NULL, testing_fucntion, >> IRQF_SHARED | >> IRQF_TRIGGER_FALLING | >> IRQF_TRIGGER_RISING, >> "testing_function", dev); >> >> Have in mind that the kernel invokes every handler registered for that >> IRQ every time an interrupt occurs on that line. So you have to check >> in each ISR if the hardware raised the interrupt. >> >> Hope it helps, >> >> -- >> Javier Martínez Canillas >> (+34) 682 39 81 69 >> Barcelona, Spain >> > > > ___ > Kernelnewbies mailing list > Kernelnewbies@kernelnewbies.org > http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies > > ___ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
Re: question in request_threaded_irq
On Mon, 2011-08-15 at 16:33 +0530, radhika bhaskaran wrote: > > For my debugging purpose i tried to register two isr's on the same > number IRQ no in the same driver. But the isr which is registerd > first is being hit. > Since the kernel is monolithic is doesn't really matter where your ISR functions are defined and where these handlers are assigned an IRQ line. > > The other isr funciton which is registered after the first isr is > never executed. > > > Any more suggestions? > Yes, does yours ISR function handlers look if they generated the interrupt and if not respond accordingly? If your device didn't raised the interrupt then they have to return IRQ_NONE. If your hardware raised the interrupt then you have to do the processing and return IRQ_HANDLED. So you have to do something like this: static irqreturn_t my_interrupt(int irq, void *data) { int result; struct my_device *my_dev = data; result = check_interrupt_raised(my_dev); if (!result) return IRQ_NONE; process_interrupt(my_dev); return IRQ_HANDLED; } Hope it helps, Javier Martinez Canillas ___ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
Re: question in request_threaded_irq
Hi javier, Thanks for the help. I have tried the suggestion made by you. But unfortunately it did not work. For my debugging purpose i tried to register two isr's on the same number IRQ no in the same driver. But the isr which is registerd first is being hit. The other isr funciton which is registered after the first isr is never executed. Any more suggestions? Regards, RB On Sun, Aug 14, 2011 at 6:31 AM, Javier Martinez Canillas < martinez.jav...@gmail.com> wrote: > On Sat, Aug 13, 2011 at 9:25 PM, radhika bhaskaran > wrote: > > Hi, > > I have a question with respect to request_threaded_irq. > > Assume that one driver has registered and ISR on some particular irq > number. > > Can i use the same irq number and register and isr in another driver. > > Because that is my requirement. > > when i try to do that is an error message with an error no as -19. > > err = request_threaded_irq(pdata->irq , NULL, testing_fucntion, > > IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING, > > "testing_function", dev); > > Can some one please help me whether it is possible. > > Regards, > > RB > > You have to use the IRQF_SHARED flag that specifies the IRQ will be > shared for two devices: > > err = request_threaded_irq(pdata->irq , NULL, testing_fucntion, IRQF_SHARED > | > IRQF_TRIGGER_FALLING | > IRQF_TRIGGER_RISING, > "testing_function", dev); > > Have in mind that the kernel invokes every handler registered for that > IRQ every time an interrupt occurs on that line. So you have to check > in each ISR if the hardware raised the interrupt. > > Hope it helps, > > -- > Javier Martínez Canillas > (+34) 682 39 81 69 > Barcelona, Spain > ___ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
Re: question in request_threaded_irq
On Sat, Aug 13, 2011 at 9:25 PM, radhika bhaskaran wrote: > Hi, > I have a question with respect to request_threaded_irq. > Assume that one driver has registered and ISR on some particular irq number. > Can i use the same irq number and register and isr in another driver. > Because that is my requirement. > when i try to do that is an error message with an error no as -19. > err = request_threaded_irq(pdata->irq , NULL, testing_fucntion, > IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING, > "testing_function", dev); > Can some one please help me whether it is possible. > Regards, > RB You have to use the IRQF_SHARED flag that specifies the IRQ will be shared for two devices: err = request_threaded_irq(pdata->irq , NULL, testing_fucntion, IRQF_SHARED | IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING, "testing_function", dev); Have in mind that the kernel invokes every handler registered for that IRQ every time an interrupt occurs on that line. So you have to check in each ISR if the hardware raised the interrupt. Hope it helps, -- Javier Martínez Canillas (+34) 682 39 81 69 Barcelona, Spain ___ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies