diff options
author | Brian Masney <masneyb@onstation.org> | 2017-01-17 04:24:56 -0500 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2017-01-22 13:21:36 +0000 |
commit | 65ac716b2013b97d68dbec4be0b5fb01a16efe7e (patch) | |
tree | d1608af6f95b93fbd489bb371beb15433331ef46 /drivers/staging/iio | |
parent | 578e02a7ed551f8d5aa8ee9fd094369bd84e7b01 (diff) |
staging: iio: isl29028: remove unnecessary error logging in isl29028_chip_init_and_power_on()
If the call to isl29028_chip_init_and_power_on() in isl29028_probe()
fails, then isl29028_probe() will log an error message. All of the
error paths in that call path already have error logging in place. This
patch removes the unnecessary logging.
Signed-off-by: Brian Masney <masneyb@onstation.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/staging/iio')
-rw-r--r-- | drivers/staging/iio/light/isl29028.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/staging/iio/light/isl29028.c b/drivers/staging/iio/light/isl29028.c index cffecf9d6bde..be1fc4a8960e 100644 --- a/drivers/staging/iio/light/isl29028.c +++ b/drivers/staging/iio/light/isl29028.c @@ -528,10 +528,8 @@ static int isl29028_probe(struct i2c_client *client, } ret = isl29028_chip_init_and_power_on(chip); - if (ret < 0) { - dev_err(&client->dev, "chip initialization failed: %d\n", ret); + if (ret < 0) return ret; - } indio_dev->info = &isl29028_info; indio_dev->channels = isl29028_channels; |