Re: [PATCH v8 01/17] iio: ABI: add attributes for altcurrent channels

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] `out_altcurrentY_raw` is incorrectly documented as an input measurement instead of an output setting.
--

--- Patch [1]: [PATCH v8 01/17] iio: ABI: add attributes for altcurrent channels ---
commit f95abfd3ec5c3cd46b3d0c4a84b154158053f62c
Author: Rodrigo Alencar <rodrigo.alencar@analog.com>

iio: ABI: add attributes for altcurrent channels
    
Add attributes applicable to altcurrent channels, e.g., frequency, phase,
scale, raw, label and powerdown. Those are going to be used by a DDS
device to be introduced shortly.

> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
> index 5cc2e82c4997e..90047189c3ebe 100644
> --- a/Documentation/ABI/testing/sysfs-bus-iio
> +++ b/Documentation/ABI/testing/sysfs-bus-iio

[ ... ]

> @@ -1736,6 +1743,8 @@ Description:
>  
>  What:		/sys/bus/iio/devices/iio:deviceX/in_currentY_raw
>  What:		/sys/bus/iio/devices/iio:deviceX/in_currentY_supply_raw
> +What:		/sys/bus/iio/devices/iio:deviceX/in_altcurrentY_raw
> +What:		/sys/bus/iio/devices/iio:deviceX/out_altcurrentY_raw

[Severity: Low]
Is out_altcurrentY_raw grouped under the wrong description here?

Since it is an output attribute, having it fall under the "Raw current
measurement from channel Y." description below seems inconsistent. Should this
be separated into its own section or grouped with an output setting?

>  KernelVersion:	3.17
>  Contact:	linux-iio@vger.kernel.org
>  Description:

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260714-ad9910-iio-driver-v8-0-36939c3c07d2@analog.com?part=1