From: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
Jerome Oufella <jerome.oufella@savoirfairelinux.com>,
platform-driver-x86 <platform-driver-x86@vger.kernel.org>,
linux-serial <linux-serial@vger.kernel.org>,
lm-sensors <lm-sensors@lm-sensors.org>
Subject: Re: [RFC 2/5] gpio: add support for Technologic Systems TS-5500 GPIOs
Date: Fri, 13 May 2011 17:33:52 -0400 [thread overview]
Message-ID: <1305322106-sup-3383@v0nbox> (raw)
In-Reply-To: <20110430111536.5cb49d59@lxorguk.ukuu.org.uk>
Excerpts from Alan Cox's message of sam avr 30 06:15:36 -0400 2011:
> > + /* Enable IRQ generation */
> > + mutex_lock(&drvdata->gpio_lock);
> > + PORT_BIT_SET(0x7A, 7); /* DIO1_13 on IRQ7 */
> > + PORT_BIT_SET(0x7D, 7); /* DIO2_13 on IRQ6 */
> > + if (use_lcdio) {
> > + PORT_BIT_CLEAR(0x7D, 4); /* Enable LCD header usage as DIO */
> > + PORT_BIT_SET(0x7D, 6); /* LCD_RS on IRQ1 */
> > + }
>
> What happens if an IRQ occurs at this point, you have no handler for it ?
The IRQ is just not handled. What would be the proper way to handle
that? Would it be possible to write those registers when the IRQ is
requested?
Thanks,
Vivien.
WARNING: multiple messages have this Message-ID (diff)
From: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
Jerome Oufella <jerome.oufella@savoirfairelinux.com>,
platform-driver-x86 <platform-driver-x86@vger.kernel.org>,
linux-serial <linux-serial@vger.kernel.org>,
lm-sensors <lm-sensors@lm-sensors.org>
Subject: Re: [lm-sensors] [RFC 2/5] gpio: add support for Technologic
Date: Fri, 13 May 2011 21:33:52 +0000 [thread overview]
Message-ID: <1305322106-sup-3383@v0nbox> (raw)
In-Reply-To: <20110430111536.5cb49d59@lxorguk.ukuu.org.uk>
Excerpts from Alan Cox's message of sam avr 30 06:15:36 -0400 2011:
> > + /* Enable IRQ generation */
> > + mutex_lock(&drvdata->gpio_lock);
> > + PORT_BIT_SET(0x7A, 7); /* DIO1_13 on IRQ7 */
> > + PORT_BIT_SET(0x7D, 7); /* DIO2_13 on IRQ6 */
> > + if (use_lcdio) {
> > + PORT_BIT_CLEAR(0x7D, 4); /* Enable LCD header usage as DIO */
> > + PORT_BIT_SET(0x7D, 6); /* LCD_RS on IRQ1 */
> > + }
>
> What happens if an IRQ occurs at this point, you have no handler for it ?
The IRQ is just not handled. What would be the proper way to handle
that? Would it be possible to write those registers when the IRQ is
requested?
Thanks,
Vivien.
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
next prev parent reply other threads:[~2011-05-13 21:33 UTC|newest]
Thread overview: 88+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-29 22:21 [RFC 0/5] Support for Technologic Systems TS-5500 board Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` [lm-sensors] " Vivien Didelot
2011-04-29 22:21 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Vivien Didelot
2011-04-29 23:32 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Greg KH
2011-04-29 23:32 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Greg KH
2011-05-02 21:07 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Vivien Didelot
2011-05-02 21:07 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Vivien Didelot
2011-05-02 21:55 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Greg KH
2011-05-02 21:55 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Greg KH
2011-05-03 9:39 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Alan Cox
2011-05-03 9:39 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Alan Cox
2011-05-03 14:13 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Greg KH
2011-05-03 14:13 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Greg KH
2011-04-30 10:07 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Alan Cox
2011-04-30 10:07 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Alan Cox
2011-05-04 15:15 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Vivien Didelot
2011-05-04 15:15 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Vivien Didelot
2011-05-04 15:29 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Alan Cox
2011-05-04 15:29 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Alan Cox
2011-05-04 20:34 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Vivien Didelot
2011-05-04 20:34 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Vivien Didelot
2011-05-05 13:38 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Alan Cox
2011-05-05 13:38 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Alan Cox
2011-05-11 22:24 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Vivien Didelot
2011-05-11 22:24 ` [lm-sensors] [RFC 1/5] platform-drivers-x86: add support for Vivien Didelot
2011-05-17 12:37 ` [RFC 1/5] platform-drivers-x86: add support for Technologic Systems TS-5xxx detection Sean Young
2011-05-18 15:03 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` [RFC 2/5] gpio: add support for Technologic Systems TS-5500 GPIOs Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` [lm-sensors] [RFC 2/5] gpio: add support for Technologic Systems Vivien Didelot
2011-04-30 10:15 ` [RFC 2/5] gpio: add support for Technologic Systems TS-5500 GPIOs Alan Cox
2011-04-30 10:15 ` [lm-sensors] [RFC 2/5] gpio: add support for Technologic Alan Cox
2011-05-13 21:33 ` Vivien Didelot [this message]
2011-05-13 21:33 ` Vivien Didelot
2011-05-13 22:03 ` [RFC 2/5] gpio: add support for Technologic Systems TS-5500 GPIOs Alan Cox
2011-05-13 22:03 ` [lm-sensors] [RFC 2/5] gpio: add support for Technologic Alan Cox
2011-05-04 16:29 ` [RFC 2/5] gpio: add support for Technologic Systems TS-5500 GPIOs Arnd Bergmann
2011-05-04 16:29 ` [lm-sensors] [RFC 2/5] gpio: add support for Technologic Arnd Bergmann
2011-04-29 22:21 ` [RFC 3/5] serial: add support for Technologic Systems TS-5500 RS-485 serial port Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` [lm-sensors] [RFC 3/5] serial: add support for Technologic Systems Vivien Didelot
2011-04-30 10:17 ` [RFC 3/5] serial: add support for Technologic Systems TS-5500 RS-485 serial port Alan Cox
2011-04-30 10:17 ` [lm-sensors] [RFC 3/5] serial: add support for Technologic Alan Cox
2011-06-06 20:48 ` [RFC 3/5] serial: add support for Technologic Systems TS-5500 RS-485 serial port Vivien Didelot
2011-06-06 20:48 ` [lm-sensors] [RFC 3/5] serial: add support for Technologic Vivien Didelot
2011-04-29 22:21 ` [RFC 3/5] serial: add support for Technologic Systems TS-5500 RS-485 serial port Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` [RFC 4/5] leds: add support for Technologic Systems TS-5500 leds Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` [lm-sensors] [RFC 4/5] leds: add support for Technologic Systems Vivien Didelot
2011-05-03 6:04 ` [RFC 4/5] leds: add support for Technologic Systems TS-5500 leds Govindraj
2011-05-03 6:16 ` [lm-sensors] [RFC 4/5] leds: add support for Technologic Govindraj
2011-04-29 22:21 ` [RFC 4/5] leds: add support for Technologic Systems TS-5500 leds Vivien Didelot
2011-04-29 22:21 ` [RFC 5/5] hwmon: add support for Technologic Systems TS-5500 A-D converter Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` Vivien Didelot
2011-04-29 22:21 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Systems Vivien Didelot
2011-04-30 3:39 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Systems TS-5500 A-D converter Guenter Roeck
2011-04-30 3:39 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Guenter Roeck
2011-04-30 3:39 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Systems TS-5500 A-D converter Guenter Roeck
2011-04-30 9:56 ` Jonathan Cameron
2011-04-30 9:56 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Jonathan Cameron
2011-05-03 15:55 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Systems TS-5500 A-D converter Vivien Didelot
2011-05-03 15:55 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Vivien Didelot
2011-05-03 17:33 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Systems TS-5500 A-D converter Guenter Roeck
2011-05-03 17:33 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Guenter Roeck
2011-05-03 17:33 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Systems TS-5500 A-D converter Guenter Roeck
2011-05-04 9:03 ` Jonathan Cameron
2011-05-04 9:03 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Jonathan Cameron
2011-05-04 9:03 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Systems TS-5500 A-D converter Jonathan Cameron
2011-06-06 22:56 ` Vivien Didelot
2011-06-07 8:37 ` Jonathan Cameron
2011-04-30 10:20 ` Alan Cox
2011-04-30 10:20 ` [lm-sensors] [RFC 5/5] hwmon: add support for Technologic Alan Cox
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1305322106-sup-3383@v0nbox \
--to=vivien.didelot@savoirfairelinux.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=jerome.oufella@savoirfairelinux.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=lm-sensors@lm-sensors.org \
--cc=platform-driver-x86@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.