On Wed, Mar 18, 2020 at 07:00:38AM +0100, Greg KH wrote: > On Wed, Mar 18, 2020 at 09:58:13AM +0530, Deepak R Varma wrote: > > Add spaces around operator symbols to improve readability. Warning > > flagged by checkpatch script. > > > > Signed-off-by: Deepak R Varma <mh12gx2825@xxxxxxxxx> > > --- > > drivers/staging/iio/adc/ad7280a.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/staging/iio/adc/ad7280a.c b/drivers/staging/iio/adc/ad7280a.c > > index 19a5f244dcae..34ca0d09db85 100644 > > --- a/drivers/staging/iio/adc/ad7280a.c > > +++ b/drivers/staging/iio/adc/ad7280a.c > > @@ -825,14 +825,14 @@ 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, > > + in_voltage - voltage_thresh_low_value, > > 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, > > + in_voltage - voltage_thresh_high_value, > > 0644, > > ad7280_read_channel_config, > > ad7280_write_channel_config, > > Did you try building this code? > > It catches everyone... yes sir, I did. See this: CC [M] drivers/staging/fbtft/fbtft.mod.o CC [M] drivers/staging/iio/adc/ad7192.mod.o LD [M] drivers/staging/iio/adc/ad7280a.ko LD [M] drivers/staging/media/ipu3/ipu3-imgu.ko LD [M] drivers/staging/fbtft/fbtft.ko LD [M] drivers/staging/iio/adc/ad7192.ko Let me know if I missed something. Thank you, Deepak. > > greg k-h