Add module parameters to control OLPC touchpad delays

The HPGK touchpad that is found on the XO driver has
historically exhibitted eratic behaviour in various
environments (very dry, very humid, etc) that can be
worked around via some delays. This patch turns those
delays into module parameters to make testing simpler.

Signed-off-by: Paul Fox <pgf@laptop.org>
Signed-off-by: Deepak Saxena <dsaxena@laptop.org>

1 files changed, 24 insertions, 4 deletions

diff --git a/drivers/input/mouse/olpc.c b/drivers/input/mouse/olpc.c
index 4d7adbfb3c01..f127dc081d39 100644
--- a/drivers/input/mouse/olpc.c
+++ b/drivers/input/mouse/olpc.c
@@ -47,6 +47,26 @@ module_param(recalib_delta, int, 0644);
 MODULE_PARM_DESC(recalib_delta,
 	"packets containing a delta this large will cause a recalibration.");
 
+static int jumpy_delay = 1000;
+module_param(jumpy_delay, int, 0644);
+MODULE_PARM_DESC(jumpy_delay,
+	"delay (ms) before recal after jumpiness detected");
+
+static int spew_delay = 1000;
+module_param(spew_delay, int, 0644);
+MODULE_PARM_DESC(spew_delay,
+	"delay (ms) before recal after packet spew detected");
+
+static int recal_guard_time = 2000;
+module_param(recal_guard_time, int, 0644);
+MODULE_PARM_DESC(recal_guard_time,
+	"interval (ms) during which recal will be restarted if packet received");
+
+static int post_interrupt_delay = 1000;
+module_param(post_interrupt_delay, int, 0644);
+MODULE_PARM_DESC(post_interrupt_delay,
+	"delay (ms) before recal after recal interrupt detected");
+
 static int autorecal = 1;
 module_param(autorecal, int, 0644);
 MODULE_PARM_DESC(autorecal, "enable recalibration in the driver?");
@@ -70,7 +90,7 @@ static void hgpk_jumpy_hack(struct psmouse *psmouse, int x, int y)
 		 * way I likes it!
 		 */
 		queue_delayed_work(kpsmoused_wq, &priv->recalib_wq,
-				msecs_to_jiffies(1000));
+				msecs_to_jiffies(jumpy_delay));
 	}
 }
 
@@ -114,7 +134,7 @@ static void hgpk_spewing_hack(struct psmouse *psmouse, int l, int r, int x,
 			 * get it back, but gave up after dickety six miles
 			 */
 			queue_delayed_work(kpsmoused_wq, &priv->recalib_wq,
-					msecs_to_jiffies(1000));
+					msecs_to_jiffies(spew_delay));
 		}
 		/* reset every 100 packets */
 		count = 0;
@@ -193,7 +213,7 @@ static psmouse_ret_t hgpk_process_byte(struct psmouse *psmouse)
 			 */
 			hgpk_dbg(psmouse, "packet inside calibration window, queueing another recalibration\n");
 			queue_delayed_work(kpsmoused_wq, &priv->recalib_wq,
-					msecs_to_jiffies(1000));
+					msecs_to_jiffies(post_interrupt_delay));
 		}
 		priv->recalib_window = 0;
 	}
@@ -249,7 +269,7 @@ static int hgpk_force_recalibrate(struct psmouse *psmouse)
 	 * If someone's finger *was* on the touchpad, it's probably
 	 * miscalibrated.  So, we should schedule another recalibration
 	 */
-	priv->recalib_window = jiffies +  msecs_to_jiffies(2000);
+	priv->recalib_window = jiffies +  msecs_to_jiffies(recal_guard_time);
 
 	return 0;
 }