* [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