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 v2 1/1] USB: EHCI: Initialize multiple USB controllers at once
Date: Tue, 14 Aug 2012 19:30:45 +0200	[thread overview]
Message-ID: <201208141930.46103.marex@denx.de> (raw)
In-Reply-To: <502A7A6B.3030100@wwwdotorg.org>

Dear Stephen Warren,

> On 08/13/2012 10:22 PM, Jim Lin wrote:
> > Marek Vasut wrote at Sunday, August 12, 2012 7:47 AM:
> >> Dear Jim Lin,
> >> 
> >>> Add support for command line "usb reset" or "usb start" to initialize
> >>> , "usb stop" to stop multiple USB controllers at once.
> >>> Other command like "usb tree" also supports multiple controllers.
> 
> ...
> 
> >>> +#ifdef CONFIG_USB_KEYBOARD
> >>> +       drv_usb_kbd_init();
> >> 
> >> Will the keyboard driver survive this?
> > 
> > Yes I have tried it after doing the following.
> > 1. Define CONFIG_USB_KEYBOARD, CONFIG_SYS_USB_EVENT_POLL,
> > CONFIG_USB_MULTI, and CONFIG_USB_MAX_CONTROLLER_COUNT 3
> > 
> >  in config header file like seaboard.h to compile
> > 
> > 2. Install USB keyboard
> > 3. Run "usb reset" in u-boot serial console
> > 4. Run "coninfo" to see usbkbd appeared
> > 5. Run "setenv stdin usbkbd"
> > 6. Typing u-boot console command on USB keyboard
> 
> I've also tested this on Springbank, which has a built-in USB keyboard
> and an external USB port via a separate host controller, which I
> connected to an Ethernet device, and it appears to all work very well
> for me.
> 
> So,
> 
> Tested-by: Stephen Warren <swarren@wwwdotorg.org>

Good, thanks! :)

Best regards,
Marek Vasut

  reply	other threads:[~2012-08-14 17:30 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-08  9:48 [U-Boot] [PATCH v2 1/1] USB: EHCI: Initialize multiple USB controllers at once Jim Lin
2012-08-11 23:46 ` Marek Vasut
2012-08-13 10:25   ` Jim Lin
2012-08-14 17:29     ` Marek Vasut
2012-08-14  4:22   ` Jim Lin
2012-08-14 16:18     ` Stephen Warren
2012-08-14 17:30       ` Marek Vasut [this message]
  -- strict thread matches above, loose matches on Subject: below --
2012-08-10  9:36 Jim Lin
2012-08-10 16:07 ` Stephen Warren

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