[PATCH] Documentation: ABI: testing: iio: minor cleanup

[PATCH] Documentation: ABI: testing: iio: minor cleanup

[Manuel Ebner]

Add parenthesis and dashes.

Signed-off-by: Manuel Ebner <manuelebner@mailbox.org>
---
 Documentation/ABI/testing/sysfs-bus-iio            | 10 +++++-----
 Documentation/ABI/testing/sysfs-bus-iio-adc-ad7192 |  2 +-
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
index d8d6d85235b0..4ea5598e7cd2 100644
--- a/Documentation/ABI/testing/sysfs-bus-iio
+++ b/Documentation/ABI/testing/sysfs-bus-iio
@@ -917,7 +917,7 @@ Description:
 
 		Note the driver will assume the last p events requested are
 		to be enabled where p is how many it supports (which may vary
-		depending on the exact set requested. So if you want to be
+		depending on the exact set requested). So if you want to be
 		sure you have set what you think you have, check the contents of
 		these attributes after everything is configured. Drivers may
 		have to buffer any parameters so that they are consistent when
@@ -973,7 +973,7 @@ Description:
 
 		Note the driver will assume the last p events requested are
 		to be enabled where p is however many it supports (which may
-		vary depending on the exact set requested. So if you want to be
+		vary depending on the exact set requested). So if you want to be
 		sure you have set what you think you have, check the contents of
 		these attributes after everything is configured. Drivers may
 		have to buffer any parameters so that they are consistent when
@@ -1528,9 +1528,9 @@ Description:
 		the unused bits, so to get a clean value the bits value must be
 		used to mask the buffer output value appropriately. The storagebits
 		value also specifies the data alignment. So s48/64>>2 will be a
-		signed 48 bit integer stored in a 64 bit location aligned to a 64
-		bit boundary. To obtain the clean value, shift right 2 and apply a
-		mask to zero the top 16 bits of the result.
+		signed 48-bit integer stored in a 64-bit location aligned to a
+		64-bit boundary. To obtain the clean value, shift right 2 and apply
+		a mask to zero the top 16 bits of the result.
 		For other storage combinations this attribute will be extended
 		appropriately.
 
diff --git a/Documentation/ABI/testing/sysfs-bus-iio-adc-ad7192 b/Documentation/ABI/testing/sysfs-bus-iio-adc-ad7192
index 28be1cabf112..d44eb5f8728b 100644
--- a/Documentation/ABI/testing/sysfs-bus-iio-adc-ad7192
+++ b/Documentation/ABI/testing/sysfs-bus-iio-adc-ad7192
@@ -16,7 +16,7 @@ Description:
 		In bridge applications, such as strain gauges and load cells,
 		the bridge itself consumes the majority of the current in the
 		system. To minimize the current consumption of the system,
-		the bridge can be disconnected (when it is not being used
+		the bridge can be disconnected (when it is not being used)
 		using the bridge_switch_en attribute.
 
 What:		/sys/bus/iio/devices/iio:deviceX/in_voltage2-voltage2_shorted_raw
-- 
2.54.0

Re: [PATCH] Documentation: ABI: testing: iio: minor cleanup

[Andy Shevchenko]

On Fri, Jun 12, 2026 at 02:13:43PM +0200, Manuel Ebner wrote:
> Add parenthesis and dashes.

LGTM,
Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>

...

>  		used to mask the buffer output value appropriately. The storagebits
>  		value also specifies the data alignment. So s48/64>>2 will be a
> -		signed 48 bit integer stored in a 64 bit location aligned to a 64
> -		bit boundary. To obtain the clean value, shift right 2 and apply a
> -		mask to zero the top 16 bits of the result.
> +		signed 48-bit integer stored in a 64-bit location aligned to a
> +		64-bit boundary. To obtain the clean value, shift right 2 and apply
> +		a mask to zero the top 16 bits of the result.

We can reduce a churn by leaving 'a' as it was before (despite being long
line). But I leave this exercise to Jonathan if he wants to.

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH] Documentation: ABI: testing: iio: minor cleanup

[Jonathan Cameron]

On Wed, 17 Jun 2026 09:52:29 +0300
Andy Shevchenko <andriy.shevchenko@intel.com> wrote:

> On Fri, Jun 12, 2026 at 02:13:43PM +0200, Manuel Ebner wrote:
> > Add parenthesis and dashes.  
> 
> LGTM,
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>
> 
> ...
> 
> >  		used to mask the buffer output value appropriately. The storagebits
> >  		value also specifies the data alignment. So s48/64>>2 will be a
> > -		signed 48 bit integer stored in a 64 bit location aligned to a 64
> > -		bit boundary. To obtain the clean value, shift right 2 and apply a
> > -		mask to zero the top 16 bits of the result.
> > +		signed 48-bit integer stored in a 64-bit location aligned to a
> > +		64-bit boundary. To obtain the clean value, shift right 2 and apply
> > +		a mask to zero the top 16 bits of the result.  
> 
> We can reduce a churn by leaving 'a' as it was before (despite being long
> line). But I leave this exercise to Jonathan if he wants to.

True enough but I'm feeling lazy today (too hot and getting hotter :()

Applied.

Thanks,

Jonathan

>