thunderbolt: Drop duplicated get_switch_at_route()
tb_switch_find_by_route() does the same already so use it instead and remove duplicated get_switch_at_route(). Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com> Reviewed-by: Lukas Wunner <lukas@wunner.de>
This commit is contained in:
@@ -258,7 +258,7 @@ static void tb_handle_hotplug(struct work_struct *work)
|
||||
if (!tcm->hotplug_active)
|
||||
goto out; /* during init, suspend or shutdown */
|
||||
|
||||
sw = get_switch_at_route(tb->root_switch, ev->route);
|
||||
sw = tb_switch_find_by_route(tb, ev->route);
|
||||
if (!sw) {
|
||||
tb_warn(tb,
|
||||
"hotplug event from non existent switch %llx:%x (unplug: %d)\n",
|
||||
@@ -269,14 +269,14 @@ static void tb_handle_hotplug(struct work_struct *work)
|
||||
tb_warn(tb,
|
||||
"hotplug event from non existent port %llx:%x (unplug: %d)\n",
|
||||
ev->route, ev->port, ev->unplug);
|
||||
goto out;
|
||||
goto put_sw;
|
||||
}
|
||||
port = &sw->ports[ev->port];
|
||||
if (tb_is_upstream_port(port)) {
|
||||
tb_warn(tb,
|
||||
"hotplug event for upstream port %llx:%x (unplug: %d)\n",
|
||||
ev->route, ev->port, ev->unplug);
|
||||
goto out;
|
||||
goto put_sw;
|
||||
}
|
||||
if (ev->unplug) {
|
||||
if (port->remote) {
|
||||
@@ -306,6 +306,9 @@ static void tb_handle_hotplug(struct work_struct *work)
|
||||
tb_activate_pcie_devices(tb);
|
||||
}
|
||||
}
|
||||
|
||||
put_sw:
|
||||
tb_switch_put(sw);
|
||||
out:
|
||||
mutex_unlock(&tb->lock);
|
||||
kfree(ev);
|
||||
|
Reference in New Issue
Block a user