diff options
| author | Martin Kepplinger <martink@posteo.de> | 2017-05-01 20:13:47 +0300 |
|---|---|---|
| committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2017-05-01 22:30:00 +0300 |
| commit | 8a038b83e012097a7ac6cfb9f6c5fac1da8fad6e (patch) | |
| tree | f9f4c54bec7e89928f90b22d64bde02f39d062ca | |
| parent | 61e29ec1c96f67605217c2a80441a75ff906db62 (diff) | |
| download | linux-8a038b83e012097a7ac6cfb9f6c5fac1da8fad6e.tar.xz | |
Input: ar1021_i2c - use BIT to check for a bit
The MSB for the first byte of touch data transmission is always 1. Make
it a little more obvious we're testing this bit by using BIT(7).
Signed-off-by: Martin Kepplinger <martink@posteo.de>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
| -rw-r--r-- | drivers/input/touchscreen/ar1021_i2c.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/input/touchscreen/ar1021_i2c.c b/drivers/input/touchscreen/ar1021_i2c.c index 21c74ee59341..f9dcbd63e598 100644 --- a/drivers/input/touchscreen/ar1021_i2c.c +++ b/drivers/input/touchscreen/ar1021_i2c.c @@ -6,6 +6,7 @@ * License: GPLv2 as published by the FSF. */ +#include <linux/bitops.h> #include <linux/module.h> #include <linux/input.h> #include <linux/of.h> @@ -42,7 +43,7 @@ static irqreturn_t ar1021_i2c_irq(int irq, void *dev_id) goto out; /* sync bit set ? */ - if ((data[0] & 0x80) == 0) + if (!(data[0] & BIT(7))) goto out; button = data[0] & BIT(0); |