All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 1/3] MAINTAINERS: add missing files to the adis lib
@ 2022-01-22 13:09 Nuno Sá
  2022-01-22 13:09 ` [PATCH v2 2/3] adis: simplify 'adis_update_bits' macros Nuno Sá
  2022-01-22 13:09 ` [PATCH v2 3/3] iio: adis: stylistic changes Nuno Sá
  0 siblings, 2 replies; 4+ messages in thread
From: Nuno Sá @ 2022-01-22 13:09 UTC (permalink / raw)
  To: linux-iio; +Cc: Jonathan Cameron, Lars-Peter Clausen, Michael Hennerich

The triggered buffer support was missing in the MAINTAINERS file. Add
them.

Signed-off-by: Nuno Sá <nuno.sa@analog.com>
---
changes in v2:
 * none.

 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 78881bbaf3c0..7527466b436d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1103,6 +1103,8 @@ M:	Nuno Sa <nuno.sa@analog.com>
 L:	linux-iio@vger.kernel.org
 S:	Supported
 F:	drivers/iio/imu/adis.c
+F:	drivers/iio/imu/adis_buffer.c
+F:	drivers/iio/imu/adis_trigger.c
 F:	include/linux/iio/imu/adis.h
 
 ANALOG DEVICES INC ADIS16460 DRIVER
-- 
2.17.1


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

* [PATCH v2 2/3] adis: simplify 'adis_update_bits' macros
  2022-01-22 13:09 [PATCH v2 1/3] MAINTAINERS: add missing files to the adis lib Nuno Sá
@ 2022-01-22 13:09 ` Nuno Sá
  2022-01-22 13:09 ` [PATCH v2 3/3] iio: adis: stylistic changes Nuno Sá
  1 sibling, 0 replies; 4+ messages in thread
From: Nuno Sá @ 2022-01-22 13:09 UTC (permalink / raw)
  To: linux-iio; +Cc: Jonathan Cameron, Lars-Peter Clausen, Michael Hennerich

There's no need to use  '__builtin_choose_expr' to choose the right
call to 'adis_update_bits_base()'. We can change the 'BUILD_BUG_ON()'
condition so that it makes sure only the supported sizes are
passed in. With that, we can just use 'sizeof(val)' as the size argument
of 'adis_update_bits_base()'.

Signed-off-by: Nuno Sá <nuno.sa@analog.com>
---
changes in v2:
 * Fix 'BUILD_BUG_ON()' condition in the locked version.

 include/linux/iio/imu/adis.h | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/include/linux/iio/imu/adis.h b/include/linux/iio/imu/adis.h
index 7c02f5292eea..11754f97d8bb 100644
--- a/include/linux/iio/imu/adis.h
+++ b/include/linux/iio/imu/adis.h
@@ -381,10 +381,8 @@ static inline int adis_update_bits_base(struct adis *adis, unsigned int reg,
  * @val can lead to undesired behavior if the register to update is 16bit.
  */
 #define adis_update_bits(adis, reg, mask, val) ({			\
-	BUILD_BUG_ON(sizeof(val) == 1 || sizeof(val) == 8);		\
-	__builtin_choose_expr(sizeof(val) == 4,				\
-		adis_update_bits_base(adis, reg, mask, val, 4),         \
-		adis_update_bits_base(adis, reg, mask, val, 2));	\
+	BUILD_BUG_ON(sizeof(val) != 2 && sizeof(val) != 4);		\
+	adis_update_bits_base(adis, reg, mask, val, sizeof(val));	\
 })
 
 /**
@@ -399,10 +397,8 @@ static inline int adis_update_bits_base(struct adis *adis, unsigned int reg,
  * @val can lead to undesired behavior if the register to update is 16bit.
  */
 #define __adis_update_bits(adis, reg, mask, val) ({			\
-	BUILD_BUG_ON(sizeof(val) == 1 || sizeof(val) == 8);		\
-	__builtin_choose_expr(sizeof(val) == 4,				\
-		__adis_update_bits_base(adis, reg, mask, val, 4),	\
-		__adis_update_bits_base(adis, reg, mask, val, 2));	\
+	BUILD_BUG_ON(sizeof(val) != 2 && sizeof(val) != 4);		\
+	__adis_update_bits_base(adis, reg, mask, val, sizeof(val));	\
 })
 
 int adis_enable_irq(struct adis *adis, bool enable);
-- 
2.17.1


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

* [PATCH v2 3/3] iio: adis: stylistic changes
  2022-01-22 13:09 [PATCH v2 1/3] MAINTAINERS: add missing files to the adis lib Nuno Sá
  2022-01-22 13:09 ` [PATCH v2 2/3] adis: simplify 'adis_update_bits' macros Nuno Sá
@ 2022-01-22 13:09 ` Nuno Sá
  2022-01-22 16:32   ` Jonathan Cameron
  1 sibling, 1 reply; 4+ messages in thread
From: Nuno Sá @ 2022-01-22 13:09 UTC (permalink / raw)
  To: linux-iio; +Cc: Jonathan Cameron, Lars-Peter Clausen, Michael Hennerich

Minor stylistic changes to address checkptach complains when called with
'--strict'.

Signed-off-by: Nuno Sá <nuno.sa@analog.com>
---
changes in v2:
 * none.

 drivers/iio/imu/adis.c         | 47 +++++++++++++++++----------------
 drivers/iio/imu/adis_buffer.c  |  6 ++---
 drivers/iio/imu/adis_trigger.c |  3 +--
 include/linux/iio/imu/adis.h   | 48 ++++++++++++++++++----------------
 4 files changed, 54 insertions(+), 50 deletions(-)

diff --git a/drivers/iio/imu/adis.c b/drivers/iio/imu/adis.c
index cb0d66bf6561..638957001653 100644
--- a/drivers/iio/imu/adis.c
+++ b/drivers/iio/imu/adis.c
@@ -30,8 +30,8 @@
  * @value: The value to write to device (up to 4 bytes)
  * @size: The size of the @value (in bytes)
  */
-int __adis_write_reg(struct adis *adis, unsigned int reg,
-	unsigned int value, unsigned int size)
+int __adis_write_reg(struct adis *adis, unsigned int reg, unsigned int value,
+		     unsigned int size)
 {
 	unsigned int page = reg / ADIS_PAGE_SIZE;
 	int ret, i;
@@ -114,7 +114,7 @@ int __adis_write_reg(struct adis *adis, unsigned int reg,
 	ret = spi_sync(adis->spi, &msg);
 	if (ret) {
 		dev_err(&adis->spi->dev, "Failed to write register 0x%02X: %d\n",
-				reg, ret);
+			reg, ret);
 	} else {
 		adis->current_page = page;
 	}
@@ -130,8 +130,8 @@ EXPORT_SYMBOL_GPL(__adis_write_reg);
  * @val: The value read back from the device
  * @size: The size of the @val buffer
  */
-int __adis_read_reg(struct adis *adis, unsigned int reg,
-	unsigned int *val, unsigned int size)
+int __adis_read_reg(struct adis *adis, unsigned int reg, unsigned int *val,
+		    unsigned int size)
 {
 	unsigned int page = reg / ADIS_PAGE_SIZE;
 	struct spi_message msg;
@@ -201,12 +201,12 @@ int __adis_read_reg(struct adis *adis, unsigned int reg,
 	ret = spi_sync(adis->spi, &msg);
 	if (ret) {
 		dev_err(&adis->spi->dev, "Failed to read register 0x%02X: %d\n",
-				reg, ret);
+			reg, ret);
 		return ret;
-	} else {
-		adis->current_page = page;
 	}
 
+	adis->current_page = page;
+
 	switch (size) {
 	case 4:
 		*val = get_unaligned_be32(adis->rx);
@@ -247,13 +247,13 @@ EXPORT_SYMBOL_GPL(__adis_update_bits_base);
 
 #ifdef CONFIG_DEBUG_FS
 
-int adis_debugfs_reg_access(struct iio_dev *indio_dev,
-	unsigned int reg, unsigned int writeval, unsigned int *readval)
+int adis_debugfs_reg_access(struct iio_dev *indio_dev, unsigned int reg,
+			    unsigned int writeval, unsigned int *readval)
 {
 	struct adis *adis = iio_device_get_drvdata(indio_dev);
 
 	if (readval) {
-		uint16_t val16;
+		u16 val16;
 		int ret;
 
 		ret = adis_read_reg_16(adis, reg, &val16);
@@ -261,9 +261,9 @@ int adis_debugfs_reg_access(struct iio_dev *indio_dev,
 			*readval = val16;
 
 		return ret;
-	} else {
-		return adis_write_reg_16(adis, reg, writeval);
 	}
+
+	return adis_write_reg_16(adis, reg, writeval);
 }
 EXPORT_SYMBOL(adis_debugfs_reg_access);
 
@@ -279,14 +279,16 @@ EXPORT_SYMBOL(adis_debugfs_reg_access);
 int adis_enable_irq(struct adis *adis, bool enable)
 {
 	int ret = 0;
-	uint16_t msc;
+	u16 msc;
 
 	mutex_lock(&adis->state_lock);
 
 	if (adis->data->enable_irq) {
 		ret = adis->data->enable_irq(adis, enable);
 		goto out_unlock;
-	} else if (adis->data->unmasked_drdy) {
+	}
+
+	if (adis->data->unmasked_drdy) {
 		if (enable)
 			enable_irq(adis->spi->irq);
 		else
@@ -322,7 +324,7 @@ EXPORT_SYMBOL(adis_enable_irq);
  */
 int __adis_check_status(struct adis *adis)
 {
-	uint16_t status;
+	u16 status;
 	int ret;
 	int i;
 
@@ -358,7 +360,7 @@ int __adis_reset(struct adis *adis)
 	const struct adis_timeout *timeouts = adis->data->timeouts;
 
 	ret = __adis_write_reg_8(adis, adis->data->glob_cmd_reg,
-			ADIS_GLOB_CMD_SW_RESET);
+				 ADIS_GLOB_CMD_SW_RESET);
 	if (ret) {
 		dev_err(&adis->spi->dev, "Failed to reset device: %d\n", ret);
 		return ret;
@@ -414,7 +416,7 @@ int __adis_initial_startup(struct adis *adis)
 {
 	const struct adis_timeout *timeouts = adis->data->timeouts;
 	struct gpio_desc *gpio;
-	uint16_t prod_id;
+	u16 prod_id;
 	int ret;
 
 	/* check if the device has rst pin low */
@@ -423,7 +425,7 @@ int __adis_initial_startup(struct adis *adis)
 		return PTR_ERR(gpio);
 
 	if (gpio) {
-		msleep(10);
+		usleep_range(10, 12);
 		/* bring device out of reset */
 		gpiod_set_value_cansleep(gpio, 0);
 		msleep(timeouts->reset_ms);
@@ -477,7 +479,8 @@ EXPORT_SYMBOL_GPL(__adis_initial_startup);
  * a error bit in the channels raw value set error_mask to 0.
  */
 int adis_single_conversion(struct iio_dev *indio_dev,
-	const struct iio_chan_spec *chan, unsigned int error_mask, int *val)
+			   const struct iio_chan_spec *chan,
+			   unsigned int error_mask, int *val)
 {
 	struct adis *adis = iio_device_get_drvdata(indio_dev);
 	unsigned int uval;
@@ -486,7 +489,7 @@ int adis_single_conversion(struct iio_dev *indio_dev,
 	mutex_lock(&adis->state_lock);
 
 	ret = __adis_read_reg(adis, chan->address, &uval,
-			chan->scan_type.storagebits / 8);
+			      chan->scan_type.storagebits / 8);
 	if (ret)
 		goto err_unlock;
 
@@ -521,7 +524,7 @@ EXPORT_SYMBOL_GPL(adis_single_conversion);
  * called.
  */
 int adis_init(struct adis *adis, struct iio_dev *indio_dev,
-	struct spi_device *spi, const struct adis_data *data)
+	      struct spi_device *spi, const struct adis_data *data)
 {
 	if (!data || !data->timeouts) {
 		dev_err(&spi->dev, "No config data or timeouts not defined!\n");
diff --git a/drivers/iio/imu/adis_buffer.c b/drivers/iio/imu/adis_buffer.c
index 351c303c8a8c..d3527cf5ed37 100644
--- a/drivers/iio/imu/adis_buffer.c
+++ b/drivers/iio/imu/adis_buffer.c
@@ -20,7 +20,7 @@
 #include <linux/iio/imu/adis.h>
 
 static int adis_update_scan_mode_burst(struct iio_dev *indio_dev,
-	const unsigned long *scan_mask)
+				       const unsigned long *scan_mask)
 {
 	struct adis *adis = iio_device_get_drvdata(indio_dev);
 	unsigned int burst_length, burst_max_length;
@@ -67,7 +67,7 @@ static int adis_update_scan_mode_burst(struct iio_dev *indio_dev,
 }
 
 int adis_update_scan_mode(struct iio_dev *indio_dev,
-	const unsigned long *scan_mask)
+			  const unsigned long *scan_mask)
 {
 	struct adis *adis = iio_device_get_drvdata(indio_dev);
 	const struct iio_chan_spec *chan;
@@ -158,7 +158,7 @@ static irqreturn_t adis_trigger_handler(int irq, void *p)
 	}
 
 	iio_push_to_buffers_with_timestamp(indio_dev, adis->buffer,
-		pf->timestamp);
+					   pf->timestamp);
 
 irq_done:
 	iio_trigger_notify_done(indio_dev->trig);
diff --git a/drivers/iio/imu/adis_trigger.c b/drivers/iio/imu/adis_trigger.c
index c461bd1e8e69..0e7fb00ba241 100644
--- a/drivers/iio/imu/adis_trigger.c
+++ b/drivers/iio/imu/adis_trigger.c
@@ -15,8 +15,7 @@
 #include <linux/iio/trigger.h>
 #include <linux/iio/imu/adis.h>
 
-static int adis_data_rdy_trigger_set_state(struct iio_trigger *trig,
-						bool state)
+static int adis_data_rdy_trigger_set_state(struct iio_trigger *trig, bool state)
 {
 	struct adis *adis = iio_trigger_get_drvdata(trig);
 
diff --git a/include/linux/iio/imu/adis.h b/include/linux/iio/imu/adis.h
index 11754f97d8bb..515ca09764fe 100644
--- a/include/linux/iio/imu/adis.h
+++ b/include/linux/iio/imu/adis.h
@@ -32,6 +32,7 @@ struct adis_timeout {
 	u16 sw_reset_ms;
 	u16 self_test_ms;
 };
+
 /**
  * struct adis_data - ADIS chip variant specific data
  * @read_delay: SPI delay for read operations in us
@@ -45,7 +46,7 @@ struct adis_timeout {
  * @self_test_mask: Bitmask of supported self-test operations
  * @self_test_reg: Register address to request self test command
  * @self_test_no_autoclear: True if device's self-test needs clear of ctrl reg
- * @status_error_msgs: Array of error messgaes
+ * @status_error_msgs: Array of error messages
  * @status_error_mask: Bitmask of errors supported by the device
  * @timeouts: Chip specific delays
  * @enable_irq: Hook for ADIS devices that have a special IRQ enable/disable
@@ -130,12 +131,12 @@ struct adis {
 	unsigned long		irq_flag;
 	void			*buffer;
 
-	uint8_t			tx[10] ____cacheline_aligned;
-	uint8_t			rx[4];
+	u8			tx[10] ____cacheline_aligned;
+	u8			rx[4];
 };
 
 int adis_init(struct adis *adis, struct iio_dev *indio_dev,
-	struct spi_device *spi, const struct adis_data *data);
+	      struct spi_device *spi, const struct adis_data *data);
 int __adis_reset(struct adis *adis);
 
 /**
@@ -156,9 +157,9 @@ static inline int adis_reset(struct adis *adis)
 }
 
 int __adis_write_reg(struct adis *adis, unsigned int reg,
-	unsigned int val, unsigned int size);
+		     unsigned int val, unsigned int size);
 int __adis_read_reg(struct adis *adis, unsigned int reg,
-	unsigned int *val, unsigned int size);
+		    unsigned int *val, unsigned int size);
 
 /**
  * __adis_write_reg_8() - Write single byte to a register (unlocked)
@@ -167,7 +168,7 @@ int __adis_read_reg(struct adis *adis, unsigned int reg,
  * @value: The value to write
  */
 static inline int __adis_write_reg_8(struct adis *adis, unsigned int reg,
-	uint8_t val)
+				     u8 val)
 {
 	return __adis_write_reg(adis, reg, val, 1);
 }
@@ -179,7 +180,7 @@ static inline int __adis_write_reg_8(struct adis *adis, unsigned int reg,
  * @value: Value to be written
  */
 static inline int __adis_write_reg_16(struct adis *adis, unsigned int reg,
-	uint16_t val)
+				      u16 val)
 {
 	return __adis_write_reg(adis, reg, val, 2);
 }
@@ -191,7 +192,7 @@ static inline int __adis_write_reg_16(struct adis *adis, unsigned int reg,
  * @value: Value to be written
  */
 static inline int __adis_write_reg_32(struct adis *adis, unsigned int reg,
-	uint32_t val)
+				      u32 val)
 {
 	return __adis_write_reg(adis, reg, val, 4);
 }
@@ -203,7 +204,7 @@ static inline int __adis_write_reg_32(struct adis *adis, unsigned int reg,
  * @val: The value read back from the device
  */
 static inline int __adis_read_reg_16(struct adis *adis, unsigned int reg,
-	uint16_t *val)
+				     u16 *val)
 {
 	unsigned int tmp;
 	int ret;
@@ -222,7 +223,7 @@ static inline int __adis_read_reg_16(struct adis *adis, unsigned int reg,
  * @val: The value read back from the device
  */
 static inline int __adis_read_reg_32(struct adis *adis, unsigned int reg,
-	uint32_t *val)
+				     u32 *val)
 {
 	unsigned int tmp;
 	int ret;
@@ -242,7 +243,7 @@ static inline int __adis_read_reg_32(struct adis *adis, unsigned int reg,
  * @size: The size of the @value (in bytes)
  */
 static inline int adis_write_reg(struct adis *adis, unsigned int reg,
-	unsigned int val, unsigned int size)
+				 unsigned int val, unsigned int size)
 {
 	int ret;
 
@@ -261,7 +262,7 @@ static inline int adis_write_reg(struct adis *adis, unsigned int reg,
  * @size: The size of the @val buffer
  */
 static int adis_read_reg(struct adis *adis, unsigned int reg,
-	unsigned int *val, unsigned int size)
+			 unsigned int *val, unsigned int size)
 {
 	int ret;
 
@@ -279,7 +280,7 @@ static int adis_read_reg(struct adis *adis, unsigned int reg,
  * @value: The value to write
  */
 static inline int adis_write_reg_8(struct adis *adis, unsigned int reg,
-	uint8_t val)
+				   u8 val)
 {
 	return adis_write_reg(adis, reg, val, 1);
 }
@@ -291,7 +292,7 @@ static inline int adis_write_reg_8(struct adis *adis, unsigned int reg,
  * @value: Value to be written
  */
 static inline int adis_write_reg_16(struct adis *adis, unsigned int reg,
-	uint16_t val)
+				    u16 val)
 {
 	return adis_write_reg(adis, reg, val, 2);
 }
@@ -303,7 +304,7 @@ static inline int adis_write_reg_16(struct adis *adis, unsigned int reg,
  * @value: Value to be written
  */
 static inline int adis_write_reg_32(struct adis *adis, unsigned int reg,
-	uint32_t val)
+				    u32 val)
 {
 	return adis_write_reg(adis, reg, val, 4);
 }
@@ -315,7 +316,7 @@ static inline int adis_write_reg_32(struct adis *adis, unsigned int reg,
  * @val: The value read back from the device
  */
 static inline int adis_read_reg_16(struct adis *adis, unsigned int reg,
-	uint16_t *val)
+				   u16 *val)
 {
 	unsigned int tmp;
 	int ret;
@@ -334,7 +335,7 @@ static inline int adis_read_reg_16(struct adis *adis, unsigned int reg,
  * @val: The value read back from the device
  */
 static inline int adis_read_reg_32(struct adis *adis, unsigned int reg,
-	uint32_t *val)
+				   u32 *val)
 {
 	unsigned int tmp;
 	int ret;
@@ -439,8 +440,8 @@ static inline void adis_dev_unlock(struct adis *adis)
 }
 
 int adis_single_conversion(struct iio_dev *indio_dev,
-	const struct iio_chan_spec *chan, unsigned int error_mask,
-	int *val);
+			   const struct iio_chan_spec *chan,
+			   unsigned int error_mask, int *val);
 
 #define ADIS_VOLTAGE_CHAN(addr, si, chan, name, info_all, bits) { \
 	.type = IIO_VOLTAGE, \
@@ -489,7 +490,7 @@ int adis_single_conversion(struct iio_dev *indio_dev,
 	.modified = 1, \
 	.channel2 = IIO_MOD_ ## mod, \
 	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) | \
-		 info_sep, \
+		 (info_sep), \
 	.info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE), \
 	.info_mask_shared_by_all = info_all, \
 	.address = (addr), \
@@ -523,7 +524,7 @@ devm_adis_setup_buffer_and_trigger(struct adis *adis, struct iio_dev *indio_dev,
 int devm_adis_probe_trigger(struct adis *adis, struct iio_dev *indio_dev);
 
 int adis_update_scan_mode(struct iio_dev *indio_dev,
-	const unsigned long *scan_mask);
+			  const unsigned long *scan_mask);
 
 #else /* CONFIG_IIO_BUFFER */
 
@@ -547,7 +548,8 @@ static inline int devm_adis_probe_trigger(struct adis *adis,
 #ifdef CONFIG_DEBUG_FS
 
 int adis_debugfs_reg_access(struct iio_dev *indio_dev,
-	unsigned int reg, unsigned int writeval, unsigned int *readval);
+			    unsigned int reg, unsigned int writeval,
+			    unsigned int *readval);
 
 #else
 
-- 
2.17.1


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

* Re: [PATCH v2 3/3] iio: adis: stylistic changes
  2022-01-22 13:09 ` [PATCH v2 3/3] iio: adis: stylistic changes Nuno Sá
@ 2022-01-22 16:32   ` Jonathan Cameron
  0 siblings, 0 replies; 4+ messages in thread
From: Jonathan Cameron @ 2022-01-22 16:32 UTC (permalink / raw)
  To: Nuno Sá; +Cc: linux-iio, Lars-Peter Clausen, Michael Hennerich

On Sat, 22 Jan 2022 14:09:05 +0100
Nuno Sá <nuno.sa@analog.com> wrote:

> Minor stylistic changes to address checkptach complains when called with
> '--strict'.
> 
> Signed-off-by: Nuno Sá <nuno.sa@analog.com>
All look good to me.

Series applied to the togreg branch of iio.git and pushed out as testing for
0-day to poke at it and see if it can find anything we missed.

Thanks,

Jonathan
> ---
> changes in v2:
>  * none.
> 
>  drivers/iio/imu/adis.c         | 47 +++++++++++++++++----------------
>  drivers/iio/imu/adis_buffer.c  |  6 ++---
>  drivers/iio/imu/adis_trigger.c |  3 +--
>  include/linux/iio/imu/adis.h   | 48 ++++++++++++++++++----------------
>  4 files changed, 54 insertions(+), 50 deletions(-)
> 
> diff --git a/drivers/iio/imu/adis.c b/drivers/iio/imu/adis.c
> index cb0d66bf6561..638957001653 100644
> --- a/drivers/iio/imu/adis.c
> +++ b/drivers/iio/imu/adis.c
> @@ -30,8 +30,8 @@
>   * @value: The value to write to device (up to 4 bytes)
>   * @size: The size of the @value (in bytes)
>   */
> -int __adis_write_reg(struct adis *adis, unsigned int reg,
> -	unsigned int value, unsigned int size)
> +int __adis_write_reg(struct adis *adis, unsigned int reg, unsigned int value,
> +		     unsigned int size)
>  {
>  	unsigned int page = reg / ADIS_PAGE_SIZE;
>  	int ret, i;
> @@ -114,7 +114,7 @@ int __adis_write_reg(struct adis *adis, unsigned int reg,
>  	ret = spi_sync(adis->spi, &msg);
>  	if (ret) {
>  		dev_err(&adis->spi->dev, "Failed to write register 0x%02X: %d\n",
> -				reg, ret);
> +			reg, ret);
>  	} else {
>  		adis->current_page = page;
>  	}
> @@ -130,8 +130,8 @@ EXPORT_SYMBOL_GPL(__adis_write_reg);
>   * @val: The value read back from the device
>   * @size: The size of the @val buffer
>   */
> -int __adis_read_reg(struct adis *adis, unsigned int reg,
> -	unsigned int *val, unsigned int size)
> +int __adis_read_reg(struct adis *adis, unsigned int reg, unsigned int *val,
> +		    unsigned int size)
>  {
>  	unsigned int page = reg / ADIS_PAGE_SIZE;
>  	struct spi_message msg;
> @@ -201,12 +201,12 @@ int __adis_read_reg(struct adis *adis, unsigned int reg,
>  	ret = spi_sync(adis->spi, &msg);
>  	if (ret) {
>  		dev_err(&adis->spi->dev, "Failed to read register 0x%02X: %d\n",
> -				reg, ret);
> +			reg, ret);
>  		return ret;
> -	} else {
> -		adis->current_page = page;
>  	}
>  
> +	adis->current_page = page;
> +
>  	switch (size) {
>  	case 4:
>  		*val = get_unaligned_be32(adis->rx);
> @@ -247,13 +247,13 @@ EXPORT_SYMBOL_GPL(__adis_update_bits_base);
>  
>  #ifdef CONFIG_DEBUG_FS
>  
> -int adis_debugfs_reg_access(struct iio_dev *indio_dev,
> -	unsigned int reg, unsigned int writeval, unsigned int *readval)
> +int adis_debugfs_reg_access(struct iio_dev *indio_dev, unsigned int reg,
> +			    unsigned int writeval, unsigned int *readval)
>  {
>  	struct adis *adis = iio_device_get_drvdata(indio_dev);
>  
>  	if (readval) {
> -		uint16_t val16;
> +		u16 val16;
>  		int ret;
>  
>  		ret = adis_read_reg_16(adis, reg, &val16);
> @@ -261,9 +261,9 @@ int adis_debugfs_reg_access(struct iio_dev *indio_dev,
>  			*readval = val16;
>  
>  		return ret;
> -	} else {
> -		return adis_write_reg_16(adis, reg, writeval);
>  	}
> +
> +	return adis_write_reg_16(adis, reg, writeval);
>  }
>  EXPORT_SYMBOL(adis_debugfs_reg_access);
>  
> @@ -279,14 +279,16 @@ EXPORT_SYMBOL(adis_debugfs_reg_access);
>  int adis_enable_irq(struct adis *adis, bool enable)
>  {
>  	int ret = 0;
> -	uint16_t msc;
> +	u16 msc;
>  
>  	mutex_lock(&adis->state_lock);
>  
>  	if (adis->data->enable_irq) {
>  		ret = adis->data->enable_irq(adis, enable);
>  		goto out_unlock;
> -	} else if (adis->data->unmasked_drdy) {
> +	}
> +
> +	if (adis->data->unmasked_drdy) {
>  		if (enable)
>  			enable_irq(adis->spi->irq);
>  		else
> @@ -322,7 +324,7 @@ EXPORT_SYMBOL(adis_enable_irq);
>   */
>  int __adis_check_status(struct adis *adis)
>  {
> -	uint16_t status;
> +	u16 status;
>  	int ret;
>  	int i;
>  
> @@ -358,7 +360,7 @@ int __adis_reset(struct adis *adis)
>  	const struct adis_timeout *timeouts = adis->data->timeouts;
>  
>  	ret = __adis_write_reg_8(adis, adis->data->glob_cmd_reg,
> -			ADIS_GLOB_CMD_SW_RESET);
> +				 ADIS_GLOB_CMD_SW_RESET);
>  	if (ret) {
>  		dev_err(&adis->spi->dev, "Failed to reset device: %d\n", ret);
>  		return ret;
> @@ -414,7 +416,7 @@ int __adis_initial_startup(struct adis *adis)
>  {
>  	const struct adis_timeout *timeouts = adis->data->timeouts;
>  	struct gpio_desc *gpio;
> -	uint16_t prod_id;
> +	u16 prod_id;
>  	int ret;
>  
>  	/* check if the device has rst pin low */
> @@ -423,7 +425,7 @@ int __adis_initial_startup(struct adis *adis)
>  		return PTR_ERR(gpio);
>  
>  	if (gpio) {
> -		msleep(10);
> +		usleep_range(10, 12);
>  		/* bring device out of reset */
>  		gpiod_set_value_cansleep(gpio, 0);
>  		msleep(timeouts->reset_ms);
> @@ -477,7 +479,8 @@ EXPORT_SYMBOL_GPL(__adis_initial_startup);
>   * a error bit in the channels raw value set error_mask to 0.
>   */
>  int adis_single_conversion(struct iio_dev *indio_dev,
> -	const struct iio_chan_spec *chan, unsigned int error_mask, int *val)
> +			   const struct iio_chan_spec *chan,
> +			   unsigned int error_mask, int *val)
>  {
>  	struct adis *adis = iio_device_get_drvdata(indio_dev);
>  	unsigned int uval;
> @@ -486,7 +489,7 @@ int adis_single_conversion(struct iio_dev *indio_dev,
>  	mutex_lock(&adis->state_lock);
>  
>  	ret = __adis_read_reg(adis, chan->address, &uval,
> -			chan->scan_type.storagebits / 8);
> +			      chan->scan_type.storagebits / 8);
>  	if (ret)
>  		goto err_unlock;
>  
> @@ -521,7 +524,7 @@ EXPORT_SYMBOL_GPL(adis_single_conversion);
>   * called.
>   */
>  int adis_init(struct adis *adis, struct iio_dev *indio_dev,
> -	struct spi_device *spi, const struct adis_data *data)
> +	      struct spi_device *spi, const struct adis_data *data)
>  {
>  	if (!data || !data->timeouts) {
>  		dev_err(&spi->dev, "No config data or timeouts not defined!\n");
> diff --git a/drivers/iio/imu/adis_buffer.c b/drivers/iio/imu/adis_buffer.c
> index 351c303c8a8c..d3527cf5ed37 100644
> --- a/drivers/iio/imu/adis_buffer.c
> +++ b/drivers/iio/imu/adis_buffer.c
> @@ -20,7 +20,7 @@
>  #include <linux/iio/imu/adis.h>
>  
>  static int adis_update_scan_mode_burst(struct iio_dev *indio_dev,
> -	const unsigned long *scan_mask)
> +				       const unsigned long *scan_mask)
>  {
>  	struct adis *adis = iio_device_get_drvdata(indio_dev);
>  	unsigned int burst_length, burst_max_length;
> @@ -67,7 +67,7 @@ static int adis_update_scan_mode_burst(struct iio_dev *indio_dev,
>  }
>  
>  int adis_update_scan_mode(struct iio_dev *indio_dev,
> -	const unsigned long *scan_mask)
> +			  const unsigned long *scan_mask)
>  {
>  	struct adis *adis = iio_device_get_drvdata(indio_dev);
>  	const struct iio_chan_spec *chan;
> @@ -158,7 +158,7 @@ static irqreturn_t adis_trigger_handler(int irq, void *p)
>  	}
>  
>  	iio_push_to_buffers_with_timestamp(indio_dev, adis->buffer,
> -		pf->timestamp);
> +					   pf->timestamp);
>  
>  irq_done:
>  	iio_trigger_notify_done(indio_dev->trig);
> diff --git a/drivers/iio/imu/adis_trigger.c b/drivers/iio/imu/adis_trigger.c
> index c461bd1e8e69..0e7fb00ba241 100644
> --- a/drivers/iio/imu/adis_trigger.c
> +++ b/drivers/iio/imu/adis_trigger.c
> @@ -15,8 +15,7 @@
>  #include <linux/iio/trigger.h>
>  #include <linux/iio/imu/adis.h>
>  
> -static int adis_data_rdy_trigger_set_state(struct iio_trigger *trig,
> -						bool state)
> +static int adis_data_rdy_trigger_set_state(struct iio_trigger *trig, bool state)
>  {
>  	struct adis *adis = iio_trigger_get_drvdata(trig);
>  
> diff --git a/include/linux/iio/imu/adis.h b/include/linux/iio/imu/adis.h
> index 11754f97d8bb..515ca09764fe 100644
> --- a/include/linux/iio/imu/adis.h
> +++ b/include/linux/iio/imu/adis.h
> @@ -32,6 +32,7 @@ struct adis_timeout {
>  	u16 sw_reset_ms;
>  	u16 self_test_ms;
>  };
> +
>  /**
>   * struct adis_data - ADIS chip variant specific data
>   * @read_delay: SPI delay for read operations in us
> @@ -45,7 +46,7 @@ struct adis_timeout {
>   * @self_test_mask: Bitmask of supported self-test operations
>   * @self_test_reg: Register address to request self test command
>   * @self_test_no_autoclear: True if device's self-test needs clear of ctrl reg
> - * @status_error_msgs: Array of error messgaes
> + * @status_error_msgs: Array of error messages
>   * @status_error_mask: Bitmask of errors supported by the device
>   * @timeouts: Chip specific delays
>   * @enable_irq: Hook for ADIS devices that have a special IRQ enable/disable
> @@ -130,12 +131,12 @@ struct adis {
>  	unsigned long		irq_flag;
>  	void			*buffer;
>  
> -	uint8_t			tx[10] ____cacheline_aligned;
> -	uint8_t			rx[4];
> +	u8			tx[10] ____cacheline_aligned;
> +	u8			rx[4];
>  };
>  
>  int adis_init(struct adis *adis, struct iio_dev *indio_dev,
> -	struct spi_device *spi, const struct adis_data *data);
> +	      struct spi_device *spi, const struct adis_data *data);
>  int __adis_reset(struct adis *adis);
>  
>  /**
> @@ -156,9 +157,9 @@ static inline int adis_reset(struct adis *adis)
>  }
>  
>  int __adis_write_reg(struct adis *adis, unsigned int reg,
> -	unsigned int val, unsigned int size);
> +		     unsigned int val, unsigned int size);
>  int __adis_read_reg(struct adis *adis, unsigned int reg,
> -	unsigned int *val, unsigned int size);
> +		    unsigned int *val, unsigned int size);
>  
>  /**
>   * __adis_write_reg_8() - Write single byte to a register (unlocked)
> @@ -167,7 +168,7 @@ int __adis_read_reg(struct adis *adis, unsigned int reg,
>   * @value: The value to write
>   */
>  static inline int __adis_write_reg_8(struct adis *adis, unsigned int reg,
> -	uint8_t val)
> +				     u8 val)
>  {
>  	return __adis_write_reg(adis, reg, val, 1);
>  }
> @@ -179,7 +180,7 @@ static inline int __adis_write_reg_8(struct adis *adis, unsigned int reg,
>   * @value: Value to be written
>   */
>  static inline int __adis_write_reg_16(struct adis *adis, unsigned int reg,
> -	uint16_t val)
> +				      u16 val)
>  {
>  	return __adis_write_reg(adis, reg, val, 2);
>  }
> @@ -191,7 +192,7 @@ static inline int __adis_write_reg_16(struct adis *adis, unsigned int reg,
>   * @value: Value to be written
>   */
>  static inline int __adis_write_reg_32(struct adis *adis, unsigned int reg,
> -	uint32_t val)
> +				      u32 val)
>  {
>  	return __adis_write_reg(adis, reg, val, 4);
>  }
> @@ -203,7 +204,7 @@ static inline int __adis_write_reg_32(struct adis *adis, unsigned int reg,
>   * @val: The value read back from the device
>   */
>  static inline int __adis_read_reg_16(struct adis *adis, unsigned int reg,
> -	uint16_t *val)
> +				     u16 *val)
>  {
>  	unsigned int tmp;
>  	int ret;
> @@ -222,7 +223,7 @@ static inline int __adis_read_reg_16(struct adis *adis, unsigned int reg,
>   * @val: The value read back from the device
>   */
>  static inline int __adis_read_reg_32(struct adis *adis, unsigned int reg,
> -	uint32_t *val)
> +				     u32 *val)
>  {
>  	unsigned int tmp;
>  	int ret;
> @@ -242,7 +243,7 @@ static inline int __adis_read_reg_32(struct adis *adis, unsigned int reg,
>   * @size: The size of the @value (in bytes)
>   */
>  static inline int adis_write_reg(struct adis *adis, unsigned int reg,
> -	unsigned int val, unsigned int size)
> +				 unsigned int val, unsigned int size)
>  {
>  	int ret;
>  
> @@ -261,7 +262,7 @@ static inline int adis_write_reg(struct adis *adis, unsigned int reg,
>   * @size: The size of the @val buffer
>   */
>  static int adis_read_reg(struct adis *adis, unsigned int reg,
> -	unsigned int *val, unsigned int size)
> +			 unsigned int *val, unsigned int size)
>  {
>  	int ret;
>  
> @@ -279,7 +280,7 @@ static int adis_read_reg(struct adis *adis, unsigned int reg,
>   * @value: The value to write
>   */
>  static inline int adis_write_reg_8(struct adis *adis, unsigned int reg,
> -	uint8_t val)
> +				   u8 val)
>  {
>  	return adis_write_reg(adis, reg, val, 1);
>  }
> @@ -291,7 +292,7 @@ static inline int adis_write_reg_8(struct adis *adis, unsigned int reg,
>   * @value: Value to be written
>   */
>  static inline int adis_write_reg_16(struct adis *adis, unsigned int reg,
> -	uint16_t val)
> +				    u16 val)
>  {
>  	return adis_write_reg(adis, reg, val, 2);
>  }
> @@ -303,7 +304,7 @@ static inline int adis_write_reg_16(struct adis *adis, unsigned int reg,
>   * @value: Value to be written
>   */
>  static inline int adis_write_reg_32(struct adis *adis, unsigned int reg,
> -	uint32_t val)
> +				    u32 val)
>  {
>  	return adis_write_reg(adis, reg, val, 4);
>  }
> @@ -315,7 +316,7 @@ static inline int adis_write_reg_32(struct adis *adis, unsigned int reg,
>   * @val: The value read back from the device
>   */
>  static inline int adis_read_reg_16(struct adis *adis, unsigned int reg,
> -	uint16_t *val)
> +				   u16 *val)
>  {
>  	unsigned int tmp;
>  	int ret;
> @@ -334,7 +335,7 @@ static inline int adis_read_reg_16(struct adis *adis, unsigned int reg,
>   * @val: The value read back from the device
>   */
>  static inline int adis_read_reg_32(struct adis *adis, unsigned int reg,
> -	uint32_t *val)
> +				   u32 *val)
>  {
>  	unsigned int tmp;
>  	int ret;
> @@ -439,8 +440,8 @@ static inline void adis_dev_unlock(struct adis *adis)
>  }
>  
>  int adis_single_conversion(struct iio_dev *indio_dev,
> -	const struct iio_chan_spec *chan, unsigned int error_mask,
> -	int *val);
> +			   const struct iio_chan_spec *chan,
> +			   unsigned int error_mask, int *val);
>  
>  #define ADIS_VOLTAGE_CHAN(addr, si, chan, name, info_all, bits) { \
>  	.type = IIO_VOLTAGE, \
> @@ -489,7 +490,7 @@ int adis_single_conversion(struct iio_dev *indio_dev,
>  	.modified = 1, \
>  	.channel2 = IIO_MOD_ ## mod, \
>  	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) | \
> -		 info_sep, \
> +		 (info_sep), \
>  	.info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE), \
>  	.info_mask_shared_by_all = info_all, \
>  	.address = (addr), \
> @@ -523,7 +524,7 @@ devm_adis_setup_buffer_and_trigger(struct adis *adis, struct iio_dev *indio_dev,
>  int devm_adis_probe_trigger(struct adis *adis, struct iio_dev *indio_dev);
>  
>  int adis_update_scan_mode(struct iio_dev *indio_dev,
> -	const unsigned long *scan_mask);
> +			  const unsigned long *scan_mask);
>  
>  #else /* CONFIG_IIO_BUFFER */
>  
> @@ -547,7 +548,8 @@ static inline int devm_adis_probe_trigger(struct adis *adis,
>  #ifdef CONFIG_DEBUG_FS
>  
>  int adis_debugfs_reg_access(struct iio_dev *indio_dev,
> -	unsigned int reg, unsigned int writeval, unsigned int *readval);
> +			    unsigned int reg, unsigned int writeval,
> +			    unsigned int *readval);
>  
>  #else
>  


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

end of thread, other threads:[~2022-01-22 16:26 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-22 13:09 [PATCH v2 1/3] MAINTAINERS: add missing files to the adis lib Nuno Sá
2022-01-22 13:09 ` [PATCH v2 2/3] adis: simplify 'adis_update_bits' macros Nuno Sá
2022-01-22 13:09 ` [PATCH v2 3/3] iio: adis: stylistic changes Nuno Sá
2022-01-22 16:32   ` Jonathan Cameron

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.