linux-kernel - Re: [PATCH] iio/gyro/bmg160: Add support for BMI088 chip

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20200322173826.12a7c28c@archlinux>
Date:   Sun, 22 Mar 2020 17:38:26 +0000
From:   Jonathan Cameron <jic23@...nel.org>
To:     Mike Looijmans <mike.looijmans@...ic.nl>
Cc:     linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, knaack.h@....de, lars@...afoo.de,
        pmeerw@...erw.net, robh+dt@...nel.org, mark.rutland@....com,
        hns@...delico.com, rui.silva@...aro.org
Subject: Re: [PATCH] iio/gyro/bmg160: Add support for BMI088 chip

On Mon, 16 Mar 2020 07:34:03 +0100
Mike Looijmans <mike.looijmans@...ic.nl> wrote:

> The BMI088 is pin-compatible with the BMI055, and provides
> both gyro and accel functions. The gyro part is similar to
> the BMI055 and this adds the chip to the list of supported
> devices for the gyro part.
> 
> The accel part of the chip is not compatible with anything
> existing already.
> 
> Signed-off-by: Mike Looijmans <mike.looijmans@...ic.nl>

Applied.

Thanks,

Jonathan

> ---
>  Documentation/devicetree/bindings/iio/gyroscope/bmg160.txt | 2 +-
>  drivers/iio/gyro/Kconfig                                   | 2 +-
>  drivers/iio/gyro/bmg160_i2c.c                              | 2 ++
>  drivers/iio/gyro/bmg160_spi.c                              | 1 +
>  4 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/iio/gyroscope/bmg160.txt b/Documentation/devicetree/bindings/iio/gyroscope/bmg160.txt
> index 78e18a1e9c1d..bb43d1ad9c9f 100644
> --- a/Documentation/devicetree/bindings/iio/gyroscope/bmg160.txt
> +++ b/Documentation/devicetree/bindings/iio/gyroscope/bmg160.txt
> @@ -2,7 +2,7 @@
>  
>  Required properties:
>  
> -  - compatible : should be "bosch,bmg160" or "bosch,bmi055_gyro"
> +  - compatible : should be "bosch,bmg160", "bosch,bmi055_gyro" or "bosch,bmi088_gyro"
>    - reg : the I2C address of the sensor (0x69)
>  
>  Optional properties:
> diff --git a/drivers/iio/gyro/Kconfig b/drivers/iio/gyro/Kconfig
> index 7eaf77707b0b..6daeddf37f60 100644
> --- a/drivers/iio/gyro/Kconfig
> +++ b/drivers/iio/gyro/Kconfig
> @@ -61,7 +61,7 @@ config BMG160
>  	help
>  	  Say yes here to build support for BOSCH BMG160 Tri-axis Gyro Sensor
>  	  driver connected via I2C or SPI. This driver also supports BMI055
> -	  gyroscope.
> +	  and BMI088 gyroscope.
>  
>  	  This driver can also be built as a module.  If so, the module
>  	  will be called bmg160_i2c or bmg160_spi.
> diff --git a/drivers/iio/gyro/bmg160_i2c.c b/drivers/iio/gyro/bmg160_i2c.c
> index 4fc9c6a3321f..de15c6fc3381 100644
> --- a/drivers/iio/gyro/bmg160_i2c.c
> +++ b/drivers/iio/gyro/bmg160_i2c.c
> @@ -42,6 +42,7 @@ static int bmg160_i2c_remove(struct i2c_client *client)
>  static const struct acpi_device_id bmg160_acpi_match[] = {
>  	{"BMG0160", 0},
>  	{"BMI055B", 0},
> +	{"BMI088B", 0},
>  	{},
>  };
>  
> @@ -50,6 +51,7 @@ MODULE_DEVICE_TABLE(acpi, bmg160_acpi_match);
>  static const struct i2c_device_id bmg160_i2c_id[] = {
>  	{"bmg160", 0},
>  	{"bmi055_gyro", 0},
> +	{"bmi088_gyro", 0},
>  	{}
>  };
>  
> diff --git a/drivers/iio/gyro/bmg160_spi.c b/drivers/iio/gyro/bmg160_spi.c
> index 182a59c42507..03e80bb2601a 100644
> --- a/drivers/iio/gyro/bmg160_spi.c
> +++ b/drivers/iio/gyro/bmg160_spi.c
> @@ -37,6 +37,7 @@ static int bmg160_spi_remove(struct spi_device *spi)
>  static const struct spi_device_id bmg160_spi_id[] = {
>  	{"bmg160", 0},
>  	{"bmi055_gyro", 0},
> +	{"bmi088_gyro", 0},
>  	{}
>  };
>