From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752603AbbKJLMj (ORCPT ); Tue, 10 Nov 2015 06:12:39 -0500 Received: from szxga01-in.huawei.com ([58.251.152.64]:21731 "EHLO szxga01-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752226AbbKJLMf (ORCPT ); Tue, 10 Nov 2015 06:12:35 -0500 Subject: Re: [PATCH v3 02/32] devicetree: bindings: scsi: HiSi SAS To: Rob Herring References: <1447086757-147706-1-git-send-email-john.garry@huawei.com> <1447086757-147706-3-git-send-email-john.garry@huawei.com> <20151109180133.GA23100@rob-hp-laptop> CC: "JBottomley@odin.com" , "pawel.moll@arm.com" , "mark.rutland@arm.com" , "ijc+devicetree@hellion.org.uk" , "galak@codeaurora.org" , "arnd@arndb.de" , "linux-scsi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Linuxarm , "john.garry2@mail.dcu.ie" , "hare@suse.de" , "xuwei (O)" , "zhangfei.gao@linaro.org" From: John Garry Message-ID: <5641D087.8050502@huawei.com> Date: Tue, 10 Nov 2015 11:09:59 +0000 User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <20151109180133.GA23100@rob-hp-laptop> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.202.137.251] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020202.5641D09C.0198,ss=1,re=0.000,fgs=0, ip=0.0.0.0, so=2013-06-18 04:22:30, dmn=2011-05-27 18:58:46 X-Mirapoint-Loop-Id: 7ec99d1d7a5ee0993b5a18a231c81d9f Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/11/2015 18:01, Rob Herring wrote: > On Tue, Nov 10, 2015 at 12:32:07AM +0800, John Garry wrote: >> Add devicetree bindings for HiSilicon SAS driver. >> >> Signed-off-by: John Garry >> Signed-off-by: Zhangfei Gao >> --- >> .../devicetree/bindings/scsi/hisilicon-sas.txt | 81 ++++++++++++++++++++++ >> 1 file changed, 81 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/scsi/hisilicon-sas.txt >> >> diff --git a/Documentation/devicetree/bindings/scsi/hisilicon-sas.txt b/Documentation/devicetree/bindings/scsi/hisilicon-sas.txt >> new file mode 100644 >> index 0000000..2333cc3 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/scsi/hisilicon-sas.txt >> @@ -0,0 +1,81 @@ >> +* HiSilicon SAS controller >> + >> +The HiSilicon SAS controller supports SAS/SATA. >> + >> +Main node required properties: >> + - compatible : value should be as follows: >> + (a) "hisilicon,sas-controller-v1" for v1 of HiSilicon SAS controller IP > > Please do a more specific compatible string with the SOC part number. > Same versions of IP blocks can have different integration/process > features/bugs. > How about "hisilicon,hip05-sas-v1"? >> + - sas-addr : array of 8 bytes for host SAS address >> + - reg : Address and length of the SAS register >> + - hisilicon,sas-syscon: phandle of syscon used for sas control >> + - ctrl-reset-reg : offset to controller reset register in ctrl reg >> + - ctrl-reset-sts-reg : offset to controller reset status register in ctrl reg >> + - ctrl-clock-ena-reg : offset to controller clock enable register in ctrl reg >> + - queue-count : number of delivery and completion queues in the controller >> + - phy-count : number of phys accessible by the controller >> + - interrupts : Interrupts for phys, completion queues, and fatal >> + sources; the interrupts are ordered in 3 groups, as follows: >> + - Phy interrupts >> + - Completion queue interrupts >> + - Fatal interrupts >> + Phy interrupts : Each phy has 3 interrupt sources: >> + - broadcast >> + - phyup >> + - abnormal >> + The phy interrupts are ordered into groups of 3 per phy >> + (broadcast, phyup, and abnormal) in increasing order. >> + Completion queue interrupts : each completion queue has 1 >> + interrupt source. The interrupts are ordered in >> + increasing order. >> + Fatal interrupts : the fatal interrupts are ordered as follows: >> + - ECC >> + - AXI bus >> + >> +* HiSilicon SAS syscon >> + >> +Required properties: >> +- compatible: should be "hisilicon,sas-ctrl", "syscon" > > Please add a more specific compatible here too. We plan to remove this node now as it will be defined in the platform dt bindings: https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/Documentation/devicetree/bindings/arm/hisilicon/hisilicon.txt#n174 > > Rob > Thanks, John