thunderbolt: Do not create DisplayPort tunnels on adapters of the same router
authorMika Westerberg <mika.westerberg@linux.intel.com>
Tue, 26 Mar 2024 08:58:15 +0000 (10:58 +0200)
committerMika Westerberg <mika.westerberg@linux.intel.com>
Tue, 2 Apr 2024 05:51:12 +0000 (08:51 +0300)
Probably due to a firmware bug Dell TB16 dock announces that one of its
DisplayPort adapters is actually DP IN. Now this is possible and used
with some external GPUs but not likely in this case as we are dealing
with a dock. Anyways the problem is that the driver tries to create a
DisplayPort tunnel between adapters of the same router which then shows
to user that there is no picture on the display (because there are no
available DP OUT adapters on the dock anymore).

Fix this by not creating DisplayPort tunnels between adapters that are
on the same router.

Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/10265
Fixes: 274baf695b08 ("thunderbolt: Add DP IN added last in the head of the list of DP resources")
Cc: Gil Fine <gil.fine@linux.intel.com>
Cc: stable@vger.kernel.org
Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
drivers/thunderbolt/tb.c

index c5ce7a694b27df774aeb0e9d6bf8fd7c1cd6a82e..360cb95f39aabcb7f3a03494b33b88572ece24a7 100644 (file)
@@ -1801,6 +1801,12 @@ static struct tb_port *tb_find_dp_out(struct tb *tb, struct tb_port *in)
                        continue;
                }
 
+               /* Needs to be on different routers */
+               if (in->sw == port->sw) {
+                       tb_port_dbg(port, "skipping DP OUT on same router\n");
+                       continue;
+               }
+
                tb_port_dbg(port, "DP OUT available\n");
 
                /*