Linux IIO development
 help / color / mirror / Atom feed
* [PATCH] iio: ABI: combine current input sections
@ 2024-12-19 19:25 David Lechner
  2024-12-20 20:09 ` Jonathan Cameron
  0 siblings, 1 reply; 2+ messages in thread
From: David Lechner @ 2024-12-19 19:25 UTC (permalink / raw)
  To: Jonathan Cameron, Lars-Peter Clausen
  Cc: linux-iio, linux-kernel, kernel test robot, David Lechner

Combine two duplicate sections describing in_currentY_raw.

This went unnoticed until we renamed in_currentX_raw to in_currentY_raw
and the kernel test robot found the duplication.

Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202412200256.OB5Hmw5Q-lkp@intel.com/
Signed-off-by: David Lechner <dlechner@baylibre.com>
---
 Documentation/ABI/testing/sysfs-bus-iio | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
index d97c35057a034c76ed433e3e6de0aff8ec4bbb70..25d366d452a55293c420300335a0271a28855107 100644
--- a/Documentation/ABI/testing/sysfs-bus-iio
+++ b/Documentation/ABI/testing/sysfs-bus-iio
@@ -168,18 +168,6 @@ Description:
 		is required is a consistent labeling.  Units after application
 		of scale and offset are millivolts.
 
-What:		/sys/bus/iio/devices/iio:deviceX/in_currentY_raw
-What:		/sys/bus/iio/devices/iio:deviceX/in_currentY_supply_raw
-KernelVersion:	3.17
-Contact:	linux-iio@vger.kernel.org
-Description:
-		Raw (unscaled no bias removal etc.) current measurement from
-		channel Y. In special cases where the channel does not
-		correspond to externally available input one of the named
-		versions may be used. The number must always be specified and
-		unique to allow association with event codes. Units after
-		application of scale and offset are milliamps.
-
 What:		/sys/bus/iio/devices/iio:deviceX/in_powerY_raw
 KernelVersion:	4.5
 Contact:	linux-iio@vger.kernel.org
@@ -1700,15 +1688,18 @@ Description:
 		or without compensation from tilt sensors.
 
 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_currentY_i_raw
 What:		/sys/bus/iio/devices/iio:deviceX/in_currentY_q_raw
-KernelVersion:	3.18
+KernelVersion:	3.17
 Contact:	linux-iio@vger.kernel.org
 Description:
 		Raw current measurement from channel Y. Units are in milliamps
 		after application of scale and offset. If no offset or scale is
 		present, output should be considered as processed with the
-		unit in milliamps.
+		unit in milliamps. In special cases where the channel does not
+		correspond to externally available input one of the named
+		versions may be used.
 
 		Channels with 'i' and 'q' modifiers always exist in pairs and both
 		channels refer to the same signal. The 'i' channel contains the in-phase

---
base-commit: fe2a04fbcbfe44694fcb36ff6212fce54b8f56cc
change-id: 20241219-iio-abi-combine-current-input-sections-c62cf5dbfa9b

Best regards,
-- 
David Lechner <dlechner@baylibre.com>


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] iio: ABI: combine current input sections
  2024-12-19 19:25 [PATCH] iio: ABI: combine current input sections David Lechner
@ 2024-12-20 20:09 ` Jonathan Cameron
  0 siblings, 0 replies; 2+ messages in thread
From: Jonathan Cameron @ 2024-12-20 20:09 UTC (permalink / raw)
  To: David Lechner
  Cc: Lars-Peter Clausen, linux-iio, linux-kernel, kernel test robot

On Thu, 19 Dec 2024 13:25:23 -0600
David Lechner <dlechner@baylibre.com> wrote:

> Combine two duplicate sections describing in_currentY_raw.
> 
> This went unnoticed until we renamed in_currentX_raw to in_currentY_raw
> and the kernel test robot found the duplication.
> 
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202412200256.OB5Hmw5Q-lkp@intel.com/
> Signed-off-by: David Lechner <dlechner@baylibre.com>
Applied.
Thanks,

J
> ---
>  Documentation/ABI/testing/sysfs-bus-iio | 19 +++++--------------
>  1 file changed, 5 insertions(+), 14 deletions(-)
> 
> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
> index d97c35057a034c76ed433e3e6de0aff8ec4bbb70..25d366d452a55293c420300335a0271a28855107 100644
> --- a/Documentation/ABI/testing/sysfs-bus-iio
> +++ b/Documentation/ABI/testing/sysfs-bus-iio
> @@ -168,18 +168,6 @@ Description:
>  		is required is a consistent labeling.  Units after application
>  		of scale and offset are millivolts.
>  
> -What:		/sys/bus/iio/devices/iio:deviceX/in_currentY_raw
> -What:		/sys/bus/iio/devices/iio:deviceX/in_currentY_supply_raw
> -KernelVersion:	3.17
> -Contact:	linux-iio@vger.kernel.org
> -Description:
> -		Raw (unscaled no bias removal etc.) current measurement from
> -		channel Y. In special cases where the channel does not
> -		correspond to externally available input one of the named
> -		versions may be used. The number must always be specified and
> -		unique to allow association with event codes. Units after
> -		application of scale and offset are milliamps.
> -
>  What:		/sys/bus/iio/devices/iio:deviceX/in_powerY_raw
>  KernelVersion:	4.5
>  Contact:	linux-iio@vger.kernel.org
> @@ -1700,15 +1688,18 @@ Description:
>  		or without compensation from tilt sensors.
>  
>  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_currentY_i_raw
>  What:		/sys/bus/iio/devices/iio:deviceX/in_currentY_q_raw
> -KernelVersion:	3.18
> +KernelVersion:	3.17
>  Contact:	linux-iio@vger.kernel.org
>  Description:
>  		Raw current measurement from channel Y. Units are in milliamps
>  		after application of scale and offset. If no offset or scale is
>  		present, output should be considered as processed with the
> -		unit in milliamps.
> +		unit in milliamps. In special cases where the channel does not
> +		correspond to externally available input one of the named
> +		versions may be used.
>  
>  		Channels with 'i' and 'q' modifiers always exist in pairs and both
>  		channels refer to the same signal. The 'i' channel contains the in-phase
> 
> ---
> base-commit: fe2a04fbcbfe44694fcb36ff6212fce54b8f56cc
> change-id: 20241219-iio-abi-combine-current-input-sections-c62cf5dbfa9b
> 
> Best regards,


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-12-20 20:09 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-19 19:25 [PATCH] iio: ABI: combine current input sections David Lechner
2024-12-20 20:09 ` Jonathan Cameron

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox