Hi Shawn,

        Thanks for your review , I have used GIC_SPI and IRQ_TYPE_xxx to my 
dtsi, please check and review.

Best Regard,
        Peng

>-----Original Message-----
>From: Shawn Guo <shawn...@kernel.org>
>Sent: 2018年12月6日 9:06
>To: Peng Ma <peng...@nxp.com>
>Cc: vk...@kernel.org; robh...@kernel.org; mark.rutl...@arm.com; Leo Li
><leoyang...@nxp.com>; dan.j.willi...@intel.com; z...@zh-kernel.org;
>dmaeng...@vger.kernel.org; devicet...@vger.kernel.org;
>linux-ker...@vger.kernel.org; linux-arm-ker...@lists.infradead.org;
>linuxppc-dev@lists.ozlabs.org; Wen He <wen.h...@nxp.com>
>Subject: Re: [v11 6/7] arm64: dts: ls1046a: add qdma device tree nodes
>
>On Tue, Oct 30, 2018 at 10:36:03AM +0800, Peng Ma wrote:
>> add the qDMA device tree nodes for LS1046A devices.
>>
>> Signed-off-by: Wen He <wen.h...@nxp.com>
>> Signed-off-by: Peng Ma <peng...@nxp.com>
>> ---
>> change in v11:
>>      - no
>>
>>  arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi |   21
>+++++++++++++++++++++
>>  1 files changed, 21 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
>b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
>> index ef83786..dc65318 100644
>> --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
>> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
>> @@ -704,6 +704,27 @@
>>                                      <0000 0 0 4 &gic GIC_SPI 154
>IRQ_TYPE_LEVEL_HIGH>;
>>              };
>>
>> +            qdma: dma-controller@8380000 {
>> +                    compatible = "fsl,ls1046a-qdma", "fsl,ls1021a-qdma";
>> +                    reg = <0x0 0x8380000 0x0 0x1000>, /* Controller regs */
>> +                          <0x0 0x8390000 0x0 0x10000>, /* Status regs */
>> +                          <0x0 0x83a0000 0x0 0x40000>; /* Block regs */
>> +                    interrupts = <0 153 0x4>,
>> +                                 <0 39 0x4>,
>> +                                 <0 40 0x4>,
>> +                                 <0 41 0x4>,
>> +                                 <0 42 0x4>;
>
>Use GIC_SPI and IRQ_TYPE_xxx defines.
>
>Shawn
>
>> +                    interrupt-names = "qdma-error", "qdma-queue0",
>> +                            "qdma-queue1", "qdma-queue2", "qdma-queue3";
>> +                    dma-channels = <8>;
>> +                    block-number = <1>;
>> +                    block-offset = <0x10000>;
>> +                    fsl,dma-queues = <2>;
>> +                    status-sizes = <64>;
>> +                    queue-sizes = <64 64>;
>> +                    big-endian;
>> +            };
>> +
>>      };
>>
>>      reserved-memory {
>> --
>> 1.7.1
>>

Reply via email to