Re: [PATCH RESEND] ARM: OMAP3: USB: Fix the EHCI ULPI PHY reset issue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxx>
> >>>
> >>> 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@xxxxxx>
> >>> Reviewed-by: Partha Basak <parthab@xxxxxxxxxxxx>
> >> 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/
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux