* [PATCH] gpio: show IRQ settings even if GPIO direction is "output"
@ 2009-11-17 11:31 Rodolfo Giometti
2009-11-17 11:45 ` Janakiram Sistla
0 siblings, 1 reply; 3+ messages in thread
From: Rodolfo Giometti @ 2009-11-17 11:31 UTC (permalink / raw)
To: linux-kernel; +Cc: Haavard Skinnemoen, Rodolfo Giometti, raffaele.recalcati
Some architectures (as PXA 27x for example) may define GPIO IRQ lines
as outputs since internal circuitry supports such configuration.
This patch simply detects if a GPIO is a IRQ line or not just looking
for IRQ line number, "desc" and "desc->action" fields.
Signed-off-by: Rodolfo Giometti <giometti@linux.it>
Cc: raffaele.recalcati@bticino.it
---
drivers/gpio/gpiolib.c | 82 ++++++++++++++++++++++++------------------------
1 files changed, 41 insertions(+), 41 deletions(-)
diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
index 50de0f5..298b66f 100644
--- a/drivers/gpio/gpiolib.c
+++ b/drivers/gpio/gpiolib.c
@@ -1381,6 +1381,8 @@ static void gpiolib_dbg_show(struct seq_file *s, struct gpio_chip *chip)
unsigned gpio = chip->base;
struct gpio_desc *gdesc = &gpio_desc[gpio];
int is_out;
+ int irq;
+ struct irq_desc *desc;
for (i = 0; i < chip->ngpio; i++, gpio++, gdesc++) {
if (!test_bit(FLAG_REQUESTED, &gdesc->flags))
@@ -1394,48 +1396,46 @@ static void gpiolib_dbg_show(struct seq_file *s, struct gpio_chip *chip)
? (chip->get(chip, i) ? "hi" : "lo")
: "? ");
- if (!is_out) {
- int irq = gpio_to_irq(gpio);
- struct irq_desc *desc = irq_to_desc(irq);
-
- /* This races with request_irq(), set_irq_type(),
- * and set_irq_wake() ... but those are "rare".
- *
- * More significantly, trigger type flags aren't
- * currently maintained by genirq.
- */
- if (irq >= 0 && desc->action) {
- char *trigger;
-
- switch (desc->status & IRQ_TYPE_SENSE_MASK) {
- case IRQ_TYPE_NONE:
- trigger = "(default)";
- break;
- case IRQ_TYPE_EDGE_FALLING:
- trigger = "edge-falling";
- break;
- case IRQ_TYPE_EDGE_RISING:
- trigger = "edge-rising";
- break;
- case IRQ_TYPE_EDGE_BOTH:
- trigger = "edge-both";
- break;
- case IRQ_TYPE_LEVEL_HIGH:
- trigger = "level-high";
- break;
- case IRQ_TYPE_LEVEL_LOW:
- trigger = "level-low";
- break;
- default:
- trigger = "?trigger?";
- break;
- }
-
- seq_printf(s, " irq-%d %s%s",
- irq, trigger,
- (desc->status & IRQ_WAKEUP)
- ? " wakeup" : "");
+ irq = gpio_to_irq(gpio);
+ desc = irq_to_desc(irq);
+
+ /* This races with request_irq(), set_irq_type(),
+ * and set_irq_wake() ... but those are "rare".
+ *
+ * More significantly, trigger type flags aren't
+ * currently maintained by genirq.
+ */
+ if (irq >= 0 && desc && desc->action) {
+ char *trigger;
+
+ switch (desc->status & IRQ_TYPE_SENSE_MASK) {
+ case IRQ_TYPE_NONE:
+ trigger = "(default)";
+ break;
+ case IRQ_TYPE_EDGE_FALLING:
+ trigger = "edge-falling";
+ break;
+ case IRQ_TYPE_EDGE_RISING:
+ trigger = "edge-rising";
+ break;
+ case IRQ_TYPE_EDGE_BOTH:
+ trigger = "edge-both";
+ break;
+ case IRQ_TYPE_LEVEL_HIGH:
+ trigger = "level-high";
+ break;
+ case IRQ_TYPE_LEVEL_LOW:
+ trigger = "level-low";
+ break;
+ default:
+ trigger = "?trigger?";
+ break;
}
+
+ seq_printf(s, " irq-%d %s%s",
+ irq, trigger,
+ (desc->status & IRQ_WAKEUP)
+ ? " wakeup" : "");
}
seq_printf(s, "\n");
--
1.6.3.3
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] gpio: show IRQ settings even if GPIO direction is "output"
2009-11-17 11:31 [PATCH] gpio: show IRQ settings even if GPIO direction is "output" Rodolfo Giometti
@ 2009-11-17 11:45 ` Janakiram Sistla
2009-11-17 12:10 ` Rodolfo Giometti
0 siblings, 1 reply; 3+ messages in thread
From: Janakiram Sistla @ 2009-11-17 11:45 UTC (permalink / raw)
To: Rodolfo Giometti; +Cc: linux-kernel, Haavard Skinnemoen, raffaele.recalcati
On Tue, Nov 17, 2009 at 5:01 PM, Rodolfo Giometti <giometti@linux.it> wrote:> Some architectures (as PXA 27x for example) may define GPIO IRQ lines> as outputs since internal circuitry supports such configuration.>> This patch simply detects if a GPIO is a IRQ line or not just looking> for IRQ line number, "desc" and "desc->action" fields.>> Signed-off-by: Rodolfo Giometti <giometti@linux.it>> Cc: raffaele.recalcati@bticino.it> ---> drivers/gpio/gpiolib.c | 82 ++++++++++++++++++++++++------------------------> 1 files changed, 41 insertions(+), 41 deletions(-)>> diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c> index 50de0f5..298b66f 100644> --- a/drivers/gpio/gpiolib.c> +++ b/drivers/gpio/gpiolib.c> @@ -1381,6 +1381,8 @@ static void gpiolib_dbg_show(struct seq_file *s, struct gpio_chip *chip)> unsigned gpio = chip->base;> struct gpio_desc *gdesc = &gpio_desc[gpio];> int is_out;> + int irq;> + struct irq_desc *desc;>> for (i = 0; i < chip->ngpio; i++, gpio++, gdesc++) {> if (!test_bit(FLAG_REQUESTED, &gdesc->flags))> @@ -1394,48 +1396,46 @@ static void gpiolib_dbg_show(struct seq_file *s, struct gpio_chip *chip)> ? (chip->get(chip, i) ? "hi" : "lo")> : "? ");>> - if (!is_out) {> - int irq = gpio_to_irq(gpio);> - struct irq_desc *desc = irq_to_desc(irq);> -> - /* This races with request_irq(), set_irq_type(),> - * and set_irq_wake() ... but those are "rare".> - *> - * More significantly, trigger type flags aren't> - * currently maintained by genirq.> - */> - if (irq >= 0 && desc->action) {> - char *trigger;> -> - switch (desc->status & IRQ_TYPE_SENSE_MASK) {> - case IRQ_TYPE_NONE:> - trigger = "(default)";> - break;> - case IRQ_TYPE_EDGE_FALLING:> - trigger = "edge-falling";> - break;> - case IRQ_TYPE_EDGE_RISING:> - trigger = "edge-rising";> - break;> - case IRQ_TYPE_EDGE_BOTH:> - trigger = "edge-both";> - break;> - case IRQ_TYPE_LEVEL_HIGH:> - trigger = "level-high";> - break;> - case IRQ_TYPE_LEVEL_LOW:> - trigger = "level-low";> - break;> - default:> - trigger = "?trigger?";> - break;> - }> -> - seq_printf(s, " irq-%d %s%s",> - irq, trigger,> - (desc->status & IRQ_WAKEUP)> - ? " wakeup" : "");> + irq = gpio_to_irq(gpio);> + desc = irq_to_desc(irq);> +> + /* This races with request_irq(), set_irq_type(),> + * and set_irq_wake() ... but those are "rare".> + *> + * More significantly, trigger type flags aren't> + * currently maintained by genirq.> + */> + if (irq >= 0 && desc && desc->action) {> + char *trigger;> +> + switch (desc->status & IRQ_TYPE_SENSE_MASK) {> + case IRQ_TYPE_NONE:> + trigger = "(default)";> + break;> + case IRQ_TYPE_EDGE_FALLING:> + trigger = "edge-falling";> + break;> + case IRQ_TYPE_EDGE_RISING:> + trigger = "edge-rising";> + break;> + case IRQ_TYPE_EDGE_BOTH:> + trigger = "edge-both";> + break;> + case IRQ_TYPE_LEVEL_HIGH:> + trigger = "level-high";> + break;> + case IRQ_TYPE_LEVEL_LOW:> + trigger = "level-low";> + break;> + default:> + trigger = "?trigger?";> + break;> }> +> + seq_printf(s, " irq-%d %s%s",> + irq, trigger,> + (desc->status & IRQ_WAKEUP)> + ? " wakeup" : "");> }>> seq_printf(s, "\n");> --> 1.6.3.3
Is there any valid test scenario that makes use of thispatch???Detecting GPIO if it is IRQ in kernel what is the advantage?
Regards,Ram.ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a¢f£¢·hšïêÿ‘êçz_è®\x03(階ŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨èÚ&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] gpio: show IRQ settings even if GPIO direction is "output"
2009-11-17 11:45 ` Janakiram Sistla
@ 2009-11-17 12:10 ` Rodolfo Giometti
0 siblings, 0 replies; 3+ messages in thread
From: Rodolfo Giometti @ 2009-11-17 12:10 UTC (permalink / raw)
To: Janakiram Sistla; +Cc: linux-kernel, Haavard Skinnemoen, raffaele.recalcati
On Tue, Nov 17, 2009 at 05:15:49PM +0530, Janakiram Sistla wrote:
> On Tue, Nov 17, 2009 at 5:01 PM, Rodolfo Giometti <giometti@linux.it> wrote:
> > Some architectures (as PXA 27x for example) may define GPIO IRQ lines
> > as outputs since internal circuitry supports such configuration.
> >
> > This patch simply detects if a GPIO is a IRQ line or not just looking
> > for IRQ line number, "desc" and "desc->action" fields.
> >
> > Signed-off-by: Rodolfo Giometti <giometti@linux.it>
> > Cc: raffaele.recalcati@bticino.it
> > ---
> > drivers/gpio/gpiolib.c | 82 ++++++++++++++++++++++++------------------------
> > 1 files changed, 41 insertions(+), 41 deletions(-)
> >
> > diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
> > index 50de0f5..298b66f 100644
> > --- a/drivers/gpio/gpiolib.c
> > +++ b/drivers/gpio/gpiolib.c
> > @@ -1381,6 +1381,8 @@ static void gpiolib_dbg_show(struct seq_file *s, struct gpio_chip *chip)
> > unsigned gpio = chip->base;
> > struct gpio_desc *gdesc = &gpio_desc[gpio];
> > int is_out;
> > + int irq;
> > + struct irq_desc *desc;
> >
> > for (i = 0; i < chip->ngpio; i++, gpio++, gdesc++) {
> > if (!test_bit(FLAG_REQUESTED, &gdesc->flags))
> > @@ -1394,48 +1396,46 @@ static void gpiolib_dbg_show(struct seq_file *s, struct gpio_chip *chip)
> > ? (chip->get(chip, i) ? "hi" : "lo")
> > : "? ");
> >
> > - if (!is_out) {
> > - int irq = gpio_to_irq(gpio);
> > - struct irq_desc *desc = irq_to_desc(irq);
> > -
> > - /* This races with request_irq(), set_irq_type(),
> > - * and set_irq_wake() ... but those are "rare".
> > - *
> > - * More significantly, trigger type flags aren't
> > - * currently maintained by genirq.
> > - */
> > - if (irq >= 0 && desc->action) {
> > - char *trigger;
> > -
> > - switch (desc->status & IRQ_TYPE_SENSE_MASK) {
> > - case IRQ_TYPE_NONE:
> > - trigger = "(default)";
> > - break;
> > - case IRQ_TYPE_EDGE_FALLING:
> > - trigger = "edge-falling";
> > - break;
> > - case IRQ_TYPE_EDGE_RISING:
> > - trigger = "edge-rising";
> > - break;
> > - case IRQ_TYPE_EDGE_BOTH:
> > - trigger = "edge-both";
> > - break;
> > - case IRQ_TYPE_LEVEL_HIGH:
> > - trigger = "level-high";
> > - break;
> > - case IRQ_TYPE_LEVEL_LOW:
> > - trigger = "level-low";
> > - break;
> > - default:
> > - trigger = "?trigger?";
> > - break;
> > - }
> > -
> > - seq_printf(s, " irq-%d %s%s",
> > - irq, trigger,
> > - (desc->status & IRQ_WAKEUP)
> > - ? " wakeup" : "");
> > + irq = gpio_to_irq(gpio);
> > + desc = irq_to_desc(irq);
> > +
> > + /* This races with request_irq(), set_irq_type(),
> > + * and set_irq_wake() ... but those are "rare".
> > + *
> > + * More significantly, trigger type flags aren't
> > + * currently maintained by genirq.
> > + */
> > + if (irq >= 0 && desc && desc->action) {
> > + char *trigger;
> > +
> > + switch (desc->status & IRQ_TYPE_SENSE_MASK) {
> > + case IRQ_TYPE_NONE:
> > + trigger = "(default)";
> > + break;
> > + case IRQ_TYPE_EDGE_FALLING:
> > + trigger = "edge-falling";
> > + break;
> > + case IRQ_TYPE_EDGE_RISING:
> > + trigger = "edge-rising";
> > + break;
> > + case IRQ_TYPE_EDGE_BOTH:
> > + trigger = "edge-both";
> > + break;
> > + case IRQ_TYPE_LEVEL_HIGH:
> > + trigger = "level-high";
> > + break;
> > + case IRQ_TYPE_LEVEL_LOW:
> > + trigger = "level-low";
> > + break;
> > + default:
> > + trigger = "?trigger?";
> > + break;
> > }
> > +
> > + seq_printf(s, " irq-%d %s%s",
> > + irq, trigger,
> > + (desc->status & IRQ_WAKEUP)
> > + ? " wakeup" : "");
> > }
> >
> > seq_printf(s, "\n");
> > --
> > 1.6.3.3
>
> Is there any valid test scenario that makes use of this
> patch???Detecting GPIO if it is IRQ in kernel what is the advantage?
My PXA270 based board uses a passive electrical circuit to detect a
CARD insert supplied by GPIO output current, so, in order to work, I
need to setup the GPIO as OUTPUT high level and then, thanks to a pull
down, I can read the CARD insert/removal event.
My solution is more generic then current one and applyes to those
architectures whose support such kind of GPIOs (as PXA27x does) with
no drawbacks for others.
Ciao,
Rodolfo
--
GNU/Linux Solutions e-mail: giometti@enneenne.com
Linux Device Driver giometti@linux.it
Embedded Systems phone: +39 349 2432127
UNIX programming skype: rodolfo.giometti
Freelance ICT Italia - Consulente ICT Italia - www.consulenti-ict.it
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2009-11-17 12:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-17 11:31 [PATCH] gpio: show IRQ settings even if GPIO direction is "output" Rodolfo Giometti
2009-11-17 11:45 ` Janakiram Sistla
2009-11-17 12:10 ` Rodolfo Giometti
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome