summaryrefslogtreecommitdiffstats
path: root/drivers/acpi/sbs.c
diff options
context:
space:
mode:
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>2014-09-28 01:27:42 +0200
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2014-09-28 01:27:42 +0200
commit5159e39bdf119613adc6556901ae2d2e5e027551 (patch)
tree655958cbe030099c84dcc7985454115149401268 /drivers/acpi/sbs.c
parent7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334 (diff)
downloadlinux-5159e39bdf119613adc6556901ae2d2e5e027551.tar.gz
linux-5159e39bdf119613adc6556901ae2d2e5e027551.tar.bz2
linux-5159e39bdf119613adc6556901ae2d2e5e027551.zip
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/sbs.c')
-rw-r--r--drivers/acpi/sbs.c2
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;