Hi! On 01/26/2017 09:33 AM, Daniel Baluta wrote: > On Thu, Jan 26, 2017 at 9:23 AM, Florian Vaussard > <florian.vaussard@xxxxxxxxx> wrote: >> Hello, >> >> On 01/25/2017 10:25 AM, Eva Rachel Retuya wrote: >>> Add basic IIO support for the Analog Devices ADXL345 3-axis accelerometer. >>> The datasheet can be found here: >>> http://www.analog.com/media/en/technical-documentation/data-sheets/ADXL345.pdf >>> >> >> There is already a driver in the kernel for this accelerometer (under >> drivers/input/misc/adxl34x*). Maybe it would be worth migrating the exiting >> driver to IIO instead of duplicating the effort? >> >> Your driver appears to support only the I2C variant, whereas the existing driver >> supports both I2C / SPI (hint: I have the SPI variant on a board :)) > > Hi Florian, > > This is our intention, to migrate the ADXL34x input driver to adxl345 > IIO driver, > supporting all the features the input driver is offering. > Great to hear that! > We sent this patch early for review in order to get a feeling of > what IIO/input folks think about this device and if moving it to IIO > is a good thing. > > Glad that you have the SPI variant, perhaps you can help us with > testing the patches :). > I probably missed the RFC tag ;) Sure, I can test with SPI variant when your driver integrates the SPI support. Please CC it to florian.vaussard@xxxxxxxxxx, otherwise I may miss it. Best, Florian -- 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