Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Jean-Baptiste Maneyrol <JManeyrol@invensense.com>
To:     Jonathan Cameron <jic23@kernel.org>
CC:     "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "robh@kernel.org" <robh@kernel.org>,
        "mchehab+huawei@kernel.org" <mchehab+huawei@kernel.org>,
        "davem@davemloft.net" <davem@davemloft.net>,
        "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v4 08/13] Documentation: ABI: add specific icm42600
 documentation
Thread-Topic: [PATCH v4 08/13] Documentation: ABI: add specific icm42600
 documentation
Thread-Index: AQHWSKsgRJzyGbHlXkO1VgJbvsWR6KnIor8AgALAXHA=
Date:   Mon, 16 Nov 2020 09:25:26 +0000
Message-ID: <BL0PR12MB5011F8499380AAF4D19DC88DC4E30@BL0PR12MB5011.namprd12.prod.outlook.com>
References: <20200622153729.12702-1-jmaneyrol@invensense.com>
        <20200622153729.12702-9-jmaneyrol@invensense.com>,<20201114152302.3b05c4ea@archlinux>
In-Reply-To: <20201114152302.3b05c4ea@archlinux>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: d1027e10-4ab5-43d3-8d56-08d88a118d68
X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Nov 2020 09:25:26.3943
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 462b3b3b-e42b-47ea-801a-f1581aac892d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: /8puLAupI1/8pF2DCkjtYSgfGdOPpq5a7gjaepOMEfIwAo+C+yf9a1JHcBTkTbBp3uq/7X8R3EYbRVkXsK5mlystqqSXMW/MPdXdwM0jZHU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB3664
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.312,18.0.737
 definitions=2020-11-16_03:2020-11-13,2020-11-16 signatures=0
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 malwarescore=0
 impostorscore=0 suspectscore=0 phishscore=0 mlxlogscore=999 clxscore=1011
 priorityscore=1501 bulkscore=0 mlxscore=0 spamscore=0 lowpriorityscore=0
 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2009150000 definitions=main-2011160055
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hello Jonathan,=0A=
=0A=
no problem for me, you can move it wherever you see fit.=0A=
=0A=
Thank you.=0A=
JB=0A=
=0A=
=0A=
From: Jonathan Cameron <jic23@kernel.org>=0A=
Sent: Saturday, November 14, 2020 16:23=0A=
To: Jean-Baptiste Maneyrol <JManeyrol@invensense.com>=0A=
Cc: robh+dt@kernel.org <robh+dt@kernel.org>; robh@kernel.org <robh@kernel.o=
rg>; mchehab+huawei@kernel.org <mchehab+huawei@kernel.org>; davem@davemloft=
.net <davem@davemloft.net>; gregkh@linuxfoundation.org <gregkh@linuxfoundat=
ion.org>; linux-iio@vger.kernel.org <linux-iio@vger.kernel.org>; devicetree=
@vger.kernel.org <devicetree@vger.kernel.org>; linux-kernel@vger.kernel.org=
 <linux-kernel@vger.kernel.org>=0A=
Subject: Re: [PATCH v4 08/13] Documentation: ABI: add specific icm42600 doc=
umentation =0A=
=A0=0A=
=A0CAUTION: This email originated from outside of the organization. Please =
make sure the sender is who they say they are and do not click links or ope=
n attachments unless you recognize the sender and know the content is safe.=
=0A=
=0A=
On Mon, 22 Jun 2020 17:37:24 +0200=0A=
Jean-Baptiste Maneyrol <jmaneyrol@invensense.com> wrote:=0A=
=0A=
> Hardware offset available as calibscale sysfs attributes are real=0A=
> physical values exprimed in SI units.=0A=
> =0A=
> calibscale_available sysfs attributes represents the range of=0A=
> acceptable values.=0A=
> =0A=
> Signed-off-by: Jean-Baptiste Maneyrol <jmaneyrol@invensense.com>=0A=
=0A=
Hi Jean-Baptiste. =0A=
=0A=
This is causing us some issues as the ABI docs are now added to the=0A=
generated html docs for the kernel.=A0 It's been a while, so I've kind=0A=
of forgotten why we have this file.=A0 Was the issue that these are=0A=
in SI units as opposed to most calibbias controls which are offsets=0A=
applied to the raw analog reading hitting the ADC?=0A=
=0A=
Would you mind if we moved this into the main doc as a note for this=0A=
particular device?=0A=
=0A=
i.e. something in sysfs-bus-iio saying=0A=
=0A=
icm42600: Hardware applied calibration offset is in SI units (rad/s or m/s^=
2 as appropriate)=0A=
=0A=
?=0A=
=0A=
Thanks,=0A=
=0A=
Jonathan=0A=
=A0=0A=
> ---=0A=
>=A0 .../ABI/testing/sysfs-bus-iio-icm42600=A0=A0=A0=A0=A0=A0=A0 | 20 +++++=
++++++++++++++=0A=
>=A0 1 file changed, 20 insertions(+)=0A=
>=A0 create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-icm42600=0A=
> =0A=
> diff --git a/Documentation/ABI/testing/sysfs-bus-iio-icm42600 b/Documenta=
tion/ABI/testing/sysfs-bus-iio-icm42600=0A=
> new file mode 100644=0A=
> index 000000000000..0bf1fd4f5bf1=0A=
> --- /dev/null=0A=
> +++ b/Documentation/ABI/testing/sysfs-bus-iio-icm42600=0A=
> @@ -0,0 +1,20 @@=0A=
> +What:=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 /sys/bus/iio/devices/=
iio:deviceX/in_accel_x_calibbias=0A=
> +What:=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 /sys/bus/iio/devices/=
iio:deviceX/in_accel_y_calibbias=0A=
> +What:=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 /sys/bus/iio/devices/=
iio:deviceX/in_accel_z_calibbias=0A=
> +What:=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 /sys/bus/iio/devices/=
iio:deviceX/in_anglvel_x_calibbias=0A=
> +What:=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 /sys/bus/iio/devices/=
iio:deviceX/in_anglvel_y_calibbias=0A=
> +What:=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 /sys/bus/iio/devices/=
iio:deviceX/in_anglvel_z_calibbias=0A=
> +KernelVersion:=A0 5.8=0A=
> +Contact:=A0=A0=A0=A0=A0=A0=A0 linux-iio@vger.kernel.org=0A=
> +Description:=0A=
> +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 Hardware applied calibration offset=
 (assumed to fix production=0A=
> +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 inaccuracies). Values represent a r=
eal physical offset expressed=0A=
> +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 in SI units (m/s^2 for acceleromete=
r and rad/s for gyroscope).=0A=
> +=0A=
> +What:=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 /sys/bus/iio/devices/=
iio:deviceX/in_accel_calibbias_available=0A=
> +What:=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 /sys/bus/iio/devices/=
iio:deviceX/in_anglvel_calibbias_available=0A=
> +KernelVersion:=A0 5.8=0A=
> +Contact:=A0=A0=A0=A0=A0=A0=A0 linux-iio@vger.kernel.org=0A=
> +Description:=0A=
> +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 Range of available values for hardw=
are offset. Values in SI=0A=
> +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 units (m/s^2 for accelerometer and =
rad/s for gyroscope).=0A=