Linux-IIO Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor
@ 2019-10-07  9:43 Lorenzo Bianconi
  2019-10-07  9:43 ` [PATCH 1/2] iio: imu: st_lsm6dsx: add support to LSM6DS0 Lorenzo Bianconi
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Lorenzo Bianconi @ 2019-10-07  9:43 UTC (permalink / raw)
  To: jic23; +Cc: linux-iio, martin, lorenzo.bianconi, devicetree

Add support to STM LSM6DS0 6-axis sensor. Just compiled, but LSM6DS0 relies on
the same register map used by LSM9DS1

Lorenzo Bianconi (2):
  iio: imu: st_lsm6dsx: add support to LSM6DS0
  dt-bindings: iio: imu: st_lsm6dsx: add lsm6ds0 device bindings

 Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt | 1 +
 drivers/iio/imu/st_lsm6dsx/Kconfig                       | 2 +-
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h                  | 2 ++
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c             | 5 ++++-
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c              | 5 +++++
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c              | 5 +++++
 6 files changed, 18 insertions(+), 2 deletions(-)

-- 
2.21.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 1/2] iio: imu: st_lsm6dsx: add support to LSM6DS0
  2019-10-07  9:43 [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor Lorenzo Bianconi
@ 2019-10-07  9:43 ` Lorenzo Bianconi
  2019-10-12 14:07   ` Jonathan Cameron
  2019-10-07  9:43 ` [PATCH 2/2] dt-bindings: iio: imu: st_lsm6dsx: add lsm6ds0 device bindings Lorenzo Bianconi
  2019-10-12 12:40 ` [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor Jonathan Cameron
  2 siblings, 1 reply; 8+ messages in thread
From: Lorenzo Bianconi @ 2019-10-07  9:43 UTC (permalink / raw)
  To: jic23; +Cc: linux-iio, martin, lorenzo.bianconi, devicetree

Add support to STM LSM6DS0 6-axis (acc + gyro) Mems sensor

Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
---
 drivers/iio/imu/st_lsm6dsx/Kconfig           | 2 +-
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h      | 2 ++
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 5 ++++-
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c  | 5 +++++
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c  | 5 +++++
 5 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/drivers/iio/imu/st_lsm6dsx/Kconfig b/drivers/iio/imu/st_lsm6dsx/Kconfig
index 77aa0e77212d..b425a356d99e 100644
--- a/drivers/iio/imu/st_lsm6dsx/Kconfig
+++ b/drivers/iio/imu/st_lsm6dsx/Kconfig
@@ -12,7 +12,7 @@ config IIO_ST_LSM6DSX
 	  Say yes here to build support for STMicroelectronics LSM6DSx imu
 	  sensor. Supported devices: lsm6ds3, lsm6ds3h, lsm6dsl, lsm6dsm,
 	  ism330dlc, lsm6dso, lsm6dsox, asm330lhh, lsm6dsr, lsm6ds3tr-c,
-	  ism330dhcx and the accelerometer/gyroscope of lsm9ds1.
+	  ism330dhcx, lsm6ds0 and the accelerometer/gyroscope of lsm9ds1.
 
 	  To compile this driver as a module, choose M here: the module
 	  will be called st_lsm6dsx.
diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
index 40532f99ba4b..74854230aecf 100644
--- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
+++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
@@ -26,6 +26,7 @@
 #define ST_LSM6DS3TRC_DEV_NAME	"lsm6ds3tr-c"
 #define ST_ISM330DHCX_DEV_NAME	"ism330dhcx"
 #define ST_LSM9DS1_DEV_NAME	"lsm9ds1-imu"
+#define ST_LSM6DS0_DEV_NAME	"lsm6ds0"
 
 enum st_lsm6dsx_hw_id {
 	ST_LSM6DS3_ID,
@@ -40,6 +41,7 @@ enum st_lsm6dsx_hw_id {
 	ST_LSM6DS3TRC_ID,
 	ST_ISM330DHCX_ID,
 	ST_LSM9DS1_ID,
+	ST_LSM6DS0_ID,
 	ST_LSM6DSX_MAX_ID,
 };
 
diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
index e8c3922f1b59..bacc908cf396 100644
--- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
+++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
@@ -32,7 +32,7 @@
  *   - Gyroscope supported full-scale [dps]: +-125/+-245/+-500/+-1000/+-2000
  *   - FIFO size: 3KB
  *
- * - LSM9DS1:
+ * - LSM9DS1/LSM6DS0:
  *   - Accelerometer supported ODR [Hz]: 10, 50, 119, 238, 476, 952
  *   - Accelerometer supported full-scale [g]: +-2/+-4/+-8/+-16
  *   - Gyroscope supported ODR [Hz]: 15, 60, 119, 238, 476, 952
@@ -104,6 +104,9 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = {
 			{
 				.hw_id = ST_LSM9DS1_ID,
 				.name = ST_LSM9DS1_DEV_NAME,
+			}, {
+				.hw_id = ST_LSM6DS0_ID,
+				.name = ST_LSM6DS0_DEV_NAME,
 			},
 		},
 		.channels = {
diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c
index f52511059545..e57744affbd0 100644
--- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c
+++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c
@@ -87,6 +87,10 @@ static const struct of_device_id st_lsm6dsx_i2c_of_match[] = {
 		.compatible = "st,lsm9ds1-imu",
 		.data = (void *)ST_LSM9DS1_ID,
 	},
+	{
+		.compatible = "st,lsm6ds0",
+		.data = (void *)ST_LSM6DS0_ID,
+	},
 	{},
 };
 MODULE_DEVICE_TABLE(of, st_lsm6dsx_i2c_of_match);
@@ -104,6 +108,7 @@ static const struct i2c_device_id st_lsm6dsx_i2c_id_table[] = {
 	{ ST_LSM6DS3TRC_DEV_NAME, ST_LSM6DS3TRC_ID },
 	{ ST_ISM330DHCX_DEV_NAME, ST_ISM330DHCX_ID },
 	{ ST_LSM9DS1_DEV_NAME, ST_LSM9DS1_ID },
+	{ ST_LSM6DS0_DEV_NAME, ST_LSM6DS0_ID },
 	{},
 };
 MODULE_DEVICE_TABLE(i2c, st_lsm6dsx_i2c_id_table);
diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c
index 344b28dddebb..933d4f9f6a4a 100644
--- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c
+++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c
@@ -87,6 +87,10 @@ static const struct of_device_id st_lsm6dsx_spi_of_match[] = {
 		.compatible = "st,lsm9ds1-imu",
 		.data = (void *)ST_LSM9DS1_ID,
 	},
+	{
+		.compatible = "st,lsm6ds0",
+		.data = (void *)ST_LSM6DS0_ID,
+	},
 	{},
 };
 MODULE_DEVICE_TABLE(of, st_lsm6dsx_spi_of_match);
@@ -104,6 +108,7 @@ static const struct spi_device_id st_lsm6dsx_spi_id_table[] = {
 	{ ST_LSM6DS3TRC_DEV_NAME, ST_LSM6DS3TRC_ID },
 	{ ST_ISM330DHCX_DEV_NAME, ST_ISM330DHCX_ID },
 	{ ST_LSM9DS1_DEV_NAME, ST_LSM9DS1_ID },
+	{ ST_LSM6DS0_DEV_NAME, ST_LSM6DS0_ID },
 	{},
 };
 MODULE_DEVICE_TABLE(spi, st_lsm6dsx_spi_id_table);
-- 
2.21.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 2/2] dt-bindings: iio: imu: st_lsm6dsx: add lsm6ds0 device bindings
  2019-10-07  9:43 [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor Lorenzo Bianconi
  2019-10-07  9:43 ` [PATCH 1/2] iio: imu: st_lsm6dsx: add support to LSM6DS0 Lorenzo Bianconi
@ 2019-10-07  9:43 ` Lorenzo Bianconi
  2019-10-12 14:09   ` Jonathan Cameron
  2019-10-12 12:40 ` [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor Jonathan Cameron
  2 siblings, 1 reply; 8+ messages in thread
From: Lorenzo Bianconi @ 2019-10-07  9:43 UTC (permalink / raw)
  To: jic23; +Cc: linux-iio, martin, lorenzo.bianconi, devicetree

Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
---
 Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt b/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt
index 1a07d38c813f..fc018ecba086 100644
--- a/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt
+++ b/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt
@@ -14,6 +14,7 @@ Required properties:
   "st,lsm6ds3tr-c"
   "st,ism330dhcx"
   "st,lsm9ds1-imu"
+  "st,lsm6ds0"
 - reg: i2c address of the sensor / spi cs line
 
 Optional properties:
-- 
2.21.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor
  2019-10-07  9:43 [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor Lorenzo Bianconi
  2019-10-07  9:43 ` [PATCH 1/2] iio: imu: st_lsm6dsx: add support to LSM6DS0 Lorenzo Bianconi
  2019-10-07  9:43 ` [PATCH 2/2] dt-bindings: iio: imu: st_lsm6dsx: add lsm6ds0 device bindings Lorenzo Bianconi
@ 2019-10-12 12:40 ` Jonathan Cameron
  2019-10-12 13:27   ` Lorenzo Bianconi
  2 siblings, 1 reply; 8+ messages in thread
From: Jonathan Cameron @ 2019-10-12 12:40 UTC (permalink / raw)
  To: Lorenzo Bianconi; +Cc: linux-iio, martin, lorenzo.bianconi, devicetree

On Mon,  7 Oct 2019 11:43:36 +0200
Lorenzo Bianconi <lorenzo@kernel.org> wrote:

> Add support to STM LSM6DS0 6-axis sensor. Just compiled, but LSM6DS0 relies on
> the same register map used by LSM9DS1
I'm feeling paranoid. Is this definitely different form the
LSM6DSO which is already supported?

If so ST loose for stupid part naming.

Jonathan

> 
> Lorenzo Bianconi (2):
>   iio: imu: st_lsm6dsx: add support to LSM6DS0
>   dt-bindings: iio: imu: st_lsm6dsx: add lsm6ds0 device bindings
> 
>  Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt | 1 +
>  drivers/iio/imu/st_lsm6dsx/Kconfig                       | 2 +-
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h                  | 2 ++
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c             | 5 ++++-
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c              | 5 +++++
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c              | 5 +++++
>  6 files changed, 18 insertions(+), 2 deletions(-)
> 


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor
  2019-10-12 12:40 ` [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor Jonathan Cameron
@ 2019-10-12 13:27   ` Lorenzo Bianconi
  2019-10-12 14:08     ` Jonathan Cameron
  0 siblings, 1 reply; 8+ messages in thread
From: Lorenzo Bianconi @ 2019-10-12 13:27 UTC (permalink / raw)
  To: Jonathan Cameron; +Cc: Lorenzo Bianconi, linux-iio, martin, devicetree

>
> On Mon,  7 Oct 2019 11:43:36 +0200
> Lorenzo Bianconi <lorenzo@kernel.org> wrote:
>
> > Add support to STM LSM6DS0 6-axis sensor. Just compiled, but LSM6DS0 relies on
> > the same register map used by LSM9DS1
> I'm feeling paranoid. Is this definitely different form the
> LSM6DSO which is already supported?
>
> If so ST loose for stupid part naming.

Yes, it is very different..AFAIK LSM6DSO is last IMU sensor from ST
while LSM6DS0 is very old (it has the same register map used in
LSM9DS1 imu).

Regards,
Lorenzo

>
> Jonathan
>
> >
> > Lorenzo Bianconi (2):
> >   iio: imu: st_lsm6dsx: add support to LSM6DS0
> >   dt-bindings: iio: imu: st_lsm6dsx: add lsm6ds0 device bindings
> >
> >  Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt | 1 +
> >  drivers/iio/imu/st_lsm6dsx/Kconfig                       | 2 +-
> >  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h                  | 2 ++
> >  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c             | 5 ++++-
> >  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c              | 5 +++++
> >  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c              | 5 +++++
> >  6 files changed, 18 insertions(+), 2 deletions(-)
> >
>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] iio: imu: st_lsm6dsx: add support to LSM6DS0
  2019-10-07  9:43 ` [PATCH 1/2] iio: imu: st_lsm6dsx: add support to LSM6DS0 Lorenzo Bianconi
@ 2019-10-12 14:07   ` Jonathan Cameron
  0 siblings, 0 replies; 8+ messages in thread
From: Jonathan Cameron @ 2019-10-12 14:07 UTC (permalink / raw)
  To: Lorenzo Bianconi; +Cc: linux-iio, martin, lorenzo.bianconi, devicetree

On Mon,  7 Oct 2019 11:43:37 +0200
Lorenzo Bianconi <lorenzo@kernel.org> wrote:

> Add support to STM LSM6DS0 6-axis (acc + gyro) Mems sensor
> 
> Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
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/Kconfig           | 2 +-
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h      | 2 ++
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 5 ++++-
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c  | 5 +++++
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c  | 5 +++++
>  5 files changed, 17 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/iio/imu/st_lsm6dsx/Kconfig b/drivers/iio/imu/st_lsm6dsx/Kconfig
> index 77aa0e77212d..b425a356d99e 100644
> --- a/drivers/iio/imu/st_lsm6dsx/Kconfig
> +++ b/drivers/iio/imu/st_lsm6dsx/Kconfig
> @@ -12,7 +12,7 @@ config IIO_ST_LSM6DSX
>  	  Say yes here to build support for STMicroelectronics LSM6DSx imu
>  	  sensor. Supported devices: lsm6ds3, lsm6ds3h, lsm6dsl, lsm6dsm,
>  	  ism330dlc, lsm6dso, lsm6dsox, asm330lhh, lsm6dsr, lsm6ds3tr-c,
> -	  ism330dhcx and the accelerometer/gyroscope of lsm9ds1.
> +	  ism330dhcx, lsm6ds0 and the accelerometer/gyroscope of lsm9ds1.
>  
>  	  To compile this driver as a module, choose M here: the module
>  	  will be called st_lsm6dsx.
> diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
> index 40532f99ba4b..74854230aecf 100644
> --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
> +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
> @@ -26,6 +26,7 @@
>  #define ST_LSM6DS3TRC_DEV_NAME	"lsm6ds3tr-c"
>  #define ST_ISM330DHCX_DEV_NAME	"ism330dhcx"
>  #define ST_LSM9DS1_DEV_NAME	"lsm9ds1-imu"
> +#define ST_LSM6DS0_DEV_NAME	"lsm6ds0"
>  
>  enum st_lsm6dsx_hw_id {
>  	ST_LSM6DS3_ID,
> @@ -40,6 +41,7 @@ enum st_lsm6dsx_hw_id {
>  	ST_LSM6DS3TRC_ID,
>  	ST_ISM330DHCX_ID,
>  	ST_LSM9DS1_ID,
> +	ST_LSM6DS0_ID,
>  	ST_LSM6DSX_MAX_ID,
>  };
>  
> diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
> index e8c3922f1b59..bacc908cf396 100644
> --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
> +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
> @@ -32,7 +32,7 @@
>   *   - Gyroscope supported full-scale [dps]: +-125/+-245/+-500/+-1000/+-2000
>   *   - FIFO size: 3KB
>   *
> - * - LSM9DS1:
> + * - LSM9DS1/LSM6DS0:
>   *   - Accelerometer supported ODR [Hz]: 10, 50, 119, 238, 476, 952
>   *   - Accelerometer supported full-scale [g]: +-2/+-4/+-8/+-16
>   *   - Gyroscope supported ODR [Hz]: 15, 60, 119, 238, 476, 952
> @@ -104,6 +104,9 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = {
>  			{
>  				.hw_id = ST_LSM9DS1_ID,
>  				.name = ST_LSM9DS1_DEV_NAME,
> +			}, {
> +				.hw_id = ST_LSM6DS0_ID,
> +				.name = ST_LSM6DS0_DEV_NAME,
>  			},
>  		},
>  		.channels = {
> diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c
> index f52511059545..e57744affbd0 100644
> --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c
> +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c
> @@ -87,6 +87,10 @@ static const struct of_device_id st_lsm6dsx_i2c_of_match[] = {
>  		.compatible = "st,lsm9ds1-imu",
>  		.data = (void *)ST_LSM9DS1_ID,
>  	},
> +	{
> +		.compatible = "st,lsm6ds0",
> +		.data = (void *)ST_LSM6DS0_ID,
> +	},
>  	{},
>  };
>  MODULE_DEVICE_TABLE(of, st_lsm6dsx_i2c_of_match);
> @@ -104,6 +108,7 @@ static const struct i2c_device_id st_lsm6dsx_i2c_id_table[] = {
>  	{ ST_LSM6DS3TRC_DEV_NAME, ST_LSM6DS3TRC_ID },
>  	{ ST_ISM330DHCX_DEV_NAME, ST_ISM330DHCX_ID },
>  	{ ST_LSM9DS1_DEV_NAME, ST_LSM9DS1_ID },
> +	{ ST_LSM6DS0_DEV_NAME, ST_LSM6DS0_ID },
>  	{},
>  };
>  MODULE_DEVICE_TABLE(i2c, st_lsm6dsx_i2c_id_table);
> diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c
> index 344b28dddebb..933d4f9f6a4a 100644
> --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c
> +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c
> @@ -87,6 +87,10 @@ static const struct of_device_id st_lsm6dsx_spi_of_match[] = {
>  		.compatible = "st,lsm9ds1-imu",
>  		.data = (void *)ST_LSM9DS1_ID,
>  	},
> +	{
> +		.compatible = "st,lsm6ds0",
> +		.data = (void *)ST_LSM6DS0_ID,
> +	},
>  	{},
>  };
>  MODULE_DEVICE_TABLE(of, st_lsm6dsx_spi_of_match);
> @@ -104,6 +108,7 @@ static const struct spi_device_id st_lsm6dsx_spi_id_table[] = {
>  	{ ST_LSM6DS3TRC_DEV_NAME, ST_LSM6DS3TRC_ID },
>  	{ ST_ISM330DHCX_DEV_NAME, ST_ISM330DHCX_ID },
>  	{ ST_LSM9DS1_DEV_NAME, ST_LSM9DS1_ID },
> +	{ ST_LSM6DS0_DEV_NAME, ST_LSM6DS0_ID },
>  	{},
>  };
>  MODULE_DEVICE_TABLE(spi, st_lsm6dsx_spi_id_table);


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor
  2019-10-12 13:27   ` Lorenzo Bianconi
@ 2019-10-12 14:08     ` Jonathan Cameron
  0 siblings, 0 replies; 8+ messages in thread
From: Jonathan Cameron @ 2019-10-12 14:08 UTC (permalink / raw)
  To: Lorenzo Bianconi; +Cc: Lorenzo Bianconi, linux-iio, martin, devicetree

On Sat, 12 Oct 2019 15:27:08 +0200
Lorenzo Bianconi <lorenzo.bianconi@redhat.com> wrote:

> >
> > On Mon,  7 Oct 2019 11:43:36 +0200
> > Lorenzo Bianconi <lorenzo@kernel.org> wrote:
> >  
> > > Add support to STM LSM6DS0 6-axis sensor. Just compiled, but LSM6DS0 relies on
> > > the same register map used by LSM9DS1  
> > I'm feeling paranoid. Is this definitely different form the
> > LSM6DSO which is already supported?
> >
> > If so ST loose for stupid part naming.  
> 
> Yes, it is very different..AFAIK LSM6DSO is last IMU sensor from ST
> while LSM6DS0 is very old (it has the same register map used in
> LSM9DS1 imu).
No way anything could possibly go wrong ;)

*dramatic sigh*

> 
> Regards,
> Lorenzo
> 
> >
> > Jonathan
> >  
> > >
> > > Lorenzo Bianconi (2):
> > >   iio: imu: st_lsm6dsx: add support to LSM6DS0
> > >   dt-bindings: iio: imu: st_lsm6dsx: add lsm6ds0 device bindings
> > >
> > >  Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt | 1 +
> > >  drivers/iio/imu/st_lsm6dsx/Kconfig                       | 2 +-
> > >  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h                  | 2 ++
> > >  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c             | 5 ++++-
> > >  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c              | 5 +++++
> > >  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c              | 5 +++++
> > >  6 files changed, 18 insertions(+), 2 deletions(-)
> > >  
> >  
> 


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] dt-bindings: iio: imu: st_lsm6dsx: add lsm6ds0 device bindings
  2019-10-07  9:43 ` [PATCH 2/2] dt-bindings: iio: imu: st_lsm6dsx: add lsm6ds0 device bindings Lorenzo Bianconi
@ 2019-10-12 14:09   ` Jonathan Cameron
  0 siblings, 0 replies; 8+ messages in thread
From: Jonathan Cameron @ 2019-10-12 14:09 UTC (permalink / raw)
  To: Lorenzo Bianconi; +Cc: linux-iio, martin, lorenzo.bianconi, devicetree

On Mon,  7 Oct 2019 11:43:38 +0200
Lorenzo Bianconi <lorenzo@kernel.org> wrote:

> Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Applied.

Now a yaml conversion would be cool if you happened to get bored ;)

Jonathan

> ---
>  Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt b/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt
> index 1a07d38c813f..fc018ecba086 100644
> --- a/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt
> +++ b/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt
> @@ -14,6 +14,7 @@ Required properties:
>    "st,lsm6ds3tr-c"
>    "st,ism330dhcx"
>    "st,lsm9ds1-imu"
> +  "st,lsm6ds0"
>  - reg: i2c address of the sensor / spi cs line
>  
>  Optional properties:


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, back to index

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-07  9:43 [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor Lorenzo Bianconi
2019-10-07  9:43 ` [PATCH 1/2] iio: imu: st_lsm6dsx: add support to LSM6DS0 Lorenzo Bianconi
2019-10-12 14:07   ` Jonathan Cameron
2019-10-07  9:43 ` [PATCH 2/2] dt-bindings: iio: imu: st_lsm6dsx: add lsm6ds0 device bindings Lorenzo Bianconi
2019-10-12 14:09   ` Jonathan Cameron
2019-10-12 12:40 ` [PATCH 0/2] add support to STM LSM6DS0 6-axis Mems sensor Jonathan Cameron
2019-10-12 13:27   ` Lorenzo Bianconi
2019-10-12 14:08     ` Jonathan Cameron

Linux-IIO Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-iio/0 linux-iio/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-iio linux-iio/ https://lore.kernel.org/linux-iio \
		linux-iio@vger.kernel.org linux-iio@archiver.kernel.org
	public-inbox-index linux-iio

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-iio


AGPL code for this site: git clone https://public-inbox.org/ public-inbox