diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2014-09-28 01:27:42 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2014-09-28 01:27:42 +0200 |
| commit | 5159e39bdf119613adc6556901ae2d2e5e027551 (patch) | |
| tree | 655958cbe030099c84dcc7985454115149401268 /drivers/acpi | |
| parent | 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334 (diff) | |
| download | linux-5159e39bdf119613adc6556901ae2d2e5e027551.tar.bz2 | |
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/acpi')
| -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) * The spec requires that bit 4 always be 1. If it's not set, assume * that the implementation doesn't support an SBS charger */ - if (!(status >> 4) & 0x1) + if (!((status >> 4) & 0x1)) return -ENODEV; sbs->charger_present = (status >> 15) & 0x1; |