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=-7.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,URIBL_BLOCKED autolearn=unavailable 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 E854AC282D7 for ; Sat, 2 Feb 2019 09:54:20 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id AEEF5204EC for ; Sat, 2 Feb 2019 09:54:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1549101260; bh=7ncYregVvPbmW2TKHxmUsxilcSC/NkPjh8CUd6rIT7c=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-ID:From; b=XEwCy+w2hwaMv1M88UrwW+ZziTNTACrd5wSwTUIcVwVOhtKfd2HWu1tx+k4D98aQD 0tV05hDj1izWy0i4wlOhu4U5waMFDxn0ank5ArrD2pdcxvlW7jJM29yqmY1I4ZrvAV IbqRI6Y5NkRQAkbgmhJuTTkc2RYpSY61zL3gahh0= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726659AbfBBJyP (ORCPT ); Sat, 2 Feb 2019 04:54:15 -0500 Received: from mail.kernel.org ([198.145.29.99]:47884 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727505AbfBBJyO (ORCPT ); Sat, 2 Feb 2019 04:54:14 -0500 Received: from archlinux (cpc91196-cmbg18-2-0-cust659.5-4.cable.virginm.net [81.96.234.148]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 5025A217D6; Sat, 2 Feb 2019 09:54:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1549101253; bh=7ncYregVvPbmW2TKHxmUsxilcSC/NkPjh8CUd6rIT7c=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=dtLdP6PuAgH3kNoSGdcZqkZNNz8Vr7PftpBV/wcRXNe/VF0TUusXlFX6NYOEvDtRh TIgL2ta7gqI2rUls+kgYg+lNECUn2GXZFHEHmxX5H7f464WPwnHIIjw/MW7BguBYvH 53XwzD60G1+ixbhdY315Culs+NlPOUPN0cud6Lmw= Date: Sat, 2 Feb 2019 09:54:06 +0000 From: Jonathan Cameron To: Randolph =?UTF-8?B?TWFhw59lbg==?= Cc: Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , Rob Herring , Mark Rutland , Brian Masney , Jonathan Marek , Jean-Baptiste Maneyrol , Martin Kelly , Douglas Fischer , Hans de Goede , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 2/2] iio: imu: mpu6050: Add support for the ICM 20602 IMU Message-ID: <20190202095406.5fa80d51@archlinux> In-Reply-To: <20190128185010.8275-2-gaireg@gaireg.de> References: <20190128185010.8275-1-gaireg@gaireg.de> <20190128185010.8275-2-gaireg@gaireg.de> X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Sender: linux-iio-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org On Mon, 28 Jan 2019 19:50:03 +0100 Randolph Maa=C3=9Fen wrote: > The Invensense ICM-20602 is a 6-axis MotionTracking device that > combines a 3-axis gyroscope and an 3-axis accelerometer. It is very > similar to the ICM-20608 imu which is already supported by the mpu6050 > driver. The main difference is that the ICM-20602 has the i2c bus > disable bit in a separate register. >=20 > Signed-off-by: Randolph Maa=C3=9Fen I'm not sure this particular approach will prove extensible for future devices (seems likely that the bit may move around within the 'different' register) but who knows and we can deal with that when it happens. 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/inv_mpu6050/Kconfig | 8 ++++---- > drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 31 ++++++++++++++++++++++++= ++++++ > drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c | 6 ++++++ > drivers/iio/imu/inv_mpu6050/inv_mpu_iio.h | 8 ++++++++ > drivers/iio/imu/inv_mpu6050/inv_mpu_spi.c | 12 +++++++++--- > 5 files changed, 58 insertions(+), 7 deletions(-) >=20 > diff --git a/drivers/iio/imu/inv_mpu6050/Kconfig b/drivers/iio/imu/inv_mp= u6050/Kconfig > index 5483b2ea754d..d2fe9dbddda7 100644 > --- a/drivers/iio/imu/inv_mpu6050/Kconfig > +++ b/drivers/iio/imu/inv_mpu6050/Kconfig > @@ -13,8 +13,8 @@ config INV_MPU6050_I2C > select INV_MPU6050_IIO > select REGMAP_I2C > help > - This driver supports the Invensense MPU6050/6500/9150 and ICM20608 > - motion tracking devices over I2C. > + This driver supports the Invensense MPU6050/6500/9150 and > + ICM20608/20602 motion tracking devices over I2C. > This driver can be built as a module. The module will be called > inv-mpu6050-i2c. > =20 > @@ -24,7 +24,7 @@ config INV_MPU6050_SPI > select INV_MPU6050_IIO > select REGMAP_SPI > help > - This driver supports the Invensense MPU6050/6500/9150 and ICM20608 > - motion tracking devices over SPI. > + This driver supports the Invensense MPU6050/6500/9150 and > + ICM20608/20602 motion tracking devices over SPI. > This driver can be built as a module. The module will be called > inv-mpu6050-spi. > diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c b/drivers/iio/imu= /inv_mpu6050/inv_mpu_core.c > index 1e428c196a82..650de0fefb7b 100644 > --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c > +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c > @@ -38,6 +38,29 @@ static const int gyro_scale_6050[] =3D {133090, 266181= , 532362, 1064724}; > */ > static const int accel_scale[] =3D {598, 1196, 2392, 4785}; > =20 > +static const struct inv_mpu6050_reg_map reg_set_icm20602 =3D { > + .sample_rate_div =3D INV_MPU6050_REG_SAMPLE_RATE_DIV, > + .lpf =3D INV_MPU6050_REG_CONFIG, > + .accel_lpf =3D INV_MPU6500_REG_ACCEL_CONFIG_2, > + .user_ctrl =3D INV_MPU6050_REG_USER_CTRL, > + .fifo_en =3D INV_MPU6050_REG_FIFO_EN, > + .gyro_config =3D INV_MPU6050_REG_GYRO_CONFIG, > + .accl_config =3D INV_MPU6050_REG_ACCEL_CONFIG, > + .fifo_count_h =3D INV_MPU6050_REG_FIFO_COUNT_H, > + .fifo_r_w =3D INV_MPU6050_REG_FIFO_R_W, > + .raw_gyro =3D INV_MPU6050_REG_RAW_GYRO, > + .raw_accl =3D INV_MPU6050_REG_RAW_ACCEL, > + .temperature =3D INV_MPU6050_REG_TEMPERATURE, > + .int_enable =3D INV_MPU6050_REG_INT_ENABLE, > + .int_status =3D INV_MPU6050_REG_INT_STATUS, > + .pwr_mgmt_1 =3D INV_MPU6050_REG_PWR_MGMT_1, > + .pwr_mgmt_2 =3D INV_MPU6050_REG_PWR_MGMT_2, > + .int_pin_cfg =3D INV_MPU6050_REG_INT_PIN_CFG, > + .accl_offset =3D INV_MPU6500_REG_ACCEL_OFFSET, > + .gyro_offset =3D INV_MPU6050_REG_GYRO_OFFSET, > + .i2c_if =3D INV_ICM20602_REG_I2C_IF, > +}; > + > static const struct inv_mpu6050_reg_map reg_set_6500 =3D { > .sample_rate_div =3D INV_MPU6050_REG_SAMPLE_RATE_DIV, > .lpf =3D INV_MPU6050_REG_CONFIG, > @@ -58,6 +81,7 @@ static const struct inv_mpu6050_reg_map reg_set_6500 = =3D { > .int_pin_cfg =3D INV_MPU6050_REG_INT_PIN_CFG, > .accl_offset =3D INV_MPU6500_REG_ACCEL_OFFSET, > .gyro_offset =3D INV_MPU6050_REG_GYRO_OFFSET, > + .i2c_if =3D 0, > }; > =20 > static const struct inv_mpu6050_reg_map reg_set_6050 =3D { > @@ -78,6 +102,7 @@ static const struct inv_mpu6050_reg_map reg_set_6050 = =3D { > .int_pin_cfg =3D INV_MPU6050_REG_INT_PIN_CFG, > .accl_offset =3D INV_MPU6050_REG_ACCEL_OFFSET, > .gyro_offset =3D INV_MPU6050_REG_GYRO_OFFSET, > + .i2c_if =3D 0, > }; > =20 > static const struct inv_mpu6050_chip_config chip_config_6050 =3D { > @@ -140,6 +165,12 @@ static const struct inv_mpu6050_hw hw_info[] =3D { > .reg =3D ®_set_6500, > .config =3D &chip_config_6050, > }, > + { > + .whoami =3D INV_ICM20602_WHOAMI_VALUE, > + .name =3D "ICM20602", > + .reg =3D ®_set_icm20602, > + .config =3D &chip_config_6050, > + }, > }; > =20 > int inv_mpu6050_switch_engine(struct inv_mpu6050_state *st, bool en, u32= mask) > diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c b/drivers/iio/imu/= inv_mpu6050/inv_mpu_i2c.c > index dd758e3d403d..e46eb4ddea21 100644 > --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c > +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c > @@ -127,6 +127,7 @@ static int inv_mpu_probe(struct i2c_client *client, > st =3D iio_priv(dev_get_drvdata(&client->dev)); > switch (st->chip_type) { > case INV_ICM20608: > + case INV_ICM20602: > /* no i2c auxiliary bus on the chip */ > break; > default: > @@ -179,6 +180,7 @@ static const struct i2c_device_id inv_mpu_id[] =3D { > {"mpu9250", INV_MPU9250}, > {"mpu9255", INV_MPU9255}, > {"icm20608", INV_ICM20608}, > + {"icm20602", INV_ICM20602}, > {} > }; > =20 > @@ -213,6 +215,10 @@ static const struct of_device_id inv_of_match[] =3D { > .compatible =3D "invensense,icm20608", > .data =3D (void *)INV_ICM20608 > }, > + { > + .compatible =3D "invensense,icm20602", > + .data =3D (void *)INV_ICM20602 > + }, > { } > }; > MODULE_DEVICE_TABLE(of, inv_of_match); > diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_iio.h b/drivers/iio/imu/= inv_mpu6050/inv_mpu_iio.h > index 6bcc11fc1b88..325afd9f5f61 100644 > --- a/drivers/iio/imu/inv_mpu6050i2c_i/inv_mpu_iio.h > +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_iio.h > @@ -44,6 +44,7 @@ > * @int_pin_cfg; Controls interrupt pin configuration. > * @accl_offset: Controls the accelerometer calibration offset. > * @gyro_offset: Controls the gyroscope calibration offset. > + * @i2c_if: Controls the i2c interface > */ > struct inv_mpu6050_reg_map { > u8 sample_rate_div; > @@ -65,6 +66,7 @@ struct inv_mpu6050_reg_map { > u8 int_pin_cfg; > u8 accl_offset; > u8 gyro_offset; > + u8 i2c_if; > }; > =20 > /*device enum */ > @@ -77,6 +79,7 @@ enum inv_devices { > INV_MPU9250, > INV_MPU9255, > INV_ICM20608, > + INV_ICM20602, > INV_NUM_PARTS > }; > =20 > @@ -195,6 +198,10 @@ struct inv_mpu6050_state { > #define INV_MPU6050_BIT_PWR_ACCL_STBY 0x38 > #define INV_MPU6050_BIT_PWR_GYRO_STBY 0x07 > =20 > +/* ICM20602 register */ > +#define INV_ICM20602_REG_I2C_IF 0x70 > +#define INV_ICM20602_BIT_I2C_IF_DIS 0x40 > + > #define INV_MPU6050_REG_FIFO_COUNT_H 0x72 > #define INV_MPU6050_REG_FIFO_R_W 0x74 > =20 > @@ -261,6 +268,7 @@ struct inv_mpu6050_state { > #define INV_MPU9255_WHOAMI_VALUE 0x73 > #define INV_MPU6515_WHOAMI_VALUE 0x74 > #define INV_ICM20608_WHOAMI_VALUE 0xAF > +#define INV_ICM20602_WHOAMI_VALUE 0x12 > =20 > /* scan element definition */ > enum inv_mpu6050_scan { > diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_spi.c b/drivers/iio/imu/= inv_mpu6050/inv_mpu_spi.c > index 227f50afff22..a112c3f45f74 100644 > --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_spi.c > +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_spi.c > @@ -31,9 +31,14 @@ static int inv_mpu_i2c_disable(struct iio_dev *indio_d= ev) > if (ret) > return ret; > =20 > - st->chip_config.user_ctrl |=3D INV_MPU6050_BIT_I2C_IF_DIS; > - ret =3D regmap_write(st->map, st->reg->user_ctrl, > - st->chip_config.user_ctrl); > + if (st->reg->i2c_if) { > + ret =3D regmap_write(st->map, st->reg->i2c_if, > + INV_ICM20602_BIT_I2C_IF_DIS); > + } else { > + st->chip_config.user_ctrl |=3D INV_MPU6050_BIT_I2C_IF_DIS; > + ret =3D regmap_write(st->map, st->reg->user_ctrl, > + st->chip_config.user_ctrl); > + } > if (ret) { > inv_mpu6050_set_power_itg(st, false); > return ret; > @@ -81,6 +86,7 @@ static const struct spi_device_id inv_mpu_id[] =3D { > {"mpu9250", INV_MPU9250}, > {"mpu9255", INV_MPU9255}, > {"icm20608", INV_ICM20608}, > + {"icm20602", INV_ICM20602}, > {} > }; > =20