* [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis @ 2016-09-14 1:04 Katie Dunne 2016-09-14 5:11 ` [Outreachy kernel] " Alison Schofield 2016-09-14 5:49 ` Julia Lawall 0 siblings, 2 replies; 7+ messages in thread From: Katie Dunne @ 2016-09-14 1:04 UTC (permalink / raw) To: outreachy-kernel; +Cc: lars, michael.hennerich, jic23 knaack.h@gmx.de, pmeerw@pmeerw.net Bcc: Subject: [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis Reply-To: Issue found by checkpatch: "Alignment should match open parenthesis" Signed-off-by: Katie Dunne <kdunne@mail.ccsf.edu> --- drivers/staging/iio/impedance-analyzer/ad5933.c | 34 ++++++++++++------------- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio/impedance-analyzer/ad5933.c index 24c348d..2307b4d 100644 --- a/drivers/staging/iio/impedance-analyzer/ad5933.c +++ b/drivers/staging/iio/impedance-analyzer/ad5933.c @@ -157,7 +157,7 @@ static const struct iio_chan_spec ad5933_channels[] = { }; static int ad5933_i2c_write(struct i2c_client *client, - u8 reg, u8 len, u8 *data) + u8 reg, u8 len, u8 *data) { int ret; @@ -172,7 +172,7 @@ static int ad5933_i2c_write(struct i2c_client *client, } static int ad5933_i2c_read(struct i2c_client *client, - u8 reg, u8 len, u8 *data) + u8 reg, u8 len, u8 *data) { int ret; @@ -269,7 +269,7 @@ static int ad5933_setup(struct ad5933_state *st) dat = cpu_to_be16(st->settling_cycles); ret = ad5933_i2c_write(st->client, - AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); + AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); if (ret < 0) return ret; @@ -294,8 +294,8 @@ static void ad5933_calc_out_ranges(struct ad5933_state *st) */ static ssize_t ad5933_show_frequency(struct device *dev, - struct device_attribute *attr, - char *buf) + struct device_attribute *attr, + char *buf) { struct iio_dev *indio_dev = dev_to_iio_dev(dev); struct ad5933_state *st = iio_priv(indio_dev); @@ -322,9 +322,9 @@ static ssize_t ad5933_show_frequency(struct device *dev, } static ssize_t ad5933_store_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 ad5933_state *st = iio_priv(indio_dev); @@ -357,8 +357,8 @@ static IIO_DEVICE_ATTR(out_voltage0_freq_increment, S_IRUGO | S_IWUSR, AD5933_REG_FREQ_INC); static ssize_t ad5933_show(struct device *dev, - struct device_attribute *attr, - char *buf) + struct device_attribute *attr, + char *buf) { struct iio_dev *indio_dev = dev_to_iio_dev(dev); struct ad5933_state *st = iio_priv(indio_dev); @@ -399,9 +399,9 @@ static ssize_t ad5933_show(struct device *dev, } static ssize_t ad5933_store(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 ad5933_state *st = iio_priv(indio_dev); @@ -451,7 +451,7 @@ static ssize_t ad5933_store(struct device *dev, dat = cpu_to_be16(val); ret = ad5933_i2c_write(st->client, - AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); + AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); break; case AD5933_FREQ_POINTS: val = clamp(val, (u16)0, (u16)511); @@ -545,8 +545,8 @@ static int ad5933_read_raw(struct iio_dev *indio_dev, goto out; ret = ad5933_i2c_read(st->client, - AD5933_REG_TEMP_DATA, 2, - (u8 *)&dat); + AD5933_REG_TEMP_DATA, 2, + (u8 *)&dat); if (ret < 0) goto out; mutex_unlock(&indio_dev->mlock); @@ -705,7 +705,7 @@ static void ad5933_work(struct work_struct *work) } static int ad5933_probe(struct i2c_client *client, - const struct i2c_device_id *id) + const struct i2c_device_id *id) { int ret, voltage_uv = 0; struct ad5933_platform_data *pdata = dev_get_platdata(&client->dev); -- 2.7.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis 2016-09-14 1:04 [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis Katie Dunne @ 2016-09-14 5:11 ` Alison Schofield 2016-09-14 14:10 ` Katie Dunne 2016-09-14 5:49 ` Julia Lawall 1 sibling, 1 reply; 7+ messages in thread From: Alison Schofield @ 2016-09-14 5:11 UTC (permalink / raw) To: Katie Dunne; +Cc: outreachy-kernel, lars, michael.hennerich, jic23 On Tue, Sep 13, 2016 at 06:04:14PM -0700, Katie Dunne wrote: > knaack.h@gmx.de, pmeerw@pmeerw.net > Bcc: > Subject: [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis > Reply-To: Hi Katie, Something odd happened just above here...with the cc list and subject repeating. It looks like some of these can be pulled into less lines. If it fits within 80 chars, it's fine on one line. See below. > > Issue found by checkpatch: "Alignment should match open parenthesis" > > Signed-off-by: Katie Dunne <kdunne@mail.ccsf.edu> > --- > drivers/staging/iio/impedance-analyzer/ad5933.c | 34 ++++++++++++------------- > 1 file changed, 17 insertions(+), 17 deletions(-) > > diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio/impedance-analyzer/ad5933.c > index 24c348d..2307b4d 100644 > --- a/drivers/staging/iio/impedance-analyzer/ad5933.c > +++ b/drivers/staging/iio/impedance-analyzer/ad5933.c > @@ -157,7 +157,7 @@ static const struct iio_chan_spec ad5933_channels[] = { > }; > > static int ad5933_i2c_write(struct i2c_client *client, > - u8 reg, u8 len, u8 *data) > + u8 reg, u8 len, u8 *data) think this one fits on one line.... static int ad5933_i2c_read(struct i2c_client *client, u8 reg, u8 len, u8 *data) some others may do same or at least pull into less lines. > { > int ret; > > @@ -172,7 +172,7 @@ static int ad5933_i2c_write(struct i2c_client *client, > } > > static int ad5933_i2c_read(struct i2c_client *client, > - u8 reg, u8 len, u8 *data) > + u8 reg, u8 len, u8 *data) > { > int ret; > > @@ -269,7 +269,7 @@ static int ad5933_setup(struct ad5933_state *st) > dat = cpu_to_be16(st->settling_cycles); > > ret = ad5933_i2c_write(st->client, > - AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > + AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > if (ret < 0) > return ret; > > @@ -294,8 +294,8 @@ static void ad5933_calc_out_ranges(struct ad5933_state *st) > */ > > static ssize_t ad5933_show_frequency(struct device *dev, > - struct device_attribute *attr, > - char *buf) > + struct device_attribute *attr, > + char *buf) > { > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > struct ad5933_state *st = iio_priv(indio_dev); > @@ -322,9 +322,9 @@ static ssize_t ad5933_show_frequency(struct device *dev, > } > > static ssize_t ad5933_store_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 ad5933_state *st = iio_priv(indio_dev); > @@ -357,8 +357,8 @@ static IIO_DEVICE_ATTR(out_voltage0_freq_increment, S_IRUGO | S_IWUSR, > AD5933_REG_FREQ_INC); > > static ssize_t ad5933_show(struct device *dev, > - struct device_attribute *attr, > - char *buf) > + struct device_attribute *attr, > + char *buf) > { > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > struct ad5933_state *st = iio_priv(indio_dev); > @@ -399,9 +399,9 @@ static ssize_t ad5933_show(struct device *dev, > } > > static ssize_t ad5933_store(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 ad5933_state *st = iio_priv(indio_dev); > @@ -451,7 +451,7 @@ static ssize_t ad5933_store(struct device *dev, > > dat = cpu_to_be16(val); > ret = ad5933_i2c_write(st->client, > - AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > + AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > break; > case AD5933_FREQ_POINTS: > val = clamp(val, (u16)0, (u16)511); > @@ -545,8 +545,8 @@ static int ad5933_read_raw(struct iio_dev *indio_dev, > goto out; > > ret = ad5933_i2c_read(st->client, > - AD5933_REG_TEMP_DATA, 2, > - (u8 *)&dat); > + AD5933_REG_TEMP_DATA, 2, > + (u8 *)&dat); > if (ret < 0) > goto out; > mutex_unlock(&indio_dev->mlock); > @@ -705,7 +705,7 @@ static void ad5933_work(struct work_struct *work) > } > > static int ad5933_probe(struct i2c_client *client, > - const struct i2c_device_id *id) > + const struct i2c_device_id *id) > { > int ret, voltage_uv = 0; > struct ad5933_platform_data *pdata = dev_get_platdata(&client->dev); > -- > 2.7.4 > > -- > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > To post to this group, send email to outreachy-kernel@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20160914010414.GA441%40katie-Inspiron-5748. > For more options, visit https://groups.google.com/d/optout. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis 2016-09-14 5:11 ` [Outreachy kernel] " Alison Schofield @ 2016-09-14 14:10 ` Katie Dunne 2016-09-14 14:18 ` Daniel Baluta 2016-09-14 16:30 ` Alison Schofield 0 siblings, 2 replies; 7+ messages in thread From: Katie Dunne @ 2016-09-14 14:10 UTC (permalink / raw) To: Alison Schofield; +Cc: outreachy-kernel, lars, michael.hennerich, jic23 On Tue, Sep 13, 2016 at 10:11:43PM -0700, Alison Schofield wrote: > On Tue, Sep 13, 2016 at 06:04:14PM -0700, Katie Dunne wrote: > > knaack.h@gmx.de, pmeerw@pmeerw.net > > Bcc: > > Subject: [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis > > Reply-To: > > Hi Katie, > Something odd happened just above here...with the cc list and subject > repeating. Thanks Alison. Something weird happened with mutt where the subject disappeared so I had type it twice. I will try to figure out what went wrong. > It looks like some of these can be pulled into less lines. If it fits > within 80 chars, it's fine on one line. See below. I wasn't sure if I could do this in the same patch, but that makes sense. I will send a v2. Katie > > > > > Issue found by checkpatch: "Alignment should match open parenthesis" > > > > Signed-off-by: Katie Dunne <kdunne@mail.ccsf.edu> > > --- > > drivers/staging/iio/impedance-analyzer/ad5933.c | 34 ++++++++++++------------- > > 1 file changed, 17 insertions(+), 17 deletions(-) > > > > diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio/impedance-analyzer/ad5933.c > > index 24c348d..2307b4d 100644 > > --- a/drivers/staging/iio/impedance-analyzer/ad5933.c > > +++ b/drivers/staging/iio/impedance-analyzer/ad5933.c > > @@ -157,7 +157,7 @@ static const struct iio_chan_spec ad5933_channels[] = { > > }; > > > > static int ad5933_i2c_write(struct i2c_client *client, > > - u8 reg, u8 len, u8 *data) > > + u8 reg, u8 len, u8 *data) > > think this one fits on one line.... > static int ad5933_i2c_read(struct i2c_client *client, u8 reg, u8 len, u8 *data) > some others may do same or at least pull into less lines. > > > > { > > int ret; > > > > @@ -172,7 +172,7 @@ static int ad5933_i2c_write(struct i2c_client *client, > > } > > > > static int ad5933_i2c_read(struct i2c_client *client, > > - u8 reg, u8 len, u8 *data) > > + u8 reg, u8 len, u8 *data) > > { > > int ret; > > > > @@ -269,7 +269,7 @@ static int ad5933_setup(struct ad5933_state *st) > > dat = cpu_to_be16(st->settling_cycles); > > > > ret = ad5933_i2c_write(st->client, > > - AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > > + AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > > if (ret < 0) > > return ret; > > > > @@ -294,8 +294,8 @@ static void ad5933_calc_out_ranges(struct ad5933_state *st) > > */ > > > > static ssize_t ad5933_show_frequency(struct device *dev, > > - struct device_attribute *attr, > > - char *buf) > > + struct device_attribute *attr, > > + char *buf) > > { > > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > > struct ad5933_state *st = iio_priv(indio_dev); > > @@ -322,9 +322,9 @@ static ssize_t ad5933_show_frequency(struct device *dev, > > } > > > > static ssize_t ad5933_store_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 ad5933_state *st = iio_priv(indio_dev); > > @@ -357,8 +357,8 @@ static IIO_DEVICE_ATTR(out_voltage0_freq_increment, S_IRUGO | S_IWUSR, > > AD5933_REG_FREQ_INC); > > > > static ssize_t ad5933_show(struct device *dev, > > - struct device_attribute *attr, > > - char *buf) > > + struct device_attribute *attr, > > + char *buf) > > { > > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > > struct ad5933_state *st = iio_priv(indio_dev); > > @@ -399,9 +399,9 @@ static ssize_t ad5933_show(struct device *dev, > > } > > > > static ssize_t ad5933_store(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 ad5933_state *st = iio_priv(indio_dev); > > @@ -451,7 +451,7 @@ static ssize_t ad5933_store(struct device *dev, > > > > dat = cpu_to_be16(val); > > ret = ad5933_i2c_write(st->client, > > - AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > > + AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > > break; > > case AD5933_FREQ_POINTS: > > val = clamp(val, (u16)0, (u16)511); > > @@ -545,8 +545,8 @@ static int ad5933_read_raw(struct iio_dev *indio_dev, > > goto out; > > > > ret = ad5933_i2c_read(st->client, > > - AD5933_REG_TEMP_DATA, 2, > > - (u8 *)&dat); > > + AD5933_REG_TEMP_DATA, 2, > > + (u8 *)&dat); > > if (ret < 0) > > goto out; > > mutex_unlock(&indio_dev->mlock); > > @@ -705,7 +705,7 @@ static void ad5933_work(struct work_struct *work) > > } > > > > static int ad5933_probe(struct i2c_client *client, > > - const struct i2c_device_id *id) > > + const struct i2c_device_id *id) > > { > > int ret, voltage_uv = 0; > > struct ad5933_platform_data *pdata = dev_get_platdata(&client->dev); > > -- > > 2.7.4 > > > > -- > > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > > To post to this group, send email to outreachy-kernel@googlegroups.com. > > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20160914010414.GA441%40katie-Inspiron-5748. > > For more options, visit https://groups.google.com/d/optout. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis 2016-09-14 14:10 ` Katie Dunne @ 2016-09-14 14:18 ` Daniel Baluta 2016-09-14 16:25 ` Katie Dunne 2016-09-14 16:30 ` Alison Schofield 1 sibling, 1 reply; 7+ messages in thread From: Daniel Baluta @ 2016-09-14 14:18 UTC (permalink / raw) To: Katie Dunne Cc: Alison Schofield, outreachy-kernel, Lars-Peter Clausen, Hennerich, Michael, Jonathan Cameron On Wed, Sep 14, 2016 at 5:10 PM, Katie Dunne <kdunne@mail.ccsf.edu> wrote: > On Tue, Sep 13, 2016 at 10:11:43PM -0700, Alison Schofield wrote: >> On Tue, Sep 13, 2016 at 06:04:14PM -0700, Katie Dunne wrote: >> > knaack.h@gmx.de, pmeerw@pmeerw.net >> > Bcc: >> > Subject: [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis >> > Reply-To: >> >> Hi Katie, >> Something odd happened just above here...with the cc list and subject >> repeating. > > Thanks Alison. Something weird happened with mutt where the subject > disappeared so I had type it twice. I will try to figure out what went > wrong. > >> It looks like some of these can be pulled into less lines. If it fits >> within 80 chars, it's fine on one line. See below. > > I wasn't sure if I could do this in the same patch, but that makes > sense. > > I will send a v2. Hi Katie, Just to make sure that everything works fine, it's always a good idea to send the email first only to yourself. Then if it looks good send it to everybody :). I usually find late problems with this approach. thanks, daniel. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis 2016-09-14 14:18 ` Daniel Baluta @ 2016-09-14 16:25 ` Katie Dunne 0 siblings, 0 replies; 7+ messages in thread From: Katie Dunne @ 2016-09-14 16:25 UTC (permalink / raw) To: Daniel Baluta; +Cc: Alison Schofield, outreachy-kernel On Wed, Sep 14, 2016 at 05:18:12PM +0300, Daniel Baluta wrote: > On Wed, Sep 14, 2016 at 5:10 PM, Katie Dunne <kdunne@mail.ccsf.edu> wrote: > > On Tue, Sep 13, 2016 at 10:11:43PM -0700, Alison Schofield wrote: > >> On Tue, Sep 13, 2016 at 06:04:14PM -0700, Katie Dunne wrote: > >> > knaack.h@gmx.de, pmeerw@pmeerw.net > >> > Bcc: > >> > Subject: [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis > >> > Reply-To: > >> > >> Hi Katie, > >> Something odd happened just above here...with the cc list and subject > >> repeating. > > > > Thanks Alison. Something weird happened with mutt where the subject > > disappeared so I had type it twice. I will try to figure out what went > > wrong. > > > >> It looks like some of these can be pulled into less lines. If it fits > >> within 80 chars, it's fine on one line. See below. > > > > I wasn't sure if I could do this in the same patch, but that makes > > sense. > > > > I will send a v2. > > Hi Katie, > > Just to make sure that everything works fine, it's always a good idea > to send the email first only to yourself. > > Then if it looks good send it to everybody :). I usually find late problems > with this approach. Thanks, Daniel. It looks like the problem was how I was formating the Cc list in mutt. I sent it only to myself first and it looked correct. But adding multiple emails addresses incorrectly changed the formatting. Katie > > thanks, > daniel. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis 2016-09-14 14:10 ` Katie Dunne 2016-09-14 14:18 ` Daniel Baluta @ 2016-09-14 16:30 ` Alison Schofield 1 sibling, 0 replies; 7+ messages in thread From: Alison Schofield @ 2016-09-14 16:30 UTC (permalink / raw) To: Katie Dunne; +Cc: outreachy-kernel, lars, michael.hennerich, jic23 On Wed, Sep 14, 2016 at 07:10:42AM -0700, Katie Dunne wrote: > On Tue, Sep 13, 2016 at 10:11:43PM -0700, Alison Schofield wrote: > > On Tue, Sep 13, 2016 at 06:04:14PM -0700, Katie Dunne wrote: > > > knaack.h@gmx.de, pmeerw@pmeerw.net > > > Bcc: > > > Subject: [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis > > > Reply-To: > > > > Hi Katie, > > Something odd happened just above here...with the cc list and subject > > repeating. > > Thanks Alison. Something weird happened with mutt where the subject > disappeared so I had type it twice. I will try to figure out what went > wrong. > > > It looks like some of these can be pulled into less lines. If it fits > > within 80 chars, it's fine on one line. See below. > > I wasn't sure if I could do this in the same patch, but that makes > sense. > > I will send a v2. Katie, Your comment is good, because that is a question we keep asking: should something be a separate patch? In this case, as you go to realign the function arguments and as a side effect remove some line wrap, it's all part of the realignment. alisons > > Katie > > > > > > > > > Issue found by checkpatch: "Alignment should match open parenthesis" > > > > > > Signed-off-by: Katie Dunne <kdunne@mail.ccsf.edu> > > > --- > > > drivers/staging/iio/impedance-analyzer/ad5933.c | 34 ++++++++++++------------- > > > 1 file changed, 17 insertions(+), 17 deletions(-) > > > > > > diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio/impedance-analyzer/ad5933.c > > > index 24c348d..2307b4d 100644 > > > --- a/drivers/staging/iio/impedance-analyzer/ad5933.c > > > +++ b/drivers/staging/iio/impedance-analyzer/ad5933.c > > > @@ -157,7 +157,7 @@ static const struct iio_chan_spec ad5933_channels[] = { > > > }; > > > > > > static int ad5933_i2c_write(struct i2c_client *client, > > > - u8 reg, u8 len, u8 *data) > > > + u8 reg, u8 len, u8 *data) > > > > think this one fits on one line.... > > static int ad5933_i2c_read(struct i2c_client *client, u8 reg, u8 len, u8 *data) > > some others may do same or at least pull into less lines. > > > > > > > { > > > int ret; > > > > > > @@ -172,7 +172,7 @@ static int ad5933_i2c_write(struct i2c_client *client, > > > } > > > > > > static int ad5933_i2c_read(struct i2c_client *client, > > > - u8 reg, u8 len, u8 *data) > > > + u8 reg, u8 len, u8 *data) > > > { > > > int ret; > > > > > > @@ -269,7 +269,7 @@ static int ad5933_setup(struct ad5933_state *st) > > > dat = cpu_to_be16(st->settling_cycles); > > > > > > ret = ad5933_i2c_write(st->client, > > > - AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > > > + AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > > > if (ret < 0) > > > return ret; > > > > > > @@ -294,8 +294,8 @@ static void ad5933_calc_out_ranges(struct ad5933_state *st) > > > */ > > > > > > static ssize_t ad5933_show_frequency(struct device *dev, > > > - struct device_attribute *attr, > > > - char *buf) > > > + struct device_attribute *attr, > > > + char *buf) > > > { > > > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > > > struct ad5933_state *st = iio_priv(indio_dev); > > > @@ -322,9 +322,9 @@ static ssize_t ad5933_show_frequency(struct device *dev, > > > } > > > > > > static ssize_t ad5933_store_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 ad5933_state *st = iio_priv(indio_dev); > > > @@ -357,8 +357,8 @@ static IIO_DEVICE_ATTR(out_voltage0_freq_increment, S_IRUGO | S_IWUSR, > > > AD5933_REG_FREQ_INC); > > > > > > static ssize_t ad5933_show(struct device *dev, > > > - struct device_attribute *attr, > > > - char *buf) > > > + struct device_attribute *attr, > > > + char *buf) > > > { > > > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > > > struct ad5933_state *st = iio_priv(indio_dev); > > > @@ -399,9 +399,9 @@ static ssize_t ad5933_show(struct device *dev, > > > } > > > > > > static ssize_t ad5933_store(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 ad5933_state *st = iio_priv(indio_dev); > > > @@ -451,7 +451,7 @@ static ssize_t ad5933_store(struct device *dev, > > > > > > dat = cpu_to_be16(val); > > > ret = ad5933_i2c_write(st->client, > > > - AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > > > + AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > > > break; > > > case AD5933_FREQ_POINTS: > > > val = clamp(val, (u16)0, (u16)511); > > > @@ -545,8 +545,8 @@ static int ad5933_read_raw(struct iio_dev *indio_dev, > > > goto out; > > > > > > ret = ad5933_i2c_read(st->client, > > > - AD5933_REG_TEMP_DATA, 2, > > > - (u8 *)&dat); > > > + AD5933_REG_TEMP_DATA, 2, > > > + (u8 *)&dat); > > > if (ret < 0) > > > goto out; > > > mutex_unlock(&indio_dev->mlock); > > > @@ -705,7 +705,7 @@ static void ad5933_work(struct work_struct *work) > > > } > > > > > > static int ad5933_probe(struct i2c_client *client, > > > - const struct i2c_device_id *id) > > > + const struct i2c_device_id *id) > > > { > > > int ret, voltage_uv = 0; > > > struct ad5933_platform_data *pdata = dev_get_platdata(&client->dev); > > > -- > > > 2.7.4 > > > > > > -- > > > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > > > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > > > To post to this group, send email to outreachy-kernel@googlegroups.com. > > > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20160914010414.GA441%40katie-Inspiron-5748. > > > For more options, visit https://groups.google.com/d/optout. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis 2016-09-14 1:04 [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis Katie Dunne 2016-09-14 5:11 ` [Outreachy kernel] " Alison Schofield @ 2016-09-14 5:49 ` Julia Lawall 1 sibling, 0 replies; 7+ messages in thread From: Julia Lawall @ 2016-09-14 5:49 UTC (permalink / raw) To: Katie Dunne; +Cc: outreachy-kernel, lars, michael.hennerich, jic23 On Tue, 13 Sep 2016, Katie Dunne wrote: > knaack.h@gmx.de, pmeerw@pmeerw.net > Bcc: > Subject: [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis > Reply-To: Something went wrong here... julia > > Issue found by checkpatch: "Alignment should match open parenthesis" > > Signed-off-by: Katie Dunne <kdunne@mail.ccsf.edu> > --- > drivers/staging/iio/impedance-analyzer/ad5933.c | 34 ++++++++++++------------- > 1 file changed, 17 insertions(+), 17 deletions(-) > > diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio/impedance-analyzer/ad5933.c > index 24c348d..2307b4d 100644 > --- a/drivers/staging/iio/impedance-analyzer/ad5933.c > +++ b/drivers/staging/iio/impedance-analyzer/ad5933.c > @@ -157,7 +157,7 @@ static const struct iio_chan_spec ad5933_channels[] = { > }; > > static int ad5933_i2c_write(struct i2c_client *client, > - u8 reg, u8 len, u8 *data) > + u8 reg, u8 len, u8 *data) > { > int ret; > > @@ -172,7 +172,7 @@ static int ad5933_i2c_write(struct i2c_client *client, > } > > static int ad5933_i2c_read(struct i2c_client *client, > - u8 reg, u8 len, u8 *data) > + u8 reg, u8 len, u8 *data) > { > int ret; > > @@ -269,7 +269,7 @@ static int ad5933_setup(struct ad5933_state *st) > dat = cpu_to_be16(st->settling_cycles); > > ret = ad5933_i2c_write(st->client, > - AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > + AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > if (ret < 0) > return ret; > > @@ -294,8 +294,8 @@ static void ad5933_calc_out_ranges(struct ad5933_state *st) > */ > > static ssize_t ad5933_show_frequency(struct device *dev, > - struct device_attribute *attr, > - char *buf) > + struct device_attribute *attr, > + char *buf) > { > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > struct ad5933_state *st = iio_priv(indio_dev); > @@ -322,9 +322,9 @@ static ssize_t ad5933_show_frequency(struct device *dev, > } > > static ssize_t ad5933_store_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 ad5933_state *st = iio_priv(indio_dev); > @@ -357,8 +357,8 @@ static IIO_DEVICE_ATTR(out_voltage0_freq_increment, S_IRUGO | S_IWUSR, > AD5933_REG_FREQ_INC); > > static ssize_t ad5933_show(struct device *dev, > - struct device_attribute *attr, > - char *buf) > + struct device_attribute *attr, > + char *buf) > { > struct iio_dev *indio_dev = dev_to_iio_dev(dev); > struct ad5933_state *st = iio_priv(indio_dev); > @@ -399,9 +399,9 @@ static ssize_t ad5933_show(struct device *dev, > } > > static ssize_t ad5933_store(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 ad5933_state *st = iio_priv(indio_dev); > @@ -451,7 +451,7 @@ static ssize_t ad5933_store(struct device *dev, > > dat = cpu_to_be16(val); > ret = ad5933_i2c_write(st->client, > - AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > + AD5933_REG_SETTLING_CYCLES, 2, (u8 *)&dat); > break; > case AD5933_FREQ_POINTS: > val = clamp(val, (u16)0, (u16)511); > @@ -545,8 +545,8 @@ static int ad5933_read_raw(struct iio_dev *indio_dev, > goto out; > > ret = ad5933_i2c_read(st->client, > - AD5933_REG_TEMP_DATA, 2, > - (u8 *)&dat); > + AD5933_REG_TEMP_DATA, 2, > + (u8 *)&dat); > if (ret < 0) > goto out; > mutex_unlock(&indio_dev->mlock); > @@ -705,7 +705,7 @@ static void ad5933_work(struct work_struct *work) > } > > static int ad5933_probe(struct i2c_client *client, > - const struct i2c_device_id *id) > + const struct i2c_device_id *id) > { > int ret, voltage_uv = 0; > struct ad5933_platform_data *pdata = dev_get_platdata(&client->dev); > -- > 2.7.4 > > -- > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > To post to this group, send email to outreachy-kernel@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20160914010414.GA441%40katie-Inspiron-5748. > For more options, visit https://groups.google.com/d/optout. > ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-09-14 16:30 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-09-14 1:04 [PATCH] staging: iio: ad5933: align arguments on new line with open parenthesis Katie Dunne 2016-09-14 5:11 ` [Outreachy kernel] " Alison Schofield 2016-09-14 14:10 ` Katie Dunne 2016-09-14 14:18 ` Daniel Baluta 2016-09-14 16:25 ` Katie Dunne 2016-09-14 16:30 ` Alison Schofield 2016-09-14 5:49 ` Julia Lawall
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.