Hi Henrik, Thanks for your comments. > -----Original Message----- > From: Henrik Rydberg [mailto:rydberg@xxxxxxxxxxx] > Sent: Wednesday, September 22, 2010 11:51 PM > > +struct bu21013_ts_data { > > + struct i2c_client *client; > > + wait_queue_head_t wait; > > + bool touch_stopped; > > + const struct bu21013_platform_device *chip; > > + struct input_dev *in_dev; > > + unsigned int intr_pin; > > + signed short x_pos[2]; > > + signed short y_pos[2]; > > + bool previous_press_reported; > > > The MT state here (x_pos, y_pos, previous_press_reported) could, in > principle, > be simplified away, since the type A protocol is stateless. > > [...] What is type A protocol? Could you elaborate. > > +static int bu21013_do_touch_report(struct bu21013_ts_data *data) > > + if (((number_of_active_x_sensors != 0) && > > + (number_of_active_y_sensors == 0)) || > > + ((number_of_active_x_sensors == 0) && > > + (number_of_active_y_sensors != 0))) > > + return 0; > > > So these should really be completely ignored, not reported as touch up? > Any > chance the above is the last event seen before a period of no activity? > This is the case one of x or y sensors is 0(when touch is not recognized properly), this should be ignored. Thanks & Regards, Naveen -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html