Commit cac8e9e5 authored by Dmitry Torokhov's avatar Dmitry Torokhov

Input: make connect and disconnect methods mandatory for serio

       drivers since that's where serio_{open|close} are called
       from to actually bind driver to a port.
Signed-off-by: default avatarDmitry Torokhov <dtor@mail.ru>
parent 7e1ec8c8
...@@ -68,7 +68,6 @@ static void serio_find_dev(struct serio *serio) ...@@ -68,7 +68,6 @@ static void serio_find_dev(struct serio *serio)
list_for_each_entry(dev, &serio_dev_list, node) { list_for_each_entry(dev, &serio_dev_list, node) {
if (serio->dev) if (serio->dev)
break; break;
if (dev->connect)
dev->connect(serio, dev); dev->connect(serio, dev);
} }
} }
...@@ -160,7 +159,7 @@ static void serio_handle_events(void) ...@@ -160,7 +159,7 @@ static void serio_handle_events(void)
/* reconnect failed - fall through to rescan */ /* reconnect failed - fall through to rescan */
case SERIO_RESCAN : case SERIO_RESCAN :
if (event->serio->dev && event->serio->dev->disconnect) if (event->serio->dev)
event->serio->dev->disconnect(event->serio); event->serio->dev->disconnect(event->serio);
serio_find_dev(event->serio); serio_find_dev(event->serio);
break; break;
...@@ -282,7 +281,7 @@ void __serio_unregister_port(struct serio *serio) ...@@ -282,7 +281,7 @@ void __serio_unregister_port(struct serio *serio)
{ {
serio_remove_pending_events(serio); serio_remove_pending_events(serio);
list_del_init(&serio->node); list_del_init(&serio->node);
if (serio->dev && serio->dev->disconnect) if (serio->dev)
serio->dev->disconnect(serio); serio->dev->disconnect(serio);
} }
...@@ -296,7 +295,7 @@ void serio_register_device(struct serio_dev *dev) ...@@ -296,7 +295,7 @@ void serio_register_device(struct serio_dev *dev)
down(&serio_sem); down(&serio_sem);
list_add_tail(&dev->node, &serio_dev_list); list_add_tail(&dev->node, &serio_dev_list);
list_for_each_entry(serio, &serio_list, node) list_for_each_entry(serio, &serio_list, node)
if (!serio->dev && dev->connect) if (!serio->dev)
dev->connect(serio, dev); dev->connect(serio, dev);
up(&serio_sem); up(&serio_sem);
} }
...@@ -309,7 +308,7 @@ void serio_unregister_device(struct serio_dev *dev) ...@@ -309,7 +308,7 @@ void serio_unregister_device(struct serio_dev *dev)
list_del_init(&dev->node); list_del_init(&dev->node);
list_for_each_entry(serio, &serio_list, node) { list_for_each_entry(serio, &serio_list, node) {
if (serio->dev == dev && dev->disconnect) if (serio->dev == dev)
dev->disconnect(serio); dev->disconnect(serio);
serio_find_dev(serio); serio_find_dev(serio);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment