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-----
next prev 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).