All of lore.kernel.org
 help / color / mirror / Atom feed
From: Armin Kuster <akuster@mvista.com>
To: Gabriel Paubert <paubert@iram.es>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	linuxppc-dev <linuxppc-dev@lists.linuxppc.org>
Subject: Re: 405 DMA api
Date: Mon, 25 Feb 2002 12:09:19 -0800	[thread overview]
Message-ID: <3C7A99EF.1AF2F99@mvista.com> (raw)
In-Reply-To: Pine.LNX.4.33.0202251548350.22001-100000@gra-lx1.iram.es


Gabriel Paubert wrote:
>
> On Mon, 25 Feb 2002, Benjamin Herrenschmidt wrote:
>
> >
> > >Hello,
> > >
> > >I would like to have the 405 dma share the same dma api as the current
> > >pmac. I propose to make the asm/dma.h _generic_ and move all the power
> > >Mac unique stuff into a seperate file such as "pmac_dma.h".  The dma.h
> > >would then be the gateway for other ppc archs by using "CONFIG_<arch>"
> > >to determine which hearder to pull in.  I made the pmac_pma.h as the
> > >default.  I have tested this on the  walnut and can only compile for the
> > >pmac.  I have included a patch and I  would like some feedback and to
> > >have the pmac_dma verified.
> > >
> > >files:
> > >include/asm-ppc/dma.h <- generic defines and config selection
> > >include/asm-ppc/pmac_dma.h     <- most code from dma.h
> > >include/asm-ppc/ppc405_dma.h <- changed func names
> > >arch/ppc/kernel/ppc405_dma.c <- changes to refect new header scheme
> >
> > Well... except this isn't pmac dma. The pmac has a DBDMA controller
> > which is much different from the legacy DMA controller used here. What
> > you see in this dma.h file is, I bet, prep stuff.
>
> True. It´s the legacy ISA DMA controller which can be found on PreP and
> CHRP, almost always in the PCI-ISA bridge. However these are not all
> created equal: the one in the WinBond 83C553 in my MVME boards, which is
> almost identical to some Intel model (82378ZB), is actually capable of 32
> bit addressing and can do descripto based scatter-gather DMA. However,
> since I never used anything that needed DMA on these boards (the floppy
> actually) I did not bother to improve the dma driver.
>
> Some historical note: the 82378ZB, with its improved DMA engine, is AFAICS
> the only Intel PCI<->ISA bridge with enhanced DMA capabilities. Also AFAIR
> this chip was exactly the one specified in the PreP documentation,
> strangely enough, the 82378 production was stopped and the enhanced DMA
> capability disappeared from all later Intel PCI<->ISA bridge. My 82378ZB
> doc is from March 1996 BTW...
>
>         Regards,
>         Gabriel.


Ok ... thanks for the info.  where do we go from here?  Can it get push
in after a better name is selected?

-- armin

** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/

      reply	other threads:[~2002-02-25 20:09 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-02-24 20:56 405 DMA api Armin Kuster
2002-02-25  7:48 ` Dan Malek
2002-02-25  8:16 ` Benjamin Herrenschmidt
2002-02-25 15:15   ` Gabriel Paubert
2002-02-25 20:09     ` Armin Kuster [this message]

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=3C7A99EF.1AF2F99@mvista.com \
    --to=akuster@mvista.com \
    --cc=benh@kernel.crashing.org \
    --cc=linuxppc-dev@lists.linuxppc.org \
    --cc=paubert@iram.es \
    /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.