Fedora kernel-2.6.17-1.2142_FC4 patched with stable patch-2.6.17.4-vs2.0.2-rc26.diff
[linux-2.6.git] / drivers / input / touchscreen / gunze.c
index e5e38dd..466da19 100644 (file)
@@ -48,14 +48,12 @@ MODULE_LICENSE("GPL");
 
 #define        GUNZE_MAX_LENGTH        10
 
-static char *gunze_name = "Gunze AHL-51S TouchScreen";
-
 /*
  * Per-touchscreen data.
  */
 
 struct gunze {
-       struct input_dev dev;
+       struct input_dev *dev;
        struct serio *serio;
        int idx;
        unsigned char data[GUNZE_MAX_LENGTH];
@@ -64,18 +62,17 @@ struct gunze {
 
 static void gunze_process_packet(struct gunze* gunze, struct pt_regs *regs)
 {
-       struct input_dev *dev = &gunze->dev;
+       struct input_dev *dev = gunze->dev;
 
        if (gunze->idx != GUNZE_MAX_LENGTH || gunze->data[5] != ',' ||
                (gunze->data[0] != 'T' && gunze->data[0] != 'R')) {
-               gunze->data[10] = 0;
-               printk(KERN_WARNING "gunze.c: bad packet: >%s<\n", gunze->data);
+               printk(KERN_WARNING "gunze.c: bad packet: >%.*s<\n", GUNZE_MAX_LENGTH, gunze->data);
                return;
        }
 
        input_regs(dev, regs);
-       input_report_abs(dev, ABS_X, simple_strtoul(gunze->data + 1, NULL, 10) * 4);
-       input_report_abs(dev, ABS_Y, 3072 - simple_strtoul(gunze->data + 6, NULL, 10) * 3);
+       input_report_abs(dev, ABS_X, simple_strtoul(gunze->data + 1, NULL, 10));
+       input_report_abs(dev, ABS_Y, 1024 - simple_strtoul(gunze->data + 6, NULL, 10));
        input_report_key(dev, BTN_TOUCH, gunze->data[0] == 'T');
        input_sync(dev);
 }
@@ -83,7 +80,7 @@ static void gunze_process_packet(struct gunze* gunze, struct pt_regs *regs)
 static irqreturn_t gunze_interrupt(struct serio *serio,
                unsigned char data, unsigned int flags, struct pt_regs *regs)
 {
-       struct gunze* gunze = serio->private;
+       struct gunze* gunze = serio_get_drvdata(serio);
 
        if (data == '\r') {
                gunze_process_packet(gunze, regs);
@@ -101,68 +98,88 @@ static irqreturn_t gunze_interrupt(struct serio *serio,
 
 static void gunze_disconnect(struct serio *serio)
 {
-       struct gunze* gunze = serio->private;
-       input_unregister_device(&gunze->dev);
+       struct gunze *gunze = serio_get_drvdata(serio);
+
+       input_get_device(gunze->dev);
+       input_unregister_device(gunze->dev);
        serio_close(serio);
+       serio_set_drvdata(serio, NULL);
+       input_put_device(gunze->dev);
        kfree(gunze);
 }
 
 /*
  * gunze_connect() is the routine that is called when someone adds a
- * new serio device. It looks whether it was registered as a Gunze touchscreen
- * and if yes, registers it as an input device.
+ * new serio device that supports Gunze protocol and registers it as
+ * an input device.
  */
 
-static void gunze_connect(struct serio *serio, struct serio_driver *drv)
+static int gunze_connect(struct serio *serio, struct serio_driver *drv)
 {
        struct gunze *gunze;
-
-       if (serio->type != (SERIO_RS232 | SERIO_GUNZE))
-               return;
-
-       if (!(gunze = kmalloc(sizeof(struct gunze), GFP_KERNEL)))
-               return;
-
-       memset(gunze, 0, sizeof(struct gunze));
-
-       init_input_dev(&gunze->dev);
-       gunze->dev.evbit[0] = BIT(EV_KEY) | BIT(EV_ABS);
-       gunze->dev.keybit[LONG(BTN_TOUCH)] = BIT(BTN_TOUCH);
-       input_set_abs_params(&gunze->dev, ABS_X, 96, 4000, 0, 0);
-       input_set_abs_params(&gunze->dev, ABS_Y, 72, 3000, 0, 0);
+       struct input_dev *input_dev;
+       int err;
+
+       gunze = kzalloc(sizeof(struct gunze), GFP_KERNEL);
+       input_dev = input_allocate_device();
+       if (!gunze || !input_dev) {
+               err = -ENOMEM;
+               goto fail;
+       }
 
        gunze->serio = serio;
-       serio->private = gunze;
-
+       gunze->dev = input_dev;
        sprintf(gunze->phys, "%s/input0", serio->phys);
 
-       gunze->dev.private = gunze;
-       gunze->dev.name = gunze_name;
-       gunze->dev.phys = gunze->phys;
-       gunze->dev.id.bustype = BUS_RS232;
-       gunze->dev.id.vendor = SERIO_GUNZE;
-       gunze->dev.id.product = 0x0051;
-       gunze->dev.id.version = 0x0100;
-
-       if (serio_open(serio, drv)) {
-               kfree(gunze);
-               return;
-       }
-
-       input_register_device(&gunze->dev);
+       input_dev->private = gunze;
+       input_dev->name = "Gunze AHL-51S TouchScreen";
+       input_dev->phys = gunze->phys;
+       input_dev->id.bustype = BUS_RS232;
+       input_dev->id.vendor = SERIO_GUNZE;
+       input_dev->id.product = 0x0051;
+       input_dev->id.version = 0x0100;
+       input_dev->evbit[0] = BIT(EV_KEY) | BIT(EV_ABS);
+       input_dev->keybit[LONG(BTN_TOUCH)] = BIT(BTN_TOUCH);
+       input_set_abs_params(input_dev, ABS_X, 24, 1000, 0, 0);
+       input_set_abs_params(input_dev, ABS_Y, 24, 1000, 0, 0);
+
+       serio_set_drvdata(serio, gunze);
+
+       err = serio_open(serio, drv);
+       if (err)
+               goto fail;
+
+       input_register_device(gunze->dev);
+       return 0;
 
-       printk(KERN_INFO "input: %s on %s\n", gunze_name, serio->phys);
+ fail: serio_set_drvdata(serio, NULL);
+       input_free_device(input_dev);
+       kfree(gunze);
+       return err;
 }
 
 /*
- * The serio device structure.
+ * The serio driver structure.
  */
 
+static struct serio_device_id gunze_serio_ids[] = {
+       {
+               .type   = SERIO_RS232,
+               .proto  = SERIO_GUNZE,
+               .id     = SERIO_ANY,
+               .extra  = SERIO_ANY,
+       },
+       { 0 }
+};
+
+MODULE_DEVICE_TABLE(serio, gunze_serio_ids);
+
 static struct serio_driver gunze_drv = {
        .driver         = {
                .name   = "gunze",
        },
        .description    = DRIVER_DESC,
+       .id_table       = gunze_serio_ids,
        .interrupt      = gunze_interrupt,
        .connect        = gunze_connect,
        .disconnect     = gunze_disconnect,
@@ -172,13 +189,13 @@ static struct serio_driver gunze_drv = {
  * The functions for inserting/removing us as a module.
  */
 
-int __init gunze_init(void)
+static int __init gunze_init(void)
 {
        serio_register_driver(&gunze_drv);
        return 0;
 }
 
-void __exit gunze_exit(void)
+static void __exit gunze_exit(void)
 {
        serio_unregister_driver(&gunze_drv);
 }