Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

Hi Leonard, On |19 Apr 16 @ 12:53|, Crestez Dan Leonard wrote: > On 04/19/2016 08:10 AM, Lucas De Marchi wrote: > > On Mon, Apr 18, 2016 at 4:51 PM, Jonathan Cameron wrote: > >> On 18/04/16 11:25, Crestez Dan Leonard wrote: > >>> On 04/18/2016 09:07 AM, Denis Ciocca wrote: >

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

Hi Leonard, On |19 Apr 16 @ 12:53|, Crestez Dan Leonard wrote: > On 04/19/2016 08:10 AM, Lucas De Marchi wrote: > > On Mon, Apr 18, 2016 at 4:51 PM, Jonathan Cameron wrote: > >> On 18/04/16 11:25, Crestez Dan Leonard wrote: > >>> On 04/18/2016 09:07 AM, Denis Ciocca wrote: > >>> Then st_combo_*

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

On 04/19/2016 08:10 AM, Lucas De Marchi wrote: > On Mon, Apr 18, 2016 at 4:51 PM, Jonathan Cameron wrote: >> On 18/04/16 11:25, Crestez Dan Leonard wrote: >>> On 04/18/2016 09:07 AM, Denis Ciocca wrote: >>> Then st_combo_* implementation functions would forward to st_magn_* or

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

On 04/19/2016 08:10 AM, Lucas De Marchi wrote: > On Mon, Apr 18, 2016 at 4:51 PM, Jonathan Cameron wrote: >> On 18/04/16 11:25, Crestez Dan Leonard wrote: >>> On 04/18/2016 09:07 AM, Denis Ciocca wrote: >>> Then st_combo_* implementation functions would forward to st_magn_* or >>> st_accel_*

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

On Mon, Apr 18, 2016 at 4:51 PM, Jonathan Cameron wrote: > On 18/04/16 11:25, Crestez Dan Leonard wrote: >> On 04/18/2016 09:07 AM, Denis Ciocca wrote: >> Then st_combo_* implementation functions would forward to st_magn_* or >> st_accel_* depending on chan->type. Does this make

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

On Mon, Apr 18, 2016 at 4:51 PM, Jonathan Cameron wrote: > On 18/04/16 11:25, Crestez Dan Leonard wrote: >> On 04/18/2016 09:07 AM, Denis Ciocca wrote: >> Then st_combo_* implementation functions would forward to st_magn_* or >> st_accel_* depending on chan->type. Does this make sense? > Perhaps.

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

On 18/04/16 11:25, Crestez Dan Leonard wrote: > On 04/18/2016 09:07 AM, Denis Ciocca wrote: >> Hi Leonard and Jonathan, >> >> basically the patch can not work. > Well, it can work as long as you don't initialize both the accel and the > magn at the same time. There is no issue with the gyro

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

On 18/04/16 11:25, Crestez Dan Leonard wrote: > On 04/18/2016 09:07 AM, Denis Ciocca wrote: >> Hi Leonard and Jonathan, >> >> basically the patch can not work. > Well, it can work as long as you don't initialize both the accel and the > magn at the same time. There is no issue with the gyro

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

On 04/18/2016 09:07 AM, Denis Ciocca wrote: > Hi Leonard and Jonathan, > > basically the patch can not work. Well, it can work as long as you don't initialize both the accel and the magn at the same time. There is no issue with the gyro driver. Would it be OK if I resend this as a [PATCH] intended

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

On 04/18/2016 09:07 AM, Denis Ciocca wrote: > Hi Leonard and Jonathan, > > basically the patch can not work. Well, it can work as long as you don't initialize both the accel and the magn at the same time. There is no issue with the gyro driver. Would it be OK if I resend this as a [PATCH] intended

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

Hi Leonard and Jonathan, basically the patch can not work. Current ST infrastructure needs to use one i2c slave per driver (accel or magn or gyro). All sensors currently supported (including lsm303agr) have one i2c address per sensor type (for example in lsm303agr, accel has one i2c address,

Re: [RFC] iio: st: Add lsm9ds0 support for gyro accel and magny

Hi Leonard and Jonathan, basically the patch can not work. Current ST infrastructure needs to use one i2c slave per driver (accel or magn or gyro). All sensors currently supported (including lsm303agr) have one i2c address per sensor type (for example in lsm303agr, accel has one i2c address,