On Thu, 2 Apr 2015 18:51:38 +0200
Martin Pieuchot <m...@openbsd.org> wrote:
> Even if that's true, nothing prevent us to commit this diff first, as
> long as it does not introduce regression and then work on the possible
> refactoring needed to support debounce packets :)
That is great to hear.
Because the way pms_sync_elantech_v3 now handles crc "mode"
pms_proc_elantech_v3 can't possibly receive a debounce packet. Therefore
I moved the debounce packet check in pms_proc_elantech_v3 into the part
that handles non-crc packets.
Below is the full diff for anyone that wants to try out these changes.
Index: sys/dev/pckbc/pms.c
===================================================================
RCS file: /cvs/src/sys/dev/pckbc/pms.c,v
retrieving revision 1.58
diff -u -p -r1.58 pms.c
--- sys/dev/pckbc/pms.c 26 Mar 2015 01:30:22 -0000 1.58
+++ sys/dev/pckbc/pms.c 2 Apr 2015 20:43:23 -0000
@@ -137,6 +137,7 @@ struct elantech_softc {
#define ELANTECH_F_HAS_ROCKER 0x02
#define ELANTECH_F_2FINGER_PACKET 0x04
#define ELANTECH_F_HW_V1_OLD 0x08
+#define ELANTECH_F_CRC_ENABLED 0x10
int fw_version;
int min_x, min_y;
@@ -1812,6 +1813,9 @@ elantech_get_hwinfo_v3(struct pms_softc
elantech->fw_version = fw_version;
elantech->flags |= ELANTECH_F_REPORTS_PRESSURE;
+ if ((fw_version & 0x4000) == 0x4000)
+ elantech->flags |= ELANTECH_F_CRC_ENABLED;
+
if (elantech_set_absolute_mode_v3(sc))
return (-1);
@@ -2164,14 +2168,23 @@ pms_sync_elantech_v2(struct pms_softc *s
int
pms_sync_elantech_v3(struct pms_softc *sc, int data)
{
+ struct elantech_softc *elantech = sc->elantech;
+
switch (sc->inputstate) {
case 0:
+ if (elantech->flags & ELANTECH_F_CRC_ENABLED)
+ break;
if ((data & 0x0c) != 0x04 && (data & 0x0c) != 0x0c)
return (-1);
break;
case 3:
- if ((data & 0xcf) != 0x02 && (data & 0xce) != 0x0c)
- return (-1);
+ if (elantech->flags & ELANTECH_F_CRC_ENABLED) {
+ if ((data & 0x09) != 0x08 && (data & 0x09) != 0x09)
+ return (-1);
+ } else {
+ if ((data & 0xcf) != 0x02 && (data & 0xce) != 0x0c)
+ return (-1);
+ }
break;
}
@@ -2256,11 +2269,6 @@ pms_proc_elantech_v3(struct pms_softc *s
struct elantech_softc *elantech = sc->elantech;
int x, y, w, z;
- /* The hardware sends this packet when in debounce state.
- * The packet should be ignored. */
- if (!memcmp(sc->packet, debounce_pkt, sizeof(debounce_pkt)))
- return;
-
x = ((sc->packet[1] & 0x0f) << 8 | sc->packet[2]);
y = ((sc->packet[4] & 0x0f) << 8 | sc->packet[5]);
z = 0;
@@ -2271,10 +2279,19 @@ pms_proc_elantech_v3(struct pms_softc *s
* and a tail packet. We report a single event and ignore
* the tail packet.
*/
- if ((sc->packet[0] & 0x0c) != 0x04 &&
- (sc->packet[3] & 0xcf) != 0x02) {
- /* not the head packet -- ignore */
- return;
+ if (elantech->flags & ELANTECH_F_CRC_ENABLED) {
+ if ((sc->packet[3] & 0x09) != 0x08)
+ return;
+ } else {
+ /* The hardware sends this packet when in debounce
state.
+ * The packet should be ignored. */
+ if (!memcmp(sc->packet, debounce_pkt,
sizeof(debounce_pkt)))
+ return;
+ if ((sc->packet[0] & 0x0c) != 0x04 &&
+ (sc->packet[3] & 0xcf) != 0x02) {
+ /* not the head packet -- ignore */
+ return;
+ }
}
}
