Input: ct82c710 - remove a redundant serio_register_port()
authorAxel Lin <axel.lin@gmail.com>
Mon, 22 Nov 2010 06:27:13 +0000 (22:27 -0800)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Mon, 22 Nov 2010 06:32:17 +0000 (22:32 -0800)
We already call serio_register_port() in ct82c710_probe(),
thus remove a redundant serio_register_port() in ct82c710_init().

Looks like this bug is introduced by
916d83cfe5da1cda454d8b0ae233f06b58bd7f91
"Input: ct82c710 - convert to the new platform device interface"

[dtor@mail.ru: also move printk to where we register port]
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
drivers/input/serio/ct82c710.c

index 4a3084695c00b554af42be46075047c9c5d9538c..448c7724beb90eda75edc0ef2d83c04e1fd9357a 100644 (file)
@@ -191,6 +191,9 @@ static int __devinit ct82c710_probe(struct platform_device *dev)
 
        serio_register_port(ct82c710_port);
 
+       printk(KERN_INFO "serio: C&T 82c710 mouse port at %#llx irq %d\n",
+               (unsigned long long)CT82C710_DATA, CT82C710_IRQ);
+
        return 0;
 }
 
@@ -237,11 +240,6 @@ static int __init ct82c710_init(void)
        if (error)
                goto err_free_device;
 
-       serio_register_port(ct82c710_port);
-
-       printk(KERN_INFO "serio: C&T 82c710 mouse port at %#llx irq %d\n",
-               (unsigned long long)CT82C710_DATA, CT82C710_IRQ);
-
        return 0;
 
  err_free_device: