Messages in this thread | | | From | <> | Date | Wed, 22 Sep 2010 13:56:15 +0200 | Subject | RE: [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/
| |