Commit 1993cbf0 authored by Linus Torvalds's avatar Linus Torvalds

Merge tag 'usb-4.17-rc3' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb

Pull USB fixes from Greg KH:
 "Here are a number of USB driver fixes for reported problems for
  4.17-rc3.

  The "largest" here is a number of phy core changes for reported
  problems with the -rc1 release. There's also the usual musb and xhci
  fixes, as well as new device id updates. There are also some usbip
  fixes for reported problems as more people start to use that code with
  containers.

  All of these have been in linux-next with no reported issues, except
  the last few new device ids, which are "obviously correct" :)"

* tag 'usb-4.17-rc3' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb: (26 commits)
  USB: musb: dsps: drop duplicate phy initialisation
  USB: musb: host: prevent core phy initialisation
  usb: core: phy: add the SPDX-License-Identifier and include guard
  xhci: Fix Kernel oops in xhci dbgtty
  usb: select USB_COMMON for usb role switch config
  usb: core: phy: add missing forward declaration for "struct device"
  usb: core: phy: make it a no-op if CONFIG_GENERIC_PHY is disabled
  usb: core: use phy_exit during suspend if wake up is not supported
  usb: core: split usb_phy_roothub_{init,alloc}
  usb: core: phy: fix return value of usb_phy_roothub_exit()
  usb: typec: ucsi: Increase command completion timeout value
  Revert "xhci: plat: Register shutdown for xhci_plat"
  usb: core: Add quirk for HP v222w 16GB Mini
  Documentation: typec.rst: Use literal-block element with ascii art
  usb: typec: ucsi: fix tracepoint related build error
  usbip: usbip_event: fix to not print kernel pointer address
  usbip: usbip_host: fix to hold parent lock for device_attach() calls
  usbip: vhci_hcd: Fix usb device and sockfd leaks
  usbip: vhci_hcd: check rhport before using in vhci_hub_control()
  USB: Increment wakeup count on remote wakeup.
  ...
parents 7ff50002 573a0948
...@@ -28,7 +28,10 @@ Required properties: ...@@ -28,7 +28,10 @@ Required properties:
- interrupts: one XHCI interrupt should be described here. - interrupts: one XHCI interrupt should be described here.
Optional properties: Optional properties:
- clocks: reference to a clock - clocks: reference to the clocks
- clock-names: mandatory if there is a second clock, in this case
the name must be "core" for the first clock and "reg" for the
second one
- usb2-lpm-disable: indicate if we don't want to enable USB2 HW LPM - usb2-lpm-disable: indicate if we don't want to enable USB2 HW LPM
- usb3-lpm-capable: determines if platform is USB3 LPM capable - usb3-lpm-capable: determines if platform is USB3 LPM capable
- quirk-broken-port-ped: set if the controller has broken port disable mechanism - quirk-broken-port-ped: set if the controller has broken port disable mechanism
......
...@@ -210,7 +210,7 @@ If the connector is dual-role capable, there may also be a switch for the data ...@@ -210,7 +210,7 @@ If the connector is dual-role capable, there may also be a switch for the data
role. USB Type-C Connector Class does not supply separate API for them. The role. USB Type-C Connector Class does not supply separate API for them. The
port drivers can use USB Role Class API with those. port drivers can use USB Role Class API with those.
Illustration of the muxes behind a connector that supports an alternate mode: Illustration of the muxes behind a connector that supports an alternate mode::
------------------------ ------------------------
| Connector | | Connector |
......
...@@ -207,5 +207,6 @@ config USB_ULPI_BUS ...@@ -207,5 +207,6 @@ config USB_ULPI_BUS
config USB_ROLE_SWITCH config USB_ROLE_SWITCH
tristate tristate
select USB_COMMON
endif # USB_SUPPORT endif # USB_SUPPORT
...@@ -2262,7 +2262,8 @@ int hcd_bus_suspend(struct usb_device *rhdev, pm_message_t msg) ...@@ -2262,7 +2262,8 @@ int hcd_bus_suspend(struct usb_device *rhdev, pm_message_t msg)
hcd->state = HC_STATE_SUSPENDED; hcd->state = HC_STATE_SUSPENDED;
if (!PMSG_IS_AUTO(msg)) if (!PMSG_IS_AUTO(msg))
usb_phy_roothub_power_off(hcd->phy_roothub); usb_phy_roothub_suspend(hcd->self.sysdev,
hcd->phy_roothub);
/* Did we race with a root-hub wakeup event? */ /* Did we race with a root-hub wakeup event? */
if (rhdev->do_remote_wakeup) { if (rhdev->do_remote_wakeup) {
...@@ -2302,7 +2303,8 @@ int hcd_bus_resume(struct usb_device *rhdev, pm_message_t msg) ...@@ -2302,7 +2303,8 @@ int hcd_bus_resume(struct usb_device *rhdev, pm_message_t msg)
} }
if (!PMSG_IS_AUTO(msg)) { if (!PMSG_IS_AUTO(msg)) {
status = usb_phy_roothub_power_on(hcd->phy_roothub); status = usb_phy_roothub_resume(hcd->self.sysdev,
hcd->phy_roothub);
if (status) if (status)
return status; return status;
} }
...@@ -2344,7 +2346,7 @@ int hcd_bus_resume(struct usb_device *rhdev, pm_message_t msg) ...@@ -2344,7 +2346,7 @@ int hcd_bus_resume(struct usb_device *rhdev, pm_message_t msg)
} }
} else { } else {
hcd->state = old_state; hcd->state = old_state;
usb_phy_roothub_power_off(hcd->phy_roothub); usb_phy_roothub_suspend(hcd->self.sysdev, hcd->phy_roothub);
dev_dbg(&rhdev->dev, "bus %s fail, err %d\n", dev_dbg(&rhdev->dev, "bus %s fail, err %d\n",
"resume", status); "resume", status);
if (status != -ESHUTDOWN) if (status != -ESHUTDOWN)
...@@ -2377,6 +2379,7 @@ void usb_hcd_resume_root_hub (struct usb_hcd *hcd) ...@@ -2377,6 +2379,7 @@ void usb_hcd_resume_root_hub (struct usb_hcd *hcd)
spin_lock_irqsave (&hcd_root_hub_lock, flags); spin_lock_irqsave (&hcd_root_hub_lock, flags);
if (hcd->rh_registered) { if (hcd->rh_registered) {
pm_wakeup_event(&hcd->self.root_hub->dev, 0);
set_bit(HCD_FLAG_WAKEUP_PENDING, &hcd->flags); set_bit(HCD_FLAG_WAKEUP_PENDING, &hcd->flags);
queue_work(pm_wq, &hcd->wakeup_work); queue_work(pm_wq, &hcd->wakeup_work);
} }
...@@ -2758,12 +2761,16 @@ int usb_add_hcd(struct usb_hcd *hcd, ...@@ -2758,12 +2761,16 @@ int usb_add_hcd(struct usb_hcd *hcd,
} }
if (!hcd->skip_phy_initialization && usb_hcd_is_primary_hcd(hcd)) { if (!hcd->skip_phy_initialization && usb_hcd_is_primary_hcd(hcd)) {
hcd->phy_roothub = usb_phy_roothub_init(hcd->self.sysdev); hcd->phy_roothub = usb_phy_roothub_alloc(hcd->self.sysdev);
if (IS_ERR(hcd->phy_roothub)) { if (IS_ERR(hcd->phy_roothub)) {
retval = PTR_ERR(hcd->phy_roothub); retval = PTR_ERR(hcd->phy_roothub);
goto err_phy_roothub_init; goto err_phy_roothub_alloc;
} }
retval = usb_phy_roothub_init(hcd->phy_roothub);
if (retval)
goto err_phy_roothub_alloc;
retval = usb_phy_roothub_power_on(hcd->phy_roothub); retval = usb_phy_roothub_power_on(hcd->phy_roothub);
if (retval) if (retval)
goto err_usb_phy_roothub_power_on; goto err_usb_phy_roothub_power_on;
...@@ -2936,7 +2943,7 @@ int usb_add_hcd(struct usb_hcd *hcd, ...@@ -2936,7 +2943,7 @@ int usb_add_hcd(struct usb_hcd *hcd,
usb_phy_roothub_power_off(hcd->phy_roothub); usb_phy_roothub_power_off(hcd->phy_roothub);
err_usb_phy_roothub_power_on: err_usb_phy_roothub_power_on:
usb_phy_roothub_exit(hcd->phy_roothub); usb_phy_roothub_exit(hcd->phy_roothub);
err_phy_roothub_init: err_phy_roothub_alloc:
if (hcd->remove_phy && hcd->usb_phy) { if (hcd->remove_phy && hcd->usb_phy) {
usb_phy_shutdown(hcd->usb_phy); usb_phy_shutdown(hcd->usb_phy);
usb_put_phy(hcd->usb_phy); usb_put_phy(hcd->usb_phy);
......
...@@ -653,12 +653,17 @@ void usb_wakeup_notification(struct usb_device *hdev, ...@@ -653,12 +653,17 @@ void usb_wakeup_notification(struct usb_device *hdev,
unsigned int portnum) unsigned int portnum)
{ {
struct usb_hub *hub; struct usb_hub *hub;
struct usb_port *port_dev;
if (!hdev) if (!hdev)
return; return;
hub = usb_hub_to_struct_hub(hdev); hub = usb_hub_to_struct_hub(hdev);
if (hub) { if (hub) {
port_dev = hub->ports[portnum - 1];
if (port_dev && port_dev->child)
pm_wakeup_event(&port_dev->child->dev, 0);
set_bit(portnum, hub->wakeup_bits); set_bit(portnum, hub->wakeup_bits);
kick_hub_wq(hub); kick_hub_wq(hub);
} }
...@@ -3434,8 +3439,11 @@ int usb_port_resume(struct usb_device *udev, pm_message_t msg) ...@@ -3434,8 +3439,11 @@ int usb_port_resume(struct usb_device *udev, pm_message_t msg)
/* Skip the initial Clear-Suspend step for a remote wakeup */ /* Skip the initial Clear-Suspend step for a remote wakeup */
status = hub_port_status(hub, port1, &portstatus, &portchange); status = hub_port_status(hub, port1, &portstatus, &portchange);
if (status == 0 && !port_is_suspended(hub, portstatus)) if (status == 0 && !port_is_suspended(hub, portstatus)) {
if (portchange & USB_PORT_STAT_C_SUSPEND)
pm_wakeup_event(&udev->dev, 0);
goto SuspendCleared; goto SuspendCleared;
}
/* see 7.1.7.7; affects power usage, but not budgeting */ /* see 7.1.7.7; affects power usage, but not budgeting */
if (hub_is_superspeed(hub->hdev)) if (hub_is_superspeed(hub->hdev))
......
...@@ -19,19 +19,6 @@ struct usb_phy_roothub { ...@@ -19,19 +19,6 @@ struct usb_phy_roothub {
struct list_head list; struct list_head list;
}; };
static struct usb_phy_roothub *usb_phy_roothub_alloc(struct device *dev)
{
struct usb_phy_roothub *roothub_entry;
roothub_entry = devm_kzalloc(dev, sizeof(*roothub_entry), GFP_KERNEL);
if (!roothub_entry)
return ERR_PTR(-ENOMEM);
INIT_LIST_HEAD(&roothub_entry->list);
return roothub_entry;
}
static int usb_phy_roothub_add_phy(struct device *dev, int index, static int usb_phy_roothub_add_phy(struct device *dev, int index,
struct list_head *list) struct list_head *list)
{ {
...@@ -45,9 +32,11 @@ static int usb_phy_roothub_add_phy(struct device *dev, int index, ...@@ -45,9 +32,11 @@ static int usb_phy_roothub_add_phy(struct device *dev, int index,
return PTR_ERR(phy); return PTR_ERR(phy);
} }
roothub_entry = usb_phy_roothub_alloc(dev); roothub_entry = devm_kzalloc(dev, sizeof(*roothub_entry), GFP_KERNEL);
if (IS_ERR(roothub_entry)) if (!roothub_entry)
return PTR_ERR(roothub_entry); return -ENOMEM;
INIT_LIST_HEAD(&roothub_entry->list);
roothub_entry->phy = phy; roothub_entry->phy = phy;
...@@ -56,28 +45,44 @@ static int usb_phy_roothub_add_phy(struct device *dev, int index, ...@@ -56,28 +45,44 @@ static int usb_phy_roothub_add_phy(struct device *dev, int index,
return 0; return 0;
} }
struct usb_phy_roothub *usb_phy_roothub_init(struct device *dev) struct usb_phy_roothub *usb_phy_roothub_alloc(struct device *dev)
{ {
struct usb_phy_roothub *phy_roothub; struct usb_phy_roothub *phy_roothub;
struct usb_phy_roothub *roothub_entry;
struct list_head *head;
int i, num_phys, err; int i, num_phys, err;
if (!IS_ENABLED(CONFIG_GENERIC_PHY))
return NULL;
num_phys = of_count_phandle_with_args(dev->of_node, "phys", num_phys = of_count_phandle_with_args(dev->of_node, "phys",
"#phy-cells"); "#phy-cells");
if (num_phys <= 0) if (num_phys <= 0)
return NULL; return NULL;
phy_roothub = usb_phy_roothub_alloc(dev); phy_roothub = devm_kzalloc(dev, sizeof(*phy_roothub), GFP_KERNEL);
if (IS_ERR(phy_roothub)) if (!phy_roothub)
return phy_roothub; return ERR_PTR(-ENOMEM);
INIT_LIST_HEAD(&phy_roothub->list);
for (i = 0; i < num_phys; i++) { for (i = 0; i < num_phys; i++) {
err = usb_phy_roothub_add_phy(dev, i, &phy_roothub->list); err = usb_phy_roothub_add_phy(dev, i, &phy_roothub->list);
if (err) if (err)
goto err_out; return ERR_PTR(err);
} }
return phy_roothub;
}
EXPORT_SYMBOL_GPL(usb_phy_roothub_alloc);
int usb_phy_roothub_init(struct usb_phy_roothub *phy_roothub)
{
struct usb_phy_roothub *roothub_entry;
struct list_head *head;
int err;
if (!phy_roothub)
return 0;
head = &phy_roothub->list; head = &phy_roothub->list;
list_for_each_entry(roothub_entry, head, list) { list_for_each_entry(roothub_entry, head, list) {
...@@ -86,14 +91,13 @@ struct usb_phy_roothub *usb_phy_roothub_init(struct device *dev) ...@@ -86,14 +91,13 @@ struct usb_phy_roothub *usb_phy_roothub_init(struct device *dev)
goto err_exit_phys; goto err_exit_phys;
} }
return phy_roothub; return 0;
err_exit_phys: err_exit_phys:
list_for_each_entry_continue_reverse(roothub_entry, head, list) list_for_each_entry_continue_reverse(roothub_entry, head, list)
phy_exit(roothub_entry->phy); phy_exit(roothub_entry->phy);
err_out: return err;
return ERR_PTR(err);
} }
EXPORT_SYMBOL_GPL(usb_phy_roothub_init); EXPORT_SYMBOL_GPL(usb_phy_roothub_init);
...@@ -111,7 +115,7 @@ int usb_phy_roothub_exit(struct usb_phy_roothub *phy_roothub) ...@@ -111,7 +115,7 @@ int usb_phy_roothub_exit(struct usb_phy_roothub *phy_roothub)
list_for_each_entry(roothub_entry, head, list) { list_for_each_entry(roothub_entry, head, list) {
err = phy_exit(roothub_entry->phy); err = phy_exit(roothub_entry->phy);
if (err) if (err)
ret = ret; ret = err;
} }
return ret; return ret;
...@@ -156,3 +160,38 @@ void usb_phy_roothub_power_off(struct usb_phy_roothub *phy_roothub) ...@@ -156,3 +160,38 @@ void usb_phy_roothub_power_off(struct usb_phy_roothub *phy_roothub)
phy_power_off(roothub_entry->phy); phy_power_off(roothub_entry->phy);
} }
EXPORT_SYMBOL_GPL(usb_phy_roothub_power_off); EXPORT_SYMBOL_GPL(usb_phy_roothub_power_off);
int usb_phy_roothub_suspend(struct device *controller_dev,
struct usb_phy_roothub *phy_roothub)
{
usb_phy_roothub_power_off(phy_roothub);
/* keep the PHYs initialized so the device can wake up the system */
if (device_may_wakeup(controller_dev))
return 0;
return usb_phy_roothub_exit(phy_roothub);
}
EXPORT_SYMBOL_GPL(usb_phy_roothub_suspend);
int usb_phy_roothub_resume(struct device *controller_dev,
struct usb_phy_roothub *phy_roothub)
{
int err;
/* if the device can't wake up the system _exit was called */
if (!device_may_wakeup(controller_dev)) {
err = usb_phy_roothub_init(phy_roothub);
if (err)
return err;
}
err = usb_phy_roothub_power_on(phy_roothub);
/* undo _init if _power_on failed */
if (err && !device_may_wakeup(controller_dev))
usb_phy_roothub_exit(phy_roothub);
return err;
}
EXPORT_SYMBOL_GPL(usb_phy_roothub_resume);
/* SPDX-License-Identifier: GPL-2.0+ */
/*
* USB roothub wrapper
*
* Copyright (C) 2018 Martin Blumenstingl <martin.blumenstingl@googlemail.com>
*/
#ifndef __USB_CORE_PHY_H_
#define __USB_CORE_PHY_H_
struct device;
struct usb_phy_roothub; struct usb_phy_roothub;
struct usb_phy_roothub *usb_phy_roothub_init(struct device *dev); struct usb_phy_roothub *usb_phy_roothub_alloc(struct device *dev);
int usb_phy_roothub_init(struct usb_phy_roothub *phy_roothub);
int usb_phy_roothub_exit(struct usb_phy_roothub *phy_roothub); int usb_phy_roothub_exit(struct usb_phy_roothub *phy_roothub);
int usb_phy_roothub_power_on(struct usb_phy_roothub *phy_roothub); int usb_phy_roothub_power_on(struct usb_phy_roothub *phy_roothub);
void usb_phy_roothub_power_off(struct usb_phy_roothub *phy_roothub); void usb_phy_roothub_power_off(struct usb_phy_roothub *phy_roothub);
int usb_phy_roothub_suspend(struct device *controller_dev,
struct usb_phy_roothub *phy_roothub);
int usb_phy_roothub_resume(struct device *controller_dev,
struct usb_phy_roothub *phy_roothub);
#endif /* __USB_CORE_PHY_H_ */
...@@ -186,6 +186,9 @@ static const struct usb_device_id usb_quirk_list[] = { ...@@ -186,6 +186,9 @@ static const struct usb_device_id usb_quirk_list[] = {
{ USB_DEVICE(0x03f0, 0x0701), .driver_info = { USB_DEVICE(0x03f0, 0x0701), .driver_info =
USB_QUIRK_STRING_FETCH_255 }, USB_QUIRK_STRING_FETCH_255 },
/* HP v222w 16GB Mini USB Drive */
{ USB_DEVICE(0x03f0, 0x3f40), .driver_info = USB_QUIRK_DELAY_INIT },
/* Creative SB Audigy 2 NX */ /* Creative SB Audigy 2 NX */
{ USB_DEVICE(0x041e, 0x3020), .driver_info = USB_QUIRK_RESET_RESUME }, { USB_DEVICE(0x041e, 0x3020), .driver_info = USB_QUIRK_RESET_RESUME },
......
...@@ -320,9 +320,11 @@ int xhci_dbc_tty_register_driver(struct xhci_hcd *xhci) ...@@ -320,9 +320,11 @@ int xhci_dbc_tty_register_driver(struct xhci_hcd *xhci)
void xhci_dbc_tty_unregister_driver(void) void xhci_dbc_tty_unregister_driver(void)
{ {
if (dbc_tty_driver) {
tty_unregister_driver(dbc_tty_driver); tty_unregister_driver(dbc_tty_driver);
put_tty_driver(dbc_tty_driver); put_tty_driver(dbc_tty_driver);
dbc_tty_driver = NULL; dbc_tty_driver = NULL;
}
} }
static void dbc_rx_push(unsigned long _port) static void dbc_rx_push(unsigned long _port)
......
...@@ -126,7 +126,10 @@ static void xhci_pci_quirks(struct device *dev, struct xhci_hcd *xhci) ...@@ -126,7 +126,10 @@ static void xhci_pci_quirks(struct device *dev, struct xhci_hcd *xhci)
if (pdev->vendor == PCI_VENDOR_ID_AMD && usb_amd_find_chipset_info()) if (pdev->vendor == PCI_VENDOR_ID_AMD && usb_amd_find_chipset_info())
xhci->quirks |= XHCI_AMD_PLL_FIX; xhci->quirks |= XHCI_AMD_PLL_FIX;
if (pdev->vendor == PCI_VENDOR_ID_AMD && pdev->device == 0x43bb) if (pdev->vendor == PCI_VENDOR_ID_AMD &&
(pdev->device == 0x15e0 ||
pdev->device == 0x15e1 ||
pdev->device == 0x43bb))
xhci->quirks |= XHCI_SUSPEND_DELAY; xhci->quirks |= XHCI_SUSPEND_DELAY;
if (pdev->vendor == PCI_VENDOR_ID_AMD) if (pdev->vendor == PCI_VENDOR_ID_AMD)
......
...@@ -157,6 +157,7 @@ static int xhci_plat_probe(struct platform_device *pdev) ...@@ -157,6 +157,7 @@ static int xhci_plat_probe(struct platform_device *pdev)
struct resource *res; struct resource *res;
struct usb_hcd *hcd; struct usb_hcd *hcd;
struct clk *clk; struct clk *clk;
struct clk *reg_clk;
int ret; int ret;
int irq; int irq;
...@@ -226,17 +227,27 @@ static int xhci_plat_probe(struct platform_device *pdev) ...@@ -226,17 +227,27 @@ static int xhci_plat_probe(struct platform_device *pdev)
hcd->rsrc_len = resource_size(res); hcd->rsrc_len = resource_size(res);
/* /*
* Not all platforms have a clk so it is not an error if the * Not all platforms have clks so it is not an error if the
* clock does not exists. * clock do not exist.
*/ */
reg_clk = devm_clk_get(&pdev->dev, "reg");
if (!IS_ERR(reg_clk)) {
ret = clk_prepare_enable(reg_clk);
if (ret)
goto put_hcd;
} else if (PTR_ERR(reg_clk) == -EPROBE_DEFER) {
ret = -EPROBE_DEFER;
goto put_hcd;
}
clk = devm_clk_get(&pdev->dev, NULL); clk = devm_clk_get(&pdev->dev, NULL);
if (!IS_ERR(clk)) { if (!IS_ERR(clk)) {
ret = clk_prepare_enable(clk); ret = clk_prepare_enable(clk);
if (ret) if (ret)
goto put_hcd; goto disable_reg_clk;
} else if (PTR_ERR(clk) == -EPROBE_DEFER) { } else if (PTR_ERR(clk) == -EPROBE_DEFER) {
ret = -EPROBE_DEFER; ret = -EPROBE_DEFER;
goto put_hcd; goto disable_reg_clk;
} }
xhci = hcd_to_xhci(hcd); xhci = hcd_to_xhci(hcd);
...@@ -252,6 +263,7 @@ static int xhci_plat_probe(struct platform_device *pdev) ...@@ -252,6 +263,7 @@ static int xhci_plat_probe(struct platform_device *pdev)
device_wakeup_enable(hcd->self.controller); device_wakeup_enable(hcd->self.controller);
xhci->clk = clk; xhci->clk = clk;
xhci->reg_clk = reg_clk;
xhci->main_hcd = hcd; xhci->main_hcd = hcd;
xhci->shared_hcd = __usb_create_hcd(driver, sysdev, &pdev->dev, xhci->shared_hcd = __usb_create_hcd(driver, sysdev, &pdev->dev,
dev_name(&pdev->dev), hcd); dev_name(&pdev->dev), hcd);
...@@ -320,9 +332,11 @@ static int xhci_plat_probe(struct platform_device *pdev) ...@@ -320,9 +332,11 @@ static int xhci_plat_probe(struct platform_device *pdev)
usb_put_hcd(xhci->shared_hcd); usb_put_hcd(xhci->shared_hcd);
disable_clk: disable_clk:
if (!IS_ERR(clk))
clk_disable_unprepare(clk); clk_disable_unprepare(clk);
disable_reg_clk:
clk_disable_unprepare(reg_clk);
put_hcd: put_hcd:
usb_put_hcd(hcd); usb_put_hcd(hcd);
...@@ -338,6 +352,7 @@ static int xhci_plat_remove(struct platform_device *dev) ...@@ -338,6 +352,7 @@ static int xhci_plat_remove(struct platform_device *dev)
struct usb_hcd *hcd = platform_get_drvdata(dev); struct usb_hcd *hcd = platform_get_drvdata(dev);
struct xhci_hcd *xhci = hcd_to_xhci(hcd); struct xhci_hcd *xhci = hcd_to_xhci(hcd);
struct clk *clk = xhci->clk; struct clk *clk = xhci->clk;
struct clk *reg_clk = xhci->reg_clk;
xhci->xhc_state |= XHCI_STATE_REMOVING; xhci->xhc_state |= XHCI_STATE_REMOVING;
...@@ -347,8 +362,8 @@ static int xhci_plat_remove(struct platform_device *dev) ...@@ -347,8 +362,8 @@ static int xhci_plat_remove(struct platform_device *dev)
usb_remove_hcd(hcd); usb_remove_hcd(hcd);
usb_put_hcd(xhci->shared_hcd); usb_put_hcd(xhci->shared_hcd);
if (!IS_ERR(clk))
clk_disable_unprepare(clk); clk_disable_unprepare(clk);
clk_disable_unprepare(reg_clk);
usb_put_hcd(hcd); usb_put_hcd(hcd);
pm_runtime_set_suspended(&dev->dev); pm_runtime_set_suspended(&dev->dev);
...@@ -420,7 +435,6 @@ MODULE_DEVICE_TABLE(acpi, usb_xhci_acpi_match); ...@@ -420,7 +435,6 @@ MODULE_DEVICE_TABLE(acpi, usb_xhci_acpi_match);
static struct platform_driver usb_xhci_driver = { static struct platform_driver usb_xhci_driver = {
.probe = xhci_plat_probe, .probe = xhci_plat_probe,
.remove = xhci_plat_remove, .remove = xhci_plat_remove,
.shutdown = usb_hcd_platform_shutdown,
.driver = { .driver = {
.name = "xhci-hcd", .name = "xhci-hcd",
.pm = &xhci_plat_pm_ops, .pm = &xhci_plat_pm_ops,
......
...@@ -1729,8 +1729,9 @@ struct xhci_hcd { ...@@ -1729,8 +1729,9 @@ struct xhci_hcd {
int page_shift; int page_shift;
/* msi-x vectors */ /* msi-x vectors */
int msix_count; int msix_count;
/* optional clock */ /* optional clocks */
struct clk *clk; struct clk *clk;
struct clk *reg_clk;
/* data structures */ /* data structures */
struct xhci_device_context_array *dcbaa; struct xhci_device_context_array *dcbaa;
struct xhci_ring *cmd_ring; struct xhci_ring *cmd_ring;
......
...@@ -451,7 +451,6 @@ static int dsps_musb_init(struct musb *musb) ...@@ -451,7 +451,6 @@ static int dsps_musb_init(struct musb *musb)
if (!rev) if (!rev)
return -ENODEV; return -ENODEV;
usb_phy_init(musb->xceiv);
if (IS_ERR(musb->phy)) { if (IS_ERR(musb->phy)) {
musb->phy = NULL; musb->phy = NULL;
} else { } else {
...@@ -501,7 +500,6 @@ static int dsps_musb_exit(struct musb *musb) ...@@ -501,7 +500,6 @@ static int dsps_musb_exit(struct musb *musb)
struct dsps_glue *glue = dev_get_drvdata(dev->parent); struct dsps_glue *glue = dev_get_drvdata(dev->parent);
del_timer_sync(&musb->dev_timer); del_timer_sync(&musb->dev_timer);
usb_phy_shutdown(musb->xceiv);
phy_power_off(musb->phy); phy_power_off(musb->phy);
phy_exit(musb->phy); phy_exit(musb->phy);
debugfs_remove_recursive(glue->dbgfs_root); debugfs_remove_recursive(glue->dbgfs_root);
......
...@@ -2754,6 +2754,7 @@ int musb_host_setup(struct musb *musb, int power_budget) ...@@ -2754,6 +2754,7 @@ int musb_host_setup(struct musb *musb, int power_budget)
hcd->self.otg_port = 1; hcd->self.otg_port = 1;
musb->xceiv->otg->host = &hcd->self; musb->xceiv->otg->host = &hcd->self;
hcd->power_budget = 2 * (power_budget ? : 250); hcd->power_budget = 2 * (power_budget ? : 250);
hcd->skip_phy_initialization = 1;
ret = usb_add_hcd(hcd, 0, 0); ret = usb_add_hcd(hcd, 0, 0);
if (ret < 0) if (ret < 0)
......
...@@ -62,6 +62,7 @@ config USB_SERIAL_SIMPLE ...@@ -62,6 +62,7 @@ config USB_SERIAL_SIMPLE
- Fundamental Software dongle. - Fundamental Software dongle.
- Google USB serial devices - Google USB serial devices
- HP4x calculators - HP4x calculators
- Libtransistor USB console
- a number of Motorola phones - a number of Motorola phones
- Motorola Tetra devices - Motorola Tetra devices
- Novatel Wireless GPS receivers - Novatel Wireless GPS receivers
......
...@@ -214,6 +214,7 @@ static const struct usb_device_id id_table[] = { ...@@ -214,6 +214,7 @@ static const struct usb_device_id id_table[] = {
{ USB_DEVICE(0x3195, 0xF190) }, /* Link Instruments MSO-19 */ { USB_DEVICE(0x3195, 0xF190) }, /* Link Instruments MSO-19 */
{ USB_DEVICE(0x3195, 0xF280) }, /* Link Instruments MSO-28 */ { USB_DEVICE(0x3195, 0xF280) }, /* Link Instruments MSO-28 */
{ USB_DEVICE(0x3195, 0xF281) }, /* Link Instruments MSO-28 */ { USB_DEVICE(0x3195, 0xF281) }, /* Link Instruments MSO-28 */
{ USB_DEVICE(0x3923, 0x7A0B) }, /* National Instruments USB Serial Console */
{ USB_DEVICE(0x413C, 0x9500) }, /* DW700 GPS USB interface */ { USB_DEVICE(0x413C, 0x9500) }, /* DW700 GPS USB interface */
{ } /* Terminating Entry */ { } /* Terminating Entry */
}; };
......
...@@ -1898,7 +1898,8 @@ static int ftdi_8u2232c_probe(struct usb_serial *serial) ...@@ -1898,7 +1898,8 @@ static int ftdi_8u2232c_probe(struct usb_serial *serial)
return ftdi_jtag_probe(serial); return ftdi_jtag_probe(serial);
if (udev->product && if (udev->product &&
(!strcmp(udev->product, "BeagleBone/XDS100V2") || (!strcmp(udev->product, "Arrow USB Blaster") ||
!strcmp(udev->product, "BeagleBone/XDS100V2") ||
!strcmp(udev->product, "SNAP Connect E10"))) !strcmp(udev->product, "SNAP Connect E10")))
return ftdi_jtag_probe(serial); return ftdi_jtag_probe(serial);
......
...@@ -63,6 +63,11 @@ DEVICE(flashloader, FLASHLOADER_IDS); ...@@ -63,6 +63,11 @@ DEVICE(flashloader, FLASHLOADER_IDS);
0x01) } 0x01) }
DEVICE(google, GOOGLE_IDS); DEVICE(google, GOOGLE_IDS);
/* Libtransistor USB console */
#define LIBTRANSISTOR_IDS() \
{ USB_DEVICE(0x1209, 0x8b00) }
DEVICE(libtransistor, LIBTRANSISTOR_IDS);
/* ViVOpay USB Serial Driver */ /* ViVOpay USB Serial Driver */
#define VIVOPAY_IDS() \ #define VIVOPAY_IDS() \
{ USB_DEVICE(0x1d5f, 0x1004) } /* ViVOpay 8800 */ { USB_DEVICE(0x1d5f, 0x1004) } /* ViVOpay 8800 */
...@@ -110,6 +115,7 @@ static struct usb_serial_driver * const serial_drivers[] = { ...@@ -110,6 +115,7 @@ static struct usb_serial_driver * const serial_drivers[] = {
&funsoft_device, &funsoft_device,
&flashloader_device, &flashloader_device,
&google_device, &google_device,
&libtransistor_device,
&vivopay_device, &vivopay_device,
&moto_modem_device, &moto_modem_device,
&motorola_tetra_device, &motorola_tetra_device,
...@@ -126,6 +132,7 @@ static const struct usb_device_id id_table[] = { ...@@ -126,6 +132,7 @@ static const struct usb_device_id id_table[] = {
FUNSOFT_IDS(), FUNSOFT_IDS(),
FLASHLOADER_IDS(), FLASHLOADER_IDS(),
GOOGLE_IDS(), GOOGLE_IDS(),
LIBTRANSISTOR_IDS(),
VIVOPAY_IDS(), VIVOPAY_IDS(),
MOTO_IDS(), MOTO_IDS(),
MOTOROLA_TETRA_IDS(), MOTOROLA_TETRA_IDS(),
......
...@@ -5,6 +5,6 @@ obj-$(CONFIG_TYPEC_UCSI) += typec_ucsi.o ...@@ -5,6 +5,6 @@ obj-$(CONFIG_TYPEC_UCSI) += typec_ucsi.o
typec_ucsi-y := ucsi.o typec_ucsi-y := ucsi.o
typec_ucsi-$(CONFIG_FTRACE) += trace.o typec_ucsi-$(CONFIG_TRACING) += trace.o
obj-$(CONFIG_UCSI_ACPI) += ucsi_acpi.o obj-$(CONFIG_UCSI_ACPI) += ucsi_acpi.o
...@@ -28,7 +28,7 @@ ...@@ -28,7 +28,7 @@
* difficult to estimate the time it takes for the system to process the command * difficult to estimate the time it takes for the system to process the command
* before it is actually passed to the PPM. * before it is actually passed to the PPM.
*/ */
#define UCSI_TIMEOUT_MS 1000 #define UCSI_TIMEOUT_MS 5000
/* /*
* UCSI_SWAP_TIMEOUT_MS - Timeout for role swap requests * UCSI_SWAP_TIMEOUT_MS - Timeout for role swap requests
......
...@@ -186,7 +186,12 @@ static ssize_t rebind_store(struct device_driver *dev, const char *buf, ...@@ -186,7 +186,12 @@ static ssize_t rebind_store(struct device_driver *dev, const char *buf,
if (!bid) if (!bid)
return -ENODEV; return -ENODEV;
/* device_attach() callers should hold parent lock for USB */
if (bid->udev->dev.parent)
device_lock(bid->udev->dev.parent);
ret = device_attach(&bid->udev->dev); ret = device_attach(&bid->udev->dev);
if (bid->udev->dev.parent)
device_unlock(bid->udev->dev.parent);
if (ret < 0) { if (ret < 0) {
dev_err(&bid->udev->dev, "rebind failed\n"); dev_err(&bid->udev->dev, "rebind failed\n");
return ret; return ret;
......
...@@ -243,7 +243,7 @@ enum usbip_side { ...@@ -243,7 +243,7 @@ enum usbip_side {
#define VUDC_EVENT_ERROR_USB (USBIP_EH_SHUTDOWN | USBIP_EH_UNUSABLE) #define VUDC_EVENT_ERROR_USB (USBIP_EH_SHUTDOWN | USBIP_EH_UNUSABLE)
#define VUDC_EVENT_ERROR_MALLOC (USBIP_EH_SHUTDOWN | USBIP_EH_UNUSABLE) #define VUDC_EVENT_ERROR_MALLOC (USBIP_EH_SHUTDOWN | USBIP_EH_UNUSABLE)
#define VDEV_EVENT_REMOVED (USBIP_EH_SHUTDOWN | USBIP_EH_BYE) #define VDEV_EVENT_REMOVED (USBIP_EH_SHUTDOWN | USBIP_EH_RESET | USBIP_EH_BYE)
#define VDEV_EVENT_DOWN (USBIP_EH_SHUTDOWN | USBIP_EH_RESET) #define VDEV_EVENT_DOWN (USBIP_EH_SHUTDOWN | USBIP_EH_RESET)
#define VDEV_EVENT_ERROR_TCP (USBIP_EH_SHUTDOWN | USBIP_EH_RESET) #define VDEV_EVENT_ERROR_TCP (USBIP_EH_SHUTDOWN | USBIP_EH_RESET)
#define VDEV_EVENT_ERROR_MALLOC (USBIP_EH_SHUTDOWN | USBIP_EH_UNUSABLE) #define VDEV_EVENT_ERROR_MALLOC (USBIP_EH_SHUTDOWN | USBIP_EH_UNUSABLE)
......
...@@ -91,10 +91,6 @@ static void event_handler(struct work_struct *work) ...@@ -91,10 +91,6 @@ static void event_handler(struct work_struct *work)
unset_event(ud, USBIP_EH_UNUSABLE); unset_event(ud, USBIP_EH_UNUSABLE);
} }
/* Stop the error handler. */
if (ud->event & USBIP_EH_BYE)
usbip_dbg_eh("removed %p\n", ud);
wake_up(&ud->eh_waitq); wake_up(&ud->eh_waitq);
} }
} }
......
...@@ -354,6 +354,8 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, ...@@ -354,6 +354,8 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
usbip_dbg_vhci_rh(" ClearHubFeature\n"); usbip_dbg_vhci_rh(" ClearHubFeature\n");
break; break;
case ClearPortFeature: case ClearPortFeature:
if (rhport < 0)
goto error;
switch (wValue) { switch (wValue) {
case USB_PORT_FEAT_SUSPEND: case USB_PORT_FEAT_SUSPEND:
if (hcd->speed == HCD_USB3) { if (hcd->speed == HCD_USB3) {
...@@ -511,11 +513,16 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, ...@@ -511,11 +513,16 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
goto error; goto error;
} }
if (rhport < 0)
goto error;
vhci_hcd->port_status[rhport] |= USB_PORT_STAT_SUSPEND; vhci_hcd->port_status[rhport] |= USB_PORT_STAT_SUSPEND;
break; break;
case USB_PORT_FEAT_POWER: case USB_PORT_FEAT_POWER:
usbip_dbg_vhci_rh( usbip_dbg_vhci_rh(
" SetPortFeature: USB_PORT_FEAT_POWER\n"); " SetPortFeature: USB_PORT_FEAT_POWER\n");
if (rhport < 0)
goto error;
if (hcd->speed == HCD_USB3) if (hcd->speed == HCD_USB3)
vhci_hcd->port_status[rhport] |= USB_SS_PORT_STAT_POWER; vhci_hcd->port_status[rhport] |= USB_SS_PORT_STAT_POWER;
else else
...@@ -524,6 +531,8 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, ...@@ -524,6 +531,8 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
case USB_PORT_FEAT_BH_PORT_RESET: case USB_PORT_FEAT_BH_PORT_RESET:
usbip_dbg_vhci_rh( usbip_dbg_vhci_rh(
" SetPortFeature: USB_PORT_FEAT_BH_PORT_RESET\n"); " SetPortFeature: USB_PORT_FEAT_BH_PORT_RESET\n");
if (rhport < 0)
goto error;
/* Applicable only for USB3.0 hub */ /* Applicable only for USB3.0 hub */
if (hcd->speed != HCD_USB3) { if (hcd->speed != HCD_USB3) {
pr_err("USB_PORT_FEAT_BH_PORT_RESET req not " pr_err("USB_PORT_FEAT_BH_PORT_RESET req not "
...@@ -534,6 +543,8 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, ...@@ -534,6 +543,8 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
case USB_PORT_FEAT_RESET: case USB_PORT_FEAT_RESET:
usbip_dbg_vhci_rh( usbip_dbg_vhci_rh(
" SetPortFeature: USB_PORT_FEAT_RESET\n"); " SetPortFeature: USB_PORT_FEAT_RESET\n");
if (rhport < 0)
goto error;
/* if it's already enabled, disable */ /* if it's already enabled, disable */
if (hcd->speed == HCD_USB3) { if (hcd->speed == HCD_USB3) {
vhci_hcd->port_status[rhport] = 0; vhci_hcd->port_status[rhport] = 0;
...@@ -554,6 +565,8 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, ...@@ -554,6 +565,8 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
default: default:
usbip_dbg_vhci_rh(" SetPortFeature: default %d\n", usbip_dbg_vhci_rh(" SetPortFeature: default %d\n",
wValue); wValue);
if (rhport < 0)
goto error;
if (hcd->speed == HCD_USB3) { if (hcd->speed == HCD_USB3) {
if ((vhci_hcd->port_status[rhport] & if ((vhci_hcd->port_status[rhport] &
USB_SS_PORT_STAT_POWER) != 0) { USB_SS_PORT_STAT_POWER) != 0) {
......
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