Re: [patch] Input: silead - remove some dead code
From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2016-08-04 15:25:45
Also in:
kernel-janitors
On Thu, Aug 04, 2016 at 08:28:59AM +0300, Dan Carpenter wrote:
buf[0] is an unsigned char. touch_nr is an int. The test for negative here doesn't make sense so I have removed it. Signed-off-by: Dan Carpenter <redacted>
Applied, thank you.
quoted hunk ↗ jump to hunk
diff --git a/drivers/input/touchscreen/silead.c b/drivers/input/touchscreen/silead.c index 5f55167..7379fe1 100644 --- a/drivers/input/touchscreen/silead.c +++ b/drivers/input/touchscreen/silead.c@@ -147,9 +147,6 @@ static void silead_ts_read_data(struct i2c_client *client) } touch_nr = buf[0]; - if (touch_nr < 0) - return; - if (touch_nr > data->max_fingers) { dev_warn(dev, "More touches reported then supported %d > %d\n", touch_nr, data->max_fingers);
-- Dmitry