Re: [PATCH] iio: st-sensors: add support for lis3lv02d accelerometer

On Mon, Aug 25, 2014 at 6:19 PM, Jonathan Cameron [off-list ref] wrote:

On 23/08/14 18:31, Linus Walleij wrote:

quoted

This adds support for the LIS3LV02 accelerometer found in the
ST Microelectronics Nomadik board series.

Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: Denis CIOCCA <redacted>
Cc: Lee Jones <redacted>
Signed-off-by: Linus Walleij <redacted>

This is a little 'interesting', given the part is already supported by
drivers/misc/lis3lv02d*.c and already has bindings under there.
Documentation/devicetree/bindings/misc/lis302.txt
(some of these are interesting).

Grrr how come I missed this... It was originally merged as a
hwmon driver.

quoted

+/* CUSTOM VALUES FOR SENSOR 4 */

I wish now we'd nipped this naming convention in the bud and insisted
on naming after a relevant part. Oh well.

Well we can always make an invasive patch fixing that. I can
probably do it using sed or something.

Do we think this is the right way to handle the alternative support
here?

Well this patch is a driver using the IIO framework and was written
by ST, but the other driver support a few other interesting features
and seems to be for the laptop usecase.

Adding the other authors to the TO: line...

Yours,
Linus Walleij