linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Igor Grinberg <grinberg@compulab.co.il>
To: balbi@ti.com
Cc: Samuel Ortiz <sameo@linux.intel.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:18:49 +0200	[thread overview]
Message-ID: <4F71CC49.4080903@compulab.co.il> (raw)
In-Reply-To: <20120320155541.GF32693@arwen.pp.htv.fi>

-----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?

What do you think, Felipe?

> 
> my bad, here's my Ack:
> 
> Acked-by: Felipe Balbi <balbi@ti.com>
> 

- -- 
Regards,
Igor.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJPccxIAAoJEBDE8YO64Efay3EP/ji+LEF3rDCDkqpZ9ELYcsWW
My+4EozWSXRHEpxFk/xb9V/3+inX9cczaRR3DRFQLn8UxxTtn0VJSVmTtFB4Uadc
vsrlBU9gi6DXYX1Jpu3vRUvLnQcfBzlXCnLTVW3fgsYyztNHcHQj7F87t/Kug5V1
UFV21DNzePYlta3+UZly8q0anqvEqlbTY0j81pWhpL5i0T/8iGAhWdTOppbgT+oH
FZ7W+YUk1TJtKHwFWABXfAIBwg/NHaIYPKDiv8z3wIFnT2Lk5IdZagIZfwYExtUE
spX2DEMtti8yKwfEJW2rQHLF8/YCWxKc9oiYegVWniNx7WBYje2wEKP1lT4eYJbe
iCPzhPu63wipvaw2yaVYFdJ6fCHZ4s8ZkE2fWhuGgSsnD/4ljAtl0i+eeVhu6fdi
kQF6E3uNOVOSh3iI1GuMUjolOVDAQiRZdQGDATstLu38sC2NVPfmxW6z6doSQcn5
TeiCkIiZaC/b+6gvglBbbWuxcNUJQCJ+2DFVweBEDmmO2WKgpgo6GGhE0H4uMxjs
1wSP6EABz6O2ihjq1YlBvXMvhP2YWO68u/iBVty1cIt5u9mOFnavpNMPLj9c02ew
AE8TM9LLS5zOYQtEYU3q0WiGQEhaSa5f87pEdQYDH0i98FH9prRRxl3Zh0IOhHa8
Skz7J4AVgxdEci8Op1Uc
=oe1q
-----END PGP SIGNATURE-----

  parent reply	other threads:[~2012-03-27 14:19 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 [this message]
2012-03-27 14:43       ` Samuel Ortiz
     [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=4F71CC49.4080903@compulab.co.il \
    --to=grinberg@compulab.co.il \
    --cc=balbi@ti.com \
    --cc=govindraj.raja@ti.com \
    --cc=keshava_mgowda@ti.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=parthab@india.ti.com \
    --cc=sameo@linux.intel.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).