diff options
| author | Jonas Aaberg <jonas.aberg@stericsson.com> | 2012-02-14 07:46:59 +0100 |
|---|---|---|
| committer | Philippe Langlais <philippe.langlais@linaro.org> | 2012-02-14 16:02:57 +0100 |
| commit | 9eb78639b93cca0c50f1b0d322e8779771c76d9a (patch) | |
| tree | 111235769fe6ed1a0552278c4014668b55c2714e | |
| parent | d0f0a5776a19be937a2e02865d963971e4408b94 (diff) | |
misc: bh1780gli: Fix some obvious short commings
Unbalanced regulator handling on error during probe
and msleep on times less than 10 ms.
ST-Ericsson Linux next: -
ST-Ericsson ID: 370799
ST-Ericsson FOSS-OUT ID: Trivial
Change-Id: Ic1080bf38a2f1e1922ec015b305b5a2de3b30f0e
Signed-off-by: Jonas Aaberg <jonas.aberg@stericsson.com>
| -rw-r--r-- | drivers/misc/bh1780gli.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/drivers/misc/bh1780gli.c b/drivers/misc/bh1780gli.c index 060441599cb..b08b271bf28 100644 --- a/drivers/misc/bh1780gli.c +++ b/drivers/misc/bh1780gli.c @@ -145,7 +145,7 @@ static ssize_t bh1780_store_power_state(struct device *dev, if (val == BH1780_POFF) regulator_disable(ddata->regulator); - msleep(BH1780_PON_DELAY); + mdelay(BH1780_PON_DELAY); ddata->power_state = val; mutex_unlock(&ddata->lock); @@ -201,7 +201,7 @@ static int __devinit bh1780_probe(struct i2c_client *client, ret = bh1780_read(ddata, BH1780_REG_PARTID, "PART ID"); if (ret < 0) { dev_err(&client->dev, "failed to read part ID\n"); - goto put_regulator; + goto disable_regulator; } #ifdef CONFIG_HAS_EARLYSUSPEND ddata->early_suspend.level = EARLY_SUSPEND_LEVEL_BLANK_SCREEN + 1; @@ -219,13 +219,15 @@ static int __devinit bh1780_probe(struct i2c_client *client, mutex_init(&ddata->lock); ret = sysfs_create_group(&client->dev.kobj, &bh1780_attr_group); - if (ret) + if (ret) { + dev_err(&client->dev, "failed to create sysfs group\n"); goto put_regulator; + } return 0; - -put_regulator: +disable_regulator: regulator_disable(ddata->regulator); +put_regulator: regulator_put(ddata->regulator); free_ddata: kfree(ddata); |