diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-04-15 17:47:20 +0300 |
|---|---|---|
| committer | Darren Hart <dvhart@linux.intel.com> | 2016-04-15 08:26:25 -0700 |
| commit | ff22b4806dd79ba06c78255b67e73c10e34d47cc (patch) | |
| tree | f21c84edd8e7640ad21a957b9173d79704084b51 /drivers/platform | |
| parent | 330a106508128056b3d7b6c940a4e906da2282dc (diff) | |
hp_accel: Silence an uninitialized variable warning
If acpi_evaluate_integer() fails then "lret" isn't initialized. I've
tweaked the error handling to avoid this issue.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
Diffstat (limited to 'drivers/platform')
| -rw-r--r-- | drivers/platform/x86/hp_accel.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/platform/x86/hp_accel.c b/drivers/platform/x86/hp_accel.c index 263002ba43eb..09356684c32f 100644 --- a/drivers/platform/x86/hp_accel.c +++ b/drivers/platform/x86/hp_accel.c @@ -127,8 +127,10 @@ static int lis3lv02d_acpi_read(struct lis3lv02d *lis3, int reg, u8 *ret) arg0.integer.value = reg; status = acpi_evaluate_integer(dev->handle, "ALRD", &args, &lret); + if (ACPI_FAILURE(status)) + return -EINVAL; *ret = lret; - return (status != AE_OK) ? -EINVAL : 0; + return 0; } /** |