public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 2/5] sunxi: add USB EHCI driver
Date: Sun, 27 Jul 2014 00:22:31 +0200	[thread overview]
Message-ID: <53D42A27.3020408@redhat.com> (raw)
In-Reply-To: <f4b90cf6a65a83e92ae1c6204f57402d395c1a67.1405895188.git.rbyshko@gmail.com>

Hi,

On Fri, 2014-07-25 at 18:31 +0200, Marek Vasut wrote:
 > On Friday, July 25, 2014 at 06:06:45 PM, Ian Campbell wrote:
 > > On Fri, 2014-07-25 at 17:41 +0200, Marek Vasut wrote:

<snip>

 > > > If you pick all but 2/5 right now, I am happy with that. It will prevent
 > > > conflicts. I will pick 2/5 only after that . Let's do it like that. Roman
 > > > will also have to resend only 2/5 once fixed.
 > >
 > > Some patches (e.g. #5) don't make much sense without #2, but they are
 > > harmless and when things eventually meet in whichever upstream tree it
 > > should all (hopefully!) burst into life. Does that sound ok?
 >
 > Yes, let's do it this way please. I will pick a new version of 2/5 only, you
 > pick the rest now.

I've been working on u-boot-sunxi next today to add support for more boards upstream,
and I thought it would be good to cherry pick this set first so that I can add the
necessary CONFIG_USB_EHCI in boards.cfg for boards which support this right away,
but this causes builds to fail, e.g. :

drivers/usb/host/built-in.o: In function `usb_lowlevel_stop':
/home/hans/projects/sunxi/u-boot/drivers/usb/host/ehci-hcd.c:923: undefined reference to `ehci_hcd_stop'
drivers/usb/host/built-in.o: In function `usb_lowlevel_init':
/home/hans/projects/sunxi/u-boot/drivers/usb/host/ehci-hcd.c:935: undefined reference to `ehci_hcd_init'
arm-linux-gnu-ld.bfd: BFD version 2.24.0-4.fc21 20140613 assertion fail ../../binutils-2.24/bfd/elf32-arm.c:7696
arm-linux-gnu-ld.bfd: BFD version 2.24.0-4.fc21 20140613 assertion fail ../../binutils-2.24/bfd/elf32-arm.c:7696
arm-linux-gnu-ld.bfd: error: required section '.rel.plt' not found in the linker script
arm-linux-gnu-ld.bfd: final link failed: Invalid operation
Makefile:986: recipe for target 'u-boot' failed

So merging 1 and 3-5 to the sunxi tree while merging 2 through your (Marek's) tree
does not seem to be the best idea. I think it would be best to take this patch
(patch 2) through the sunxi tree too, with your ack (once your happy with it).

Roman, this means that it would be nice to get a v4 from you soon, so that we can line up
the next bunch of patches for mainline.

Thanks & Regards,

Hans

  parent reply	other threads:[~2014-07-26 22:22 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-24 20:54 [U-Boot] [PATCH v3 0/5] ARM: Allwinner sunxi USB Host EHCI support Roman Byshko
2014-07-24 20:54 ` [U-Boot] [PATCH v3 1/5] sunxi: add defines to control USB Host clocks/resets Roman Byshko
2014-07-24 20:54 ` [U-Boot] [PATCH v3 2/5] sunxi: add USB EHCI driver Roman Byshko
2014-07-25 15:01   ` Marek Vasut
2014-07-25 15:38     ` Ian Campbell
2014-07-25 15:41       ` Marek Vasut
2014-07-25 16:06         ` Ian Campbell
2014-07-25 16:31           ` Marek Vasut
2014-07-25 16:36             ` Ian Campbell
2014-07-26 16:25   ` Hans de Goede
2014-07-26 22:22   ` Hans de Goede [this message]
2014-07-27  0:55     ` Marek Vasut
2014-07-27  8:29       ` Ian Campbell
2014-07-27 10:03         ` Hans de Goede
2014-07-27 15:35           ` Marek Vasut
2014-07-27 17:32           ` [U-Boot] [PATCH v4 " Roman Byshko
2014-07-27 17:57             ` Marek Vasut
2014-07-27 20:14               ` Hans de Goede
2014-07-27 20:41                 ` Marek Vasut
2014-07-24 20:54 ` [U-Boot] [PATCH v3 3/5] sunxi: add general USB settings Roman Byshko
2014-07-24 20:54 ` [U-Boot] [PATCH v3 4/5] sun7i: add USB EHCI settings Roman Byshko
2014-07-24 20:54 ` [U-Boot] [PATCH v3 5/5] sun7i: cubietruck: enable USB EHCI Roman Byshko
2014-07-25 20:33   ` Ian Campbell
2014-07-26 22:34     ` Hans de Goede
2014-07-25  9:31 ` [U-Boot] [PATCH v3 0/5] ARM: Allwinner sunxi USB Host EHCI support Roman B.

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=53D42A27.3020408@redhat.com \
    --to=hdegoede@redhat.com \
    --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