usb: acpi: fix device link removal

The device link to the USB4 host interface has to be removed
manually since it's no longer auto removed.

Fixes: 623dae3e7084 ("usb: acpi: fix boot hang due to early incorrect 'tunneled' USB3 device links")
Cc: stable <stable@kernel.org>
Signed-off-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Link: https://lore.kernel.org/r/20250611111415.2707865-1-heikki.krogerus@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c
index 5c12dfdef5692ca5b3d55afb6ce616f7395fc01b..6bb6e92cb0a495855f2d4ec2885ded6a776dbbf0 100644 (file)
--- a/drivers/usb/core/hub.c
+++ b/drivers/usb/core/hub.c
@@ -2368,6 +2368,9 @@ void usb_disconnect(struct usb_device **pdev)
        usb_remove_ep_devs(&udev->ep0);
        usb_unlock_device(udev);
 
+       if (udev->usb4_link)
+               device_link_del(udev->usb4_link);
+
        /* Unregister the device.  The device driver is responsible
         * for de-configuring the device and invoking the remove-device
         * notifier chain (used by usbfs and possibly others).

diff --git a/drivers/usb/core/usb-acpi.c b/drivers/usb/core/usb-acpi.c
index ea1ce8beb0cbb27adea84f640c1625a75f5861be..489dbdc96f94ad4dabbf40f2bb195f99e621d514 100644 (file)
--- a/drivers/usb/core/usb-acpi.c
+++ b/drivers/usb/core/usb-acpi.c
@@ -157,7 +157,7 @@ EXPORT_SYMBOL_GPL(usb_acpi_set_power_state);
  */
 static int usb_acpi_add_usb4_devlink(struct usb_device *udev)
 {
-       const struct device_link *link;
+       struct device_link *link;
        struct usb_port *port_dev;
        struct usb_hub *hub;
 
@@ -188,6 +188,8 @@ static int usb_acpi_add_usb4_devlink(struct usb_device *udev)
        dev_dbg(&port_dev->dev, "Created device link from %s to %s\n",
                dev_name(&port_dev->child->dev), dev_name(nhi_fwnode->dev));
 
+       udev->usb4_link = link;
+
        return 0;
 }
 

diff --git a/include/linux/usb.h b/include/linux/usb.h
index 1b2545b4363bcf3ef97a53a004ebf456eb9d5d05..92c752f5446ff37ef09b9296f7711e1a622680ea 100644 (file)
--- a/include/linux/usb.h
+++ b/include/linux/usb.h
@@ -614,6 +614,7 @@ struct usb3_lpm_parameters {
  *     FIXME -- complete doc
  * @authenticated: Crypto authentication passed
  * @tunnel_mode: Connection native or tunneled over USB4
+ * @usb4_link: device link to the USB4 host interface
  * @lpm_capable: device supports LPM
  * @lpm_devinit_allow: Allow USB3 device initiated LPM, exit latency is in range
  * @usb2_hw_lpm_capable: device can perform USB2 hardware LPM
@@ -724,6 +725,7 @@ struct usb_device {
        unsigned reset_resume:1;
        unsigned port_is_suspended:1;
        enum usb_link_tunnel_mode tunnel_mode;
+       struct device_link *usb4_link;
 
        int slot_id;
        struct usb2_lpm_parameters l1_params;