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

[Denis CIOCCA <denis.ciocca-qxv4g6HH51o@public.gmane.org>]

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 <jic23@kernel.org> wrote:
>> On 23/08/14 18:31, Linus Walleij wrote:
>>> This adds support for the LIS3LV02 accelerometer found in the
>>> ST Microelectronics Nomadik board series.
>>>
>>> Cc: devicetree@vger.kernel.org
>>> Cc: Denis CIOCCA <denis.ciocca@st.com>
>>> Cc: Lee Jones <lee.jones@linaro.org>
>>> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
>> 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.
>
>
>>> +/* 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