From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <1506539806.53049.56.camel@linux.intel.com> Subject: Re: [PATCH v3 0/8] iio: accel: kxcjk1003: support Kionix KXTF9 From: Srinivas Pandruvada To: =?UTF-8?Q?Micha=C5=82_Miros=C5=82aw?= , Jonathan Cameron Cc: linux-iio@vger.kernel.org, Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler Date: Wed, 27 Sep 2017 12:16:46 -0700 In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 List-ID: On Sun, 2017-09-17 at 17:00 +0200, Michał Mirosław wrote: > This series adds support for KXTF9 accelerometer. It is mostly > compatible > to KXCJK, but replaces motion direction interrupt with tap detection > (not supported in this implementation, yet). > > First 4 patches are refactorings with no outside-visible effects. > Patch 5 fixes sysfs attribute name of sampling_frequency_avail > to match sampling_frequency attribute it describes. > Patch 6 adds support for KXTF9 accelerometers. > Patch 7 drops -1013 suffix from driver name. > I am reviewing this. I finished first three and will get to the remaining. Thanks, Srinivas > --- > v3: > - renamed samp_freq_avail[] > - dropped direction bits rename > - added driver rename > - rebased against v4.14-rc1 > v2: > - removed register renaming > - split report_motion_event() patch > --- > > Michał Mirosław (8): >   iio: accel: kxcjk1003: refactor ODR setting >   iio: accel: kxcjk1013: fix INT_CTRL/INT_SRC1 bit names >   iio: accel: kxcjk1013: extract report_motion_event() from interrupt >     handler >   iio: accel: kxcjk1013: make sysfs/sampling_frequency_avail dynamic >   iio: accel: kxcjk1013: make sampling_frequency_avail per-type >   iio: accel: kxcjk1013: add support for KXTF9 >   iio: accel: kxcjk1013: remove unused platform data struct >   iio: accel: kxcjk1013: drop variation number from driver's name > >  drivers/iio/accel/Kconfig            |    9 +- >  drivers/iio/accel/Makefile           |    2 +- >  drivers/iio/accel/kxcjk-1013.c       | 1426 ------------------------ > -------- >  drivers/iio/accel/kxcjk.c            | 1514 > ++++++++++++++++++++++++++++++++++ >  include/linux/iio/accel/kxcjk_1013.h |   22 - >  5 files changed, 1519 insertions(+), 1454 deletions(-) >  delete mode 100644 drivers/iio/accel/kxcjk-1013.c >  create mode 100644 drivers/iio/accel/kxcjk.c >  delete mode 100644 include/linux/iio/accel/kxcjk_1013.h >