All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthias Fuchs <matthias.fuchs@esd-electronics.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] reg ISP 1561 integration with u-boot1.1.6
Date: Mon, 15 Jan 2007 07:08:47 +0100	[thread overview]
Message-ID: <200701150708.47499.matthias.fuchs@esd-electronics.com> (raw)
In-Reply-To: <87ps9ksdun.fsf@denx.de>

Hi Markus,

On Friday 12 January 2007 12:07, Markus Klotzb?cher wrote:
> Hi Matthias,
> 
> Matthias Fuchs <matthias.fuchs@esd-electronics.com> writes:
> 
> > Sounds reasonable to me. We still need a more pretty way to handle the
> > +/- CFG_PCIRAM_BASE.  What about some cpu_to_bus/bus_to_cpu macros?
> 
> Hmm, that sounds a bit confusing too if you're only accessing a on chip
> controller. How about just renaming the define to something more generic
> such as CFG_OHCI_RAM_OFFSET and adding a nice explanation in the README?
> That way it wouldn't seem so strongly related to pci.
When dealing with a pci OHCI controller the offset if pci related. But you are 
right, when not running on a PCI platform your name ist better.

BTW, considering Wolfgang's desire for multi ohci controller support, the 
constant will change into a field in the ohci struct.

Matthias

  reply	other threads:[~2007-01-15  6:08 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
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 [this message]
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=200701150708.47499.matthias.fuchs@esd-electronics.com \
    --to=matthias.fuchs@esd-electronics.com \
    --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.