diff options
author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2014-09-27 19:27:42 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2014-09-27 19:27:42 -0400 |
commit | 5159e39bdf119613adc6556901ae2d2e5e027551 (patch) | |
tree | 655958cbe030099c84dcc7985454115149401268 /drivers | |
parent | 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334 (diff) |
ACPI / SBS: Fix check in acpi_ac_get_present()
Parentheses are missing under an if () statement in
acpi_ac_get_present() which makes the check work differently
from what was intended (at least according to the comment right
above it). Add the missing parens.
The problem was found by sparse.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Cc: All applicable <stable@vger.kernel.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/acpi/sbs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/acpi/sbs.c b/drivers/acpi/sbs.c index 32aeceae10e3..a7a3edd28beb 100644 --- a/drivers/acpi/sbs.c +++ b/drivers/acpi/sbs.c | |||
@@ -441,7 +441,7 @@ static int acpi_ac_get_present(struct acpi_sbs *sbs) | |||
441 | * The spec requires that bit 4 always be 1. If it's not set, assume | 441 | * The spec requires that bit 4 always be 1. If it's not set, assume |
442 | * that the implementation doesn't support an SBS charger | 442 | * that the implementation doesn't support an SBS charger |
443 | */ | 443 | */ |
444 | if (!(status >> 4) & 0x1) | 444 | if (!((status >> 4) & 0x1)) |
445 | return -ENODEV; | 445 | return -ENODEV; |
446 | 446 | ||
447 | sbs->charger_present = (status >> 15) & 0x1; | 447 | sbs->charger_present = (status >> 15) & 0x1; |