diff options
| author | Vivien Didelot <vivien.didelot@savoirfairelinux.com> | 2012-01-26 15:59:00 -0500 |
|---|---|---|
| committer | Simone Willett <swillett@nvidia.com> | 2012-02-08 14:06:52 -0800 |
| commit | 4fe22be3084aeac7c823b5b26de84a8085ecca95 (patch) | |
| tree | 3aaab70720113e97d400982c4a777752ec18debc /drivers/hwmon | |
| parent | e3703d249a6f14ff3be78ae8324f9c2324ee3020 (diff) | |
hwmon: (sht15) fix bad error code
commit 6edf3c30af01854c416f8654d3d5d2652470afd4 upstream.
When no platform data was supplied, returned error code was 0.
Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Signed-off-by: Guenter Roeck <guenter.roeck@ericsson.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Varun Wadekar <vwadekar@nvidia.com>
Change-Id: I8d12ab55a13d5c97df4d4f185a268ef5c973a66a
Reviewed-on: http://git-master/r/79685
Reviewed-by: Automatic_Commit_Validation_User
Diffstat (limited to 'drivers/hwmon')
| -rw-r--r-- | drivers/hwmon/sht15.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/hwmon/sht15.c b/drivers/hwmon/sht15.c index fe4104c6b764..535792550c6d 100644 --- a/drivers/hwmon/sht15.c +++ b/drivers/hwmon/sht15.c @@ -883,7 +883,7 @@ static int sht15_invalidate_voltage(struct notifier_block *nb, static int __devinit sht15_probe(struct platform_device *pdev) { - int ret = 0; + int ret; struct sht15_data *data = kzalloc(sizeof(*data), GFP_KERNEL); u8 status = 0; @@ -901,6 +901,7 @@ static int __devinit sht15_probe(struct platform_device *pdev) init_waitqueue_head(&data->wait_queue); if (pdev->dev.platform_data == NULL) { + ret = -EINVAL; dev_err(&pdev->dev, "no platform data supplied\n"); goto err_free_data; } |