Re: [PATCH v2 3/3] iio: imu: adis16475.c: Add delta angle and delta velocity channels

[Jonathan Cameron]

On Wed, 26 Jul 2023 10:23:02 +0000
"Bolboaca, Ramona" <Ramona.Bolboaca@xxxxxxxxxx> wrote:

> > On Wed, 19 Jul 2023 15:31:52 +0300
> > Ramona Bolboaca <ramona.bolboaca@xxxxxxxxxx> wrote:
> >   
> > > Add support for delta angle and delta velocity raw and buffer
> > > readings to adis16475 driver.
> > >
> > > Signed-off-by: Ramona Bolboaca <ramona.bolboaca@xxxxxxxxxx>  
> > As noted in my reply to Nuno's review, the question of channel type
> > is still unresolved.
> >   
> 
> Hi Jonathan,
> 
> I think Nuno already offered the definition from the data-sheet, but let me
> mention it here again for context:
> 
> Delta angle:
> In addition to the angular rate of rotation (gyroscope) measurements around
> each axis (x, y, and z), the ADIS16505 also provides delta angle measurements
> that represent a calculation of angular displacement between each sample update.
> 
> Delta velocity:
> In addition to the linear acceleration measurements along each axis (x, y, and
> z), the ADIS16505 also provides delta velocity measurements that represent a
> calculation of linear velocity change between each sample update.
> 
> Seeing how we cannot find something more suitable, do you think it would make
> sense to add these new channels to IIO: something like IIO_DELTA_VELOCITY and
> IIO_DELTA_ANGLE and specify in ABI docs that these measurements are relative
> to the previous sample? Or do you have something else in mind?

Yes - I think we need these new channel types to avoid inconsistent use of the
existing types.  In the documentation I would give a little context, particularly
around the common use of these being to simply sum them over time.

Jonathan


> 
> Thank you,
> Ramona Bolboaca
> 
> > https://urldefense.com/v3/__https://lore.kernel.org/all/20230720193457.27
> > 2f02a9@jic23-
> > huawei/__;!!A3Ni8CS0y2Y!52Dhq_Yj9srV_lE8JgPysp2b3Lbg9U2w94VIGTMa8
> > 3hepJgv-NCnFWomV83Ud5C3TGp3tMMI0IDVRYZV5N8$
> > 
> > Thanks,
> > 
> > Jonathan