From: Felipe Balbi <balbi@kernel.org>
To: Roger Quadros <rogerq@ti.com>
Cc: vivek.gautam@codeaurora.org, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org, Roger Quadros <rogerq@ti.com>
Subject: Re: [PATCH v2 4/4] usb: dwc3: Workaround for super-speed host on dra7 in dual-role mode
Date: Tue, 28 Mar 2017 14:10:48 +0300 [thread overview]
Message-ID: <87o9wlhd1j.fsf@linux.intel.com> (raw)
In-Reply-To: <1487250377-13653-5-git-send-email-rogerq@ti.com>
[-- Attachment #1: Type: text/plain, Size: 638 bytes --]
Hi,
Roger Quadros <rogerq@ti.com> writes:
> dra7 OTG core limits the host controller to USB2.0 (high-speed) mode
> when we're operating in dual-role.
yeah, that's not a quirk. DRA7 supports OTGv2, not OTGv3. There was no
USB3 when OTGv2 was written.
DRA7 just shouldn't use OTG core altogether. In fact, this is the very
thing I've been saying for a long time. Make the simplest implementation
possible. The dead simple, does-one-thing-only sort of implementation.
All we need for Dual-Role (without OTG extras) is some input for ID and
VBUS, then we add/remove HCD/UDC conditionally and set PRTCAPDIR.
--
balbi
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]
next prev parent reply other threads:[~2017-03-28 11:11 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-16 13:06 [PATCH v2 0/4] usb: dwc3: dual-role support Roger Quadros
2017-02-16 13:06 ` [PATCH v2 1/4] usb: dwc3: core.h: add some register definitions Roger Quadros
2017-02-16 13:06 ` [PATCH v2 2/4] usb: dwc3: omap: don't miss events during suspend/resume Roger Quadros
2017-02-16 13:06 ` [PATCH v2 3/4] usb: dwc3: add dual-role support Roger Quadros
2017-03-28 11:07 ` Felipe Balbi
2017-03-29 11:33 ` Roger Quadros
2017-03-29 13:15 ` Felipe Balbi
2017-03-30 6:40 ` Roger Quadros
2017-03-30 9:27 ` Felipe Balbi
2017-04-03 5:31 ` John Youn
2017-02-16 13:06 ` [PATCH v2 4/4] usb: dwc3: Workaround for super-speed host on dra7 in dual-role mode Roger Quadros
2017-02-23 8:34 ` Vivek Gautam
2017-02-24 0:57 ` Peter Chen
2017-02-24 3:08 ` Vivek Gautam
2017-02-24 12:02 ` Roger Quadros
2017-02-25 3:46 ` Chanwoo Choi
2017-02-25 3:50 ` Chanwoo Choi
2017-02-28 13:54 ` Vivek Gautam
2017-02-25 3:35 ` Chanwoo Choi
2017-02-28 15:17 ` Roger Quadros
2017-03-28 11:10 ` Felipe Balbi [this message]
2017-03-29 9:57 ` Roger Quadros
2017-03-29 10:32 ` Felipe Balbi
2017-03-29 12:00 ` Roger Quadros
2017-03-29 13:21 ` Felipe Balbi
2017-03-29 13:58 ` Roger Quadros
2017-03-30 9:32 ` Felipe Balbi
2017-03-30 10:11 ` Roger Quadros
2017-03-31 7:43 ` Roger Quadros
2017-03-31 7:46 ` Felipe Balbi
2017-03-31 11:50 ` Roger Quadros
2017-03-31 12:00 ` Felipe Balbi
2017-03-31 12:21 ` Roger Quadros
2017-03-31 12:58 ` Felipe Balbi
2017-03-13 8:33 ` [PATCH v2 0/4] usb: dwc3: dual-role support Roger Quadros
2017-03-28 10:27 ` Felipe Balbi
2017-03-29 9:50 ` Roger Quadros
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=87o9wlhd1j.fsf@linux.intel.com \
--to=balbi@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=rogerq@ti.com \
--cc=vivek.gautam@codeaurora.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).