From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.kernel.org ([198.145.29.99]:40472 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752668AbdIXPu6 (ORCPT ); Sun, 24 Sep 2017 11:50:58 -0400 Date: Sun, 24 Sep 2017 16:50:54 +0100 From: Jonathan Cameron To: Lorenzo Bianconi Cc: linux-iio@vger.kernel.org, lorenzo.bianconi@st.com Subject: Re: [PATCH 2/2] iio: imu: st_lsm6dsx: remove rounding configuration Message-ID: <20170924165054.7280ddd3@archlinux> In-Reply-To: <20170923191052.14419-3-lorenzo.bianconi@st.com> References: <20170923191052.14419-1-lorenzo.bianconi@st.com> <20170923191052.14419-3-lorenzo.bianconi@st.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On Sat, 23 Sep 2017 21:10:52 +0200 Lorenzo Bianconi wrote: > Remove rounding configuration since it is enabled by default for the > FIFO output registers > > Signed-off-by: Lorenzo Bianconi Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > --- > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 7 ------- > 1 file changed, 7 deletions(-) > > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > index 37d6e324ae74..debf4064f474 100644 > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > @@ -54,8 +54,6 @@ > #define ST_LSM6DSX_REG_BDU_MASK BIT(6) > #define ST_LSM6DSX_REG_INT2_ON_INT1_ADDR 0x13 > #define ST_LSM6DSX_REG_INT2_ON_INT1_MASK BIT(5) > -#define ST_LSM6DSX_REG_ROUNDING_ADDR 0x16 > -#define ST_LSM6DSX_REG_ROUNDING_MASK BIT(2) > > #define ST_LSM6DSX_REG_ACC_ODR_ADDR 0x10 > #define ST_LSM6DSX_REG_ACC_ODR_MASK GENMASK(7, 4) > @@ -611,11 +609,6 @@ static int st_lsm6dsx_init_device(struct st_lsm6dsx_hw *hw) > if (err < 0) > return err; > > - err = st_lsm6dsx_write_with_mask(hw, ST_LSM6DSX_REG_ROUNDING_ADDR, > - ST_LSM6DSX_REG_ROUNDING_MASK, 1); > - if (err < 0) > - return err; > - > /* enable FIFO watermak interrupt */ > err = st_lsm6dsx_get_drdy_reg(hw, &drdy_int_reg); > if (err < 0)