linux-kernel - Re: [v3 05/13] iio: document pitch, yaw, roll modifiers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20220219163149.6c33318c@jic23-huawei>
Date:   Sat, 19 Feb 2022 16:31:49 +0000
From:   Jonathan Cameron <jic23@...nel.org>
To:     Andrea Merello <andrea.merello@...il.com>
Cc:     mchehab+huawei@...nel.org, linux-iio@...r.kernel.org,
        linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
        lars@...afoo.de, robh+dt@...nel.org, andy.shevchenko@...il.com,
        matt.ranostay@...sulko.com, ardeleanalex@...il.com,
        jacopo@...ndi.org, Andrea Merello <andrea.merello@....it>
Subject: Re: [v3 05/13] iio: document pitch, yaw, roll modifiers

On Thu, 17 Feb 2022 17:27:02 +0100
Andrea Merello <andrea.merello@...il.com> wrote:

> This patch introduces ABI documentation for new modifiers used for
> reporting rotations expressed as euler angles (i.e. yaw, pitch, roll).
> 
> Signed-off-by: Andrea Merello <andrea.merello@....it>
I've lost track of what we already discussed about this, but why have these
as measured in degrees?

Gah. I thought we'd maintained consistency on this, but seems incli is
in degrees, angl is in radians and rot isn't documented at all.

Checking drivers the adis16209 has rot in degrees and I can't immediately figure
out what the hid sensors driver is using.  The underlying spec supports the
hardware returning in either radians or degrees I think. 

Ah well, at least these are new so given the existing mess means at least some
units are already in degrees we aren't making things worse.

Perhaps it's worth a comment on the unit inconsistency in this patch description
so we can at least look back at the history if it comes up for a future driver.

Thanks,

Jonathan


> ---
>  Documentation/ABI/testing/sysfs-bus-iio | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
> index b2fb4d9abcd1..1b8d77577608 100644
> --- a/Documentation/ABI/testing/sysfs-bus-iio
> +++ b/Documentation/ABI/testing/sysfs-bus-iio
> @@ -2008,3 +2008,12 @@ Description:
>  		Available range for the forced calibration value, expressed as:
>  
>  		- a range specified as "[min step max]"
> +
> +What:		/sys/bus/iio/devices/iio:deviceX/in_rot_yaw_raw
> +What:		/sys/bus/iio/devices/iio:deviceX/in_rot_pitch_raw
> +What:		/sys/bus/iio/devices/iio:deviceX/in_rot_roll_raw
> +KernelVersion:	5.17
> +Contact:	linux-iio@...r.kernel.org
> +Description:
> +		Raw (unscaled) euler angles readings. Units after
> +		application of scale are deg.