diff options
author | Amit Shah <amit.shah@redhat.com> | 2010-09-02 08:41:47 -0400 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2010-10-21 03:14:02 -0400 |
commit | 7a2853178dfba9553d58f356113f47fd582e9cc6 (patch) | |
tree | f140719154ca00c80a6bc27f28fda37efe85f2d3 /drivers | |
parent | f4028119714e452f9b49377ec55e0ed1e5d1dfa4 (diff) |
virtio: console: remove_port() should return void
When a port is removed, we have to assume the port is gone. So a
success/failure return value doesn't make sense.
Signed-off-by: Amit Shah <amit.shah@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/char/virtio_console.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c index 70f1c38fa14c..21b621343033 100644 --- a/drivers/char/virtio_console.c +++ b/drivers/char/virtio_console.c | |||
@@ -1131,7 +1131,7 @@ fail: | |||
1131 | } | 1131 | } |
1132 | 1132 | ||
1133 | /* Remove all port-specific data. */ | 1133 | /* Remove all port-specific data. */ |
1134 | static int remove_port(struct port *port) | 1134 | static void remove_port(struct port *port) |
1135 | { | 1135 | { |
1136 | struct port_buffer *buf; | 1136 | struct port_buffer *buf; |
1137 | 1137 | ||
@@ -1181,7 +1181,6 @@ static int remove_port(struct port *port) | |||
1181 | debugfs_remove(port->debugfs_file); | 1181 | debugfs_remove(port->debugfs_file); |
1182 | 1182 | ||
1183 | kfree(port); | 1183 | kfree(port); |
1184 | return 0; | ||
1185 | } | 1184 | } |
1186 | 1185 | ||
1187 | /* Any private messages that the Host and Guest want to share */ | 1186 | /* Any private messages that the Host and Guest want to share */ |