Re: [PATCH] OMAP: USB : Fix the EHCI enumeration and core retention issue

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

 



On Thu, Jul 12, 2012 at 12:11 PM, Munegowda, Keshava
<keshava_mgowda@xxxxxx> wrote:
> On Wed, Jul 11, 2012 at 7:53 PM, Kevin Hilman <khilman@xxxxxx> wrote:
>> "Munegowda, Keshava" <keshava_mgowda@xxxxxx> writes:
>>
>>> On Wed, Jul 11, 2012 at 3:59 PM, Samuel Ortiz <sameo@xxxxxxxxxxxxxxx> wrote:
>>>> Hi Keshava, Kevin,
>>>>
>>>> On Fri, Jul 06, 2012 at 05:29:00PM +0530, Munegowda, Keshava wrote:
>>>>> Samuel
>>>>>           I have sent that patch to disable the ehci in
>>>>> omap2plus_defconfig; after merging that
>>>>> please merge this patch too. This will fix the crashes in during boot
>>>>> with NFS in beagleXM
>>>> I'm going to apply and push this patch for 3.5, and the defconfig patch can be
>>>> pushed through Tony's tree.
>>>> Kevin, could you please ACK it ?
>>>>
>>>> Cheers,
>>>> Samuel.
>>>>
>>>
>>> Thanks Samuel
>>>
>>> Kevin,
>>>     need your ack for this.
>>
>> You never answered earlier questions from myself or Russ Dill about the
>> more targetted patches from Russ:
>>
>>    ARM: OMAP: USB: Fixup ehci_hcd_omap_probe error path
>>    Fix OMAP EHCI suspend/resume failure (i693) '354ab856' causes
>>
>> Also, your current $SUBJECT patch is large and not well
>> described. e.g. what is "not correct" and why.  Why does it fix the
>> problems mentioned?  The original changelog mentions the "core retention
>> issue" but this patch does nothing to address that.  If you want an Ack
>> from me, especially because I'm not an expert in this IP, you'll have to
>> describe things in a way that I can understand.
>>
>> IMO, at this point of the dev cycle (trying to stabilize v3.5), the full
>> cleanup/fix of this feature will need to be done for v3.6.  For v3.5, I
>> think the two patches from Russ Dill should be merged.   They are
>> targetted fixes and very well described.
>>
>> Kevin
>>
>
>
> Hi Kevin
>         The usb2 host of omap3/4/5 silicons has the following ips
>
> 1. UHH   (  /drivers/mfd/omap-usb-host.c ) -- platform driver
> 2. TLL    ( /drivers/mfd/omap-usb-host.c )
> 3. ehci    ( /drivers/usb/host/ehci-omap.c)  - platform driver
> 4. ohci ( /drivers/usb/host/ohci-omap3.c ) - platform drivers
>
> The 3 platform drivers exists to make the ehci/ohci functional.
>
> The  UHH-TLL or usb host core driver is the parent platform driver of
> ehci and ohci.
> This parent driver doe the clock enable/disable which common for both
> ehci and ohci.
> takes care of common port setting and clocks during suspend and resume
> and ensures
> that there is no overwrites by ehci and ohci platform drivers.
>
> The commit id  354ab8567ae3107a8cbe7228c3181990ba598aac titled
> "Fix OMAP EHCI suspend/resume failure (i693)" was handling the clocks in
> the ehci driver it self, instead it should be handled by usb host core
> driver  as per above
> explanation. so, the UHH-TLL Driver should handle the changes done by
> 354ab8567ae3107a8cbe7228c3181990ba598aac.
>
>
> hence this patch removes the changed done by the commit id
> 354ab8567ae3107a8cbe7228c3181990ba598aac
>
> suppose if this patch is not included, then it will cause the
> following two problems
>
> 1.  crash during the system boot
>                        - observed in beagle xm , with NFS file system
>                                            the Ethernet is through
> ehci driver , since the ehci ports clocks are not handled properly
>                          by this commit id
> 354ab8567ae3107a8cbe7228c3181990ba598aac,  it leads to crash
>
> 2. crash during suspend/resume
>                   - observed in beagle xm with ram fs
>                                      if the ehci is driver is included
> and if it tries to suspend it leads to crash
>
> regards
> keshava


hi Felipe
        I request you to ack this patch;
this will enable the boot issue in beagle xm with NFS.
I will rework the patch with commit id
354ab8567ae3107a8cbe7228c3181990ba598aac by Anand gadiyar
after the TLL driver gets merged.

regards
keshava
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux