linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: maxime.ripard@free-electrons.com (maxime.ripard at free-electrons.com)
To: linux-arm-kernel@lists.infradead.org
Subject: Chipidea usb otg support for IMX/MXS (device functionality)
Date: Fri, 7 Jun 2013 11:37:46 +0200	[thread overview]
Message-ID: <20130607093746.GL14209@lukather> (raw)
In-Reply-To: <51A77BAB.4060101@digi.com>

Hi Hector,

On Thu, May 30, 2013 at 06:17:47PM +0200, Hector Palacios wrote:
> Dear Maxime,
> 
> On 05/29/2013 09:50 AM, maxime.ripard at free-electrons.com wrote:
> >Hi,
> >
> >On Wed, May 29, 2013 at 07:11:30AM +0000, Chen Peter-B29397 wrote:
> >>
> >>>Hello,
> >>>
> >>>Am I right in assuming that the MXS USB on-the-go port does not currently
> >>>support the
> >>>device (gadget) functionality?
> >>>Anybody out there working on that?
> >>>
> >>
> >>As far as I know, Maxime Ripard may already let the chipidea durl-role function
> >>work ok at mx28. It may need my chipidea otg patch
> >>
> >>https://github.com/hzpeterchen/linux-usb.git
> >
> >Indeed, I've been using the patchset "Add tested id switch and vbus
> >connect detect support for Chipidea" from Peter for quite some time on
> >top of 3.9 and it works like a charm for the gadget mode on an MX28
> >platform.
> >
> >BTW, Peter, I've seen that these patches are still not merged in 3.10,
> >is there a reason for that? do you plan on sending a version rebased on
> >top of 3.10 some time in the future? I tried to do the rebasing myself,
> >but the chipidea driver seems to have changed quite heavily, which makes
> >the process quite difficult when you don't know what you're doing :)
> 
> I guess you didn't get rid of the 'possible circular locking
> dependency' you talked about at [1], right?

Indeed

> I experimented the same and also a BUG [2] after cable reconnection.
> Despite those, I ran a simple test of serial, ethernet, and
> mass_storage gadgets and they worked fine.

I never experienced the BUG you're mentionning, everything was working
fine except the circular dependency.

> I didn't use the new properties (phy_type, dr_mode...) in the DT of
> my mx28 platform, did you?

Neither did I.

Maxime

      reply	other threads:[~2013-06-07  9:37 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <51A5A808.2070803@digi.com>
     [not found] ` <F281D0F91ED19E4D8E63A7504E8A649803E8A3F3@039-SN2MPN1-021.039d.mgd.msft.net>
2013-05-29  7:50   ` Chipidea usb otg support for IMX/MXS (device functionality) maxime.ripard at free-electrons.com
2013-05-29  8:07     ` Chen Peter-B29397
2013-05-29 10:13       ` Michael Grzeschik
2013-05-29 12:19         ` maxime.ripard at free-electrons.com
2013-05-29 12:41           ` Michael Grzeschik
2013-05-30 13:53             ` maxime.ripard at free-electrons.com
2013-06-21 11:26         ` Marek Vasut
2013-06-21 12:22           ` Michael Grzeschik
2013-06-21 12:56             ` Marek Vasut
2013-06-22  2:33               ` Marek Vasut
2013-06-22  8:28                 ` Chen Peter-B29397
2013-06-22 13:29                   ` Marek Vasut
2013-06-24  1:37                     ` Chen Peter-B29397
2013-06-24  7:08                       ` Shawn Guo
2013-06-24 13:51                       ` Marek Vasut
2013-06-25  1:48                         ` Chen Peter-B29397
2013-06-25 18:40                           ` Marek Vasut
2013-06-27 13:49                           ` Marek Vasut
2013-06-28  6:48                             ` Chen Peter-B29397
2013-06-28  9:42                               ` Marek Vasut
2013-06-28 10:46                                 ` Chen Peter-B29397
2013-06-28 11:56                                   ` Marek Vasut
2013-06-28 13:04                                     ` Chen Peter-B29397
2013-06-28 13:11                                       ` Fabio Estevam
2013-07-01  2:06                                         ` Chen Peter-B29397
2013-07-07  1:10                                   ` Marek Vasut
2013-07-08  1:21                                     ` Chen Peter-B29397
2013-07-08 20:56                                       ` Marek Vasut
2013-07-09  1:05                                         ` Chen Peter-B29397
2013-07-09 10:38                                           ` Marek Vasut
2013-06-25 14:23                         ` Fabio Estevam
2013-06-25 15:39                           ` Fabio Estevam
2013-06-25 18:42                             ` Marek Vasut
2013-07-09 15:07                       ` Marek Vasut
2013-05-30 16:17     ` Hector Palacios
2013-06-07  9:37       ` maxime.ripard at free-electrons.com [this message]

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=20130607093746.GL14209@lukather \
    --to=maxime.ripard@free-electrons.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).