aboutsummaryrefslogtreecommitdiffstats
path: root/target/linux/s3c24xx/patches-2.6.24/1210-fix-touchscreen-meddling-divde.patch.patch
diff options
context:
space:
mode:
Diffstat (limited to 'target/linux/s3c24xx/patches-2.6.24/1210-fix-touchscreen-meddling-divde.patch.patch')
-rw-r--r--target/linux/s3c24xx/patches-2.6.24/1210-fix-touchscreen-meddling-divde.patch.patch52
1 files changed, 52 insertions, 0 deletions
diff --git a/target/linux/s3c24xx/patches-2.6.24/1210-fix-touchscreen-meddling-divde.patch.patch b/target/linux/s3c24xx/patches-2.6.24/1210-fix-touchscreen-meddling-divde.patch.patch
new file mode 100644
index 0000000000..2348784fb6
--- /dev/null
+++ b/target/linux/s3c24xx/patches-2.6.24/1210-fix-touchscreen-meddling-divde.patch.patch
@@ -0,0 +1,52 @@
+From e0dc30dfda233d4ebe6f88cd1162e5952fc69027 Mon Sep 17 00:00:00 2001
+From: Andy Green <andy@openmoko.com>
+Date: Wed, 2 Jul 2008 22:43:38 +0100
+Subject: [PATCH] fix-touchscreen-meddling-divde.patch
+ Reported-by: Holger Freyther <zecke@openmoko.org>
+
+length can be zero... blowing a divide by zero exception...
+which somehow I don't get (?) Anyway the code is wrong and
+this should fix it.
+
+Signed-off-by: Andy Green <andy@openmoko.com>
+---
+ drivers/input/touchscreen/s3c2410_ts.c | 12 +++++++++---
+ 1 files changed, 9 insertions(+), 3 deletions(-)
+
+diff --git a/drivers/input/touchscreen/s3c2410_ts.c b/drivers/input/touchscreen/s3c2410_ts.c
+index 83e7aff..9fb95c1 100644
+--- a/drivers/input/touchscreen/s3c2410_ts.c
++++ b/drivers/input/touchscreen/s3c2410_ts.c
+@@ -226,12 +226,18 @@ static irqreturn_t stylus_action(int irq, void *dev_id)
+ unsigned long x;
+ unsigned long y;
+ int length = (ts.head_raw_fifo - ts.tail_raw_fifo) & (ts.extent - 1);
+- int scaled_avg_x = ts.raw_running_avg.x / length;
+- int scaled_avg_y = ts.raw_running_avg.y / length;
++ int scaled_avg_x;
++ int scaled_avg_y;
+
+ x = readl(base_addr + S3C2410_ADCDAT0) & S3C2410_ADCDAT0_XPDATA_MASK;
+ y = readl(base_addr + S3C2410_ADCDAT1) & S3C2410_ADCDAT1_YPDATA_MASK;
+
++ if (!length)
++ goto store_sample;
++
++ scaled_avg_x = ts.raw_running_avg.x / length;
++ scaled_avg_y = ts.raw_running_avg.y / length;
++
+ /* we appear to accept every sample into both the running average FIFO
+ * and the summing average. BUT, if the last sample crossed a
+ * machine-set threshold, each time we do a beauty contest
+@@ -280,7 +286,7 @@ static irqreturn_t stylus_action(int irq, void *dev_id)
+ else
+ ts.flag_previous_exceeded_threshold = 1;
+
+- /* accepted */
++store_sample:
+ ts.xp += x;
+ ts.yp += y;
+ ts.count++;
+--
+1.5.6.5
+