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

When I have time yes for sure...:D

Denis

On 09/10/14 15:36, Pavel Machek wrote:

Hi!

quoted

I'm sorry but I was in business trip. I think that current lis3lv02d
supports more things, I want to support everything in the current IIO
drivers but always I have no time.

Ok then, can you add hwmon interfaces from the other driver to your one? That way,
we can deprecate one of them...

									Pavel

quoted

BR,
Denis


On 08/27/14 15:38, Linus Walleij wrote:

quoted

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

quoted

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

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.

quoted

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