All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Markus Klotzbücher" <mk@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] reg ISP 1561 integration with u-boot1.1.6
Date: Wed, 10 Jan 2007 23:01:31 +0100	[thread overview]
Message-ID: <873b6izglg.fsf@denx.de> (raw)
In-Reply-To: <200701101813.45863.matthias.fuchs@esd-electronics.com> (Matthias Fuchs's message of "Wed, 10 Jan 2007 18:13:45 +0100")

Hi Matthias,

Matthias Fuchs <matthias.fuchs@esd-electronics.com> writes:

> it's working. I modified the testing-USB branch to run against a PCI OHCI 
> controller (Philips ISP1561) on the APC405 board (PowerPC PPC405GPr). 
>
> Enumeration and USB storage is working. I do not dare to say 'working well'.
>
> The main changes concern endianess fixes. The former code only supports OHCI 
> controller with the same endianess as the CPU. The other fix allows a offset 

IIRC same problem with the 440EP and MPC5200, thats what the extra
#ifdef in drivers/usb_ohci.c:103 is for.

> to access the system RAM by the busmastering OHCI controller. I called this 
> constant CFG_PCIRAM_BASE. But the name is a little bit confusion, isn't it?
>
> Here are my (first) questions: 
> 1) What do I have to do in usb_board_stop() and usb_board_init_fail()?
> My current code can only initialize the OHCI controller once. A 2nd 'usb 
> start' does not find any devices.

You might not have to do anything, but maybe you'll need to do some
board dependant cleanup for example. If not just leave them empty.

> 2) I had to disable the 'return -1' statement for an unfinished urb in 
> sohci_submit_job(). Why isn't it finished?

I really don't know, I guess only you can answer this!

> Please do not blame me for the board dependent code still in the usb_ohci.c 
> file. I will move it later.

Who else shall we blame then ;-) ?

Regards

Markus Klotzbuecher

  reply	other threads:[~2007-01-10 22:01 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-04 11:58 [U-Boot-Users] reg ISP 1561 integration with u-boot1.1.6 mahendra varman
2007-01-04 17:45 ` Matthias Fuchs
     [not found]   ` <4ac2955e0701042143w51a8067u80297938c7f4997d@mail.gmail.com>
2007-01-10 17:13     ` Matthias Fuchs
2007-01-10 22:01       ` Markus Klotzbücher [this message]
2007-01-11  8:50         ` Matthias Fuchs
2007-01-11 10:17           ` Markus Klotzbücher
2007-01-11 12:39             ` Matthias Fuchs
2007-01-11 13:15               ` Wolfgang Denk
2007-01-11 15:00                 ` Matthias Fuchs
2007-01-11 14:11               ` Markus Klotzbücher
2007-01-11 14:43                 ` Wolfgang Denk
2007-01-11 15:06                 ` Matthias Fuchs
2007-01-12 11:07                   ` Markus Klotzbücher
2007-01-15  6:08                     ` Matthias Fuchs
2007-01-11 13:13             ` Wolfgang Denk
2007-01-11 13:13           ` Wolfgang Denk
2007-01-11 14:58             ` Matthias Fuchs

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=873b6izglg.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.