HI Gabriele, On Sun, Mar 22, 2015 at 03:43:55PM +0100, Gabriele Mazzotta wrote: > The driver was not reporting widths for image sensors and cr48 sensors > despite it was calculating them. > > Link: https://bugzilla.kernel.org/show_bug.cgi?id=77161 > Signed-off-by: Gabriele Mazzotta <gabriele.mzt@xxxxxxxxx> > --- > drivers/input/mouse/synaptics.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c > index ff47084..4e86ba6 100644 > --- a/drivers/input/mouse/synaptics.c > +++ b/drivers/input/mouse/synaptics.c > @@ -927,6 +927,7 @@ static void synaptics_report_mt_data(struct psmouse *psmouse, > input_report_abs(dev, ABS_MT_POSITION_X, pos[i].x); > input_report_abs(dev, ABS_MT_POSITION_Y, pos[i].y); > input_report_abs(dev, ABS_MT_PRESSURE, hw[i]->z); > + input_report_abs(dev, ABS_MT_TOUCH_MAJOR, hw[i]->w); As I mentioned in the other email ABS_MT_TOUCH_MAJOR should use surface units for reporting. > } > > input_mt_drop_unused(dev); > @@ -1192,8 +1193,9 @@ static void set_input_params(struct psmouse *psmouse, > if (SYN_CAP_IMAGE_SENSOR(priv->ext_cap_0c)) { > set_abs_position_params(dev, priv, ABS_MT_POSITION_X, > ABS_MT_POSITION_Y); > - /* Image sensors can report per-contact pressure */ > + /* Image sensors can report per-contact pressure and width */ > input_set_abs_params(dev, ABS_MT_PRESSURE, 0, 255, 0, 0); > + input_set_abs_params(dev, ABS_MT_TOUCH_MAJOR, 4, 15, 0, 0); > input_mt_init_slots(dev, 2, INPUT_MT_POINTER | INPUT_MT_TRACK); > /* Image sensors can signal 4 and 5 finger clicks */ > __set_bit(BTN_TOOL_QUADTAP, dev->keybit); > @@ -1202,6 +1204,7 @@ static void set_input_params(struct psmouse *psmouse, > set_abs_position_params(dev, priv, ABS_MT_POSITION_X, > ABS_MT_POSITION_Y); > input_set_abs_params(dev, ABS_MT_PRESSURE, 0, 255, 0, 0); > + input_set_abs_params(dev, ABS_MT_TOUCH_MAJOR, 4, 15, 0, 0); > /* > * Profile sensor in CR-48 tracks contacts reasonably well, > * other non-image sensors with AGM use semi-mt. > -- > 2.1.4 > Thanks. -- Dmitry -- 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
