mbox series

[0/2] add support to STM LSM6DS0 6-axis Mems sensor

Message ID cover.1570441034.git.lorenzo@kernel.org (mailing list archive)
Headers show
Series add support to STM LSM6DS0 6-axis Mems sensor | expand

Message

Lorenzo Bianconi Oct. 7, 2019, 9:43 a.m. UTC
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(-)

Comments

Jonathan Cameron Oct. 12, 2019, 12:40 p.m. UTC | #1
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(-)
>
Lorenzo Bianconi Oct. 12, 2019, 1:27 p.m. UTC | #2
>
> 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(-)
> >
>
Jonathan Cameron Oct. 12, 2019, 2:08 p.m. UTC | #3
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(-)
> > >  
> >  
>