From: "Markus Klotzbücher" <mk@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Contribution: EHCI USB HC driver
Date: Sat, 01 Mar 2008 09:35:05 +0100 [thread overview]
Message-ID: <878x12suyu.fsf@denx.de> (raw)
In-Reply-To: <m2mypj8ymd.fsf@ohwell.denx.de> (Detlev Zundel's message of "Fri\, 29 Feb 2008 18\:24\:10 +0100")
Hi Marcel, Detlev,
I somehow missed this mail, thanks for following up!
Detlev Zundel <dzu@denx.de> writes:
>> Juniper Networks wrote a driver for the EHCI UCB HC and I'd like to
>> contribute that back if there's interest. Besides a new piece of code,
>> this also changes some of the common USB code so that high-speed
>> devices and split-transactions are supported.
>
> Cool!
Very nice indeed, thanks for sharing Marcel!
>> At this time the EHCI driver is exclusive in that there's no support
>> for having the companion controller active. I presume this is one of
>> the major themes for adding EHCI support, so that's definitely open
>> for discussion.
This is definitely an issue, but not one that can or should be solved
for the USB subsystem solely. What we really need to implement such
features in a clean manner is a well thought out U-Boot Driver Model...
>> Attached a patch with the common code changes as well as the EHCI
>> driver itself. The patch does not contain any changes to get it built
>> (i.e. included in libusb, etc). A seasoned U-Boot developer should
>> have no problem playing with it, while the details are being worked
>> out for getting this alongside a USB 1.1 HC driver.
Nice work! One thing we definitely will want to do is separate the
low-level board/cpu specific and generic ehci code such as done for the
generic ohci driver. From a first look it doesn't seem this would be too
hard.
> I think this will have to go through Markus' USB tree, so I put him on
> CC. Markus, could you perhaps add a branch in your repository to give
> this (and the upcoming) patches a better visibility even during
> development?
Yes, I'll do that.
> I am actually pretty happy, that finally someone is actively starting to
> put effort into moving the usb support in U-Boot forward - and before
> this gets interpreted wrongly - in my opinion Markus does a great job as
> a USB maintainer and I definitely do not want lessen his work here, but
> as we talked about a few times, most of his effort got sucked up in
> maintaining the current code base.
;-) No offence taken! I'm also glad to see some effort in this
direction!
Best regards
Markus Klotzb?cher
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
next prev parent reply other threads:[~2008-03-01 8:35 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-25 20:07 [U-Boot-Users] Contribution: EHCI USB HC driver Marcel Moolenaar
2008-02-29 17:24 ` Detlev Zundel
2008-02-29 19:33 ` Marcel Moolenaar
2008-03-01 8:38 ` Markus Klotzbücher
2008-03-01 10:18 ` Stefan Roese
2008-03-03 18:46 ` Marcel Moolenaar
2008-03-01 8:35 ` Markus Klotzbücher [this message]
2008-03-03 18:18 ` Marcel Moolenaar
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=878x12suyu.fsf@denx.de \
--to=mk@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.