[PATCH v1] iio: adc: Fix typos in comments across various files

Yu Jiaoliang posted 1 patch 2 months ago
drivers/iio/adc/ad7091r-base.h       | 2 +-
drivers/iio/adc/ad7606.h             | 2 +-
drivers/iio/adc/ad7887.c             | 2 +-
drivers/iio/adc/ad_sigma_delta.c     | 4 ++--
drivers/iio/adc/max34408.c           | 2 +-
drivers/iio/adc/pac1921.c            | 2 +-
drivers/iio/adc/palmas_gpadc.c       | 2 +-
drivers/iio/adc/ti-ads1298.c         | 2 +-
drivers/iio/adc/ti_am335x_adc.c      | 2 +-
drivers/iio/adc/twl4030-madc.c       | 2 +-
drivers/iio/adc/xilinx-xadc-events.c | 2 +-
11 files changed, 12 insertions(+), 12 deletions(-)
[PATCH v1] iio: adc: Fix typos in comments across various files
Posted by Yu Jiaoliang 2 months ago
This commit fixes several typographical errors in comments within
the drivers/iio/adc directory. No functional changes are made.

Detected using codespell.

Signed-off-by: Yu Jiaoliang <yujiaoliang@vivo.com>
---
 drivers/iio/adc/ad7091r-base.h       | 2 +-
 drivers/iio/adc/ad7606.h             | 2 +-
 drivers/iio/adc/ad7887.c             | 2 +-
 drivers/iio/adc/ad_sigma_delta.c     | 4 ++--
 drivers/iio/adc/max34408.c           | 2 +-
 drivers/iio/adc/pac1921.c            | 2 +-
 drivers/iio/adc/palmas_gpadc.c       | 2 +-
 drivers/iio/adc/ti-ads1298.c         | 2 +-
 drivers/iio/adc/ti_am335x_adc.c      | 2 +-
 drivers/iio/adc/twl4030-madc.c       | 2 +-
 drivers/iio/adc/xilinx-xadc-events.c | 2 +-
 11 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/drivers/iio/adc/ad7091r-base.h b/drivers/iio/adc/ad7091r-base.h
index 696bf7a897bb..092ddea0f395 100644
--- a/drivers/iio/adc/ad7091r-base.h
+++ b/drivers/iio/adc/ad7091r-base.h
@@ -65,7 +65,7 @@ struct ad7091r_state {
 	struct regulator *vref;
 	const struct ad7091r_chip_info *chip_info;
 	enum ad7091r_mode mode;
-	struct mutex lock; /*lock to prevent concurent reads */
+	struct mutex lock; /*lock to prevent concurrent reads */
 	__be16 tx_buf __aligned(IIO_DMA_MINALIGN);
 	__be16 rx_buf;
 };
diff --git a/drivers/iio/adc/ad7606.h b/drivers/iio/adc/ad7606.h
index 6649e84d25de..02195bedb574 100644
--- a/drivers/iio/adc/ad7606.h
+++ b/drivers/iio/adc/ad7606.h
@@ -46,7 +46,7 @@
  *			oversampling ratios.
  * @oversampling_num	number of elements stored in oversampling_avail array
  * @os_req_reset	some devices require a reset to update oversampling
- * @init_delay_ms	required delay in miliseconds for initialization
+ * @init_delay_ms	required delay in milliseconds for initialization
  *			after a restart
  */
 struct ad7606_chip_info {
diff --git a/drivers/iio/adc/ad7887.c b/drivers/iio/adc/ad7887.c
index b301da9b88b1..69add1dc4b53 100644
--- a/drivers/iio/adc/ad7887.c
+++ b/drivers/iio/adc/ad7887.c
@@ -41,7 +41,7 @@ enum ad7887_channels {
 };
 
 /**
- * struct ad7887_chip_info - chip specifc information
+ * struct ad7887_chip_info - chip specific information
  * @int_vref_mv:	the internal reference voltage
  * @channels:		channels specification
  * @num_channels:	number of channels
diff --git a/drivers/iio/adc/ad_sigma_delta.c b/drivers/iio/adc/ad_sigma_delta.c
index e2bed2d648f2..e30c7f8fcbec 100644
--- a/drivers/iio/adc/ad_sigma_delta.c
+++ b/drivers/iio/adc/ad_sigma_delta.c
@@ -469,7 +469,7 @@ static irqreturn_t ad_sd_trigger_handler(int irq, void *p)
 		/*
 		 * Data array after transfer will look like (if status is appended):
 		 * data[] = { [0][sample][sample][sample][status] }
-		 * Keeping the first byte 0 shifts the status postion by 1 byte to the right.
+		 * Keeping the first byte 0 shifts the status position by 1 byte to the right.
 		 */
 		status_pos = reg_size + 1;
 
@@ -656,7 +656,7 @@ int ad_sd_init(struct ad_sigma_delta *sigma_delta, struct iio_dev *indio_dev,
 	sigma_delta->spi = spi;
 	sigma_delta->info = info;
 
-	/* If the field is unset in ad_sigma_delta_info, asume there can only be 1 slot. */
+	/* If the field is unset in ad_sigma_delta_info, assume there can only be 1 slot. */
 	if (!info->num_slots)
 		sigma_delta->num_slots = 1;
 	else
diff --git a/drivers/iio/adc/max34408.c b/drivers/iio/adc/max34408.c
index ffec22be2d59..971e6e5dee9b 100644
--- a/drivers/iio/adc/max34408.c
+++ b/drivers/iio/adc/max34408.c
@@ -161,7 +161,7 @@ static int max34408_read_raw(struct iio_dev *indio_dev,
 		return IIO_VAL_INT;
 	case IIO_CHAN_INFO_SCALE:
 		/*
-		 * calcluate current for 8bit ADC with Rsense
+		 * calculate current for 8bit ADC with Rsense
 		 * value.
 		 * 10 mV * 1000 / Rsense uOhm = max current
 		 * (max current * adc val * 1000) / (2^8 - 1) mA
diff --git a/drivers/iio/adc/pac1921.c b/drivers/iio/adc/pac1921.c
index 4c2a1c07bc39..c49175f2c0b7 100644
--- a/drivers/iio/adc/pac1921.c
+++ b/drivers/iio/adc/pac1921.c
@@ -1077,7 +1077,7 @@ static int pac1921_init(struct pac1921_priv *priv)
 	/*
 	 * Init control register:
 	 * - VPower free run integration mode
-	 * - OUT pin full scale range: 3V (HW detault)
+	 * - OUT pin full scale range: 3V (HW default)
 	 * - no timeout, no sleep, no sleep override, no recalc (HW defaults)
 	 */
 	val = FIELD_PREP(PAC1921_CONTROL_MXSL_MASK,
diff --git a/drivers/iio/adc/palmas_gpadc.c b/drivers/iio/adc/palmas_gpadc.c
index 203cbbc70719..67d567ee21b4 100644
--- a/drivers/iio/adc/palmas_gpadc.c
+++ b/drivers/iio/adc/palmas_gpadc.c
@@ -456,7 +456,7 @@ static int palmas_gpadc_get_calibrated_code(struct palmas_gpadc *adc,
  *   raw high threshold = (ideal threshold + INL) * gain error + offset error
  *
  * The gain error include both gain error, as specified in the datasheet, and
- * the gain error drift. These paramenters vary depending on device and whether
+ * the gain error drift. These parameters vary depending on device and whether
  * the channel is calibrated (trimmed) or not.
  */
 static int palmas_gpadc_threshold_with_tolerance(int val, const int INL,
diff --git a/drivers/iio/adc/ti-ads1298.c b/drivers/iio/adc/ti-ads1298.c
index 13cb32125eef..a6432ef1fa90 100644
--- a/drivers/iio/adc/ti-ads1298.c
+++ b/drivers/iio/adc/ti-ads1298.c
@@ -294,7 +294,7 @@ static int ads1298_get_scale(struct ads1298_private *priv,
 		if (ret)
 			return ret;
 
-		/* Refererence in millivolts */
+		/* Reference in millivolts */
 		*val = regval & ADS1298_MASK_CONFIG3_VREF_4V ? 4000 : 2400;
 	}
 
diff --git a/drivers/iio/adc/ti_am335x_adc.c b/drivers/iio/adc/ti_am335x_adc.c
index 426e3c9f88a1..d362eba6cd7c 100644
--- a/drivers/iio/adc/ti_am335x_adc.c
+++ b/drivers/iio/adc/ti_am335x_adc.c
@@ -494,7 +494,7 @@ static int tiadc_read_raw(struct iio_dev *indio_dev,
 	/*
 	 * We check the complete FIFO. We programmed just one entry but in case
 	 * something went wrong we left empty handed (-EAGAIN previously) and
-	 * then the value apeared somehow in the FIFO we would have two entries.
+	 * then the value appeared somehow in the FIFO we would have two entries.
 	 * Therefore we read every item and keep only the latest version of the
 	 * requested channel.
 	 */
diff --git a/drivers/iio/adc/twl4030-madc.c b/drivers/iio/adc/twl4030-madc.c
index 0253064fadec..563478e9c5eb 100644
--- a/drivers/iio/adc/twl4030-madc.c
+++ b/drivers/iio/adc/twl4030-madc.c
@@ -248,7 +248,7 @@ static const struct s16_fract twl4030_divider_ratios[16] = {
 	{15, 100},	/* CHANNEL 11 */
 	{1, 4},		/* CHANNEL 12 */
 	{1, 1},		/* CHANNEL 13 Reserved channels */
-	{1, 1},		/* CHANNEL 14 Reseved channels */
+	{1, 1},		/* CHANNEL 14 Reserved channels */
 	{5, 11},	/* CHANNEL 15 */
 };
 
diff --git a/drivers/iio/adc/xilinx-xadc-events.c b/drivers/iio/adc/xilinx-xadc-events.c
index 1bd375fb10e0..90f62377c34d 100644
--- a/drivers/iio/adc/xilinx-xadc-events.c
+++ b/drivers/iio/adc/xilinx-xadc-events.c
@@ -220,7 +220,7 @@ int xadc_write_event_value(struct iio_dev *indio_dev,
 		/*
 		 * Since we store the hysteresis as relative (to the threshold)
 		 * value, but the hardware expects an absolute value we need to
-		 * recalcualte this value whenever the hysteresis or the
+		 * recalculate this value whenever the hysteresis or the
 		 * threshold changes.
 		 */
 		if (xadc->threshold[offset] < xadc->temp_hysteresis)
-- 
2.34.1
Re: [PATCH v1] iio: adc: Fix typos in comments across various files
Posted by Jonathan Cameron 2 months ago
On Thu, 26 Sep 2024 11:43:54 +0800
Yu Jiaoliang <yujiaoliang@vivo.com> wrote:

> This commit fixes several typographical errors in comments within
> the drivers/iio/adc directory. No functional changes are made.
> 
> Detected using codespell.
> 
> Signed-off-by: Yu Jiaoliang <yujiaoliang@vivo.com>
Applied.

Thanks,

Jonathan
Re: [PATCH v1] iio: adc: Fix typos in comments across various files
Posted by Luca Ceresoli 2 months ago
Hello Yu,

On Thu, 26 Sep 2024 11:43:54 +0800
Yu Jiaoliang <yujiaoliang@vivo.com> wrote:

> This commit fixes several typographical errors in comments within
> the drivers/iio/adc directory. No functional changes are made.

I think it would be useful to add those typo patterns to
scripts/spelling.txt, so checkpatch.pl will check them, preventing the
same typos from happening in the future.

With that added (perhaps in a separate patch), looks good.

Luca

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
Re: [PATCH v1] iio: adc: Fix typos in comments across various files
Posted by Yu Jiaoliang 2 months ago
在 2024/9/26 16:19, Luca Ceresoli 写道:
> Hello Yu,
>
> On Thu, 26 Sep 2024 11:43:54 +0800
> Yu Jiaoliang <yujiaoliang@vivo.com> wrote:
>
>> This commit fixes several typographical errors in comments within
>> the drivers/iio/adc directory. No functional changes are made.
> I think it would be useful to add those typo patterns to
> scripts/spelling.txt, so checkpatch.pl will check them, preventing the
> same typos from happening in the future.
>
> With that added (perhaps in a separate patch), looks good.

Hi Luca,

Thanks for the suggestion. I'll prepare a new patch that collects the 
typos we've encountered so far and adds them to |scripts/spelling.txt|.

>
> Luca

Best regards,
Yu