linux-toradex.git/drivers/usb/chipidea, branch v4.11

usb: chipidea: msm: Fix return value check in ci_hdrc_msm_probe()

2017-02-03T08:48:47+00:00

In case of error, the function devm_ioremap_resource() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should
be replaced with IS_ERR().

Fixes: 2fc305be364e ("usb: chipidea: msm: Mux over secondary phy at the
right time")
Signed-off-by: Wei Yongjun 
Reviewed-by: Stephen Boyd 
Signed-off-by: Peter Chen

usb: chipidea: Configure phy for appropriate mode

2017-02-03T08:48:33+00:00

When the qcom chipidea controller is used with an extcon, we need
to signal device mode or host mode to the phy so it can configure
itself for the correct mode. This should be done after the phy is
powered up, so that the register writes work correctly. Add in
the appropriate phy_set_mode() call here.

Cc: Peter Chen 
Cc: Greg Kroah-Hartman 
Signed-off-by: Stephen Boyd 
Signed-off-by: Peter Chen

usb: chipidea: usb2: delete the redundant setting default DMA mask code

2017-01-23T09:02:39+00:00

Similar as commit 2b2fe36def08 ("usb: chipidea: imx: delete the
redundant setting default DMA mask code"), the ci_hdrc_usb2 platform
device is also created by device tree, the default DMA mask should be
already set by of_dma_configure when the device are created. So delete
the redundant code at driver.

Signed-off-by: Jisheng Zhang 
Signed-off-by: Peter Chen

usb: chipidea: delete an useless header include

2017-01-23T09:02:36+00:00

 is for net phy drivers, we don't need it.

Signed-off-by: Jisheng Zhang 
Signed-off-by: Peter Chen

usb: chipidea: Drop lock across event_notify during gadget stop

2017-01-20T07:27:43+00:00

The CI_HDRC_CONTROLLER_STOPPED_EVENT may want to call sleeping
APIs similar to how _gadget_stop_activity() may. Let's drop the
lock across the event so that glue drivers can make sleeping
calls.

Cc: Peter Chen 
Cc: Greg Kroah-Hartman 
Signed-off-by: Stephen Boyd 
Signed-off-by: Peter Chen

usb: chipidea: msm: Be silent on probe defer errors

2017-01-20T07:27:38+00:00

If something fails in ci_hdrc_add_device() due to probe defer, we
shouldn't print an error message. Be silent in this case as we'll
try probe again later.

Acked-by: Peter Chen 
Cc: Greg Kroah-Hartman 
Signed-off-by: Stephen Boyd 
Signed-off-by: Peter Chen

usb: chipidea: msm: Handle phy power states

2017-01-20T07:27:35+00:00

The ULPI phy on qcom platforms needs to be initialized and
powered on after a USB reset and before we toggle the run/stop
bit. Otherwise, the phy locks up and doesn't work properly. Hook
the phy initialization into the RESET event and the phy power off
into the STOPPED event.

Acked-by: Peter Chen 
Cc: Greg Kroah-Hartman 
Signed-off-by: Stephen Boyd 
Signed-off-by: Peter Chen

usb: chipidea: msm: Add reset controller for PHY POR bit

2017-01-20T07:26:47+00:00

The MSM chipidea wrapper has two bits that are used to reset the
first or second phy. Add support for these bits via the reset
controller framework, so that phy drivers can reset their
hardware at the right time during initialization.

Acked-by: Peter Chen 
Cc: Greg Kroah-Hartman 
Signed-off-by: Stephen Boyd 
Signed-off-by: Peter Chen

usb: chipidea: msm: Make platform data driver local instead of global

2017-01-20T07:26:42+00:00

If two devices are probed with this same driver, they'll share
the same platform data structure, while the chipidea core layer
writes and modifies it. This can lead to interesting results
especially if one device is an OTG type chipidea controller and
another is a host. Let's create a copy of this structure per each
device instance so that odd things don't happen.

Acked-by: Peter Chen 
Cc: Greg Kroah-Hartman 
Signed-off-by: Stephen Boyd 
Signed-off-by: Peter Chen

usb: chipidea: msm: Restore wrapper settings after reset

2017-01-20T07:26:39+00:00

When the RESET bit is set in the USBCMD register it resets quite
a few of the wrapper's registers to their reset state. This
includes the GENCONFIG and GENCONFIG2 registers. Currently this
is done by the usb phy and ehci-msm drivers writing into the
controller wrapper's MMIO address space. Let's consolidate the
register writes into the wrapper driver instead so that we
clearly split the wrapper from the phys.

Acked-by: Peter Chen 
Cc: Greg Kroah-Hartman 
Signed-off-by: Stephen Boyd 
Signed-off-by: Peter Chen