From: John Stultz <john.stultz@linaro.org>
To: ilkka.koskinen@nokia.com
Cc: tony@atomide.com, a.zummo@towertech.it,
rtc-linux@googlegroups.com, linux-omap@vger.kernel.org,
balajitk@ti.com, ilkkakos@kolumbus.fi
Subject: RE: [PATCH] rtc-twl: Switch to using threaded irq
Date: Thu, 12 May 2011 16:24:24 -0700 [thread overview]
Message-ID: <1305242664.2680.28.camel@work-vm> (raw)
In-Reply-To: <4F5287B0C366D047BF44025F9ED7BB43E9D7@008-AM1MPN1-001.mgdnok.nokia.com>
On Thu, 2011-05-05 at 07:51 +0000, ilkka.koskinen@nokia.com wrote:
> Hi,
>
> Tony and John: What would be the appropriate path for this patch?
I'd probably push it through omap maintainer path, as its hardware
specific and can be better tested there.
thanks
-john
> On Apr 13, 2011 Krishnamoorthy, Balaji T wrote:
> >On Wed, Mar 16, 2011 at 9:37 PM, Ilkka Koskinen
> <ilkka.koskinen@nokia.com> wrote:
> >>
> >> The driver is accessing to i2c bus in interrupt handler.
> >> Therefore, it should use threaded irq.
>
> >Acked-by: Balaji T K <balajitk@ti.com>
>
> >>
> >> Signed-off-by: Ilkka Koskinen <ilkka.koskinen@nokia.com>
> >> ---
> >> drivers/rtc/rtc-twl.c | 2 +-
> >> 1 files changed, 1 insertions(+), 1 deletions(-)
> >>
> >> diff --git a/drivers/rtc/rtc-twl.c b/drivers/rtc/rtc-twl.c
> >> index ed1b868..2715b96 100644
> >> --- a/drivers/rtc/rtc-twl.c
> >> +++ b/drivers/rtc/rtc-twl.c
> >> @@ -475,7 +475,7 @@ static int __devinit twl_rtc_probe(struct platform_device *pdev)
> >> if (ret < 0)
> >> goto out1;
> >>
> >> - ret = request_irq(irq, twl_rtc_interrupt,
> >> + ret = request_threaded_irq(irq, NULL, twl_rtc_interrupt,
> >> IRQF_TRIGGER_RISING,
> >> dev_name(&rtc->dev), rtc);
> >> if (ret < 0) {
> >> --
> >> 1.7.0.4
> >>
next prev parent reply other threads:[~2011-05-12 23:24 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-16 16:07 [PATCH] rtc-twl: Switch to using threaded irq Ilkka Koskinen
2011-04-13 7:41 ` Krishnamoorthy, Balaji T
2011-05-05 7:51 ` ilkka.koskinen
2011-05-12 23:24 ` John Stultz [this message]
2011-05-18 19:20 ` Ilkka Koskinen
2011-05-31 8:51 ` Sebastian Reichel
2011-06-27 16:45 ` Sebastian Reichel
2011-08-10 22:44 ` John Stultz
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=1305242664.2680.28.camel@work-vm \
--to=john.stultz@linaro.org \
--cc=a.zummo@towertech.it \
--cc=balajitk@ti.com \
--cc=ilkka.koskinen@nokia.com \
--cc=ilkkakos@kolumbus.fi \
--cc=linux-omap@vger.kernel.org \
--cc=rtc-linux@googlegroups.com \
--cc=tony@atomide.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).