| 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
| ||
|
Message-ID: <20250927165100.5a9d3136@jic23-huawei>
Date: Sat, 27 Sep 2025 16:51:00 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: Andy Shevchenko <andriy.shevchenko@...el.com>
Cc: Gustavo Silva <gustavograzs@...il.com>, Alex Lanzano
<lanzano.alex@...il.com>, David Lechner <dlechner@...libre.com>, Nuno
Sá <nuno.sa@...log.com>, Andy Shevchenko <andy@...nel.org>,
linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v6 2/2] iio: ABI: document accelerometer event
attributes
On Wed, 17 Sep 2025 11:47:30 +0300
Andy Shevchenko <andriy.shevchenko@...el.com> wrote:
> On Tue, Sep 16, 2025 at 08:38:24PM -0300, Gustavo Silva wrote:
> > Add ABI documentation for accelerometer event-related sysfs attributes
> > exposed by the bmi270 driver. These include threshold, period, and
> > enable controls for adaptive magnitude (any-motion) and rate of change
> > (no-motion) event detection.
>
> ...
>
> > What: /sys/.../iio:deviceX/events/in_accel_y_roc_rising_en
>
> > What: /sys/.../iio:deviceX/events/in_accel_y_roc_falling_en
> > What: /sys/.../iio:deviceX/events/in_accel_z_roc_rising_en
> > What: /sys/.../iio:deviceX/events/in_accel_z_roc_falling_en
> > +What: /sys/.../iio:deviceX/events/in_accel_x&y&z_roc_rising_en
> > What: /sys/.../iio:deviceX/events/in_anglvel_x_roc_rising_en
> > What: /sys/.../iio:deviceX/events/in_anglvel_x_roc_falling_en
> > What: /sys/.../iio:deviceX/events/in_anglvel_y_roc_rising_en
>
> > What: /sys/.../events/in_accel_y_raw_thresh_rising_value
>
> > What: /sys/.../events/in_accel_y_raw_thresh_falling_value
> > What: /sys/.../events/in_accel_z_raw_thresh_rising_value
> > What: /sys/.../events/in_accel_z_raw_thresh_falling_value
> > +What: /sys/.../events/in_accel_mag_adaptive_rising_value
> > What: /sys/.../events/in_anglvel_x_raw_thresh_rising_value
> > What: /sys/.../events/in_anglvel_x_raw_thresh_falling_value
> > What: /sys/.../events/in_anglvel_y_raw_thresh_rising_value
>
> > +What: /sys/.../events/in_accel_roc_rising_value
> > What: /sys/.../events/in_accel_x_raw_roc_rising_value
> > What: /sys/.../events/in_accel_x_raw_roc_falling_value
> > What: /sys/.../events/in_accel_y_raw_roc_rising_value
>
> > What: /sys/.../events/in_accel_x_thresh_rising_period
> > What: /sys/.../events/in_accel_x_thresh_falling_period
> > +What: /sys/.../events/in_accel_roc_rising_period
> > What: /sys/.../events/in_accel_x_roc_rising_period
> > What: /sys/.../events/in_accel_x_roc_falling_period
> > What: /sys/.../events/in_accel_y_thresh_rising_period
>
> With the given context (above and below) I'm not sure this is the best place of
> putting an attribute in the list. Perhaps something below will not disrupt the
> xyz structure (like mag cases)?
I took a look at the broader context and it was a bit tricky but I think this
is the best we can do to maintain current ordering (which is fuzzily defined at best).
At some point that whole doc needs a rethink wrt to how we break the different blocks
up and order the elements within them.
With this change, series applied to the testing branch of iio.git.
That will be rebased on rc1 once available and then become the togreg branch.
thanks,
Jonathan
diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
index 89b4740dcfa1..352ab7b8476c 100644
--- a/Documentation/ABI/testing/sysfs-bus-iio
+++ b/Documentation/ABI/testing/sysfs-bus-iio
@@ -926,6 +926,7 @@ What: /sys/.../iio:deviceX/events/in_accel_y_roc_rising_en
What: /sys/.../iio:deviceX/events/in_accel_y_roc_falling_en
What: /sys/.../iio:deviceX/events/in_accel_z_roc_rising_en
What: /sys/.../iio:deviceX/events/in_accel_z_roc_falling_en
+What: /sys/.../iio:deviceX/events/in_accel_x&y&z_roc_rising_en
What: /sys/.../iio:deviceX/events/in_anglvel_x_roc_rising_en
What: /sys/.../iio:deviceX/events/in_anglvel_x_roc_falling_en
What: /sys/.../iio:deviceX/events/in_anglvel_y_roc_rising_en
@@ -1001,6 +1002,7 @@ Description:
to the raw signal, allowing slow tracking to resume and the
adaptive threshold event detection to function as expected.
+What: /sys/.../events/in_accel_mag_adaptive_rising_value
What: /sys/.../events/in_accel_thresh_rising_value
What: /sys/.../events/in_accel_thresh_falling_value
What: /sys/.../events/in_accel_x_raw_thresh_rising_value
@@ -1147,6 +1149,7 @@ Description:
will get activated once in_voltage0_raw goes above 1200 and will become
deactivated again once the value falls below 1150.
+What: /sys/.../events/in_accel_roc_rising_value
What: /sys/.../events/in_accel_x_raw_roc_rising_value
What: /sys/.../events/in_accel_x_raw_roc_falling_value
What: /sys/.../events/in_accel_y_raw_roc_rising_value
@@ -1193,6 +1196,8 @@ Description:
value is in raw device units or in processed units (as _raw
and _input do on sysfs direct channel read attributes).
+What: /sys/.../events/in_accel_mag_adaptive_rising_period
+What: /sys/.../events/in_accel_roc_rising_period
What: /sys/.../events/in_accel_x_thresh_rising_period
What: /sys/.../events/in_accel_x_thresh_falling_period
What: /sys/.../events/in_accel_x_roc_rising_period
@@ -1362,6 +1367,15 @@ Description:
number or direction is not specified, applies to all channels of
this type.
+What: /sys/.../iio:deviceX/events/in_accel_x_mag_adaptive_rising_en
t
>
> > What: /sys/.../events/in_accel_z_thresh_rising_period
>
> > What: /sys/.../events/in_accel_z_thresh_falling_period
> > What: /sys/.../events/in_accel_z_roc_rising_period
> > What: /sys/.../events/in_accel_z_roc_falling_period
> > +What: /sys/.../events/in_accel_mag_adaptive_rising_period
> > What: /sys/.../events/in_anglvel_x_thresh_rising_period
> > What: /sys/.../events/in_anglvel_x_thresh_falling_period
> > What: /sys/.../events/in_anglvel_x_roc_rising_period
>
Powered by blists - more mailing lists