public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Detlev Zundel <dzu@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Contribution: EHCI USB HC driver
Date: Fri, 29 Feb 2008 18:24:10 +0100	[thread overview]
Message-ID: <m2mypj8ymd.fsf@ohwell.denx.de> (raw)
In-Reply-To: <EFFDEE36-9229-4C51-B6EC-C9218F534B4E@juniper.net> (Marcel Moolenaar's message of "Mon, 25 Feb 2008 12:07:31 -0800")

Hi Marcel,

> 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!

> 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.
>
> 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.

Glancing over the patch I fail to see what hardware and configuration
you are using for your development.  Can you enlighten us on this?

> Comments and suggestions are welcome. Please let me know how best to
> move forward with adding this to U-Boot, provided there's interest.
> FYI,

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?

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.

So thanks again Marcel and I am looking forward to a shiny USB support
in U-Boot ;)

Cheers
  Detlev

-- 
BUGS:
  It is not yet possible to change operating system by writing to
  /proc/sys/kernel/ostype.
                                     -- Linux sysctl(2) man page
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de

  reply	other threads:[~2008-02-29 17:24 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 [this message]
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
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=m2mypj8ymd.fsf@ohwell.denx.de \
    --to=dzu@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