From: Colin Ian King <colin.i.king@xxxxxxxxx> There is a check on non-zero ret that is redundant because the same check is being performed in a previous if statement and also before that. The check is not required, remove it. Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx> --- drivers/iio/pressure/rohm-bm1390.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/iio/pressure/rohm-bm1390.c b/drivers/iio/pressure/rohm-bm1390.c index ccaa07a569c9..f24d9f927681 100644 --- a/drivers/iio/pressure/rohm-bm1390.c +++ b/drivers/iio/pressure/rohm-bm1390.c @@ -410,23 +410,20 @@ static int __bm1390_fifo_flush(struct iio_dev *idev, unsigned int samples, if (ret) return ret; if (test_bit(BM1390_CHAN_TEMP, idev->active_scan_mask)) { ret = regmap_bulk_read(data->regmap, BM1390_REG_TEMP_HI, &temp, sizeof(temp)); if (ret) return ret; } - if (ret) - return ret; - for (i = 0; i < smp_lvl; i++) { buffer[i].temp = temp; iio_push_to_buffers(idev, &buffer[i]); } return smp_lvl; } static int bm1390_fifo_flush(struct iio_dev *idev, unsigned int samples) { -- 2.39.5