iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

iio: adis: stylistic changes

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

Signed-off-by: Nuno Sá <nuno.sa@analog.com>
Link: https://lore.kernel.org/r/20220122130905.99-3-nuno.sa@analog.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
This commit is contained in:
Nuno Sá
2022-01-22 14:09:05 +01:00
committed by Jonathan Cameron
parent da59367705
commit c39010ea6b
4 changed files with 54 additions and 50 deletions
Download Patch File Download Diff File Expand all files Collapse all files

37
drivers/iio/imu/adis.c
View File

@@ -30,8 +30,8 @@
* @value: The value to write to device (up to 4 bytes) * @value: The value to write to device (up to 4 bytes)
* @size: The size of the @value (in bytes) * @size: The size of the @value (in bytes)
*/ */
int __adis_write_reg(struct adis *adis, unsigned int reg, int __adis_write_reg(struct adis *adis, unsigned int reg, unsigned int value,
unsigned int value, unsigned int size) unsigned int size)
{ {
unsigned int page = reg / ADIS_PAGE_SIZE; unsigned int page = reg / ADIS_PAGE_SIZE;
int ret, i; int ret, i;
@@ -130,8 +130,8 @@ EXPORT_SYMBOL_GPL(__adis_write_reg);
* @val: The value read back from the device * @val: The value read back from the device
* @size: The size of the @val buffer * @size: The size of the @val buffer
*/ */
int __adis_read_reg(struct adis *adis, unsigned int reg, int __adis_read_reg(struct adis *adis, unsigned int reg, unsigned int *val,
unsigned int *val, unsigned int size) unsigned int size)
{ {
unsigned int page = reg / ADIS_PAGE_SIZE; unsigned int page = reg / ADIS_PAGE_SIZE;
struct spi_message msg; struct spi_message msg;
@@ -203,10 +203,10 @@ int __adis_read_reg(struct adis *adis, unsigned int reg,
dev_err(&adis->spi->dev, "Failed to read register 0x%02X: %d\n", dev_err(&adis->spi->dev, "Failed to read register 0x%02X: %d\n",
reg, ret); reg, ret);
return ret; return ret;
} else {
adis->current_page = page;
} }
adis->current_page = page;
switch (size) { switch (size) {
case 4: case 4:
*val = get_unaligned_be32(adis->rx); *val = get_unaligned_be32(adis->rx);
@@ -247,13 +247,13 @@ EXPORT_SYMBOL_GPL(__adis_update_bits_base);
#ifdef CONFIG_DEBUG_FS #ifdef CONFIG_DEBUG_FS
int adis_debugfs_reg_access(struct iio_dev *indio_dev, int adis_debugfs_reg_access(struct iio_dev *indio_dev, unsigned int reg,
unsigned int reg, unsigned int writeval, unsigned int *readval) unsigned int writeval, unsigned int *readval)
{ {
struct adis *adis = iio_device_get_drvdata(indio_dev); struct adis *adis = iio_device_get_drvdata(indio_dev);
if (readval) { if (readval) {
uint16_t val16; u16 val16;
int ret; int ret;
ret = adis_read_reg_16(adis, reg, &val16); ret = adis_read_reg_16(adis, reg, &val16);
@@ -261,9 +261,9 @@ int adis_debugfs_reg_access(struct iio_dev *indio_dev,
*readval = val16; *readval = val16;
return ret; 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); 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 adis_enable_irq(struct adis *adis, bool enable)
{ {
int ret = 0; int ret = 0;
uint16_t msc; u16 msc;
mutex_lock(&adis->state_lock); mutex_lock(&adis->state_lock);
if (adis->data->enable_irq) { if (adis->data->enable_irq) {
ret = adis->data->enable_irq(adis, enable); ret = adis->data->enable_irq(adis, enable);
goto out_unlock; goto out_unlock;
} else if (adis->data->unmasked_drdy) { }
if (adis->data->unmasked_drdy) {
if (enable) if (enable)
enable_irq(adis->spi->irq); enable_irq(adis->spi->irq);
else else
@@ -322,7 +324,7 @@ EXPORT_SYMBOL(adis_enable_irq);
*/ */
int __adis_check_status(struct adis *adis) int __adis_check_status(struct adis *adis)
{ {
uint16_t status; u16 status;
int ret; int ret;
int i; int i;
@@ -414,7 +416,7 @@ int __adis_initial_startup(struct adis *adis)
{ {
const struct adis_timeout *timeouts = adis->data->timeouts; const struct adis_timeout *timeouts = adis->data->timeouts;
struct gpio_desc *gpio; struct gpio_desc *gpio;
uint16_t prod_id; u16 prod_id;
int ret; int ret;
/* check if the device has rst pin low */ /* check if the device has rst pin low */
@@ -423,7 +425,7 @@ int __adis_initial_startup(struct adis *adis)
return PTR_ERR(gpio); return PTR_ERR(gpio);
if (gpio) { if (gpio) {
msleep(10); usleep_range(10, 12);
/* bring device out of reset */ /* bring device out of reset */
gpiod_set_value_cansleep(gpio, 0); gpiod_set_value_cansleep(gpio, 0);
msleep(timeouts->reset_ms); 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. * a error bit in the channels raw value set error_mask to 0.
*/ */
int adis_single_conversion(struct iio_dev *indio_dev, 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); struct adis *adis = iio_device_get_drvdata(indio_dev);
unsigned int uval; unsigned int uval;

3
drivers/iio/imu/adis_trigger.c
View File

@@ -15,8 +15,7 @@
#include <linux/iio/trigger.h> #include <linux/iio/trigger.h>
#include <linux/iio/imu/adis.h> #include <linux/iio/imu/adis.h>
static int adis_data_rdy_trigger_set_state(struct iio_trigger *trig, static int adis_data_rdy_trigger_set_state(struct iio_trigger *trig, bool state)
bool state)
{ {
struct adis *adis = iio_trigger_get_drvdata(trig); struct adis *adis = iio_trigger_get_drvdata(trig);

36
include/linux/iio/imu/adis.h
View File

@@ -32,6 +32,7 @@ struct adis_timeout {
u16 sw_reset_ms; u16 sw_reset_ms;
u16 self_test_ms; u16 self_test_ms;
}; };
/** /**
* struct adis_data - ADIS chip variant specific data * struct adis_data - ADIS chip variant specific data
* @read_delay: SPI delay for read operations in us * @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_mask: Bitmask of supported self-test operations
* @self_test_reg: Register address to request self test command * @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 * @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 * @status_error_mask: Bitmask of errors supported by the device
* @timeouts: Chip specific delays * @timeouts: Chip specific delays
* @enable_irq: Hook for ADIS devices that have a special IRQ enable/disable * @enable_irq: Hook for ADIS devices that have a special IRQ enable/disable
@@ -130,8 +131,8 @@ struct adis {
unsigned long irq_flag; unsigned long irq_flag;
void *buffer; void *buffer;
uint8_t tx[10] ____cacheline_aligned; u8 tx[10] ____cacheline_aligned;
uint8_t rx[4]; u8 rx[4];
}; };
int adis_init(struct adis *adis, struct iio_dev *indio_dev, int adis_init(struct adis *adis, struct iio_dev *indio_dev,
@@ -167,7 +168,7 @@ int __adis_read_reg(struct adis *adis, unsigned int reg,
* @value: The value to write * @value: The value to write
*/ */
static inline int __adis_write_reg_8(struct adis *adis, unsigned int reg, 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); 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 * @value: Value to be written
*/ */
static inline int __adis_write_reg_16(struct adis *adis, unsigned int reg, 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); 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 * @value: Value to be written
*/ */
static inline int __adis_write_reg_32(struct adis *adis, unsigned int reg, 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); 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 * @val: The value read back from the device
*/ */
static inline int __adis_read_reg_16(struct adis *adis, unsigned int reg, static inline int __adis_read_reg_16(struct adis *adis, unsigned int reg,
uint16_t *val) u16 *val)
{ {
unsigned int tmp; unsigned int tmp;
int ret; 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 * @val: The value read back from the device
*/ */
static inline int __adis_read_reg_32(struct adis *adis, unsigned int reg, static inline int __adis_read_reg_32(struct adis *adis, unsigned int reg,
uint32_t *val) u32 *val)
{ {
unsigned int tmp; unsigned int tmp;
int ret; int ret;
@@ -279,7 +280,7 @@ static int adis_read_reg(struct adis *adis, unsigned int reg,
* @value: The value to write * @value: The value to write
*/ */
static inline int adis_write_reg_8(struct adis *adis, unsigned int reg, 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); 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 * @value: Value to be written
*/ */
static inline int adis_write_reg_16(struct adis *adis, unsigned int reg, 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); 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 * @value: Value to be written
*/ */
static inline int adis_write_reg_32(struct adis *adis, unsigned int reg, 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); 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 * @val: The value read back from the device
*/ */
static inline int adis_read_reg_16(struct adis *adis, unsigned int reg, static inline int adis_read_reg_16(struct adis *adis, unsigned int reg,
uint16_t *val) u16 *val)
{ {
unsigned int tmp; unsigned int tmp;
int ret; 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 * @val: The value read back from the device
*/ */
static inline int adis_read_reg_32(struct adis *adis, unsigned int reg, static inline int adis_read_reg_32(struct adis *adis, unsigned int reg,
uint32_t *val) u32 *val)
{ {
unsigned int tmp; unsigned int tmp;
int ret; int ret;
@@ -439,8 +440,8 @@ static inline void adis_dev_unlock(struct adis *adis)
} }
int adis_single_conversion(struct iio_dev *indio_dev, int adis_single_conversion(struct iio_dev *indio_dev,
const struct iio_chan_spec *chan, unsigned int error_mask, const struct iio_chan_spec *chan,
int *val); unsigned int error_mask, int *val);
#define ADIS_VOLTAGE_CHAN(addr, si, chan, name, info_all, bits) { \ #define ADIS_VOLTAGE_CHAN(addr, si, chan, name, info_all, bits) { \
.type = IIO_VOLTAGE, \ .type = IIO_VOLTAGE, \
@@ -489,7 +490,7 @@ int adis_single_conversion(struct iio_dev *indio_dev,
.modified = 1, \ .modified = 1, \
.channel2 = IIO_MOD_ ## mod, \ .channel2 = IIO_MOD_ ## mod, \
.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) | \ .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_type = BIT(IIO_CHAN_INFO_SCALE), \
.info_mask_shared_by_all = info_all, \ .info_mask_shared_by_all = info_all, \
.address = (addr), \ .address = (addr), \
@@ -547,7 +548,8 @@ static inline int devm_adis_probe_trigger(struct adis *adis,
#ifdef CONFIG_DEBUG_FS #ifdef CONFIG_DEBUG_FS
int adis_debugfs_reg_access(struct iio_dev *indio_dev, 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 #else