]> err.no Git - linux-2.6/commitdiff
Input: tsdev - schedule removal
authorRichard Purdie <rpurdie@rpsys.net>
Sat, 10 Feb 2007 06:29:11 +0000 (01:29 -0500)
committerDmitry Torokhov <dtor@insightbb.com>
Sat, 10 Feb 2007 06:29:11 +0000 (01:29 -0500)
Compaq touchscreen emulation (drivers/input/tsdev.c) is old,
was obsolete when it was written by the authors own admission
and much better userspace solutions like tslib now exist.
The name is also confusing.

Signed-off-by: Richard Purdie <rpurdie@rpsys.net>
Acked-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Documentation/feature-removal-schedule.txt
drivers/input/tsdev.c

index fa844fd7bded4004f1c050e8cc5bc719210c1452..5d140dc0ea2b0a63289913591048e91698bfaa52 100644 (file)
@@ -312,3 +312,18 @@ Why:    In kernel tree version of driver is unmaintained. Sk98lin driver
        replaced by the skge driver. 
 Who:    Stephen Hemminger <shemminger@osdl.org>
 
+---------------------------
+
+What:  Compaq touchscreen device emulation
+When:  Oct 2007
+Files: drivers/input/tsdev.c
+Why:   The code says it was obsolete when it was written in 2001.
+       tslib is a userspace library which does anything tsdev can do and
+       much more besides in userspace where this code belongs. There is no
+       longer any need for tsdev and applications should have converted to
+       use tslib by now.
+       The name "tsdev" is also extremely confusing and lots of people have
+       it loaded when they don't need/use it.
+Who:   Richard Purdie <rpurdie@rpsys.net>
+
+---------------------------
index a730c461227f78608d8c461195f33b80f11210fb..0300dca8591d9cdb076f0413eee62a6b56823fc8 100644 (file)
@@ -151,6 +151,10 @@ static int tsdev_open(struct inode *inode, struct file *file)
        int i = iminor(inode) - TSDEV_MINOR_BASE;
        struct tsdev_list *list;
 
+       printk(KERN_WARNING "tsdev (compaq touchscreen emulation) is scheduled "
+               "for removal.\nSee Documentation/feature-removal-schedule.txt "
+               "for details.\n");
+
        if (i >= TSDEV_MINORS || !tsdev_table[i & TSDEV_MINOR_MASK])
                return -ENODEV;