From: Daniel Kurtz Date: Wed, 9 May 2012 05:40:26 +0000 (-0700) Subject: Input: atmel_mxt_ts - do not read extra (checksum) byte X-Git-Tag: upstream/snapshot3+hdmi~3244^2~131^2~13 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=822115ff4bd7141a6069dcc17b35dfd823a44dff;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git Input: atmel_mxt_ts - do not read extra (checksum) byte atmel_mxt devices will send a checksum byte at the end of a message if the MSB of the object address is set. However, since this driver does not set this bit, the checksum byte isn't actually sent, so don't even try to read it. Signed-off-by: Daniel Kurtz Signed-off-by: Dmitry Torokhov --- diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c b/drivers/input/touchscreen/atmel_mxt_ts.c index 8f0dc77..12a6ba2 100644 --- a/drivers/input/touchscreen/atmel_mxt_ts.c +++ b/drivers/input/touchscreen/atmel_mxt_ts.c @@ -236,7 +236,6 @@ struct mxt_object { struct mxt_message { u8 reportid; u8 message[7]; - u8 checksum; }; struct mxt_finger { @@ -336,7 +335,6 @@ static void mxt_dump_message(struct device *dev, dev_dbg(dev, "message5:\t0x%x\n", message->message[4]); dev_dbg(dev, "message6:\t0x%x\n", message->message[5]); dev_dbg(dev, "message7:\t0x%x\n", message->message[6]); - dev_dbg(dev, "checksum:\t0x%x\n", message->checksum); } static int mxt_check_bootloader(struct i2c_client *client,