public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/4] USB multi controller
Date: Wed, 5 Sep 2012 08:30:24 +0200	[thread overview]
Message-ID: <201209050830.25141.marex@denx.de> (raw)
In-Reply-To: <1346826104.1487.11.camel@tellur>

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.

> HTH,
> Lucas

Best regards,
Marek Vasut

  reply	other threads:[~2012-09-05  6:30 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 [this message]
2012-09-05 19:11       ` Stephen Warren
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=201209050830.25141.marex@denx.de \
    --to=marex@denx.de \
    --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