input: edt-ft5x06: Only look at the number of points reported
authorDave Stevenson <dave.stevenson@raspberrypi.com>
Tue, 30 Nov 2021 17:28:50 +0000 (17:28 +0000)
committerPhil Elwell <8911409+pelwell@users.noreply.github.com>
Thu, 2 Dec 2021 11:52:15 +0000 (11:52 +0000)
Register 0x02 in the FT5x06 is TD_STATUS containing the number
of valid touch points being reported.

Iterate over that number of points rather than all that are
supported on the device.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
drivers/input/touchscreen/edt-ft5x06.c

index e33b5f0775defd30d1ec086491d6e5ea7bfe5cc7..9be7a6aba4097dc1ee8c155a7fe73edf0abf6f55 100644 (file)
@@ -201,6 +201,7 @@ static irqreturn_t edt_ft5x06_ts_isr(int irq, void *dev_id)
        unsigned int active_ids = 0, known_ids = tsdata->known_ids;
        long released_ids;
        int b = 0;
+       unsigned int num_points;
 
        switch (tsdata->version) {
        case EDT_M06:
@@ -248,9 +249,15 @@ static irqreturn_t edt_ft5x06_ts_isr(int irq, void *dev_id)
 
                if (!edt_ft5x06_ts_check_crc(tsdata, rdbuf, datalen))
                        goto out;
+               num_points = tsdata->max_support_points;
+       } else {
+               /* Register 2 is TD_STATUS, containing the number of touch
+                * points.
+                */
+               num_points = min(rdbuf[2] & 0xf, tsdata->max_support_points);
        }
 
-       for (i = 0; i < tsdata->max_support_points; i++) {
+       for (i = 0; i < num_points; i++) {
                u8 *buf = &rdbuf[i * tplen + offset];
 
                type = buf[0] >> 6;