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

Ciao guys,

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.

BR,
Denis


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

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@vger.kernel.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