Commit 09d692e6 authored by Linus Torvalds's avatar Linus Torvalds

Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input

* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input:
  Input: appletouch - remove extra KERN_DEBUG use from dprintk
  Input: bu21013_ts - fix null dereference in error handling
  Input: ad7879 - prevent invalid finger data reports
parents ce9d8d9f d745b532
...@@ -630,7 +630,7 @@ static void atp_complete_geyser_3_4(struct urb *urb) ...@@ -630,7 +630,7 @@ static void atp_complete_geyser_3_4(struct urb *urb)
/* Just update the base values (i.e. touchpad in untouched state) */ /* Just update the base values (i.e. touchpad in untouched state) */
if (dev->data[dev->info->datalen - 1] & ATP_STATUS_BASE_UPDATE) { if (dev->data[dev->info->datalen - 1] & ATP_STATUS_BASE_UPDATE) {
dprintk(KERN_DEBUG "appletouch: updated base values\n"); dprintk("appletouch: updated base values\n");
memcpy(dev->xy_old, dev->xy_cur, sizeof(dev->xy_old)); memcpy(dev->xy_old, dev->xy_cur, sizeof(dev->xy_old));
goto exit; goto exit;
......
...@@ -129,6 +129,9 @@ struct ad7879 { ...@@ -129,6 +129,9 @@ struct ad7879 {
u16 cmd_crtl1; u16 cmd_crtl1;
u16 cmd_crtl2; u16 cmd_crtl2;
u16 cmd_crtl3; u16 cmd_crtl3;
int x;
int y;
int Rt;
}; };
static int ad7879_read(struct ad7879 *ts, u8 reg) static int ad7879_read(struct ad7879 *ts, u8 reg)
...@@ -175,13 +178,32 @@ static int ad7879_report(struct ad7879 *ts) ...@@ -175,13 +178,32 @@ static int ad7879_report(struct ad7879 *ts)
Rt /= z1; Rt /= z1;
Rt = (Rt + 2047) >> 12; Rt = (Rt + 2047) >> 12;
if (!timer_pending(&ts->timer)) /*
input_report_key(input_dev, BTN_TOUCH, 1); * Sample found inconsistent, pressure is beyond
* the maximum. Don't report it to user space.
*/
if (Rt > ts->pressure_max)
return -EINVAL;
input_report_abs(input_dev, ABS_X, x); /*
input_report_abs(input_dev, ABS_Y, y); * Note that we delay reporting events by one sample.
input_report_abs(input_dev, ABS_PRESSURE, Rt); * This is done to avoid reporting last sample of the
* touch sequence, which may be incomplete if finger
* leaves the surface before last reading is taken.
*/
if (timer_pending(&ts->timer)) {
/* Touch continues */
input_report_key(input_dev, BTN_TOUCH, 1);
input_report_abs(input_dev, ABS_X, ts->x);
input_report_abs(input_dev, ABS_Y, ts->y);
input_report_abs(input_dev, ABS_PRESSURE, ts->Rt);
input_sync(input_dev); input_sync(input_dev);
}
ts->x = x;
ts->y = y;
ts->Rt = Rt;
return 0; return 0;
} }
......
...@@ -514,7 +514,7 @@ static int __devinit bu21013_probe(struct i2c_client *client, ...@@ -514,7 +514,7 @@ static int __devinit bu21013_probe(struct i2c_client *client,
err_cs_disable: err_cs_disable:
pdata->cs_dis(pdata->cs_pin); pdata->cs_dis(pdata->cs_pin);
err_free_mem: err_free_mem:
input_free_device(bu21013_data->in_dev); input_free_device(in_dev);
kfree(bu21013_data); kfree(bu21013_data);
return error; return error;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment