Re: [RFC PATCH 1/7] iio:ABI docs: Fix up duplicate *_calibbias_* documentation for icm42600

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, 21 Feb 2021 16:05:29 +0000
Jonathan Cameron <jic23@xxxxxxxxxx> wrote:

> On Sun, 17 Jan 2021 15:38:10 +0000
> Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
> 
> > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>  
> @Jean-Baptiste.
> 
> Whilst this is 'fairly obviously' fine, could you take a quick glance at it.
> I'm never keen to take my own patches without someone having agreed
> I haven't done anything particularly silly ;)

As these are still frustrating Mauro's attempt to get the docs to build
cleanly and it should be safe enough.

Applied to the togreg branch of iio.git and pushed out as testing for
the autobuilders to poke at it.

Jonathan

> 
> Jonathan
> 
> > 
> > This device has the unusual characteristic that the calibbias values
> > have well defined units (more commonly they are tweaks to a DAC)
> > Unfortunately the previous approach of having more specific documentation
> > in sysfs-bus-iio-icm42600 results in warnings during the documentation
> > build and random ordering in the resulting documentation.
> > 
> > To avoid this, add a note to the main documentation on this special
> > characteristic for the icm42600.   The _available for calibbias was
> > missing from the main sysfs-bus-iio docs so also add that, allowing
> > us to drop the icm42600 specific file.
> > 
> > Fixes
> > $ scripts/get_abi.pl validate warning:
> > Warning: /sys/bus/iio/devices/iio:deviceX/in_accel_x_calibbias is defined 2 times:  ./Documentation/ABI/testing/sysfs-bus-iio-icm42600:0  ./Documentation/ABI/testing/sysfs-bus-iio:394
> > Warning: /sys/bus/iio/devices/iio:deviceX/in_accel_y_calibbias is defined 2 times:  ./Documentation/ABI/testing/sysfs-bus-iio-icm42600:1  ./Documentation/ABI/testing/sysfs-bus-iio:395
> > Warning: /sys/bus/iio/devices/iio:deviceX/in_accel_z_calibbias is defined 2 times:  ./Documentation/ABI/testing/sysfs-bus-iio-icm42600:2  ./Documentation/ABI/testing/sysfs-bus-iio:396
> > Warning: /sys/bus/iio/devices/iio:deviceX/in_anglvel_x_calibbias is defined 2 times:  ./Documentation/ABI/testing/sysfs-bus-iio-icm42600:3  ./Documentation/ABI/testing/sysfs-bus-iio:397
> > Warning: /sys/bus/iio/devices/iio:deviceX/in_anglvel_y_calibbias is defined 2 times:  ./Documentation/ABI/testing/sysfs-bus-iio-icm42600:4  ./Documentation/ABI/testing/sysfs-bus-iio:398
> > Warning: /sys/bus/iio/devices/iio:deviceX/in_anglvel_z_calibbias is defined 2 times:  ./Documentation/ABI/testing/sysfs-bus-iio-icm42600:5  ./Documentation/ABI/testing/sysfs-bus-iio:399
> > 
> > Cc: Jean-Baptiste Maneyrol <jmaneyrol@xxxxxxxxxxxxxx>
> > Reported-by: Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx>
> > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
> > ---
> >  Documentation/ABI/testing/sysfs-bus-iio       | 13 ++++++++++++
> >  .../ABI/testing/sysfs-bus-iio-icm42600        | 20 -------------------
> >  2 files changed, 13 insertions(+), 20 deletions(-)
> > 
> > diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
> > index d957f5da5c04..d2dd9cc280f9 100644
> > --- a/Documentation/ABI/testing/sysfs-bus-iio
> > +++ b/Documentation/ABI/testing/sysfs-bus-iio
> > @@ -408,6 +408,19 @@ Contact:	linux-iio@xxxxxxxxxxxxxxx
> >  Description:
> >  		Hardware applied calibration offset (assumed to fix production
> >  		inaccuracies).
> > +		icm42600: For this device values are real physical offsets
> > +		expressed in SI units (m/s^2 for accelerometers and rad/s
> > +		for gyroscope)/
> > +
> > +What:		/sys/bus/iio/devices/iio:deviceX/in_accel_calibbias_available
> > +What:		/sys/bus/iio/devices/iio:deviceX/in_anglvel_calibbias_available
> > +KernelVersion:  5.8
> > +Contact:        linux-iio@xxxxxxxxxxxxxxx
> > +Description:
> > +		Available values of calibbias. Maybe expressed as either of:
> > +
> > +		- a small discrete set of values like "0 2 4 6 8"
> > +		- a range specified as "[min step max]"
> >  
> >  What:		/sys/bus/iio/devices/iio:deviceX/in_voltageY_calibscale
> >  What:		/sys/bus/iio/devices/iio:deviceX/in_voltageY_supply_calibscale
> > diff --git a/Documentation/ABI/testing/sysfs-bus-iio-icm42600 b/Documentation/ABI/testing/sysfs-bus-iio-icm42600
> > deleted file mode 100644
> > index 0bf1fd4f5bf1..000000000000
> > --- a/Documentation/ABI/testing/sysfs-bus-iio-icm42600
> > +++ /dev/null
> > @@ -1,20 +0,0 @@
> > -What:		/sys/bus/iio/devices/iio:deviceX/in_accel_x_calibbias
> > -What:		/sys/bus/iio/devices/iio:deviceX/in_accel_y_calibbias
> > -What:		/sys/bus/iio/devices/iio:deviceX/in_accel_z_calibbias
> > -What:		/sys/bus/iio/devices/iio:deviceX/in_anglvel_x_calibbias
> > -What:		/sys/bus/iio/devices/iio:deviceX/in_anglvel_y_calibbias
> > -What:		/sys/bus/iio/devices/iio:deviceX/in_anglvel_z_calibbias
> > -KernelVersion:  5.8
> > -Contact:        linux-iio@xxxxxxxxxxxxxxx
> > -Description:
> > -		Hardware applied calibration offset (assumed to fix production
> > -		inaccuracies). Values represent a real physical offset expressed
> > -		in SI units (m/s^2 for accelerometer and rad/s for gyroscope).
> > -
> > -What:		/sys/bus/iio/devices/iio:deviceX/in_accel_calibbias_available
> > -What:		/sys/bus/iio/devices/iio:deviceX/in_anglvel_calibbias_available
> > -KernelVersion:  5.8
> > -Contact:        linux-iio@xxxxxxxxxxxxxxx
> > -Description:
> > -		Range of available values for hardware offset. Values in SI
> > -		units (m/s^2 for accelerometer and rad/s for gyroscope).  
> 




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux