diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-04-15 10:47:20 -0400 |
---|---|---|
committer | Darren Hart <dvhart@linux.intel.com> | 2016-04-15 11:26:25 -0400 |
commit | ff22b4806dd79ba06c78255b67e73c10e34d47cc (patch) | |
tree | f21c84edd8e7640ad21a957b9173d79704084b51 | |
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>
-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) | |||
127 | arg0.integer.value = reg; | 127 | arg0.integer.value = reg; |
128 | 128 | ||
129 | status = acpi_evaluate_integer(dev->handle, "ALRD", &args, &lret); | 129 | status = acpi_evaluate_integer(dev->handle, "ALRD", &args, &lret); |
130 | if (ACPI_FAILURE(status)) | ||
131 | return -EINVAL; | ||
130 | *ret = lret; | 132 | *ret = lret; |
131 | return (status != AE_OK) ? -EINVAL : 0; | 133 | return 0; |
132 | } | 134 | } |
133 | 135 | ||
134 | /** | 136 | /** |