Re: [PATCH v2] [input-hid] Add hid-egalax driver to the unified hid-multitouch framework.

quoted

quoted

 struct mt_slot {
      __s32 x, y, p, w, h;

@@ -63,6 +64,9 @@ struct mt_class {

      __s32 sn_move;  /* Signal/noise ratio for move events */
      __s32 sn_pressure;      /* Signal/noise ratio for pressure events */
      __u8 maxcontacts;
+     __u8 override_logical_limits;  /* correct the reported X/Y range */
+     __u32 logical_min[2];
+     __u32 logical_max[2];

Please think about byte alignment here, keeping elements of the same
size together. Also, the override needs a specific name, given that it
applies to the whole class, not just the x and y positions. Perhaps
the override should be triggered with a quirk instead?

I'm not in favor of a quirk in this particular case: the information
is already here: max > 0.

Only if min == 0, which is far from always the case. And setting a
special value for a special hardware does what a quirk does, so maybe
it is a quirk after all.

Thanks,
Henrik
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html