From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lorenzo Bianconi Subject: Re: [PATCH v3 0/5] Add support to LIS2DW12 accel sensor Date: Mon, 2 Oct 2017 12:37:33 +0200 Message-ID: References: <20170830115043.3738-1-lorenzo.bianconi@st.com> <20170903175210.231b90a8@archlinux> <20171002113312.00000064@huawei.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <20171002113312.00000064-hv44wF8Li93QT0dZR+AlfA@public.gmane.org> Sender: linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jonathan Cameron Cc: Jonathan Cameron , linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Linus Walleij , Lorenzo BIANCONI , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org > On Sun, 1 Oct 2017 15:15:26 +0200 > Lorenzo Bianconi wrote: > >> > On Wed, 30 Aug 2017 13:50:38 +0200 >> > Lorenzo Bianconi wrote: >> > >> >> Modify management of drdy line and status register in order to support >> >> LIS2DW12 device >> >> >> >> Changes since v2: >> >> - swap status register patch (3rd in v2) with irq{1,2} definition one >> >> (1st in v2) >> >> - patchset rebased onto 'fix drdy line configuration for LIS3MDL' patch >> >> Changes since v1: >> >> - remove drdy_irq parameter from LPS001WP settings since the sensor >> >> does not support data-ready interrupts >> >> >> >> Lorenzo Bianconi (5): >> >> iio: st_sensors: add register mask for status register >> >> iio: st_sensors: decouple irq1 configuration parameters from the irq2 >> >> ones >> >> iio: st_sensors: do not always write enable_axis register >> >> iio: accel: add support to LIS2DW12 >> >> dt-bindings: iio: accel: add LIS2DW12 sensor device binding >> >> >> >> .../devicetree/bindings/iio/st-sensors.txt | 1 + >> >> drivers/iio/accel/st_accel.h | 2 + >> >> drivers/iio/accel/st_accel_core.c | 180 +++++++++++++++++---- >> >> drivers/iio/accel/st_accel_i2c.c | 5 + >> >> drivers/iio/accel/st_accel_spi.c | 5 + >> >> drivers/iio/common/st_sensors/st_sensors_core.c | 35 ++-- >> >> drivers/iio/common/st_sensors/st_sensors_trigger.c | 16 +- >> >> drivers/iio/gyro/st_gyro_core.c | 33 ++-- >> >> drivers/iio/magnetometer/st_magn_core.c | 16 +- >> >> drivers/iio/pressure/st_pressure_core.c | 46 ++++-- >> >> include/linux/iio/common/st_sensors.h | 23 ++- >> >> 11 files changed, 270 insertions(+), 92 deletions(-) >> >> >> > >> > I'm happy with the whole series, however it doesn't currently apply >> > due to various fixes working their way through the fixes branches >> > of the iio git tree. I'll try to pick it up once those have unwound. >> > There is too much going on with this code at the moment to do this >> > cleanly now. >> > >> > Give me a bump if I look like I have forgotten it once it will >> > apply cleanly to the togreg branch of iio.git. >> > >> > If I was being really fussy this should have had a dependency list >> > for everything it was dependant on that wasn't currently in mainline. >> > >> > Jonathan >> >> Hi Jonathan, >> >> what about that patchset? >> Regards, >> >> Lorenzo >> > > I think I'm still waiting for the fixes to get back round via mainline > to our upstream tree... Will check again later this week. > > Jonathan Fine, thanks. A couple of series queued waiting for that one :) Regards, Lorenzo -- UNIX is Sexy: who | grep -i blonde | talk; cd ~; wine; talk; touch; unzip; touch; strip; gasp; finger; gasp; mount; fsck; more; yes; gasp; umount; make clean; sleep