Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753407AbcCIK5z (ORCPT ); Wed, 9 Mar 2016 05:57:55 -0500 Received: from szxga02-in.huawei.com ([119.145.14.65]:42732 "EHLO szxga02-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752960AbcCIK5h (ORCPT ); Wed, 9 Mar 2016 05:57:37 -0500 Subject: Re: [PATCH 2/4] infiniband: hns: add Hisilicon RoCE support(binding) To: Sergei Shtylyov , "Wei Hu(Xavier)" , , , , , , , References: <1457080877-13456-1-git-send-email-xavier.huwei@huawei.com> <1457080877-13456-3-git-send-email-xavier.huwei@huawei.com> <56D992E2.3060504@cogentembedded.com> CC: , , , , , , , , , , , From: oulijun Message-ID: <56E00170.2050405@huawei.com> Date: Wed, 9 Mar 2016 18:56:48 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <56D992E2.3060504@cogentembedded.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.61.25.147] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020201.56E00185.00CE,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: d2d7283ab546b0245d64907d9f1194c4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2147 Lines: 64 Hi Sergei Shtylyov, thanks for reviewing I have modified the binding referred to your advice and the bindings of other module. I will send a new patch in future. Thanks Lijun Ou On 2016/3/4 21:51, Sergei Shtylyov wrote: > Hello. > > On 3/4/2016 11:41 AM, Wei Hu(Xavier) wrote: > >> This submit add binding file and dts file. >> >> Signed-off-by: Wei Hu(Xavier) >> Signed-off-by: oulijun >> --- >> .../bindings/infiniband/hisilicon-hns-roce.txt | 68 ++++++++++++++++++++++ >> 1 file changed, 68 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/infiniband/hisilicon-hns-roce.txt >> >> diff --git a/Documentation/devicetree/bindings/infiniband/hisilicon-hns-roce.txt b/Documentation/devicetree/bindings/infiniband/hisilicon-hns-roce.txt >> new file mode 100644 >> index 0000000..8004641 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/infiniband/hisilicon-hns-roce.txt >> @@ -0,0 +1,68 @@ >> +HiSilicon RoCE DT description >> + >> +HiSilicon RoCE engine is a part of network subsystem. >> +It works depending on other part of network wubsytem, such as, gmac and >> +dsa fabric. >> + >> +Additional properties are described here: >> + >> +Required properties: >> +- compatible: Should contain "hisilicon,hns-roce-v1". >> +- reg: Physical base address of the roce driver and >> +length of memory mapped region. >> +- eth-handle: phandle, specifies a reference to a node >> +representing a ethernet device. >> +- dsaf-handle: phandle, specifies a reference to a node >> +representing a dsaf device. >> +- #address-cells: must be 2 >> +- #size-cells: must be 2 >> +Optional properties: >> +- dma-coherent: Present if DMA operations are coherent. >> +- interrupt-parent: the interrupt parent of this device. >> +- interrupts: should contain 32 completion event irq,1 async event irq >> +and 1 event overflow irq. > > The "interrupt-names" prop is very desirable for so many interrupts. > >> +Example: >> + rocee@0xc4000000 { > > The node names should be generic and "0x" should be omitted, i.e. "infiniband@c4000000". > > MBR, Sergei > > > . >