From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3D119C43461 for ; Thu, 17 Sep 2020 03:30:50 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id ED97A2065D for ; Thu, 17 Sep 2020 03:30:49 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726185AbgIQDas (ORCPT ); Wed, 16 Sep 2020 23:30:48 -0400 Received: from szxga06-in.huawei.com ([45.249.212.32]:53772 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1725267AbgIQDar (ORCPT ); Wed, 16 Sep 2020 23:30:47 -0400 Received: from DGGEMS402-HUB.china.huawei.com (unknown [172.30.72.59]) by Forcepoint Email with ESMTP id 8F9AAFC03350B75F5D2C; Thu, 17 Sep 2020 11:30:42 +0800 (CST) Received: from [127.0.0.1] (10.174.177.253) by DGGEMS402-HUB.china.huawei.com (10.3.19.202) with Microsoft SMTP Server id 14.3.487.0; Thu, 17 Sep 2020 11:30:34 +0800 Subject: Re: [PATCH 1/2] dt-bindings: interrupt-controller: add Hisilicon SD5203 vector interrupt controller From: "Leizhen (ThunderTown)" To: Rob Herring CC: Thomas Gleixner , Jason Cooper , Marc Zyngier , devicetree , linux-kernel , Kefeng Wang References: <20200903120504.2308-1-thunder.leizhen@huawei.com> <20200903120504.2308-2-thunder.leizhen@huawei.com> <20200914203110.GA185427@bogus> <4873e234-d06c-930e-6645-a832164e18ea@huawei.com> Message-ID: <0928ac6a-2295-4e87-5191-4fd4580f87b5@huawei.com> Date: Thu, 17 Sep 2020 11:30:33 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 MIME-Version: 1.0 In-Reply-To: <4873e234-d06c-930e-6645-a832164e18ea@huawei.com> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.174.177.253] X-CFilter-Loop: Reflected Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2020/9/15 14:12, Leizhen (ThunderTown) wrote: > > > On 2020/9/15 4:31, Rob Herring wrote: >> On Thu, Sep 03, 2020 at 08:05:03PM +0800, Zhen Lei wrote: >>> Add DT bindings for the Hisilicon SD5203 vector interrupt controller. >>> >>> Signed-off-by: Zhen Lei >>> --- >>> .../hisilicon,sd5203-vic.txt | 27 +++++++++++++++++++ >> >> Bindings should be in DT schema format now. Do I need to change the existing "snps,dw-apb-ictl.txt" to DT schema format? > > Hi, Rob Herring: > > As Marc Zyngier's suggestion, I discarded adding an independent SD5203-VIC > driver, but make the dw-apb-ictl irqchip driver to support hierarchy irq domain. > So this new file was also dropped. Now, I updated the descriptions in the existing > file "snps,dw-apb-ictl.txt" in the following versions. > >> >>> 1 file changed, 27 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/hisilicon,sd5203-vic.txt >>> >>> diff --git a/Documentation/devicetree/bindings/interrupt-controller/hisilicon,sd5203-vic.txt b/Documentation/devicetree/bindings/interrupt-controller/hisilicon,sd5203-vic.txt >>> new file mode 100644 >>> index 000000000000..a08292e868b0 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/interrupt-controller/hisilicon,sd5203-vic.txt >>> @@ -0,0 +1,27 @@ >>> +Hisilicon SD5203 vector interrupt controller (VIC) >>> + >>> +Hisilicon SD5203 VIC based on Synopsys DesignWare APB interrupt controller, but >>> +there's something special: >>> +1. The maximum number of irqs supported is 32. The registers ENABLE, MASK and >>> + FINALSTATUS are 32 bits. >>> +2. There is only one VIC, it's used as primary interrupt controller. >>> + >>> +Required properties: >>> +- compatible: shall be "hisilicon,sd5203-vic" >>> +- reg: physical base address of the controller and length of memory mapped >>> + region starting with ENABLE_LOW register >>> +- interrupt-controller: identifies the node as an interrupt controller >>> +- #interrupt-cells: number of cells to encode an interrupt-specifier, shall be 1 >>> + >>> +The interrupt sources map to the corresponding bits in the interrupt >>> +registers, i.e. >>> +- 0 maps to bit 0 of low interrupts, >>> +- 1 maps to bit 1 of low interrupts, >>> + >>> +Example: >>> + vic: interrupt-controller@10130000 { >>> + compatible = "hisilicon,sd5203-vic"; >>> + reg = <0x10130000 0x1000>; >>> + interrupt-controller; >>> + #interrupt-cells = <1>; >>> + }; >>> -- >>> 2.26.0.106.g9fadedd >>> >>> >> >> . >>