From: peter.chen@freescale.com (Peter Chen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 3/7] usb: phy: mxs: Add VF610 USB PHY support
Date: Sat, 2 Aug 2014 08:36:16 +0800 [thread overview]
Message-ID: <20140802003615.GA2833@peterchendt> (raw)
In-Reply-To: <acb7c055ded38c74291d0f0d8d2fdca54a8b95f8.1406558450.git.stefan@agner.ch>
On Mon, Jul 28, 2014 at 04:57:29PM +0200, Stefan Agner wrote:
> This adds support for the USB PHY in Vybrid VF610. We assume that
> the disconnection without VBUS is also needed for Vybrid.
>
> Tests showed, without MXS_PHY_NEED_IP_FIX, enumeration of devices
> behind a USB Hub fails with errors:
>
> [ 215.163507] usb usb1-port1: cannot reset (err = -32)
> [ 215.170498] usb usb1-port1: cannot reset (err = -32)
> [ 215.185120] usb usb1-port1: cannot reset (err = -32)
> [ 215.191345] usb usb1-port1: cannot reset (err = -32)
> [ 215.202487] usb usb1-port1: cannot reset (err = -32)
> [ 215.207718] usb usb1-port1: Cannot enable. Maybe the USB cable is bad?
> [ 215.219317] usb usb1-port1: unable to enumerate USB device
>
> Hence we also enable the MXS_PHY_NEED_IP_FIX flag.
>
> Signed-off-by: Stefan Agner <stefan@agner.ch>
> ---
> Documentation/devicetree/bindings/usb/mxs-phy.txt | 1 +
> drivers/usb/phy/phy-mxs-usb.c | 6 ++++++
> 2 files changed, 7 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/usb/mxs-phy.txt b/Documentation/devicetree/bindings/usb/mxs-phy.txt
> index cef181a..fe3eed8 100644
> --- a/Documentation/devicetree/bindings/usb/mxs-phy.txt
> +++ b/Documentation/devicetree/bindings/usb/mxs-phy.txt
> @@ -5,6 +5,7 @@ Required properties:
> * "fsl,imx23-usbphy" for imx23 and imx28
> * "fsl,imx6q-usbphy" for imx6dq and imx6dl
> * "fsl,imx6sl-usbphy" for imx6sl
> + * "fsl,vf610-usbphy" for Vybrid vf610
> "fsl,imx23-usbphy" is still a fallback for other strings
> - reg: Should contain registers location and length
> - interrupts: Should contain phy interrupt
> diff --git a/drivers/usb/phy/phy-mxs-usb.c b/drivers/usb/phy/phy-mxs-usb.c
> index c42bdf0..8c2f23b 100644
> --- a/drivers/usb/phy/phy-mxs-usb.c
> +++ b/drivers/usb/phy/phy-mxs-usb.c
> @@ -125,10 +125,16 @@ static const struct mxs_phy_data imx6sl_phy_data = {
> MXS_PHY_NEED_IP_FIX,
> };
>
> +static const struct mxs_phy_data vf610_phy_data = {
> + .flags = MXS_PHY_DISCONNECT_LINE_WITHOUT_VBUS |
> + MXS_PHY_NEED_IP_FIX,
> +};
> +
> static const struct of_device_id mxs_phy_dt_ids[] = {
> { .compatible = "fsl,imx6sl-usbphy", .data = &imx6sl_phy_data, },
> { .compatible = "fsl,imx6q-usbphy", .data = &imx6q_phy_data, },
> { .compatible = "fsl,imx23-usbphy", .data = &imx23_phy_data, },
> + { .compatible = "fsl,vf610-usbphy", .data = &vf610_phy_data, },
> { /* sentinel */ }
> };
> MODULE_DEVICE_TABLE(of, mxs_phy_dt_ids);
> --
> 2.0.2
>
Acked-by: Peter Chen <peter.chen@freescale.com>
--
Best Regards,
Peter Chen
next prev parent reply other threads:[~2014-08-02 0:36 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-28 14:57 [PATCH v2 0/7] vf610: Add USB support Stefan Agner
2014-07-28 14:57 ` [PATCH v2 1/7] ARM: dts: vf610: Add USB PHY and controller Stefan Agner
2014-07-28 14:57 ` [PATCH v2 2/7] ARM: imx: clk-vf610: Add USBPHY clocks Stefan Agner
2014-07-28 14:57 ` [PATCH v2 3/7] usb: phy: mxs: Add VF610 USB PHY support Stefan Agner
2014-08-02 0:36 ` Peter Chen [this message]
2014-08-02 14:23 ` Stefan Agner
2014-07-28 14:57 ` [PATCH v2 4/7] ARM: dts: vf610: Add usbmisc for non-core registers Stefan Agner
2014-07-28 14:57 ` [PATCH v2 5/7] chipidea: usbmisc_imx: Add USB support for VF610 SoCs Stefan Agner
2014-08-02 1:08 ` Peter Chen
2014-07-28 14:57 ` [PATCH v2 6/7] ARM: dts: vf610-colibri: Add USB support Stefan Agner
2014-07-28 14:57 ` [PATCH v2 7/7] ARM: dts: vf610-twr: " Stefan Agner
2014-07-29 9:22 ` [PATCH v2 0/7] vf610: " Shawn Guo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140802003615.GA2833@peterchendt \
--to=peter.chen@freescale.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).