]> rtime.felk.cvut.cz Git - sojka/nv-tegra/linux-3.10.git/commitdiff
Revert "staging: iio: light: cm3217: check event's validity"
authorSri Krishna chowdary <schowdary@nvidia.com>
Sat, 21 Jun 2014 18:36:55 +0000 (00:06 +0530)
committerDhiren Parmar <dparmar@nvidia.com>
Sun, 22 Jun 2014 11:25:08 +0000 (04:25 -0700)
This reverts commit 106b781a903756f574c9f7130c3db36d6bfe97c3.
From SQA testing it is found that this causes regression in auto-brightness.

Bug 1521699

Change-Id: I39e972002d1d87092e10205a9bed50f44fcfa512
Signed-off-by: Sri Krishna chowdary <schowdary@nvidia.com>
Reviewed-on: http://git-master/r/426899
Reviewed-by: Dhiren Parmar <dparmar@nvidia.com>
drivers/staging/iio/light/cm3217.c

index e40f612e290702bd3ef34c2e5515db723a80cd5f..a104341b4c2865f3a97835f08160ba9ea142a05d 100644 (file)
@@ -320,7 +320,6 @@ static ssize_t cm3217_enable_store(struct device *dev,
        mutex_lock(&indio_dev->mlock);
        if (enable) {
                err = cm3217_cmd_wr(inf, 0, 0);
-               inf->raw_illuminance_val = -EINVAL;
                queue_delayed_work(inf->wq, &inf->dw, CM3217_HW_DELAY);
        } else {
                cancel_delayed_work_sync(&inf->dw);
@@ -343,8 +342,6 @@ static ssize_t cm3217_raw_illuminance_val_show(struct device *dev,
        if (inf->als_state != CHIP_POWER_ON_ALS_ON)
                return sprintf(buf, "-1\n");
        queue_delayed_work(inf->wq, &inf->dw, 0);
-       if (inf->raw_illuminance_val == -EINVAL)
-               return sprintf(buf, "-1\n");
        return sprintf(buf, "%d\n", inf->raw_illuminance_val);
 }
 
@@ -500,7 +497,6 @@ static int cm3217_probe(struct i2c_client *client,
                goto err_iio_register;
        }
 
-       inf->raw_illuminance_val = -EINVAL;
        dev_info(&client->dev, "%s success\n", __func__);
        return 0;