linux-kernel - [PATCH] psmouse: ESD workaround fix for OLPC XO touchpad

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20090514011603.GA12494@plexity.net>
Date:	Thu, 14 May 2009 01:16:03 +0000
From:	Deepak Saxena <dsaxena@...xity.net>
To:	Alessandro Rubini <rubini@...vis.unipv.it>
Cc:	linux-kernel@...r.kernel.org
Subject: [PATCH] psmouse:  ESD workaround fix for OLPC XO touchpad


It appears that when the XO touchpad unit resets from ESD, it sends AA 
AA instead of AA 00, the psmouse-base code handles the case of AA 00 by
triggering a serio reconnect for the port, causing a full reprobe of
the device.
    
Testing with OFW shows that this is likely to solve the problem, so
the attached patch simply expands the existing test to also catch AA AA.
    
Author: Zephaniah E. Hull <warp@...allh.com>
Signed-off-by: Andres Salomon <dilinger@...ian.org>
Signed-off-by: Deepak Saxena <dsaxena@...top.org>

---
This issue was found during testing back in 2007, before I
was involved, and the fix has been carried in the OLPC tree 
since then. Not sure how this would affect other systems so 
wondering if we should wrap this specific check around 
'#ifdef OLPC'?

diff --git a/drivers/input/mouse/psmouse-base.c b/drivers/input/mouse/psmouse-base.c
index f8f86de..cf0f4fc 100644
--- a/drivers/input/mouse/psmouse-base.c
+++ b/drivers/input/mouse/psmouse-base.c
@@ -327,7 +327,7 @@ static irqreturn_t psmouse_interrupt(struct serio *serio,
 			goto out;
 		}
 
-		if (psmouse->packet[1] == PSMOUSE_RET_ID) {
+		if (psmouse->packet[1] == PSMOUSE_RET_ID || psmouse->packet[1] == PSMOUSE_RET_BAT) {
 			__psmouse_set_state(psmouse, PSMOUSE_IGNORE);
 			serio_reconnect(serio);
 			goto out;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/