From: Scott Wood <scottwood@freescale.com>
To: Zhang Wei-r63237 <Wei.Zhang@freescale.com>
Cc: linuxppc-dev@ozlabs.org, paulus@samba.org
Subject: Re: [PATCH 1/4] Add DMA sector to Documentation/powerpc/booting-without-of.txt file.
Date: Wed, 11 Jul 2007 10:23:52 -0500 [thread overview]
Message-ID: <4694F608.2090503@freescale.com> (raw)
In-Reply-To: <46B96294322F7D458F9648B60E15112C6F3D1E@zch01exm26.fsl.freescale.net>
Zhang Wei-r63237 wrote:
>>Or don't call it anything. The ability to do extended chain mode is
>>implicit in being compatible with fsl,mpc8548-dma.
>
>
> The basic mode could be used for mpc83xx silicons.
Yes, and the 83xx device trees will list the compatible as
fsl,mpc8349-dma, not fsl,mpc8548-dma. This will tell the driver whether
extended chain mode can be used (though I don't really see the benefit
to using it even if it is there).
>>I'd rather just treat the different DMA channels as
>>independent devices,
>>rather than children of a dma "bus", and change the compatible name if
>>they're not general purpose. There's only one register that's shared
>>among the channels, and it's a superfluous status summary register.
>>
>
>
> Your and my ideas are both sides of a coin. :-)
I think there's a substantive difference between them. Making each
channel an independent device makes it easier for other drivers to use them.
-Scott
next prev parent reply other threads:[~2007-07-11 15:23 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-07-10 9:44 [PATCH 0/4] DMA engine driver for Freescale MPC8xxx processor Zhang Wei
2007-07-10 9:44 ` [PATCH 1/4] Add DMA sector to Documentation/powerpc/booting-without-of.txt file Zhang Wei
2007-07-10 9:44 ` [PATCH 2/4] Add dma sector to mpc8641hpcn board dts Zhang Wei
2007-07-10 13:55 ` Segher Boessenkool
2007-07-11 7:16 ` Zhang Wei-r63237
2007-07-11 11:27 ` Segher Boessenkool
2007-07-12 9:51 ` Zhang Wei-r63237
2007-07-10 13:57 ` Segher Boessenkool
2007-07-11 7:17 ` Zhang Wei-r63237
2007-07-10 14:01 ` [PATCH 1/4] Add DMA sector to Documentation/powerpc/booting-without-of.txt file Segher Boessenkool
2007-07-10 16:11 ` Scott Wood
2007-07-11 10:00 ` Zhang Wei-r63237
2007-07-11 15:23 ` Scott Wood [this message]
2007-07-11 17:53 ` Segher Boessenkool
2007-07-12 9:48 ` Zhang Wei-r63237
2007-07-12 17:12 ` Phil Terry
2007-07-12 19:10 ` Scott Wood
2007-07-16 14:56 ` Segher Boessenkool
2007-07-16 14:54 ` Segher Boessenkool
2007-07-17 11:17 ` Paul Mackerras
2007-07-17 15:36 ` Segher Boessenkool
2007-07-11 11:18 ` Segher Boessenkool
2007-07-11 15:30 ` Scott Wood
2007-07-11 18:01 ` Segher Boessenkool
2007-07-11 18:18 ` Scott Wood
2007-07-11 18:43 ` Segher Boessenkool
2007-07-11 19:03 ` Scott Wood
2007-07-11 19:19 ` Segher Boessenkool
2007-07-11 19:27 ` Scott Wood
2007-07-11 20:27 ` Segher Boessenkool
2007-07-11 10:06 ` Zhang Wei-r63237
2007-07-11 11:40 ` Segher Boessenkool
2007-07-12 9:45 ` Zhang Wei-r63237
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=4694F608.2090503@freescale.com \
--to=scottwood@freescale.com \
--cc=Wei.Zhang@freescale.com \
--cc=linuxppc-dev@ozlabs.org \
--cc=paulus@samba.org \
/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.