Commit bbcf40b3 authored by Mika Westerberg's avatar Mika Westerberg

thunderbolt: Do not tunnel USB3 if link is not USB4

USB3 tunneling is possible only over USB4 link so don't create USB3
tunnels if that's not the case.
Signed-off-by: default avatarMika Westerberg <mika.westerberg@linux.intel.com>
parent e876f34a
...@@ -235,6 +235,9 @@ static int tb_tunnel_usb3(struct tb *tb, struct tb_switch *sw) ...@@ -235,6 +235,9 @@ static int tb_tunnel_usb3(struct tb *tb, struct tb_switch *sw)
if (!up) if (!up)
return 0; return 0;
if (!sw->link_usb4)
return 0;
/* /*
* Look up available down port. Since we are chaining it should * Look up available down port. Since we are chaining it should
* be found right above this switch. * be found right above this switch.
......
...@@ -97,6 +97,7 @@ struct tb_switch_tmu { ...@@ -97,6 +97,7 @@ struct tb_switch_tmu {
* @device_name: Name of the device (or %NULL if not known) * @device_name: Name of the device (or %NULL if not known)
* @link_speed: Speed of the link in Gb/s * @link_speed: Speed of the link in Gb/s
* @link_width: Width of the link (1 or 2) * @link_width: Width of the link (1 or 2)
* @link_usb4: Upstream link is USB4
* @generation: Switch Thunderbolt generation * @generation: Switch Thunderbolt generation
* @cap_plug_events: Offset to the plug events capability (%0 if not found) * @cap_plug_events: Offset to the plug events capability (%0 if not found)
* @cap_lc: Offset to the link controller capability (%0 if not found) * @cap_lc: Offset to the link controller capability (%0 if not found)
...@@ -136,6 +137,7 @@ struct tb_switch { ...@@ -136,6 +137,7 @@ struct tb_switch {
const char *device_name; const char *device_name;
unsigned int link_speed; unsigned int link_speed;
unsigned int link_width; unsigned int link_width;
bool link_usb4;
unsigned int generation; unsigned int generation;
int cap_plug_events; int cap_plug_events;
int cap_lc; int cap_lc;
......
...@@ -290,6 +290,7 @@ struct tb_regs_port_header { ...@@ -290,6 +290,7 @@ struct tb_regs_port_header {
/* USB4 port registers */ /* USB4 port registers */
#define PORT_CS_18 0x12 #define PORT_CS_18 0x12
#define PORT_CS_18_BE BIT(8) #define PORT_CS_18_BE BIT(8)
#define PORT_CS_18_TCM BIT(9)
#define PORT_CS_19 0x13 #define PORT_CS_19 0x13
#define PORT_CS_19_PC BIT(3) #define PORT_CS_19_PC BIT(3)
......
...@@ -192,6 +192,20 @@ static int usb4_switch_op(struct tb_switch *sw, u16 opcode, u8 *status) ...@@ -192,6 +192,20 @@ static int usb4_switch_op(struct tb_switch *sw, u16 opcode, u8 *status)
return 0; return 0;
} }
static bool link_is_usb4(struct tb_port *port)
{
u32 val;
if (!port->cap_usb4)
return false;
if (tb_port_read(port, &val, TB_CFG_PORT,
port->cap_usb4 + PORT_CS_18, 1))
return false;
return !(val & PORT_CS_18_TCM);
}
/** /**
* usb4_switch_setup() - Additional setup for USB4 device * usb4_switch_setup() - Additional setup for USB4 device
* @sw: USB4 router to setup * @sw: USB4 router to setup
...@@ -205,6 +219,7 @@ static int usb4_switch_op(struct tb_switch *sw, u16 opcode, u8 *status) ...@@ -205,6 +219,7 @@ static int usb4_switch_op(struct tb_switch *sw, u16 opcode, u8 *status)
*/ */
int usb4_switch_setup(struct tb_switch *sw) int usb4_switch_setup(struct tb_switch *sw)
{ {
struct tb_port *downstream_port;
struct tb_switch *parent; struct tb_switch *parent;
bool tbt3, xhci; bool tbt3, xhci;
u32 val = 0; u32 val = 0;
...@@ -217,6 +232,11 @@ int usb4_switch_setup(struct tb_switch *sw) ...@@ -217,6 +232,11 @@ int usb4_switch_setup(struct tb_switch *sw)
if (ret) if (ret)
return ret; return ret;
parent = tb_switch_parent(sw);
downstream_port = tb_port_at(tb_route(sw), parent);
sw->link_usb4 = link_is_usb4(downstream_port);
tb_sw_dbg(sw, "link: %s\n", sw->link_usb4 ? "USB4" : "TBT3");
xhci = val & ROUTER_CS_6_HCI; xhci = val & ROUTER_CS_6_HCI;
tbt3 = !(val & ROUTER_CS_6_TNS); tbt3 = !(val & ROUTER_CS_6_TNS);
...@@ -227,9 +247,7 @@ int usb4_switch_setup(struct tb_switch *sw) ...@@ -227,9 +247,7 @@ int usb4_switch_setup(struct tb_switch *sw)
if (ret) if (ret)
return ret; return ret;
parent = tb_switch_parent(sw); if (sw->link_usb4 && tb_switch_find_port(parent, TB_TYPE_USB3_DOWN)) {
if (tb_switch_find_port(parent, TB_TYPE_USB3_DOWN)) {
val |= ROUTER_CS_5_UTO; val |= ROUTER_CS_5_UTO;
xhci = false; xhci = false;
} }
......
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