On Fri, Mar 08, 2019 at 04:46:52PM -0300, Anderson Reis wrote: > From: Lucas Oshiro <lucasseikioshiro@xxxxxxxxx> > > Remove unnecessary parentheses on line 116, and solve these checkpatch.pl No need to explicitly point finger at changed lines. This is what context lines are for. > CHECKs: > > - lmp91000.c:116: CHECK: Unnecessary parentheses around 'state != channel' > - lmp91000.c:116: CHECK: Unnecessary parentheses around 'channel == LMP91000_REG_MODECN_TEMP' > > Signed-off-by: Lucas Oshiro <lucasseikioshiro@xxxxxxxxx> > Signed-off-by: Anderson Reis <andersonreisrosa@xxxxxxxxx> > --- > drivers/iio/potentiostat/lmp91000.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/potentiostat/lmp91000.c b/drivers/iio/potentiostat/lmp91000.c > index 90e895adf997..03d277621861 100644 > --- a/drivers/iio/potentiostat/lmp91000.c > +++ b/drivers/iio/potentiostat/lmp91000.c > @@ -113,7 +113,7 @@ static int lmp91000_read(struct lmp91000_data *data, int channel, int *val) > return -EINVAL; > > /* delay till first temperature reading is complete */ > - if ((state != channel) && (channel == LMP91000_REG_MODECN_TEMP)) > + if (state != channel && channel == LMP91000_REG_MODECN_TEMP) > usleep_range(3000, 4000); > > data->chan_select = channel != LMP91000_REG_MODECN_3LEAD; > -- > 2.20.1 >
