public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/4] USB multi controller
Date: Wed, 05 Sep 2012 13:11:36 -0600	[thread overview]
Message-ID: <5047A3E8.7070304@wwwdotorg.org> (raw)
In-Reply-To: <201209050830.25141.marex@denx.de>

On 09/05/2012 12:30 AM, Marek Vasut wrote:
> Dear Lucas Stach,
> 
>> Hi Stephen,
>>
>> Am Dienstag, den 04.09.2012, 17:05 -0600 schrieb Stephen Warren:
>>> On 08/30/2012 10:03 AM, Lucas Stach wrote:
>>>> Hi all,
>>>>
>>>> this is a follow up on the patch "USB: EHCI: Initialize multiple
>>>> USB controllers at once" from Jim Lin. It takes some of the
>>>> code but has undergone some heavy reworking.
>>>>
>>>> When we remove the ifdef horror from the above mentioned patch it's
>>>> mostly a big interface change to the usb subsystem. As this creates
>>>> a lot of churn I've split this up into a series. Every patch is self
>>>> contained so it doesn't break compiles and *should* not regress
>>>> any functionality on it's own. At least the series is bisectable in
>>>> case anything goes wrong. I've compile tested all the ARM configs.
>>>>
>>>> Both the lowlevel usb and ehci interface change are backward
>>>> compatible, so implementations that only use one controller can
>>>> choose to ignore the new interface. All implementations are
>>>> updated to work with the new function prototypes.
>>>>
>>>> For Tegra I've included a patch to actually use the new ehci
>>>> interface. Patches are based on a Tegra tree with some relevant
>>>> changes from u-boot-usb picked over, so they should apply to
>>>> u-boot-usb/master.
>>>
>>> Can you explain what this series is based on in a little more detail? I
>>> tried applying it to Tegra's for-next today, and it wouldn't apply. I
>>> managed to apply using plain "patch" rather then "git am", but then I
>>> get a bunch of compile errors:-(
>>
>> At the time I wrote those patches the u-boot-usb and u-boot-tegra trees
>> didn't merge cleanly and I wasted quite some time trying to fix up the
>> failed merge. In the end I ended up just picking the following over to
>> my tegra tree, which should be enough to avoid any conflicts with the
>> usb tree:
>>
>> cdeb916120a4 ehci: cosmetic: Define the number of qt_buffers
>> 44ae0be7461f ehci: Fail for multi-transaction interrupt transfers
>> 14eb79b7a086 ehci: cosmetic: Define used constants
>> 5cec214ecd7d ehci-hcd: Boost transfer speed
>> cffcc5035809 usb_storage: Restore non-EHCI support
>> 4bee5c83ea46 usb_storage: Remove EHCI constraints
>> 3e8581bb9589 usb_stor_BBB_transport: Do not delay when not required
>> db19134615dd ehci: Optimize qTD allocations
> 
> u-boot-usb master is updated to master and pushed for your enjoyment.

The series doesn't appear to apply to u-boot-usb/master, nor to
u-boot-tegra/master plus those cherry-picks listed above. Perhaps you
can rebase on something (although I'm not sure which branch it's meant
to go through) so I can apply/test it?

  reply	other threads:[~2012-09-05 19:11 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-30 16:03 [U-Boot] [PATCH 0/4] USB multi controller Lucas Stach
2012-08-30 16:03 ` [U-Boot] [PATCH 1/4] usb: lowlevel interface change to support multiple controllers Lucas Stach
2012-08-30 21:03   ` Marek Vasut
2012-08-30 16:03 ` [U-Boot] [PATCH 2/4] usb: ehci: rework to take advantage of new lowlevel interface Lucas Stach
2012-08-30 21:09   ` Marek Vasut
2012-08-30 21:51     ` Lucas Stach
2012-08-30 21:55       ` Marek Vasut
2012-08-30 16:03 ` [U-Boot] [PATCH 3/4] tegra20: port to new ehci interface Lucas Stach
2012-08-30 16:03 ` [U-Boot] [PATCH 4/4] usb: add support for multiple usb controllers Lucas Stach
2012-08-30 21:12   ` Marek Vasut
2012-09-04 23:05 ` [U-Boot] [PATCH 0/4] USB multi controller Stephen Warren
2012-09-05  0:30   ` Marek Vasut
2012-09-05  6:21   ` Lucas Stach
2012-09-05  6:30     ` Marek Vasut
2012-09-05 19:11       ` Stephen Warren [this message]
2012-09-05 19:27         ` Marek Vasut
2012-09-05 22:13           ` Lucas Stach
  -- strict thread matches above, loose matches on Subject: below --
2012-09-25 22:14 Lucas Stach

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=5047A3E8.7070304@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --cc=u-boot@lists.denx.de \
    /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