On Fri, 22 Feb 2019 17:31:59 -0300 Bárbara Fernandes <barbara.fernandes@xxxxxx> wrote: > Solves checkpath.pl's message: > > CHECK: Comparison to NULL could be written "!indio_dev" > > Signed-off-by: Bárbara Fernandes <barbara.fernandes@xxxxxx> Applied, thanks. It's nice to get some cleanup for drivers that aren't in staging sometimes. After all, in recent years the kernel coding style and tools to check it have improved considerably. Thanks, Jonathan > --- > drivers/iio/adc/ad7923.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/adc/ad7923.c b/drivers/iio/adc/ad7923.c > index dbece44e26e4..cb7b854df00c 100644 > --- a/drivers/iio/adc/ad7923.c > +++ b/drivers/iio/adc/ad7923.c > @@ -272,7 +272,7 @@ static int ad7923_probe(struct spi_device *spi) > int ret; > > indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*st)); > - if (indio_dev == NULL) > + if (!indio_dev) > return -ENOMEM; > > st = iio_priv(indio_dev);