* [PATCH 1/2] Input: tsc2007: fix locking method against the current rc kernel
@ 2009-05-11 11:41 ` Kwangwoo Lee
0 siblings, 0 replies; 5+ messages in thread
From: Kwangwoo Lee @ 2009-05-11 11:41 UTC (permalink / raw)
To: Dmitry Torokhov, Thierry Reding
Cc: Trilok Soni, linux-input, linux-kernel, linux-omap
Hi Dmitry,
The following patch fixes bug in tsc2007.
It is not working correctly in the current rc kernels.
IMHO, this patch should go in the 2.6.30
Can you advise me about "From: email" part?
I splited Thierry's patch. So the original "From: email" is followed:
From: Thierry Reding <thierry.reding@avionic-design.de>
Thank you,
Kwangwoo Lee
>From 749ee3a4a2c2fe1f41601481e47cb26d8ccec20d Mon Sep 17 00:00:00 2001
From: Kwangwoo Lee <kwangwoo.lee@gmail.com>
Date: Mon, 11 May 2009 20:00:01 +0900
Subject: [PATCH 1/2] Input: tsc2007: fix locking method against the
current rc kernel.
Without this patch, current driver will emit panic message below:
Kernel panic - not syncing: Fatal exception in interrupt.
The original patch is written by Thierry.
Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
Signed-off-by: Kwangwoo Lee <kwangwoo.lee@gmail.com>
---
drivers/input/touchscreen/tsc2007.c | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/input/touchscreen/tsc2007.c
b/drivers/input/touchscreen/tsc2007.c
index 536668f..948e167 100644
--- a/drivers/input/touchscreen/tsc2007.c
+++ b/drivers/input/touchscreen/tsc2007.c
@@ -200,8 +200,9 @@ static int tsc2007_read_values(struct tsc2007 *tsc)
static enum hrtimer_restart tsc2007_timer(struct hrtimer *handle)
{
struct tsc2007 *ts = container_of(handle, struct tsc2007, timer);
+ unsigned long flags;
- spin_lock_irq(&ts->lock);
+ spin_lock_irqsave(&ts->lock, flags);
if (unlikely(!ts->get_pendown_state() && ts->pendown)) {
struct input_dev *input = ts->input;
@@ -222,7 +223,7 @@ static enum hrtimer_restart tsc2007_timer(struct
hrtimer *handle)
tsc2007_send_event(ts);
}
- spin_unlock_irq(&ts->lock);
+ spin_unlock_irqrestore(&ts->lock, flags);
return HRTIMER_NORESTART;
}
--
1.5.6.5
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH 1/2] Input: tsc2007: fix locking method against the current rc kernel
@ 2009-05-11 11:41 ` Kwangwoo Lee
0 siblings, 0 replies; 5+ messages in thread
From: Kwangwoo Lee @ 2009-05-11 11:41 UTC (permalink / raw)
To: Dmitry Torokhov, Thierry Reding
Cc: Trilok Soni, linux-input, linux-kernel, linux-omap
Hi Dmitry,
The following patch fixes bug in tsc2007.
It is not working correctly in the current rc kernels.
IMHO, this patch should go in the 2.6.30
Can you advise me about "From: email" part?
I splited Thierry's patch. So the original "From: email" is followed:
From: Thierry Reding <thierry.reding@avionic-design.de>
Thank you,
Kwangwoo Lee
>From 749ee3a4a2c2fe1f41601481e47cb26d8ccec20d Mon Sep 17 00:00:00 2001
From: Kwangwoo Lee <kwangwoo.lee@gmail.com>
Date: Mon, 11 May 2009 20:00:01 +0900
Subject: [PATCH 1/2] Input: tsc2007: fix locking method against the
current rc kernel.
Without this patch, current driver will emit panic message below:
Kernel panic - not syncing: Fatal exception in interrupt.
The original patch is written by Thierry.
Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
Signed-off-by: Kwangwoo Lee <kwangwoo.lee@gmail.com>
---
drivers/input/touchscreen/tsc2007.c | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/input/touchscreen/tsc2007.c
b/drivers/input/touchscreen/tsc2007.c
index 536668f..948e167 100644
--- a/drivers/input/touchscreen/tsc2007.c
+++ b/drivers/input/touchscreen/tsc2007.c
@@ -200,8 +200,9 @@ static int tsc2007_read_values(struct tsc2007 *tsc)
static enum hrtimer_restart tsc2007_timer(struct hrtimer *handle)
{
struct tsc2007 *ts = container_of(handle, struct tsc2007, timer);
+ unsigned long flags;
- spin_lock_irq(&ts->lock);
+ spin_lock_irqsave(&ts->lock, flags);
if (unlikely(!ts->get_pendown_state() && ts->pendown)) {
struct input_dev *input = ts->input;
@@ -222,7 +223,7 @@ static enum hrtimer_restart tsc2007_timer(struct
hrtimer *handle)
tsc2007_send_event(ts);
}
- spin_unlock_irq(&ts->lock);
+ spin_unlock_irqrestore(&ts->lock, flags);
return HRTIMER_NORESTART;
}
--
1.5.6.5
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH 1/2] Input: tsc2007: fix locking method against the current rc kernel
2009-05-11 11:41 ` Kwangwoo Lee
(?)
@ 2009-05-12 1:50 ` Dmitry Torokhov
2009-05-12 2:07 ` Kwangwoo Lee
-1 siblings, 1 reply; 5+ messages in thread
From: Dmitry Torokhov @ 2009-05-12 1:50 UTC (permalink / raw)
To: Kwangwoo Lee
Cc: Thierry Reding, Trilok Soni, linux-input, linux-kernel,
linux-omap
Hi Kwangwoo,
On Monday 11 May 2009 04:41:35 Kwangwoo Lee wrote:
> Hi Dmitry,
>
> The following patch fixes bug in tsc2007.
> It is not working correctly in the current rc kernels.
> IMHO, this patch should go in the 2.6.30
>
I agree, it is in my for-linus branch now.
> Can you advise me about "From: email" part?
> I splited Thierry's patch. So the original "From: email" is followed:
>
> From: Thierry Reding <thierry.reding@avionic-design.de>
>
> Thank you,
> Kwangwoo Lee
>
> From 749ee3a4a2c2fe1f41601481e47cb26d8ccec20d Mon Sep 17 00:00:00 2001
> From: Kwangwoo Lee <kwangwoo.lee@gmail.com>
This is the last "from" tag in the e-mail so my scripts picked that up.
I adjusted the commit so now it shows Thierry as the original author.
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] Input: tsc2007: fix locking method against the current rc kernel
2009-05-12 1:50 ` Dmitry Torokhov
@ 2009-05-12 2:07 ` Kwangwoo Lee
0 siblings, 0 replies; 5+ messages in thread
From: Kwangwoo Lee @ 2009-05-12 2:07 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: Thierry Reding, Trilok Soni, linux-input, linux-kernel,
linux-omap
Hi,
On Tue, May 12, 2009 at 10:50 AM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
> Hi Kwangwoo,
>
> On Monday 11 May 2009 04:41:35 Kwangwoo Lee wrote:
>> Hi Dmitry,
>>
>> The following patch fixes bug in tsc2007.
>> It is not working correctly in the current rc kernels.
>> IMHO, this patch should go in the 2.6.30
>>
>
> I agree, it is in my for-linus branch now.
>
>> Can you advise me about "From: email" part?
>> I splited Thierry's patch. So the original "From: email" is followed:
>>
>> From: Thierry Reding <thierry.reding@avionic-design.de>
>>
>> Thank you,
>> Kwangwoo Lee
>>
>> From 749ee3a4a2c2fe1f41601481e47cb26d8ccec20d Mon Sep 17 00:00:00 2001
>> From: Kwangwoo Lee <kwangwoo.lee@gmail.com>
>
> This is the last "from" tag in the e-mail so my scripts picked that up.
> I adjusted the commit so now it shows Thierry as the original author.
That's good. Thanks.
--
Kwangwoo Lee <kwangwoo.lee@gmail.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] Input: tsc2007: fix locking method against the current rc kernel
@ 2009-05-12 2:07 ` Kwangwoo Lee
0 siblings, 0 replies; 5+ messages in thread
From: Kwangwoo Lee @ 2009-05-12 2:07 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: Thierry Reding, Trilok Soni, linux-input, linux-kernel,
linux-omap
Hi,
On Tue, May 12, 2009 at 10:50 AM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
> Hi Kwangwoo,
>
> On Monday 11 May 2009 04:41:35 Kwangwoo Lee wrote:
>> Hi Dmitry,
>>
>> The following patch fixes bug in tsc2007.
>> It is not working correctly in the current rc kernels.
>> IMHO, this patch should go in the 2.6.30
>>
>
> I agree, it is in my for-linus branch now.
>
>> Can you advise me about "From: email" part?
>> I splited Thierry's patch. So the original "From: email" is followed:
>>
>> From: Thierry Reding <thierry.reding@avionic-design.de>
>>
>> Thank you,
>> Kwangwoo Lee
>>
>> From 749ee3a4a2c2fe1f41601481e47cb26d8ccec20d Mon Sep 17 00:00:00 2001
>> From: Kwangwoo Lee <kwangwoo.lee@gmail.com>
>
> This is the last "from" tag in the e-mail so my scripts picked that up.
> I adjusted the commit so now it shows Thierry as the original author.
That's good. Thanks.
--
Kwangwoo Lee <kwangwoo.lee@gmail.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-05-12 2:08 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-05-11 11:41 [PATCH 1/2] Input: tsc2007: fix locking method against the current rc kernel Kwangwoo Lee
2009-05-11 11:41 ` Kwangwoo Lee
2009-05-12 1:50 ` Dmitry Torokhov
2009-05-12 2:07 ` Kwangwoo Lee
2009-05-12 2:07 ` Kwangwoo Lee
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.