lkml.org 
[lkml]   [2010]   [Sep]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRE: [PATCH resent] lis3: Fix Oops with NULL platform data

Hi

>-----Original Message-----
>From: ext Takashi Iwai [mailto:tiwai@suse.de]
>Sent: 22 September, 2010 14:34
>To: Andrew Morton
>Cc: Onkalo Samu.P (Nokia-MS/Tampere); Éric Piel; linux-
>kernel@vger.kernel.org
>Subject: [PATCH resent] lis3: Fix Oops with NULL platform data
>
>The recent addition of threaded irq handler causes a NULL dereference
>when used with hp_accel driver, which has NULL pdata.
>
>Cc: <stable@kernel.org>
>Signed-off-by: Takashi Iwai <tiwai@suse.de>
>---
>
>This should go to 2.6.36 and 2.6.35-stable tree.
>
> drivers/hwmon/lis3lv02d.c | 6 ++++--
> 1 files changed, 4 insertions(+), 2 deletions(-)
>
>diff --git a/drivers/hwmon/lis3lv02d.c b/drivers/hwmon/lis3lv02d.c
>index 5e15967..1c266ec 100644
>--- a/drivers/hwmon/lis3lv02d.c
>+++ b/drivers/hwmon/lis3lv02d.c
>@@ -354,7 +354,8 @@ static irqreturn_t lis302dl_interrupt_thread1_8b(int
>irq, void *data)
>
> struct lis3lv02d *lis3 = data;
>
>- if ((lis3->pdata->irq_cfg & LIS3_IRQ1_MASK) == LIS3_IRQ1_CLICK)
>+ if (lis3->pdata &&
>+ (lis3->pdata->irq_cfg & LIS3_IRQ1_MASK) == LIS3_IRQ1_CLICK)
> lis302dl_interrupt_handle_click(lis3);
> else
> lis302dl_interrupt_handle_ff_wu(lis3);
>@@ -367,7 +368,8 @@ static irqreturn_t lis302dl_interrupt_thread2_8b(int
>irq, void *data)
>
> struct lis3lv02d *lis3 = data;
>
>- if ((lis3->pdata->irq_cfg & LIS3_IRQ2_MASK) == LIS3_IRQ2_CLICK)
>+ if (lis3->pdata &&
>+ (lis3->pdata->irq_cfg & LIS3_IRQ2_MASK) == LIS3_IRQ2_CLICK)
> lis302dl_interrupt_handle_click(lis3);
> else
> lis302dl_interrupt_handle_ff_wu(lis3);

Yes, that will remove the kernel oops definitely, but I'm wondering if
this thread should be executed at all if there is no pdata available.
Eric, is it so that in laptops interrupts are used for free fall detection
and polled input device is for joystick?

Interrupts can (and are used) also for other purposes but in that case
some additional configurations are done based on platform data.

With this patch, there are just some additional input event refreshes in
top of polled device updates. It should really not matter. So,

Acked-by: Samu Onkalo <samu.p.onkalo@nokia.com>

-Samu



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2010-09-22 13:59    [W:0.046 / U:0.616 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site