public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: David Brownell <david-b@pacbell.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [patch] csb337 board support (ARM at91rm9200)
Date: Mon, 29 Jun 2009 14:46:25 -0700	[thread overview]
Message-ID: <200906291446.26085.david-b@pacbell.net> (raw)
In-Reply-To: <20090629211601.GE23292@game.jcrosoft.org>

On Monday 29 June 2009, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > > >   - USB didn't work; the software wouldn't detect usb-storage devices.
> > > >     So it's not yet enabled.
> > >
> > > what is the power on the USB?
> > 
> > I don't understand the question.  5V of course.  Not switchable.
>
> is the usb provide enough power to supply the usb key

Well all the devices run on Linux just fine.  Two were
flash devices, one was an honest-to-gosh 200 MByte disk.

So it's fairly clear the issue relates to OHCI support
in u-boot, perhaps specific to the rm9200.  Not hardware.

  
> > > > +#if 0
> > > please 
> > > if CONFIG_CMD_USB
> > > > +/* REVISIT:  USB failed to enumerate devices of any kind ... sw issues */
> > 
> > An #if 0 won't work without completely re-whacking things.
> > 
> > Right now the commands kick in *MUCH LATER* depending on
> > whether the driver is available.  And here, the driver
> > (CONFIG_USB_OHCI_NEW driver) and its support is disabled
> > since it's not working...
> so move it

That's what we call "way too much work".  It involves
completely restructuring things ... so that instead of
enabling commands based on driver support (which may
not be viable) it enables drivers (which may not work)
based on commands (which should never have been enabled
unless the relevant driver works).

  reply	other threads:[~2009-06-29 21:46 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-12 21:58 [U-Boot] [patch] csb337 board support (ARM at91rm9200) David Brownell
2009-06-25 21:56 ` Jean-Christophe PLAGNIOL-VILLARD
2009-06-29  6:20   ` David Brownell
2009-06-29 21:16     ` Jean-Christophe PLAGNIOL-VILLARD
2009-06-29 21:46       ` David Brownell [this message]
2009-06-29 22:44         ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-10 22:43 ` Wolfgang Denk

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=200906291446.26085.david-b@pacbell.net \
    --to=david-b@pacbell.net \
    --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