[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <VI1PR04MB4431F221997731B3032CEA37ED6B0@VI1PR04MB4431.eurprd04.prod.outlook.com>
Date: Wed, 23 Oct 2019 02:38:50 +0000
From: Peng Ma <peng.ma@....com>
To: Vinod Koul <vkoul@...nel.org>
CC: Anders Roxell <anders.roxell@...aro.org>,
"dan.j.williams@...el.com" <dan.j.williams@...el.com>,
Leo Li <leoyang.li@....com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"dmaengine@...r.kernel.org" <dmaengine@...r.kernel.org>
Subject: RE: [EXT] Re: [V5 1/2] dmaengine: fsl-dpaa2-qdma: Add the DPDMAI(Data
Path DMA Interface) support
Hi Vinod,
>-----Original Message-----
>From: Vinod Koul <vkoul@...nel.org>
>Sent: 2019年10月23日 0:10
>To: Peng Ma <peng.ma@....com>
>Cc: Anders Roxell <anders.roxell@...aro.org>; dan.j.williams@...el.com; Leo Li
><leoyang.li@....com>; linux-kernel@...r.kernel.org;
>dmaengine@...r.kernel.org
>Subject: Re: [EXT] Re: [V5 1/2] dmaengine: fsl-dpaa2-qdma: Add the
>DPDMAI(Data Path DMA Interface) support
>
>Caution: EXT Email
>
>Please *do* *not* top post!
>
>On 22-10-19, 10:19, Peng Ma wrote:
>> Hi Anders && Viod,
>
>Its Vinod!
>
[Peng Ma] I am very sorry to spell your name wrong, I will pay attention to from now!
>>
>> I sent v6 patch to fix the build error, please check.
>> Patchwork link:
>> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
>>
>hwork.kernel.org%2Fproject%2Flinux-dmaengine%2Flist%2F%3Fseries%3D191
>3
>>
>97&data=02%7C01%7Cpeng.ma%40nxp.com%7C19ac47d605bf44aa99d
>e08d7570a
>>
>5e64%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637073574351
>786723&a
>>
>mp;sdata=%2B%2FazVuPMJZUonfvz8gLWGVeakKW0tRFYyjj344IbDg4%3D&am
>p;reserv
>> ed=0
>
>No I have already applied v5, please send fixes on top on dmaengine-next!
>Would also make sense to give credit to Anders using Reported-by tag
>
[Peng Ma] Ok, Got it.
The patchwork link:
https://patchwork.kernel.org/patch/11205611/
>>
>> Best Regards,
>> Peng
>> >-----Original Message-----
>> >From: Anders Roxell <anders.roxell@...aro.org>
>> >Sent: 2019年10月22日 17:27
>> >To: Peng Ma <peng.ma@....com>
>> >Cc: Vinod Koul <vkoul@...nel.org>; dan.j.williams@...el.com; Leo Li
>> ><leoyang.li@....com>; linux-kernel@...r.kernel.org;
>> >dmaengine@...r.kernel.org
>> >Subject: Re: [EXT] Re: [V5 1/2] dmaengine: fsl-dpaa2-qdma: Add the
>> >DPDMAI(Data Path DMA Interface) support
>> >
>> >Caution: EXT Email
>> >
>> >On Thu, 17 Oct 2019 at 08:16, Peng Ma <peng.ma@....com> wrote:
>> >>
>> >> Hi Vinod,
>> >>
>> >> Thanks very much for your reply.
>> >>
>> >> Best Regards,
>> >> Peng
>> >> >-----Original Message-----
>> >> >From: Vinod Koul <vkoul@...nel.org>
>> >> >Sent: 2019年10月17日 12:11
>> >> >To: Peng Ma <peng.ma@....com>
>> >> >Cc: dan.j.williams@...el.com; Leo Li <leoyang.li@....com>;
>> >> >linux-kernel@...r.kernel.org; dmaengine@...r.kernel.org
>> >> >Subject: [EXT] Re: [V5 1/2] dmaengine: fsl-dpaa2-qdma: Add the
>> >> >DPDMAI(Data Path DMA Interface) support
>> >> >
>> >> >Caution: EXT Email
>> >> >
>> >> >On 30-09-19, 02:04, Peng Ma wrote:
>> >> >> The MC(Management Complex) exports the DPDMAI(Data Path DMA
>> >> >Interface)
>> >> >> object as an interface to operate the DPAA2(Data Path
>> >> >> Acceleration Architecture 2) qDMA Engine. The DPDMAI enables
>> >> >> sending frame-based requests to qDMA and receiving back
>> >> >> confirmation response on transaction completion, utilizing the
>> >> >> DPAA2 QBMan(Queue Manager and Buffer Manager
>> >> >> hardware) infrastructure. DPDMAI object provides up to two
>> >> >> priorities for processing qDMA requests.
>> >> >> The following list summarizes the DPDMAI main features and
>capabilities:
>> >> >> 1. Supports up to two scheduling priorities for processing
>> >> >> service requests.
>> >> >> - Each DPDMAI transmit queue is mapped to one of two
>service
>> >> >> priorities, allowing further prioritization in hardware between
>> >> >> requests from different DPDMAI objects.
>> >> >> 2. Supports up to two receive queues for incoming transaction
>> >> >> completion confirmations.
>> >> >> - Each DPDMAI receive queue is mapped to one of two receive
>> >> >> priorities, allowing further prioritization between other
>> >> >> interfaces when associating the DPDMAI receive queues to
>DPIO
>> >> >> or DPCON(Data Path Concentrator) objects.
>> >> >> 3. Supports different scheduling options for processing
>received
>> >> >> packets:
>> >> >> - Queues can be configured either in 'parked' mode (default),
>> >> >> or attached to a DPIO object, or attached to DPCON object.
>> >> >> 4. Allows interaction with one or more DPIO objects for
>> >> >> dequeueing/enqueueing frame descriptors(FD) and for
>> >> >> acquiring/releasing buffers.
>> >> >> 5. Supports enable, disable, and reset operations.
>> >> >>
>> >> >> Add dpdmai to support some platforms with dpaa2 qdma engine.
>> >> >
>> >> >Applied both, thanks
>> >
>> >I see this error when I'm building.
>> >
>> >WARNING: modpost: missing MODULE_LICENSE() in
>> >drivers/dma/fsl-dpaa2-qdma/dpdmai.o
>> >see include/linux/module.h for more information
>> >ERROR: "dpdmai_enable" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko]
>> >undefined!
>> >ERROR: "dpdmai_set_rx_queue"
>> >[drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined!
>> >ERROR: "dpdmai_get_tx_queue"
>> >[drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined!
>> >ERROR: "dpdmai_get_rx_queue"
>> >[drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined!
>> >ERROR: "dpdmai_get_attributes"
>> >[drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined!
>> >ERROR: "dpdmai_open" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko]
>> >undefined!
>> >ERROR: "dpdmai_close" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko]
>> >undefined!
>> >ERROR: "dpdmai_disable" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko]
>> >undefined!
>> >ERROR: "dpdmai_reset" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko]
>> >undefined!
>> >make[2]: *** [../scripts/Makefile.modpost:95: __modpost] Error 1
>> >make[1]: *** [/srv/src/kernel/next/Makefile:1282: modules] Error 2
>> >make: *** [Makefile:179: sub-make] Error 2
>> >make: Target 'Image' not remade because of errors.
>> >make: Target 'modules' not remade because of errors.
>> >
>> >any other that see the same ?
>> >
>> >Cheers,
>> >Anders
>
>--
>~Vinod
Powered by blists - more mailing lists