linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Samuel Ortiz <sameo@linux.intel.com>
To: Igor Grinberg <grinberg@compulab.co.il>
Cc: balbi@ti.com, Keshava Munegowda <keshava_mgowda@ti.com>,
	linux-omap@vger.kernel.org, linux-usb@vger.kernel.org,
	parthab@india.ti.com, govindraj.raja@ti.com
Subject: Re: [PATCH RESEND] ARM: OMAP3: USB: Fix the EHCI ULPI PHY reset issue
Date: Tue, 27 Mar 2012 16:43:49 +0200	[thread overview]
Message-ID: <20120327144349.GH17616@sortiz-mobl> (raw)
In-Reply-To: <4F71CC49.4080903@compulab.co.il>

Hi Igor,

On Tue, Mar 27, 2012 at 04:18:49PM +0200, Igor Grinberg wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Hi Felipe, Samuel,
> 
> On 03/20/12 17:55, Felipe Balbi wrote:
> > On Tue, Mar 20, 2012 at 04:53:53PM +0100, Samuel Ortiz wrote:
> >> Hi Keshava,
> >>
> >> On Mon, Mar 19, 2012 at 12:12:47PM +0530, Keshava Munegowda wrote:
> >>> From: Keshava Munegowda <Keshava_mgowda@ti.com>
> >>>
> >>> It is observed that the echi ports of 3430 sdp board
> >>> are not working due to the random timing of programming
> >>> the associated GPIOs of the ULPI PHYs of the EHCI for reset.
> >>> If the PHYs are reset at during usbhs core driver, host ports will
> >>> not work because EHCI driver is loaded after the resetting PHYs.
> >>> The PHYs should be in reset state while initializing the EHCI
> >>> controller.
> >>> The code which does the GPIO pins associated with the PHYs
> >>> are programmed to reset is moved from the USB host core driver
> >>> to EHCI driver.
> >>>
> >>> Signed-off-by: Keshava Munegowda <keshava_mgowda@ti.com>
> >>> Reviewed-by: Partha Basak <parthab@india.ti.com>
> >> Felipe, are you ok with that patch ?
> >> I'll most likely queue it after this merge window is closed though.
> 
> I've just sent a patch that depends on this one (ehci-omap.c part),
> though it needs a review, but Samuel, you still haven't applied/pushed
> the patch to your git.kernel.org, so can this one go through Felipe's
> tree to minimize the dependencies/conflicts?
Once (and if) Linus pulls from my for-next branch, I'll start taking patches
again. It's a matter of days, so you can either wait for me to apply this
patch or have Felipe taking both.
I'm fine either ways.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/

  reply	other threads:[~2012-03-27 14:35 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-19  6:42 [PATCH RESEND] ARM: OMAP3: USB: Fix the EHCI ULPI PHY reset issue Keshava Munegowda
2012-03-19 13:34 ` Raja, Govindraj
2012-03-19 14:43   ` Munegowda, Keshava
2012-05-02  8:47   ` Russ Dill
2012-05-02  8:49     ` Koen Kooi
2012-05-02 10:38     ` Raja, Govindraj
2012-05-03  1:25       ` Russ Dill
     [not found]         ` <CA+Bv8XYFxL8QhCvu2xLz=hsozjtzAVnZXUZvQsLBr9yuXYrfEQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-05-03  7:16           ` Raja, Govindraj
     [not found]       ` <CAMrsUdL18b2Ox3CkzuNjQQW1JT1+-SAso67EkrwE-BTJbavYPA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-05-02 10:50         ` Koen Kooi
     [not found]           ` <A53CAEEE-E425-46B7-8E7B-B4AFB163ECAC-QLwJDigV5abLmq1fohREcCpxlwaOVQ5f@public.gmane.org>
2012-05-02 11:22             ` Raja, Govindraj
2012-05-03 19:08         ` Russ Dill
     [not found]           ` <CA+Bv8XYodOf_TKR6__aP=p8L7ruKhn6imWnOEbN+3UsBWPmoiw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-05-04  6:03             ` Igor Grinberg
2012-05-04  9:06               ` Russ Dill
2012-05-04  9:33                 ` Igor Grinberg
2012-03-20 15:53 ` Samuel Ortiz
2012-03-20 15:55   ` Felipe Balbi
2012-03-21  6:34     ` Munegowda, Keshava
2012-03-27 14:18     ` Igor Grinberg
2012-03-27 14:43       ` Samuel Ortiz [this message]
     [not found] ` <1332139367-1302-1-git-send-email-keshava_mgowda-l0cyMroinI0@public.gmane.org>
2012-03-27 15:10   ` Igor Grinberg
2012-04-11 11:45     ` Munegowda, Keshava
     [not found]       ` <CAP05o4JWjoh-NoDeA3vYaS1yY_SGdm3TZvHA=pkQsP-Xo-kWnQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-04-16 16:46         ` Samuel Ortiz
2012-04-17  6:30           ` Munegowda, Keshava
2012-04-22  6:36           ` Igor Grinberg

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=20120327144349.GH17616@sortiz-mobl \
    --to=sameo@linux.intel.com \
    --cc=balbi@ti.com \
    --cc=govindraj.raja@ti.com \
    --cc=grinberg@compulab.co.il \
    --cc=keshava_mgowda@ti.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=parthab@india.ti.com \
    /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).