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 | staging: iio: isl29028: made error messages consistent (diff) | |
| download | linux-dev-65ac716b2013b97d68dbec4be0b5fb01a16efe7e.tar.xz linux-dev-65ac716b2013b97d68dbec4be0b5fb01a16efe7e.zip | |
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; |