Hi, Randy,
I'll post it soon and hope to get much feedback from you.
Thanks!
Wei.
> -Original Message-
> From: Randy Vinson [mailto:[EMAIL PROTECTED]
>
> Zhang Wei-r63237 wrote:
> > Yes, I'm working on it. Do not worry about it.
> How's this go
Yes, I'm working on it. Do not worry about it.
Wei.
> -Original Message-
> From: Kumar Gala [mailto:[EMAIL PROTECTED]
> Sent: Tuesday, October 30, 2007 4:31 AM
> To: [EMAIL PROTECTED]
> Cc: Zhang Wei-r63237; [EMAIL PROTECTED];
> [EMAIL PROTECTED]; linuxppc-de
Hi,
> > +static void fsl_dma_set_src(dma_addr_t addr,
> > + struct dma_async_tx_descriptor
> *tx, int index)
> > +{
>
> What is index supposed to mean? It's not used, or documented
> anywhere than
> I can see.
I've also got more document here. Hi, Dan, could you giv
> that time kumar
> suggest to upgrade the kernel verison 2.6.23-rc3.
> Zhang did u suspect the problem is related to uboot?.
> by
> sivaji
>
>
> Zhang Wei-r63237 wrote:
> >
> > Yes, It's too old. Maybe not fully supports FDT. You can
> try the
Yes, It's too old. Maybe not fully supports FDT. You can try the version
of Kumar said or in the BSP of Freescale released.
- zw
> -Original Message-
> From: [EMAIL PROTECTED]
> [mailto:[EMAIL PROTECTED]
> g] On Behalf Of Kumar Gala
> Sent: Thursday, September 13, 2007 1:11 PM
> To: siva
Hi,
Your flat device tree address seems too low. Try to move the dtb to
higher address such as 0x200.
And which version of your u-boot?
Cheers!
- zw
> -Original Message-
> From: [EMAIL PROTECTED]
> [mailto:[EMAIL PROTECTED]
> g] On Behalf Of sivaji
> Sent: Thursday, September 13,
> > >
> > > On Fri, Sep 07, 2007 at 04:43:35PM +0200, Segher
> Boessenkool wrote:
> > > > > + l) Freescale DMA
> > > >
> > > > > +- compatible : Should be "fsl,dma".
> > > >
> > > > Please choose some more specific name. "fsl,mpc8540-dma" would
> > > > be a reasonable choice perhaps.
> >
Hi, Dan,
Does I have followed your new API? :-)
> > ---
> Greetings,
>
> Please copy me on any updates to this driver, drivers/dma, or
> crypto/async_tx.
Ok.
>
> Below are a few review comments...
>
> Regards,
> Dan
>
> > +/**
> > + * fsl_dma_alloc_descriptor - Allocate descriptor from
>
>
> If this is experimental, perhaps you should mark the depends line as
> such
> depends on on DMA_ENGINE && PPC && EXPERIMENTAL
I'll add EXPERIMENTAL for MPC83xx only.
>
> [...]
>
> >+
> >+fsl_dma_memcpy_issue_pending(chan);
> >+while (fsl_dma_is_complete(chan, cookie, NULL, NU
I'll merge them.
Thanks!
- zw
> -Original Message-
> From: David Gibson [mailto:[EMAIL PROTECTED]
> Sent: Monday, September 10, 2007 11:28 AM
> To: Zhang Wei-r63237
> Cc: [EMAIL PROTECTED]; linuxppc-dev@ozlabs.org
> Subject: Re: [PATCH 2/5] Add Freescale DMA e
Hi,
> --- /dev/null
> > +++ b/drivers/dma/fsldma.c
> > @@ -0,0 +1,995 @@
>
> Thanks for using kernel-doc notation. However, ...
>
> > +/**
> > + * fsl_dma_alloc_descriptor - Allocate descriptor from
> channel's DMA pool.
>
> Function parameters need to be listed & described here.
> See Docume
>
> On Fri, Sep 07, 2007 at 04:43:35PM +0200, Segher Boessenkool wrote:
> > > + l) Freescale DMA
> >
> > > +- compatible : Should be "fsl,dma".
> >
> > Please choose some more specific name. "fsl,mpc8540-dma" would
> > be a reasonable choice perhaps.
>
> More precisely, the compatible
Oops!
Could you give us a live show version? :D
Cheers!
- zw
> -Original Message-
> From: [EMAIL PROTECTED]
> [mailto:[EMAIL PROTECTED]
> g] On Behalf Of Timur Tabi
> Sent: Wednesday, September 05, 2007 10:54 AM
> To: linuxppc-dev@ozlabs.org
> Subject: Keep On Debugging You
>
> I wrot
Hi, Michael,
I get them. Thanks for your work! :)
Best Regards,
Zhang Wei
> -Original Message-
> From: Michael Ellerman [mailto:[EMAIL PROTECTED]
> Sent: Tuesday, August 07, 2007 11:19 AM
> To: Zang Roy-r61911
> Cc: Kumar Gala; linuxppc-dev list; Paul Mackerras; Z
Hi, Paul,
How about following patches?
[PATCH 1/2] Add sysdev/pci_quirks.c file into PowerPC architecture with
ULI chip quirk functions.
URL: http://ozlabs.org/pipermail/linuxppc-dev/2007-August/040363.html
[PATCH 2/2] Remove ULI chip quirks functions from MPC8641HPCN and
MPC8544DS boards.
URL: h
Hi, Ben,
>
> Are we sure that any powerpc machine that uses that ULI chip will need
> the same quirk setting ?
>
We can ensure current platforms (as far as we know, MPC8544DS and
MPC8641HPCN) using ULI chips in powerpc of kernel need the same quirk
setting.
And we hope to make them more generi
Hi, Arnd,
I can change it as you metioned now.
Thanks!
-zw
> -Original Message-
> From: [EMAIL PROTECTED]
> [mailto:[EMAIL PROTECTED] On Behalf Of Arnd Bergmann
> Sent: Sunday, July 29, 2007 9:57 PM
> To: linuxppc-dev@ozlabs.org
> Cc: Zhang Wei-r63237; [EMAIL PRO
Hi, Kumar,
> + RapidIO is a definition of a system interconnect. This node add
> > + the support for RapidIO processor in kernel. The node name is
> > + suggested to be 'rapidio'.
> > +
> > + Required properties:
> > +
> > +- compatible : Using "fsl,rapidio-delta" for Freescale PowerPC
Hi, Kumar,
> -Original Message-
> From: Kumar Gala [mailto:[EMAIL PROTECTED]
> On Jul 26, 2007, at 3:42 AM, Zhang Wei wrote:
>
> > This patch adds the RapidIO support to the powerpc architecture.
> > Some files are moved from ppc. OF-tree and OF-device supports are
> > added.
> > New
Hi, Ben,
Thanks!
> Sure, hence the word 'optional' which would provide a default name.
>
About the default name, since platforms we touched are very limited, we
do not know which name should be proper default name. We just assign
those default name to 'NULL' for passing the compiling process.
Hi, Ben,
Thanks! Please see my inline comments.
>
> Well, you would have to look at a cell blade or pSeries machine
> device-tree, I don't have one at hand in DTS form... but basically,
> those don't have a 1:1 relationship between hosts and nodes.
Is there any problem when the relationship be
> -Original Message-
> From: Wood Scott-B07421
>
> On Tue, Jul 10, 2007 at 05:45:26PM +0800, Zhang Wei wrote:
> > +config FSL_DMA
> > + bool "Freescale MPC8xxx DMA support"
> > + depends on DMA_ENGINE && (PPC_86xx || PPC_85xx)
>
> Remove the dependency on specific PPC chips... le
> -Original Message-
> From: Segher Boessenkool [mailto:[EMAIL PROTECTED]
>
> >>> + [EMAIL PROTECTED]
> >>> + compatible = "fsl,mpc8xxx-dma";
> >>
> >> Please use a real name, not this "xxx" stuff.
> >
> > Does the "xxx" seems to be an evil name? :-)
>
> You ar
> -Original Message-
> From: Segher Boessenkool [mailto:[EMAIL PROTECTED]
> Sent: Thursday, July 12, 2007 1:54 AM
> To: Wood Scott-B07421
> Cc: Zhang Wei-r63237; linuxppc-dev@ozlabs.org; [EMAIL PROTECTED]
> Subject: Re: [PATCH 1/4] Add DMA sector to
> Document
Hi,
> -Original Message-
> From: Segher Boessenkool [mailto:[EMAIL PROTECTED]
> >> Call it "extended-chain-mode", perhaps?
> >
> > Is it maybe too long?
>
> Not really no. It's only 19 characters, and it isn't used
> all over the place, so that should be fine.
Ok,
>
> >>> +- res
Hi, Dan,
Thanks! I get it.
It's so lucky we have the same target.
When your patch could be accepted?
Cheers,
Wei.
> -Original Message-
> From: [EMAIL PROTECTED]
> [mailto:[EMAIL PROTECTED] On Behalf Of Dan Williams
> Sent: Thursday, July 12, 2007 12:57 AM
> To: Z
Hi,
> -Original Message-
> From: Segher Boessenkool [mailto:[EMAIL PROTECTED]
>
> > + k) DMA
> > +
> > + This sector define DMA for dma-engine driver of Freescale
>
> It's called a "device node" or "node", not a "sector".
Ok, "node".
>
> > +- compatible : Should be "fsl,mpc8
Hi,
> -Original Message-
> From: Wood Scott-B07421
>
> On Tue, Jul 10, 2007 at 04:01:19PM +0200, Segher Boessenkool wrote:
> > > +- compatible : Should be "fsl,mpc8xxx-dma"
> >
> > Should _include_, not should _be_. And none of this xxx
> > business, of course.
>
> Especially sin
Hi, Dan,
Do you mention here: http://marc.info/?l=linux-raid&m=118290909614463&w=2 ?
I see the async_tx is located at crypto/ of the above page, but my patch is for
DMA engine in drivers/dma and for DMA engine driver.
Thanks!
Wei.
> -Original Message-
> Subject: Re: [PATCH 3/4] Extend t
> > + [EMAIL PROTECTED]
>
> "dma-controller" btw. And a space before the "{" (here and
> elsewhere) :-)
All right! :-)
Wei.
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
Hi, Segher,
> -Original Message-
> From: Segher Boessenkool [mailto:[EMAIL PROTECTED]
>
> > + [EMAIL PROTECTED]
> > + compatible = "fsl,mpc8xxx-dma";
>
> Please use a real name, not this "xxx" stuff.
Does the "xxx" seems to be an evil name? :-)
>
> > +
Hi, Segher,
> DTS sector to the document of booting-without-of.txt file.
>
> >>> +- #address-cells : Address representation for
> >> "rapidio" devices.
> >>> + This field represents the number of cells needed
> to represent
> >>> + the RapidIO address of the registers. For
> >> s
Hi, Segher,
> > +- #address-cells : Address representation for
> "rapidio" devices.
> > + This field represents the number of cells needed to represent
> > + the RapidIO address of the registers. For
> supporting more than
> > + 32-bits RapidIO address, this field should be
Hi, Arnd,
>
> On Wednesday 27 June 2007, Zhang Wei wrote:
> > +static struct of_device_id mpc86xx_of_ids[] = {
> > + { .type = "soc", },
> > + { .compatible = "fsl,rapidio-delta", },
> > + {},
> > +};
> > +
> > +static __init int mpc86xx_of_device_init(void)
> > +{
> > +
34 matches
Mail list logo
