Re: [RFC PATCH] genirq: implement read_irq_line for interrupt lines
From: Abhijeet Dharmapurikar
Date: Fri Apr 15 2011 - 18:08:43 EST
Thomas Gleixner wrote:
On Fri, 15 Apr 2011, Abhijeet Dharmapurikar wrote:
Some drivers need to know what the status of the interrupt line is.
This is especially true for drivers that register a handler with
IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING and in the handler they
need to know which edge transition it was invoked for.
What's the purpose of this? How is that going to be used?
The main purpose is to know in the handler if it was invoked becuase of
a rising edge or the falling edge.
Now one could say that the driver should maintain some state and toggle
it upon each invocation of this handler. That scheme quickly goes out of
sync because we might ignore interrupts while suspended.
That hunk does not apply.
Will rebase and resend it again, my bad.
+int irq_read_line(unsigned int irq)
+ struct irq_desc *desc = irq_to_desc(irq);
+ unsigned long flags;
+ int val;
Please use irq_get_desc_buslock()
yes will fix this in the next revision of this patch.
yes will do
Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm
Innovation Center, Inc. is a member of the Code Aurora Forum.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/