From: Igor Grinberg <grinberg-UTxiZqZC01RS1MOuV/RT9w@public.gmane.org>
To: Russ Dill <Russ.Dill-l0cyMroinI0@public.gmane.org>
Cc: "Raja, Govindraj" <govindraj.raja-l0cyMroinI0@public.gmane.org>,
Keshava Munegowda <keshava_mgowda-l0cyMroinI0@public.gmane.org>,
linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
balbi-l0cyMroinI0@public.gmane.org,
sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org,
parthab-PpE0FKYn9XJWk0Htik3J/w@public.gmane.org
Subject: Re: [PATCH RESEND] ARM: OMAP3: USB: Fix the EHCI ULPI PHY reset issue
Date: Fri, 04 May 2012 09:03:02 +0300 [thread overview]
Message-ID: <4FA37116.4020608@compulab.co.il> (raw)
In-Reply-To: <CA+Bv8XYodOf_TKR6__aP=p8L7ruKhn6imWnOEbN+3UsBWPmoiw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
Hi Russ,
On 05/03/12 22:08, Russ Dill wrote:
> On Wed, May 2, 2012 at 3:38 AM, Raja, Govindraj <govindraj.raja-l0cyMroinI0@public.gmane.org> wrote:
>> On Wed, May 2, 2012 at 2:17 PM, Russ Dill <Russ.Dill-l0cyMroinI0@public.gmane.org> wrote:
>>> On Mon, Mar 19, 2012 at 6:34 AM, Raja, Govindraj <govindraj.raja-l0cyMroinI0@public.gmane.org> wrote:
>>>> On Mon, Mar 19, 2012 at 12:12 PM, Keshava Munegowda
>>>> <keshava_mgowda-l0cyMroinI0@public.gmane.org> wrote:
>>>>> From: Keshava Munegowda <Keshava_mgowda-l0cyMroinI0@public.gmane.org>
>>>>>
>>>>> 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.
>>>>
>>>> I tested on beagle xm where gpio nreset is requested from
>>>> board file.
>>>> (Basic enumertaion after gpio nreset seems to work fine,
>>>> Hub and smsc lan chip get detected afetr boot up)
>>>>
>>>
>>> What base did you test this on top of? my xM is failing USB-wise when
>>> I apply this on v3.3.3 (with the UART mux fix patch) and where it is
>>> applied within master (3.4-rc4, again, with the UART mux fix patch).
>>> Additionally, reverting this patch from 3.4-rc5 causes rc5 to work
>>> properly.
>>>
>>
>> Works for me be on 3.4-rc5 Beagle-XM even without reverting the patch
>>
>> Logs as in here [1].
>>
>> --
>> Thanks,
>> Govindraj.R
>>
>> [1]:
>> http://pastebin.pandaboard.org/index.php/view/20343533
>
> I've tracked down the difference. I'm loading my uEnv.txt and uImage
> in u-boot from the network which means initializing USB. You are
> loading straight from MMC. If I switch to loading straight from MMC
> everything works.
>
> Can everyone do a 'usb start' in u-boot before booting and re-test?
> I'm pretty sure this is a regression, but the bug could be a strange
> u-boot/kernel interaction.
Probably, kernel code does not reinitialize EHCI correctly if it was already enabled?
Can you try the below sequence:
usb start
usb stop
boot Linux
?
--
Regards,
Igor.
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2012-05-04 6:03 UTC|newest]
Thread overview: 25+ 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
[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 [this message]
2012-05-04 9:06 ` Russ Dill
2012-05-04 9:33 ` Igor Grinberg
2012-05-04 9:33 ` [U-Boot] " Igor Grinberg
2012-05-03 1:25 ` Russ Dill
[not found] ` <CA+Bv8XYFxL8QhCvu2xLz=hsozjtzAVnZXUZvQsLBr9yuXYrfEQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-05-03 7:16 ` Raja, Govindraj
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
[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=4FA37116.4020608@compulab.co.il \
--to=grinberg-utxizqzc01rs1mouv/rt9w@public.gmane.org \
--cc=Russ.Dill-l0cyMroinI0@public.gmane.org \
--cc=balbi-l0cyMroinI0@public.gmane.org \
--cc=govindraj.raja-l0cyMroinI0@public.gmane.org \
--cc=keshava_mgowda-l0cyMroinI0@public.gmane.org \
--cc=linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=parthab-PpE0FKYn9XJWk0Htik3J/w@public.gmane.org \
--cc=sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.