Linux IIO development
 help / color / mirror / Atom feed
* [PATCH v2] iio: imu: fix typos in comments
       [not found] <20260701151811.22130-1-anxkhn28@gmail.com>
@ 2026-07-01 16:26 ` Anas Khan
  2026-07-01 17:49   ` Jonathan Cameron
  0 siblings, 1 reply; 4+ messages in thread
From: Anas Khan @ 2026-07-01 16:26 UTC (permalink / raw)
  To: Nuno Sá, Michael Hennerich, Jonathan Cameron,
	Jean-Baptiste Maneyrol, Remi Buisson
  Cc: Andy Shevchenko, David Lechner, linux, linux-iio, linux-kernel,
	Anas Khan

Fix several comment misspellings across drivers/iio/imu/, found with
codespell and manual review. The corrected words are coefficient,
temperature, sensitivity, access, chosen, and buses.

Signed-off-by: Anas Khan <anxkhn28@gmail.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>
---
v2: expand from the single adis16480 temperature fix (v1) to cover all
    comment typos in drivers/iio/imu/, as requested by Andy Shevchenko.
    The Reviewed-by is carried over as the changes remain trivial typo
    fixes.
 drivers/iio/imu/adis16480.c                        | 4 ++--
 drivers/iio/imu/bmi160/bmi160_core.c               | 2 +-
 drivers/iio/imu/fxos8700_core.c                    | 2 +-
 drivers/iio/imu/inv_icm42600/inv_icm42600_buffer.c | 2 +-
 drivers/iio/imu/inv_icm45600/inv_icm45600.h        | 2 +-
 drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c         | 4 ++--
 6 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/iio/imu/adis16480.c b/drivers/iio/imu/adis16480.c
index 543d5c4bfb11..c7163540cd02 100644
--- a/drivers/iio/imu/adis16480.c
+++ b/drivers/iio/imu/adis16480.c
@@ -111,7 +111,7 @@
 
 #define ADIS16480_REG_SERIAL_NUM		ADIS16480_REG(0x04, 0x20)
 
-/* Each filter coefficent bank spans two pages */
+/* Each filter coefficient bank spans two pages */
 #define ADIS16480_FIR_COEF(page) (x < 60 ? ADIS16480_REG(page, (x) + 8) : \
 		ADIS16480_REG((page) + 1, (x) - 60 + 8))
 #define ADIS16480_FIR_COEF_A(x)			ADIS16480_FIR_COEF(0x05, (x))
@@ -1468,7 +1468,7 @@ static irqreturn_t adis16480_trigger_handler(int irq, void *p)
 			 * We need to perform the padding to have the buffer
 			 * elements naturally aligned in case there are any
 			 * 32-bit storage size channels enabled which are added
-			 * in the buffer after the temprature data. In case
+			 * in the buffer after the temperature data. In case
 			 * there is no data being added after the temperature
 			 * data, the padding is harmless.
 			 */
diff --git a/drivers/iio/imu/bmi160/bmi160_core.c b/drivers/iio/imu/bmi160/bmi160_core.c
index 4abb83b75e2e..2c0d21eb853a 100644
--- a/drivers/iio/imu/bmi160/bmi160_core.c
+++ b/drivers/iio/imu/bmi160/bmi160_core.c
@@ -5,7 +5,7 @@
  * Copyright (c) 2016, Intel Corporation.
  * Copyright (c) 2019, Martin Kelly.
  *
- * IIO core driver for BMI160, with support for I2C/SPI busses
+ * IIO core driver for BMI160, with support for I2C/SPI buses
  *
  * TODO: magnetometer, hardware FIFO
  */
diff --git a/drivers/iio/imu/fxos8700_core.c b/drivers/iio/imu/fxos8700_core.c
index 281ebfd9c15a..9d2eb6fbab4a 100644
--- a/drivers/iio/imu/fxos8700_core.c
+++ b/drivers/iio/imu/fxos8700_core.c
@@ -2,7 +2,7 @@
 /*
  * FXOS8700 - NXP IMU (accelerometer plus magnetometer)
  *
- * IIO core driver for FXOS8700, with support for I2C/SPI busses
+ * IIO core driver for FXOS8700, with support for I2C/SPI buses
  *
  * TODO: Buffer, trigger, and IRQ support
  */
diff --git a/drivers/iio/imu/inv_icm42600/inv_icm42600_buffer.c b/drivers/iio/imu/inv_icm42600/inv_icm42600_buffer.c
index 68a395758031..91b7f1ecba55 100644
--- a/drivers/iio/imu/inv_icm42600/inv_icm42600_buffer.c
+++ b/drivers/iio/imu/inv_icm42600/inv_icm42600_buffer.c
@@ -186,7 +186,7 @@ static unsigned int inv_icm42600_wm_truncate(unsigned int watermark,
  * smallest latency but this is not as simple as choosing the smallest watermark
  * value. Latency depends on watermark and ODR. It requires several steps:
  * 1) compute gyro and accel latencies and choose the smallest value.
- * 2) adapt the choosen latency so that it is a multiple of both gyro and accel
+ * 2) adapt the chosen latency so that it is a multiple of both gyro and accel
  *    ones. Otherwise it is possible that you don't meet a requirement. (for
  *    example with gyro @100Hz wm 4 and accel @100Hz with wm 6, choosing the
  *    value of 4 will not meet accel latency requirement because 6 is not a
diff --git a/drivers/iio/imu/inv_icm45600/inv_icm45600.h b/drivers/iio/imu/inv_icm45600/inv_icm45600.h
index 1c796d4b2a40..95fa934a42b4 100644
--- a/drivers/iio/imu/inv_icm45600/inv_icm45600.h
+++ b/drivers/iio/imu/inv_icm45600/inv_icm45600.h
@@ -190,7 +190,7 @@ struct inv_icm45600_sensor_state {
 #define INV_ICM45600_REG_IREG_ADDR			0x7C
 #define INV_ICM45600_REG_IREG_DATA			0x7E
 
-/* Direct acces registers */
+/* Direct access registers */
 #define INV_ICM45600_REG_MISC2				0x007F
 #define INV_ICM45600_MISC2_SOFT_RESET			BIT(1)
 
diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c b/drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c
index 47394594d17a..6b858fdfd1c6 100644
--- a/drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c
+++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c
@@ -106,9 +106,9 @@ static int inv_magn_init(struct inv_mpu6050_state *st)
 		return ret;
 
 	/*
-	 * Sensor sentivity
+	 * Sensor sensitivity
 	 * 1 uT = 0.01 G and value is in micron (1e6)
-	 * sensitvity = x uT * 0.01 * 1e6
+	 * sensitivity = x uT * 0.01 * 1e6
 	 */
 	switch (st->chip_type) {
 	case INV_MPU9150:
-- 
2.54.0


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

* Re: [PATCH v2] iio: imu: fix typos in comments
  2026-07-01 16:26 ` [PATCH v2] iio: imu: fix typos in comments Anas Khan
@ 2026-07-01 17:49   ` Jonathan Cameron
  2026-07-02  7:22     ` Andy Shevchenko
  0 siblings, 1 reply; 4+ messages in thread
From: Jonathan Cameron @ 2026-07-01 17:49 UTC (permalink / raw)
  To: Anas Khan
  Cc: Nuno Sá, Michael Hennerich, Jean-Baptiste Maneyrol,
	Remi Buisson, Andy Shevchenko, David Lechner, linux, linux-iio,
	linux-kernel

On Wed,  1 Jul 2026 21:56:59 +0530
Anas Khan <anxkhn28@gmail.com> wrote:

> Fix several comment misspellings across drivers/iio/imu/, found with
> codespell and manual review. The corrected words are coefficient,
> temperature, sensitivity, access, chosen, and buses.
> 
> Signed-off-by: Anas Khan <anxkhn28@gmail.com>
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>

Andy can confirm, but generally adding more typo fixes that Andy
has not reviewed would be a reason to drop his RB tag.

Changes look fine, but I'll wait on a reply from Andy to apply
this.

Thanks,

Jonathan

> ---
> v2: expand from the single adis16480 temperature fix (v1) to cover all
>     comment typos in drivers/iio/imu/, as requested by Andy Shevchenko.
>     The Reviewed-by is carried over as the changes remain trivial typo
>     fixes.
>  drivers/iio/imu/adis16480.c                        | 4 ++--
>  drivers/iio/imu/bmi160/bmi160_core.c               | 2 +-
>  drivers/iio/imu/fxos8700_core.c                    | 2 +-
>  drivers/iio/imu/inv_icm42600/inv_icm42600_buffer.c | 2 +-
>  drivers/iio/imu/inv_icm45600/inv_icm45600.h        | 2 +-
>  drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c         | 4 ++--
>  6 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/iio/imu/adis16480.c b/drivers/iio/imu/adis16480.c
> index 543d5c4bfb11..c7163540cd02 100644
> --- a/drivers/iio/imu/adis16480.c
> +++ b/drivers/iio/imu/adis16480.c
> @@ -111,7 +111,7 @@
>  
>  #define ADIS16480_REG_SERIAL_NUM		ADIS16480_REG(0x04, 0x20)
>  
> -/* Each filter coefficent bank spans two pages */
> +/* Each filter coefficient bank spans two pages */
>  #define ADIS16480_FIR_COEF(page) (x < 60 ? ADIS16480_REG(page, (x) + 8) : \
>  		ADIS16480_REG((page) + 1, (x) - 60 + 8))
>  #define ADIS16480_FIR_COEF_A(x)			ADIS16480_FIR_COEF(0x05, (x))
> @@ -1468,7 +1468,7 @@ static irqreturn_t adis16480_trigger_handler(int irq, void *p)
>  			 * We need to perform the padding to have the buffer
>  			 * elements naturally aligned in case there are any
>  			 * 32-bit storage size channels enabled which are added
> -			 * in the buffer after the temprature data. In case
> +			 * in the buffer after the temperature data. In case
>  			 * there is no data being added after the temperature
>  			 * data, the padding is harmless.
>  			 */
> diff --git a/drivers/iio/imu/bmi160/bmi160_core.c b/drivers/iio/imu/bmi160/bmi160_core.c
> index 4abb83b75e2e..2c0d21eb853a 100644
> --- a/drivers/iio/imu/bmi160/bmi160_core.c
> +++ b/drivers/iio/imu/bmi160/bmi160_core.c
> @@ -5,7 +5,7 @@
>   * Copyright (c) 2016, Intel Corporation.
>   * Copyright (c) 2019, Martin Kelly.
>   *
> - * IIO core driver for BMI160, with support for I2C/SPI busses
> + * IIO core driver for BMI160, with support for I2C/SPI buses
>   *
>   * TODO: magnetometer, hardware FIFO
>   */
> diff --git a/drivers/iio/imu/fxos8700_core.c b/drivers/iio/imu/fxos8700_core.c
> index 281ebfd9c15a..9d2eb6fbab4a 100644
> --- a/drivers/iio/imu/fxos8700_core.c
> +++ b/drivers/iio/imu/fxos8700_core.c
> @@ -2,7 +2,7 @@
>  /*
>   * FXOS8700 - NXP IMU (accelerometer plus magnetometer)
>   *
> - * IIO core driver for FXOS8700, with support for I2C/SPI busses
> + * IIO core driver for FXOS8700, with support for I2C/SPI buses
>   *
>   * TODO: Buffer, trigger, and IRQ support
>   */
> diff --git a/drivers/iio/imu/inv_icm42600/inv_icm42600_buffer.c b/drivers/iio/imu/inv_icm42600/inv_icm42600_buffer.c
> index 68a395758031..91b7f1ecba55 100644
> --- a/drivers/iio/imu/inv_icm42600/inv_icm42600_buffer.c
> +++ b/drivers/iio/imu/inv_icm42600/inv_icm42600_buffer.c
> @@ -186,7 +186,7 @@ static unsigned int inv_icm42600_wm_truncate(unsigned int watermark,
>   * smallest latency but this is not as simple as choosing the smallest watermark
>   * value. Latency depends on watermark and ODR. It requires several steps:
>   * 1) compute gyro and accel latencies and choose the smallest value.
> - * 2) adapt the choosen latency so that it is a multiple of both gyro and accel
> + * 2) adapt the chosen latency so that it is a multiple of both gyro and accel
>   *    ones. Otherwise it is possible that you don't meet a requirement. (for
>   *    example with gyro @100Hz wm 4 and accel @100Hz with wm 6, choosing the
>   *    value of 4 will not meet accel latency requirement because 6 is not a
> diff --git a/drivers/iio/imu/inv_icm45600/inv_icm45600.h b/drivers/iio/imu/inv_icm45600/inv_icm45600.h
> index 1c796d4b2a40..95fa934a42b4 100644
> --- a/drivers/iio/imu/inv_icm45600/inv_icm45600.h
> +++ b/drivers/iio/imu/inv_icm45600/inv_icm45600.h
> @@ -190,7 +190,7 @@ struct inv_icm45600_sensor_state {
>  #define INV_ICM45600_REG_IREG_ADDR			0x7C
>  #define INV_ICM45600_REG_IREG_DATA			0x7E
>  
> -/* Direct acces registers */
> +/* Direct access registers */
>  #define INV_ICM45600_REG_MISC2				0x007F
>  #define INV_ICM45600_MISC2_SOFT_RESET			BIT(1)
>  
> diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c b/drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c
> index 47394594d17a..6b858fdfd1c6 100644
> --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c
> +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c
> @@ -106,9 +106,9 @@ static int inv_magn_init(struct inv_mpu6050_state *st)
>  		return ret;
>  
>  	/*
> -	 * Sensor sentivity
> +	 * Sensor sensitivity
>  	 * 1 uT = 0.01 G and value is in micron (1e6)
> -	 * sensitvity = x uT * 0.01 * 1e6
> +	 * sensitivity = x uT * 0.01 * 1e6
>  	 */
>  	switch (st->chip_type) {
>  	case INV_MPU9150:


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

* Re: [PATCH v2] iio: imu: fix typos in comments
  2026-07-01 17:49   ` Jonathan Cameron
@ 2026-07-02  7:22     ` Andy Shevchenko
  2026-07-02 17:07       ` Jonathan Cameron
  0 siblings, 1 reply; 4+ messages in thread
From: Andy Shevchenko @ 2026-07-02  7:22 UTC (permalink / raw)
  To: Jonathan Cameron
  Cc: Anas Khan, Nuno Sá, Michael Hennerich,
	Jean-Baptiste Maneyrol, Remi Buisson, David Lechner, linux,
	linux-iio, linux-kernel

On Wed, Jul 01, 2026 at 06:49:35PM +0100, Jonathan Cameron wrote:
> On Wed,  1 Jul 2026 21:56:59 +0530
> Anas Khan <anxkhn28@gmail.com> wrote:
> 
> > Fix several comment misspellings across drivers/iio/imu/, found with
> > codespell and manual review. The corrected words are coefficient,
> > temperature, sensitivity, access, chosen, and buses.
> > 
> > Signed-off-by: Anas Khan <anxkhn28@gmail.com>
> > Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>
> 
> Andy can confirm, but generally adding more typo fixes that Andy
> has not reviewed would be a reason to drop his RB tag.

Right, please don't add tags when the excessive changes been made.

> Changes look fine, but I'll wait on a reply from Andy to apply
> this.

...

Yeah, everything looks fine.

-- 
With Best Regards,
Andy Shevchenko



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

* Re: [PATCH v2] iio: imu: fix typos in comments
  2026-07-02  7:22     ` Andy Shevchenko
@ 2026-07-02 17:07       ` Jonathan Cameron
  0 siblings, 0 replies; 4+ messages in thread
From: Jonathan Cameron @ 2026-07-02 17:07 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Anas Khan, Nuno Sá, Michael Hennerich,
	Jean-Baptiste Maneyrol, Remi Buisson, David Lechner, linux,
	linux-iio, linux-kernel

On Thu, 2 Jul 2026 10:22:21 +0300
Andy Shevchenko <andriy.shevchenko@intel.com> wrote:

> On Wed, Jul 01, 2026 at 06:49:35PM +0100, Jonathan Cameron wrote:
> > On Wed,  1 Jul 2026 21:56:59 +0530
> > Anas Khan <anxkhn28@gmail.com> wrote:
> >   
> > > Fix several comment misspellings across drivers/iio/imu/, found with
> > > codespell and manual review. The corrected words are coefficient,
> > > temperature, sensitivity, access, chosen, and buses.
> > > 
> > > Signed-off-by: Anas Khan <anxkhn28@gmail.com>
> > > Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>  
> > 
> > Andy can confirm, but generally adding more typo fixes that Andy
> > has not reviewed would be a reason to drop his RB tag.  
> 
> Right, please don't add tags when the excessive changes been made.
> 
> > Changes look fine, but I'll wait on a reply from Andy to apply
> > this.  
> 
> ...
> 
> Yeah, everything looks fine.
> 


Applied to the testing branch of iio.git with Andy's tag
there given the looks fine above.

Thanks,

Jonathan

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

end of thread, other threads:[~2026-07-02 17:07 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20260701151811.22130-1-anxkhn28@gmail.com>
2026-07-01 16:26 ` [PATCH v2] iio: imu: fix typos in comments Anas Khan
2026-07-01 17:49   ` Jonathan Cameron
2026-07-02  7:22     ` Andy Shevchenko
2026-07-02 17:07       ` Jonathan Cameron

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