From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4E291C76186 for ; Sat, 27 Jul 2019 21:32:02 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 1EB69206B8 for ; Sat, 27 Jul 2019 21:32:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726253AbfG0VcB convert rfc822-to-8bit (ORCPT ); Sat, 27 Jul 2019 17:32:01 -0400 Received: from saturn.retrosnub.co.uk ([46.235.226.198]:45740 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725280AbfG0VcB (ORCPT ); Sat, 27 Jul 2019 17:32:01 -0400 Received: from archlinux (cpc149474-cmbg20-2-0-cust94.5-4.cable.virginm.net [82.4.196.95]) by saturn.retrosnub.co.uk (Postfix; Retrosnub mail submission) with ESMTPSA id 8CE479E6A7A; Sat, 27 Jul 2019 22:31:59 +0100 (BST) Date: Sat, 27 Jul 2019 22:31:58 +0100 From: Jonathan Cameron To: Jean-Baptiste Maneyrol Cc: linux-iio Subject: Re: [PATCH v2] iio: imu: mpu6050: add missing available scan masks Message-ID: <20190727223158.363646ea@archlinux> In-Reply-To: References: <20190627131918.19619-1-jmaneyrol@invensense.com> <20190714182026.5edebf23@archlinux> X-Mailer: Claws Mail 3.17.4 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-iio-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org On Wed, 17 Jul 2019 13:45:07 +0000 Jean-Baptiste Maneyrol wrote: > Hello Jonathan, > > I am still not seeing this patch in the fixes-togreg branch or in the testing branch. Neither in the testing-fixes branch. > > Is this expected? I've been in a bit of a mess due to travel etc, so more than possible I hadn't pushed it out as I should have done. https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git/commit/?h=fixes-togreg&id=1244a720572fd1680ac8d6b8a4235f2e8557b810 I didn't see this before I'd pushed out for Arnd's patch hence not sure if it was there before or not! Sorry about that, Jonathan > > Thanks for your response. > JB > > From: linux-iio-owner@vger.kernel.org on behalf of Jonathan Cameron > > Sent: Sunday, July 14, 2019 19:20 > > To: Jean-Baptiste Maneyrol > > Cc: linux-iio@vger.kernel.org; stable@vger.kernel.org > > Subject: Re: [PATCH v2] iio: imu: mpu6050: add missing available scan masks > >   > > >  CAUTION: This email originated from outside of the organization. Please make sure the sender is who they say they are and do not click links or open attachments unless you recognize the sender and know the content is safe. > > > > On Thu, 27 Jun 2019 13:19:53 +0000 > > Jean-Baptiste Maneyrol wrote: > > > > > Driver only supports 3-axis gyro and/or 3-axis accel. > > > For icm20602, temp data is mandatory for all configurations. > > > > > > Fix all single and double axis configurations (almost never used) and more > > > importantly fix 3-axis gyro and 6-axis accel+gyro buffer on icm20602 when > > > temp data is not enabled. > > > > > > Signed-off-by: Jean-Baptiste Maneyrol > > > Fixes: 1615fe41a195 ("iio: imu: mpu6050: Fix FIFO layout for ICM20602") > > Something odd happened in this email that meant my client saved it as garbage. > > Oh well, cut and pasted worked ;) > > > > Applied to the fixes-togreg branch of iio.git and marked for stable. > > > > Thanks, > > > > Jonathan > > > > > --- > > > Changes in v2: > > >   - Use more explicit scan defines for masks > > > > > >  drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 43 ++++++++++++++++++++++ > > >  1 file changed, 43 insertions(+) > > > > > > diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c b/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c > > > index 385f14a4d5a7..66629c3adc21 100644 > > > --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c > > > +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c > > > @@ -851,6 +851,25 @@ static const struct iio_chan_spec inv_mpu_channels[] = { > > >        INV_MPU6050_CHAN(IIO_ACCEL, IIO_MOD_Z, INV_MPU6050_SCAN_ACCL_Z), > > >  }; > > >  > > > +static const unsigned long inv_mpu_scan_masks[] = { > > > +     /* 3-axis accel */ > > > +     BIT(INV_MPU6050_SCAN_ACCL_X) > > > +             | BIT(INV_MPU6050_SCAN_ACCL_Y) > > > +             | BIT(INV_MPU6050_SCAN_ACCL_Z), > > > +     /* 3-axis gyro */ > > > +     BIT(INV_MPU6050_SCAN_GYRO_X) > > > +             | BIT(INV_MPU6050_SCAN_GYRO_Y) > > > +             | BIT(INV_MPU6050_SCAN_GYRO_Z), > > > +     /* 6-axis accel + gyro */ > > > +     BIT(INV_MPU6050_SCAN_ACCL_X) > > > +             | BIT(INV_MPU6050_SCAN_ACCL_Y) > > > +             | BIT(INV_MPU6050_SCAN_ACCL_Z) > > > +             | BIT(INV_MPU6050_SCAN_GYRO_X) > > > +             | BIT(INV_MPU6050_SCAN_GYRO_Y) > > > +             | BIT(INV_MPU6050_SCAN_GYRO_Z), > > > +     0, > > > +}; > > > + > > >  static const struct iio_chan_spec inv_icm20602_channels[] = { > > >        IIO_CHAN_SOFT_TIMESTAMP(INV_ICM20602_SCAN_TIMESTAMP), > > >        { > > > @@ -877,6 +896,28 @@ static const struct iio_chan_spec inv_icm20602_channels[] = { > > >        INV_MPU6050_CHAN(IIO_ACCEL, IIO_MOD_Z, INV_ICM20602_SCAN_ACCL_Z), > > >  }; > > >  > > > +static const unsigned long inv_icm20602_scan_masks[] = { > > > +     /* 3-axis accel + temp (mandatory) */ > > > +     BIT(INV_ICM20602_SCAN_ACCL_X) > > > +             | BIT(INV_ICM20602_SCAN_ACCL_Y) > > > +             | BIT(INV_ICM20602_SCAN_ACCL_Z) > > > +             | BIT(INV_ICM20602_SCAN_TEMP), > > > +     /* 3-axis gyro + temp (mandatory) */ > > > +     BIT(INV_ICM20602_SCAN_GYRO_X) > > > +             | BIT(INV_ICM20602_SCAN_GYRO_Y) > > > +             | BIT(INV_ICM20602_SCAN_GYRO_Z) > > > +             | BIT(INV_ICM20602_SCAN_TEMP), > > > +     /* 6-axis accel + gyro + temp (mandatory) */ > > > +     BIT(INV_ICM20602_SCAN_ACCL_X) > > > +             | BIT(INV_ICM20602_SCAN_ACCL_Y) > > > +             | BIT(INV_ICM20602_SCAN_ACCL_Z) > > > +             | BIT(INV_ICM20602_SCAN_GYRO_X) > > > +             | BIT(INV_ICM20602_SCAN_GYRO_Y) > > > +             | BIT(INV_ICM20602_SCAN_GYRO_Z) > > > +             | BIT(INV_ICM20602_SCAN_TEMP), > > > +     0, > > > +}; > > > + > > >  /* > > >   * The user can choose any frequency between INV_MPU6050_MIN_FIFO_RATE and > > >   * INV_MPU6050_MAX_FIFO_RATE, but only these frequencies are matched by the > > > @@ -1136,9 +1177,11 @@ int inv_mpu_core_probe(struct regmap *regmap, int irq, const char *name, > > >        if (chip_type == INV_ICM20602) { > > >                indio_dev->channels = inv_icm20602_channels; > > >                indio_dev->num_channels = ARRAY_SIZE(inv_icm20602_channels); > > > +             indio_dev->available_scan_masks = inv_icm20602_scan_masks; > > >        } else { > > >                indio_dev->channels = inv_mpu_channels; > > >                indio_dev->num_channels = ARRAY_SIZE(inv_mpu_channels); > > > +             indio_dev->available_scan_masks = inv_mpu_scan_masks; > > >        } > > >  > > >        indio_dev->info = &mpu_info; > > >