We don't need to special-case the device, this device may be used with a pen
that includes an eraser. Removing the special case means we expect all
currently known devices to have at lest one stylus, the code from
4a7a7f16564869b3af6701aa943e5c88927f451b tests those styli for additional
eraser.

Quote Ping Cheng, 17 May 2012,
linuxwacom-devel, Thread: Add support for the DTI-520
"DTI-520 is a protocol 4 (V4) device. It does not support serial number
and tool IDs. [...] This is the same as Graphires. Even if the pen included
in the box does not have eraser, user may use a pen with eraser."

Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
---
Changes to v1:
- expect 1 stylus only, let other code test the erasers

 test/tablet-validity.c |   14 +-------------
 1 file changed, 1 insertion(+), 13 deletions(-)

diff --git a/test/tablet-validity.c b/test/tablet-validity.c
index 8f33cb5..38fd0ef 100644
--- a/test/tablet-validity.c
+++ b/test/tablet-validity.c
@@ -59,7 +59,6 @@ static void verify_tablet(WacomDeviceDatabase *db, 
WacomDevice *device)
 {
        const char *name;
        const int *styli;
-       unsigned int product;
        int nstyli, i;
 
        name = libwacom_get_name(device);
@@ -85,18 +84,7 @@ static void verify_tablet(WacomDeviceDatabase *db, 
WacomDevice *device)
 
        styli = libwacom_get_supported_styli(device, &nstyli);
        assert(styli != NULL);
-       assert(nstyli > 0);
-
-       product = libwacom_get_vendor_id(device) << 16 | 
libwacom_get_product_id(device);
-       switch(product) {
-               /* Devices known not have erasers. */
-               case 0x056a003a: /* DTI520*/
-                       assert(nstyli == 1);
-                       break;
-               default:
-                       assert(nstyli > 1);
-                       break;
-       }
+       assert(nstyli >= 1);
 
        switch(libwacom_get_class(device)) {
                case WCLASS_BAMBOO:
-- 
1.7.10.1


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to