On Fri, 12 Oct 2012, Peter Chen wrote: > @@ -4187,9 +4190,17 @@ static void hub_port_connect_change(struct usb_hub *hub, int port1, > } > } > > + if (hcd->phy && !hdev->parent && > + !(portstatus & USB_PORT_STAT_CONNECTION)) > + usb_phy_notify_disconnect(hcd->phy, udev->speed); > + These lines must be removed. They aren't needed, because of the new code below. > /* Disconnect any existing devices under this port */ > - if (udev) > + if (udev) { > + if (hcd->phy && !hdev->parent && > + !(portstatus & USB_PORT_STAT_CONNECTION)) > + usb_phy_notify_disconnect(hcd->phy, udev->speed); > usb_disconnect(&hub->ports[port1 - 1]->child); > + } Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html