lkml.org 
[lkml]   [2019]   [Mar]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH v2 1/5] staging: iio: adc: Tab alignment
On Sun, Mar 24, 2019 at 12:35:52PM +0000, Jonathan Cameron wrote:
> On Sat, 23 Mar 2019 20:21:36 +0100
> Cristian Sicilia <sicilia.cristian@gmail.com> wrote:
>
> > Aligned some parameters.
> >
> > Signed-off-by: Cristian Sicilia <sicilia.cristian@gmail.com>
> > ---
> > drivers/staging/iio/adc/ad7280a.c | 36 ++++++++++++++++++------------------
> > 1 file changed, 18 insertions(+), 18 deletions(-)
> >
> > diff --git a/drivers/staging/iio/adc/ad7280a.c b/drivers/staging/iio/adc/ad7280a.c
> > index d9df126..55b5879 100644
> > --- a/drivers/staging/iio/adc/ad7280a.c
> > +++ b/drivers/staging/iio/adc/ad7280a.c
> > @@ -830,30 +830,30 @@ static irqreturn_t ad7280_event_handler(int irq, void *private)
> > }
> >
> > static IIO_DEVICE_ATTR_NAMED(in_thresh_low_value,
> > - in_voltage-voltage_thresh_low_value,
> > - 0644,
> > - ad7280_read_channel_config,
> > - ad7280_write_channel_config,
> > - AD7280A_CELL_UNDERVOLTAGE);
> > + in_voltage - voltage_thresh_low_value,
> Firstly, that isn't in your description and secondly you just
> broke the userspace ABI. Take a very good look at what is happening here.

Yes sorry, it will badly stringify, I fix it


>
> > + 0644,
> > + ad7280_read_channel_config,
> > + ad7280_write_channel_config,
> > + AD7280A_CELL_UNDERVOLTAGE);
> >
> > static IIO_DEVICE_ATTR_NAMED(in_thresh_high_value,
> > - in_voltage-voltage_thresh_high_value,
> > - 0644,
> > - ad7280_read_channel_config,
> > - ad7280_write_channel_config,
> > - AD7280A_CELL_OVERVOLTAGE);
> > + in_voltage - voltage_thresh_high_value,
> > + 0644,
> > + ad7280_read_channel_config,
> > + ad7280_write_channel_config,
> > + AD7280A_CELL_OVERVOLTAGE);
> >
> > static IIO_DEVICE_ATTR(in_temp_thresh_low_value,
> > - 0644,
> > - ad7280_read_channel_config,
> > - ad7280_write_channel_config,
> > - AD7280A_AUX_ADC_UNDERVOLTAGE);
> > + 0644,
> > + ad7280_read_channel_config,
> > + ad7280_write_channel_config,
> > + AD7280A_AUX_ADC_UNDERVOLTAGE);
> >
> > static IIO_DEVICE_ATTR(in_temp_thresh_high_value,
> > - 0644,
> > - ad7280_read_channel_config,
> > - ad7280_write_channel_config,
> > - AD7280A_AUX_ADC_OVERVOLTAGE);
> > + 0644,
> > + ad7280_read_channel_config,
> > + ad7280_write_channel_config,
> > + AD7280A_AUX_ADC_OVERVOLTAGE);
> >
> > static struct attribute *ad7280_event_attributes[] = {
> > &iio_dev_attr_in_thresh_low_value.dev_attr.attr,
>

\
 
 \ /
  Last update: 2019-03-24 17:35    [W:0.046 / U:10.520 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site