usb: acpi: fix boot hang due to early incorrect 'tunneled' USB3 device links

Fix a boot hang issue triggered when a USB3 device is incorrectly assumed
to be tunneled over USB4, thus attempting to create a device link between
the USB3 "consumer" device and the USB4 "supplier" Host Interface before
the USB4 side is properly bound to a driver.

This could happen if xhci isn't capable of detecting tunneled devices,
but ACPI tables contain all info needed to assume device is tunneled.
i.e. udev->tunnel_mode == USB_LINK_UNKNOWN.

It turns out that even for actual tunneled USB3 devices it can't be
assumed that the thunderbolt driver providing the tunnel is loaded
before the tunneled USB3 device is created.
The tunnel can be created by BIOS and remain in use by thunderbolt/USB4
host driver once it loads.

Solve this by making the device link "stateless", which doesn't create
a driver presence order dependency between the supplier and consumer
drivers.
It still guarantees correct suspend/resume and shutdown ordering.

cc: Mario Limonciello <mario.limonciello@amd.com>
Fixes: f1bfb4a6fed6 ("usb: acpi: add device link between tunneled USB3 device and USB4 Host Interface")
Tested-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Tested-by: Mario Limonciello <mario.limonciello@amd.com>
Link: https://lore.kernel.org/r/20241024131355.3836538-1-mathias.nyman@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

authored by Mathias Nyman and committed by Greg Kroah-Hartman 623dae3e f3b31132

+2 -2
+2 -2
drivers/usb/core/usb-acpi.c
··· 170 170 struct fwnode_handle *nhi_fwnode __free(fwnode_handle) = 171 171 fwnode_find_reference(dev_fwnode(&port_dev->dev), "usb4-host-interface", 0); 172 172 173 - if (IS_ERR(nhi_fwnode)) 173 + if (IS_ERR(nhi_fwnode) || !nhi_fwnode->dev) 174 174 return 0; 175 175 176 176 link = device_link_add(&port_dev->child->dev, nhi_fwnode->dev, 177 - DL_FLAG_AUTOREMOVE_CONSUMER | 177 + DL_FLAG_STATELESS | 178 178 DL_FLAG_RPM_ACTIVE | 179 179 DL_FLAG_PM_RUNTIME); 180 180 if (!link) {