Re: [lm-sensors] [PATCH v2] lis3: Fix Oops with NULL platform data

From: Jean Delvare
Date: Fri Sep 24 2010 - 10:30:30 EST


On Fri, 24 Sep 2010 16:14:26 +0200, Takashi Iwai wrote:
> At Fri, 24 Sep 2010 06:58:02 -0700,
> Guenter Roeck wrote:
> >
> > On Thu, Sep 23, 2010 at 01:58:33AM -0400, Takashi Iwai wrote:
> > > The recent addition of threaded irq handler causes a NULL dereference
> > > when used with hp_accel driver, which has NULL pdata.
> > >
> > > Acked-by: Samu Onkalo <samu.p.onkalo@xxxxxxxxx>
> > > Acked-by: Acked-by: Guenter Roeck <guenter.roeck@xxxxxxxxxxxx>
> > > Cc: <stable@xxxxxxxxxx>
> > > Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
> >
> > Applied.
>
> Oh, does this driver belong to hwmon tree?
> Jean told me that this isn't managed in hwmon, so I had an impression
> that it's an orphan driver.

Well, as long as the driver lives in drivers/hwmon and the hwmon
subsystem is maintained, it seems fair to consider the driver
maintained.

That being said, I indeed would like all non-hwmon drivers to go away
from drivers/hwmon. Originally we were waiting for iio to settle first,
but apparently this is taking forever. The 4 drivers I would like to
kick are: ams, hdaps, lis3lv02d and applesmc. They are primarily
accelerometer device drivers. Not sure where to put them,
drivers/accel(erometer), drivers/misc, drivers/misc/accel(erometer),
drivers/input/accel(erometer)... Opinions welcome.

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