Commit 4c39d4b9 authored by Arnd Bergmann's avatar Arnd Bergmann Committed by Greg Kroah-Hartman

usb: xhci: use bus->sysdev for DMA configuration

For xhci-hcd platform device, all the DMA parameters are not
configured properly, notably dma ops for dwc3 devices. So, set
the dma for xhci from sysdev. sysdev is pointing to device that
is known to the system firmware or hardware.

Cc: Baolin Wang <baolin.wang@linaro.org>
Tested-by: default avatarAlexander Sverdlin <alexander.sverdlin@nokia.com>
Tested-by: default avatarVivek Gautam <vivek.gautam@codeaurora.org>
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
Signed-off-by: default avatarSriram Dash <sriram.dash@nxp.com>
Signed-off-by: default avatarMathias Nyman <mathias.nyman@linux.intel.com>
Signed-off-by: default avatarPeter Chen <peter.chen@nxp.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 8123e495
...@@ -586,7 +586,7 @@ static void xhci_free_stream_ctx(struct xhci_hcd *xhci, ...@@ -586,7 +586,7 @@ static void xhci_free_stream_ctx(struct xhci_hcd *xhci,
unsigned int num_stream_ctxs, unsigned int num_stream_ctxs,
struct xhci_stream_ctx *stream_ctx, dma_addr_t dma) struct xhci_stream_ctx *stream_ctx, dma_addr_t dma)
{ {
struct device *dev = xhci_to_hcd(xhci)->self.controller; struct device *dev = xhci_to_hcd(xhci)->self.sysdev;
size_t size = sizeof(struct xhci_stream_ctx) * num_stream_ctxs; size_t size = sizeof(struct xhci_stream_ctx) * num_stream_ctxs;
if (size > MEDIUM_STREAM_ARRAY_SIZE) if (size > MEDIUM_STREAM_ARRAY_SIZE)
...@@ -614,7 +614,7 @@ static struct xhci_stream_ctx *xhci_alloc_stream_ctx(struct xhci_hcd *xhci, ...@@ -614,7 +614,7 @@ static struct xhci_stream_ctx *xhci_alloc_stream_ctx(struct xhci_hcd *xhci,
unsigned int num_stream_ctxs, dma_addr_t *dma, unsigned int num_stream_ctxs, dma_addr_t *dma,
gfp_t mem_flags) gfp_t mem_flags)
{ {
struct device *dev = xhci_to_hcd(xhci)->self.controller; struct device *dev = xhci_to_hcd(xhci)->self.sysdev;
size_t size = sizeof(struct xhci_stream_ctx) * num_stream_ctxs; size_t size = sizeof(struct xhci_stream_ctx) * num_stream_ctxs;
if (size > MEDIUM_STREAM_ARRAY_SIZE) if (size > MEDIUM_STREAM_ARRAY_SIZE)
...@@ -1686,7 +1686,7 @@ void xhci_slot_copy(struct xhci_hcd *xhci, ...@@ -1686,7 +1686,7 @@ void xhci_slot_copy(struct xhci_hcd *xhci,
static int scratchpad_alloc(struct xhci_hcd *xhci, gfp_t flags) static int scratchpad_alloc(struct xhci_hcd *xhci, gfp_t flags)
{ {
int i; int i;
struct device *dev = xhci_to_hcd(xhci)->self.controller; struct device *dev = xhci_to_hcd(xhci)->self.sysdev;
int num_sp = HCS_MAX_SCRATCHPAD(xhci->hcs_params2); int num_sp = HCS_MAX_SCRATCHPAD(xhci->hcs_params2);
xhci_dbg_trace(xhci, trace_xhci_dbg_init, xhci_dbg_trace(xhci, trace_xhci_dbg_init,
...@@ -1758,7 +1758,7 @@ static void scratchpad_free(struct xhci_hcd *xhci) ...@@ -1758,7 +1758,7 @@ static void scratchpad_free(struct xhci_hcd *xhci)
{ {
int num_sp; int num_sp;
int i; int i;
struct device *dev = xhci_to_hcd(xhci)->self.controller; struct device *dev = xhci_to_hcd(xhci)->self.sysdev;
if (!xhci->scratchpad) if (!xhci->scratchpad)
return; return;
...@@ -1831,7 +1831,7 @@ void xhci_free_command(struct xhci_hcd *xhci, ...@@ -1831,7 +1831,7 @@ void xhci_free_command(struct xhci_hcd *xhci,
void xhci_mem_cleanup(struct xhci_hcd *xhci) void xhci_mem_cleanup(struct xhci_hcd *xhci)
{ {
struct device *dev = xhci_to_hcd(xhci)->self.controller; struct device *dev = xhci_to_hcd(xhci)->self.sysdev;
int size; int size;
int i, j, num_ports; int i, j, num_ports;
...@@ -2373,7 +2373,7 @@ static int xhci_setup_port_arrays(struct xhci_hcd *xhci, gfp_t flags) ...@@ -2373,7 +2373,7 @@ static int xhci_setup_port_arrays(struct xhci_hcd *xhci, gfp_t flags)
int xhci_mem_init(struct xhci_hcd *xhci, gfp_t flags) int xhci_mem_init(struct xhci_hcd *xhci, gfp_t flags)
{ {
dma_addr_t dma; dma_addr_t dma;
struct device *dev = xhci_to_hcd(xhci)->self.controller; struct device *dev = xhci_to_hcd(xhci)->self.sysdev;
unsigned int val, val2; unsigned int val, val2;
u64 val_64; u64 val_64;
struct xhci_segment *seg; struct xhci_segment *seg;
......
...@@ -14,6 +14,7 @@ ...@@ -14,6 +14,7 @@
#include <linux/clk.h> #include <linux/clk.h>
#include <linux/dma-mapping.h> #include <linux/dma-mapping.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/pci.h>
#include <linux/of.h> #include <linux/of.h>
#include <linux/platform_device.h> #include <linux/platform_device.h>
#include <linux/usb/phy.h> #include <linux/usb/phy.h>
...@@ -148,6 +149,7 @@ static int xhci_plat_probe(struct platform_device *pdev) ...@@ -148,6 +149,7 @@ static int xhci_plat_probe(struct platform_device *pdev)
{ {
const struct of_device_id *match; const struct of_device_id *match;
const struct hc_driver *driver; const struct hc_driver *driver;
struct device *sysdev;
struct xhci_hcd *xhci; struct xhci_hcd *xhci;
struct resource *res; struct resource *res;
struct usb_hcd *hcd; struct usb_hcd *hcd;
...@@ -164,22 +166,39 @@ static int xhci_plat_probe(struct platform_device *pdev) ...@@ -164,22 +166,39 @@ static int xhci_plat_probe(struct platform_device *pdev)
if (irq < 0) if (irq < 0)
return -ENODEV; return -ENODEV;
/*
* sysdev must point to a device that is known to the system firmware
* or PCI hardware. We handle these three cases here:
* 1. xhci_plat comes from firmware
* 2. xhci_plat is child of a device from firmware (dwc3-plat)
* 3. xhci_plat is grandchild of a pci device (dwc3-pci)
*/
sysdev = &pdev->dev;
if (sysdev->parent && !sysdev->of_node && sysdev->parent->of_node)
sysdev = sysdev->parent;
#ifdef CONFIG_PCI
else if (sysdev->parent && sysdev->parent->parent &&
sysdev->parent->parent->bus == &pci_bus_type)
sysdev = sysdev->parent->parent;
#endif
/* Try to set 64-bit DMA first */ /* Try to set 64-bit DMA first */
if (!pdev->dev.dma_mask) if (WARN_ON(!sysdev->dma_mask))
/* Platform did not initialize dma_mask */ /* Platform did not initialize dma_mask */
ret = dma_coerce_mask_and_coherent(&pdev->dev, ret = dma_coerce_mask_and_coherent(sysdev,
DMA_BIT_MASK(64)); DMA_BIT_MASK(64));
else else
ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64)); ret = dma_set_mask_and_coherent(sysdev, DMA_BIT_MASK(64));
/* If seting 64-bit DMA mask fails, fall back to 32-bit DMA mask */ /* If seting 64-bit DMA mask fails, fall back to 32-bit DMA mask */
if (ret) { if (ret) {
ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32)); ret = dma_set_mask_and_coherent(sysdev, DMA_BIT_MASK(32));
if (ret) if (ret)
return ret; return ret;
} }
hcd = usb_create_hcd(driver, &pdev->dev, dev_name(&pdev->dev)); hcd = __usb_create_hcd(driver, sysdev, &pdev->dev,
dev_name(&pdev->dev), NULL);
if (!hcd) if (!hcd)
return -ENOMEM; return -ENOMEM;
...@@ -222,20 +241,20 @@ static int xhci_plat_probe(struct platform_device *pdev) ...@@ -222,20 +241,20 @@ static int xhci_plat_probe(struct platform_device *pdev)
xhci->clk = clk; xhci->clk = clk;
xhci->main_hcd = hcd; xhci->main_hcd = hcd;
xhci->shared_hcd = usb_create_shared_hcd(driver, &pdev->dev, xhci->shared_hcd = __usb_create_hcd(driver, sysdev, &pdev->dev,
dev_name(&pdev->dev), hcd); dev_name(&pdev->dev), hcd);
if (!xhci->shared_hcd) { if (!xhci->shared_hcd) {
ret = -ENOMEM; ret = -ENOMEM;
goto disable_clk; goto disable_clk;
} }
if (device_property_read_bool(&pdev->dev, "usb3-lpm-capable")) if (device_property_read_bool(sysdev, "usb3-lpm-capable"))
xhci->quirks |= XHCI_LPM_SUPPORT; xhci->quirks |= XHCI_LPM_SUPPORT;
if (device_property_read_bool(&pdev->dev, "quirk-broken-port-ped")) if (device_property_read_bool(&pdev->dev, "quirk-broken-port-ped"))
xhci->quirks |= XHCI_BROKEN_PORT_PED; xhci->quirks |= XHCI_BROKEN_PORT_PED;
hcd->usb_phy = devm_usb_get_phy_by_phandle(&pdev->dev, "usb-phy", 0); hcd->usb_phy = devm_usb_get_phy_by_phandle(sysdev, "usb-phy", 0);
if (IS_ERR(hcd->usb_phy)) { if (IS_ERR(hcd->usb_phy)) {
ret = PTR_ERR(hcd->usb_phy); ret = PTR_ERR(hcd->usb_phy);
if (ret == -EPROBE_DEFER) if (ret == -EPROBE_DEFER)
......
...@@ -237,6 +237,9 @@ static int xhci_free_msi(struct xhci_hcd *xhci) ...@@ -237,6 +237,9 @@ static int xhci_free_msi(struct xhci_hcd *xhci)
static int xhci_setup_msi(struct xhci_hcd *xhci) static int xhci_setup_msi(struct xhci_hcd *xhci)
{ {
int ret; int ret;
/*
* TODO:Check with MSI Soc for sysdev
*/
struct pci_dev *pdev = to_pci_dev(xhci_to_hcd(xhci)->self.controller); struct pci_dev *pdev = to_pci_dev(xhci_to_hcd(xhci)->self.controller);
ret = pci_enable_msi(pdev); ret = pci_enable_msi(pdev);
...@@ -263,7 +266,7 @@ static int xhci_setup_msi(struct xhci_hcd *xhci) ...@@ -263,7 +266,7 @@ static int xhci_setup_msi(struct xhci_hcd *xhci)
*/ */
static void xhci_free_irq(struct xhci_hcd *xhci) static void xhci_free_irq(struct xhci_hcd *xhci)
{ {
struct pci_dev *pdev = to_pci_dev(xhci_to_hcd(xhci)->self.controller); struct pci_dev *pdev = to_pci_dev(xhci_to_hcd(xhci)->self.sysdev);
int ret; int ret;
/* return if using legacy interrupt */ /* return if using legacy interrupt */
...@@ -748,7 +751,7 @@ void xhci_shutdown(struct usb_hcd *hcd) ...@@ -748,7 +751,7 @@ void xhci_shutdown(struct usb_hcd *hcd)
struct xhci_hcd *xhci = hcd_to_xhci(hcd); struct xhci_hcd *xhci = hcd_to_xhci(hcd);
if (xhci->quirks & XHCI_SPURIOUS_REBOOT) if (xhci->quirks & XHCI_SPURIOUS_REBOOT)
usb_disable_xhci_ports(to_pci_dev(hcd->self.controller)); usb_disable_xhci_ports(to_pci_dev(hcd->self.sysdev));
spin_lock_irq(&xhci->lock); spin_lock_irq(&xhci->lock);
xhci_halt(xhci); xhci_halt(xhci);
...@@ -765,7 +768,7 @@ void xhci_shutdown(struct usb_hcd *hcd) ...@@ -765,7 +768,7 @@ void xhci_shutdown(struct usb_hcd *hcd)
/* Yet another workaround for spurious wakeups at shutdown with HSW */ /* Yet another workaround for spurious wakeups at shutdown with HSW */
if (xhci->quirks & XHCI_SPURIOUS_WAKEUP) if (xhci->quirks & XHCI_SPURIOUS_WAKEUP)
pci_set_power_state(to_pci_dev(hcd->self.controller), PCI_D3hot); pci_set_power_state(to_pci_dev(hcd->self.sysdev), PCI_D3hot);
} }
#ifdef CONFIG_PM #ifdef CONFIG_PM
...@@ -4801,7 +4804,11 @@ int xhci_get_frame(struct usb_hcd *hcd) ...@@ -4801,7 +4804,11 @@ int xhci_get_frame(struct usb_hcd *hcd)
int xhci_gen_setup(struct usb_hcd *hcd, xhci_get_quirks_t get_quirks) int xhci_gen_setup(struct usb_hcd *hcd, xhci_get_quirks_t get_quirks)
{ {
struct xhci_hcd *xhci; struct xhci_hcd *xhci;
struct device *dev = hcd->self.controller; /*
* TODO: Check with DWC3 clients for sysdev according to
* quirks
*/
struct device *dev = hcd->self.sysdev;
int retval; int retval;
/* Accept arbitrarily long scatter-gather lists */ /* Accept arbitrarily long scatter-gather lists */
......
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