On Sat, 10 Feb 2018 18:23:15 -0200 rodrigosiqueira <rodrigosiqueiramelo@xxxxxxxxx> wrote: > This patch fixes the CHECKs reported by checkpatch.pl for "alignment > should match open parenthesis" > > Signed-off-by: rodrigosiqueira <rodrigosiqueiramelo@xxxxxxxxx> Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > --- > drivers/staging/iio/meter/ade7759.c | 80 ++++++++++++++++++------------------- > 1 file changed, 38 insertions(+), 42 deletions(-) > > diff --git a/drivers/staging/iio/meter/ade7759.c b/drivers/staging/iio/meter/ade7759.c > index d99cf508d8d0..1decb2b8afab 100644 > --- a/drivers/staging/iio/meter/ade7759.c > +++ b/drivers/staging/iio/meter/ade7759.c > @@ -72,8 +72,8 @@ struct ade7759_state { > }; > > static int ade7759_spi_write_reg_8(struct device *dev, > - u8 reg_address, > - u8 val) > + u8 reg_address, > + u8 val) > { > int ret; > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > @@ -91,8 +91,8 @@ static int ade7759_spi_write_reg_8(struct device *dev, > > /*Unlocked version of ade7759_spi_write_reg_16 function */ > static int __ade7759_spi_write_reg_16(struct device *dev, > - u8 reg_address, > - u16 value) > + u8 reg_address, > + u16 value) > { > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > struct ade7759_state *st = iio_priv(indio_dev); > @@ -104,8 +104,8 @@ static int __ade7759_spi_write_reg_16(struct device *dev, > } > > static int ade7759_spi_write_reg_16(struct device *dev, > - u8 reg_address, > - u16 value) > + u8 reg_address, > + u16 value) > { > int ret; > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > @@ -119,8 +119,8 @@ static int ade7759_spi_write_reg_16(struct device *dev, > } > > static int ade7759_spi_read_reg_8(struct device *dev, > - u8 reg_address, > - u8 *val) > + u8 reg_address, > + u8 *val) > { > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > struct ade7759_state *st = iio_priv(indio_dev); > @@ -128,8 +128,9 @@ static int ade7759_spi_read_reg_8(struct device *dev, > > ret = spi_w8r8(st->us, ADE7759_READ_REG(reg_address)); > if (ret < 0) { > - dev_err(&st->us->dev, "problem when reading 8 bit register 0x%02X", > - reg_address); > + dev_err(&st->us->dev, > + "problem when reading 8 bit register 0x%02X", > + reg_address); > return ret; > } > *val = ret; > @@ -138,8 +139,8 @@ static int ade7759_spi_read_reg_8(struct device *dev, > } > > static int ade7759_spi_read_reg_16(struct device *dev, > - u8 reg_address, > - u16 *val) > + u8 reg_address, > + u16 *val) > { > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > struct ade7759_state *st = iio_priv(indio_dev); > @@ -158,8 +159,8 @@ static int ade7759_spi_read_reg_16(struct device *dev, > } > > static int ade7759_spi_read_reg_40(struct device *dev, > - u8 reg_address, > - u64 *val) > + u8 reg_address, > + u64 *val) > { > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > struct ade7759_state *st = iio_priv(indio_dev); > @@ -179,8 +180,9 @@ static int ade7759_spi_read_reg_40(struct device *dev, > > ret = spi_sync_transfer(st->us, xfers, ARRAY_SIZE(xfers)); > if (ret) { > - dev_err(&st->us->dev, "problem when reading 40 bit register 0x%02X", > - reg_address); > + dev_err(&st->us->dev, > + "problem when reading 40 bit register 0x%02X", > + reg_address); > goto error_ret; > } > *val = ((u64)st->rx[1] << 32) | ((u64)st->rx[2] << 24) | > @@ -192,8 +194,8 @@ static int ade7759_spi_read_reg_40(struct device *dev, > } > > static ssize_t ade7759_read_8bit(struct device *dev, > - struct device_attribute *attr, > - char *buf) > + struct device_attribute *attr, > + char *buf) > { > int ret; > u8 val = 0; > @@ -207,8 +209,8 @@ static ssize_t ade7759_read_8bit(struct device *dev, > } > > static ssize_t ade7759_read_16bit(struct device *dev, > - struct device_attribute *attr, > - char *buf) > + struct device_attribute *attr, > + char *buf) > { > int ret; > u16 val = 0; > @@ -222,8 +224,8 @@ static ssize_t ade7759_read_16bit(struct device *dev, > } > > static ssize_t ade7759_read_40bit(struct device *dev, > - struct device_attribute *attr, > - char *buf) > + struct device_attribute *attr, > + char *buf) > { > int ret; > u64 val = 0; > @@ -237,9 +239,9 @@ static ssize_t ade7759_read_40bit(struct device *dev, > } > > static ssize_t ade7759_write_8bit(struct device *dev, > - struct device_attribute *attr, > - const char *buf, > - size_t len) > + struct device_attribute *attr, > + const char *buf, > + size_t len) > { > struct iio_dev_attr *this_attr = to_iio_dev_attr(attr); > int ret; > @@ -255,9 +257,9 @@ static ssize_t ade7759_write_8bit(struct device *dev, > } > > static ssize_t ade7759_write_16bit(struct device *dev, > - struct device_attribute *attr, > - const char *buf, > - size_t len) > + struct device_attribute *attr, > + const char *buf, > + size_t len) > { > struct iio_dev_attr *this_attr = to_iio_dev_attr(attr); > int ret; > @@ -277,9 +279,7 @@ static int ade7759_reset(struct device *dev) > int ret; > u16 val; > > - ret = ade7759_spi_read_reg_16(dev, > - ADE7759_MODE, > - &val); > + ret = ade7759_spi_read_reg_16(dev, ADE7759_MODE, &val); > if (ret < 0) > return ret; > > @@ -365,9 +365,7 @@ static int ade7759_stop_device(struct device *dev) > int ret; > u16 val; > > - ret = ade7759_spi_read_reg_16(dev, > - ADE7759_MODE, > - &val); > + ret = ade7759_spi_read_reg_16(dev, ADE7759_MODE, &val); > if (ret < 0) { > dev_err(dev, "unable to power down the device, error: %d\n", > ret); > @@ -404,16 +402,14 @@ static int ade7759_initial_setup(struct iio_dev *indio_dev) > } > > static ssize_t ade7759_read_frequency(struct device *dev, > - struct device_attribute *attr, > - char *buf) > + struct device_attribute *attr, > + char *buf) > { > int ret; > u16 t; > int sps; > > - ret = ade7759_spi_read_reg_16(dev, > - ADE7759_MODE, > - &t); > + ret = ade7759_spi_read_reg_16(dev, ADE7759_MODE, &t); > if (ret) > return ret; > > @@ -424,9 +420,9 @@ static ssize_t ade7759_read_frequency(struct device *dev, > } > > static ssize_t ade7759_write_frequency(struct device *dev, > - struct device_attribute *attr, > - const char *buf, > - size_t len) > + struct device_attribute *attr, > + const char *buf, > + size_t len) > { > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > struct ade7759_state *st = iio_priv(indio_dev); -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html