On Mon, Aug 25, 2014 at 6:19 PM, Jonathan Cameron <jic23@xxxxxxxxxx> 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@xxxxxxxxxxxxxxx >> Cc: Denis CIOCCA <denis.ciocca@xxxxxx> >> Cc: Lee Jones <lee.jones@xxxxxxxxxx> >> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
